sb0 changed the topic of #m-labs to: ARTIQ, Migen, MiSoC, Mixxeo & other M-Labs projects :: fka #milkymist :: Logs http://irclog.whitequark.org/m-labs
<GitHub150>
[smoltcp] dlrobertson opened pull request #59: Implement set_ttl for Tcp and Udp sockets (master...add_ttl) https://git.io/vd16o
<GitHub123>
[smoltcp] dlrobertson commented on pull request #59 c499cf9: This isn't super useful. I'd be okay with backing this out. https://git.io/vd1ie
<GitHub72>
[smoltcp] dlrobertson commented on pull request #59 c499cf9: This isn't ergonomic to have to specify the `ttl` member on initialization. https://git.io/vd16h
<GitHub125>
[smoltcp] dlrobertson commented on pull request #59 c499cf9: This is totally unrelated to the TTL value change. I'm 100% fine with backing this out. https://git.io/vd16j
_whitelogger has joined #m-labs
_whitelogger has joined #m-labs
_whitelogger has joined #m-labs
_whitelogger has joined #m-labs
_whitelogger has joined #m-labs
artiq__ has joined #m-labs
<artiq__>
force build --props=package=artiq-kc705-phaser artiq-board
<artiq__>
Is there a buildbot available for the phaser environment that works ubuntu?
<sb0>
artiq__, works ubuntu?
<sb0>
the phaser stuff itself runs on the fpga board and is independent of the OS of your computer
<GitHub198>
[smoltcp] whitequark commented on pull request #59 c499cf9: `an explicitly set TTL value`, `IANA recommended` https://git.io/vd1Hf
<GitHub11>
[smoltcp] whitequark commented on pull request #59 c499cf9: `Set the time-to-live (IPv4) or hop limit (IPv6) value used in outgoing packets.` https://git.io/vd1Hv
<GitHub157>
[smoltcp] whitequark commented on pull request #59 c499cf9: Same as above. https://git.io/vd1He
<GitHub113>
[smoltcp] whitequark commented on pull request #59 c499cf9: `The time-to-live (IPv4) or hop limit (IPv6) value used in outgoing packets.` https://git.io/vd19j
<artiq__>
Hi sb0
<artiq__>
Could you please activate a buildbot for phaser environment? Or if there is one active already how can I call it?
<GitHub1>
[smoltcp] whitequark commented on issue #54: @little-dude Sure why not, if it's up to smoltcp's code quality. We can always split it afterwards.... https://git.io/vd1HT
<whitequark>
bb-m-labs: force build --props=package=artiq-kc705-phaser artiq-board
<bb-m-labs>
build forced [ETA 13m39s]
<bb-m-labs>
I'll give a shout when the build finishes
<GitHub174>
[smoltcp] whitequark commented on issue #55: So... why does _smoltcp_ need to support multiple interfaces? It's not only perfectly doable but also IMO more convenient to build routing on *top* of it. That is, packets originating in the OS or destined to the OS are routed according to the OS' routing table, and the rest is handled using raw sockets, which also provide buffering, because it's not really possible to do routing without buffering with
<artiq__>
force build --props=package=artiq-kc705-phaser artiq-board
<sb0>
artiq__, whitequark just built that package
<artiq__>
I am new to buildbot. I installed buildbot. Can you tell me what commands do I need to make the buildbot program to install on my station the built?
<sb0>
what exactly are you trying to do?
<artiq__>
I want to install phaser environment to be run on kc7 board.
<sb0>
ok, you don't need to install buildbot for that
<artiq__>
OR this one: conda create -n artiq-main artiq-kc705-nist_clock ?
<sb0>
yes
<artiq__>
So that would just become conda create -n artiq-main artiq-kc705-phaser ?
<sb0>
sure
<sb0>
it's just a conda package name, you can look them up on anaconda.org
<artiq__>
great! thanks
<GitHub122>
[smoltcp] whitequark closed issue #56: rustfmt.toml for smoltcp https://git.io/vdiX0
<GitHub176>
[smoltcp] whitequark commented on issue #56: I looked at the diff produced by rustfmt. It is 11937 line long and readability suffers, particularly badly with tests. Thank you for your work but I don't think I'm going to use rustfmt for my projects. https://git.io/vd15e
_whitelogger has joined #m-labs
<felix_>
mithro: iirc vivado tries to detect if its being straced or debugged from some version on and then just prints some error message. iirc clifford had that problem a while ago; not sure how he solved it though
<mithro>
An alternative option is a fuse filesystem which traces file accesses...
<whitequark>
you could just disable hat check
<GitHub180>
[smoltcp] whitequark commented on pull request #57 61737cd: `writer` is a thin pointer so why not https://git.io/vd1NJ
<GitHub118>
[smoltcp] whitequark commented on pull request #57 61737cd: Oh, this also works. Sweet, this is the 1st time I find higher-ranker lifetimes useful, personally. https://git.io/vd1NI
<GitHub155>
[smoltcp] whitequark commented on pull request #57 61737cd: This is an implementation detail so just initialize it with struct initialization syntax. No need for a separate `new` method etc. https://git.io/vd1Nk
<GitHub26>
[smoltcp] whitequark commented on pull request #57 61737cd: I think this can just be `tx_token: DeviceT::TxToken` https://git.io/vd1Nt
<GitHub165>
[smoltcp] whitequark commented on pull request #57 61737cd: You should document the reason for `InterfaceInner`'s existence. https://git.io/vd1Nq
<GitHub144>
[smoltcp] whitequark commented on pull request #57 61737cd: Isn't this condition inverted? I think you can just move it into the `else` branch. https://git.io/vd1Nf
<GitHub135>
[smoltcp] whitequark commented on pull request #57 61737cd: You need to *first* call `f` and *then* record the packet or you'll record junk. https://git.io/vd1NU
<GitHub129>
[smoltcp] whitequark commented on pull request #57 61737cd: You need a mutex because state is mutable. Without a mutex it will always perform the same action on every packet. https://git.io/vd1NT
<GitHub91>
[smoltcp] whitequark commented on pull request #57 61737cd: Why is this necessary? https://git.io/vd1Nm
<GitHub16>
[smoltcp] whitequark commented on pull request #57 61737cd: Is this really needed? I thought it's enough that `Device` is using `'a`. https://git.io/vd1NL
<GitHub169>
[smoltcp] whitequark commented on pull request #57 61737cd: :+1: Pays off already! https://git.io/vd1NY
<GitHub182>
[smoltcp] whitequark commented on issue #57: @phil-opp looks like github has marked a few comments as "outdated" where I didn't intend that, so please expand them all. https://git.io/vd1NO
<GitHub31>
[smoltcp] whitequark pushed 1 new commit to master: https://git.io/vd1Nc
<GitHub31>
smoltcp/master bbb54fe whitequark: Fix a (future) syntax error in the net_log macro.
<travis-ci>
m-labs/smoltcp#316 (master - bbb54fe : whitequark): The build passed.
<GitHub96>
[smoltcp] batonius commented on issue #55: That would mean I have to use an instance of `SocketSet` for each `EthernetInterface`, so sockets bound to 0.0.0.0 gonna be problematic, but I think I can just duplicate them. I'll try to implement this approach and report back with results. https://git.io/vdMn3
<GitHub109>
[smoltcp] whitequark commented on issue #55: Once you figure out a design we might lift it into smoltcp, but right now I feel like it's not the right direction for this project to delve into. https://git.io/vdMcl