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
<GitHub-m-labs> [artiq] sbourdeauducq commented on issue #1059: DRTIO targets (master, satellite) will also have the option to build without SAWG, so this creates a lot of variants.... https://github.com/m-labs/artiq/issues/1059#issuecomment-398248891
kaolpr has quit [Ping timeout: 276 seconds]
kaolpr has joined #m-labs
<GitHub60> [smoltcp] dlrobertson opened issue #241: Add the NDISC cache entry states to Neighbors in the NeighborCache https://github.com/m-labs/smoltcp/issues/241
<GitHub88> [smoltcp] dlrobertson commented on issue #241: @squidarth since you've already done some work on the neighbor cache, this could be a good follow-up issue. It will be a good bit more work as the use of cache entries in `EthernetInterface` will need to be updated, but I think it is a reasonable next issue to work on, once #234 is merged. https://github.com/m-labs/smoltcp/issues/241#issuecomment-398254380
<GitHub3> [smoltcp] dlrobertson commented on issue #240: @m-labs-homu r+ https://github.com/m-labs/smoltcp/pull/240#issuecomment-398259840
<GitHub89> [smoltcp] m-labs-homu commented on issue #240: :pushpin: Commit 3f1a66a has been approved by `dlrobertson`
<GitHub83> [smoltcp] m-labs-homu pushed 1 new commit to auto: https://github.com/m-labs/smoltcp/commit/bf249333cdfbe026b3d0a8a9fb00e2251c7812d3
<GitHub83> smoltcp/auto bf24933 Dan Robertson: Add clippy to CI...
<GitHub96> [smoltcp] m-labs-homu commented on issue #240: :hourglass: Testing commit 3f1a66a02fe1eff2a0c807dc88c3fc7664502b2b with merge bf249333cdfbe026b3d0a8a9fb00e2251c7812d3... https://github.com/m-labs/smoltcp/pull/240#issuecomment-398259876
<travis-ci> m-labs/smoltcp#1041 (auto - bf24933 : Dan Robertson): The build passed.
<GitHub55> [smoltcp] m-labs-homu commented on issue #240: :sunny: Test successful - [status-travis](https://travis-ci.org/m-labs/smoltcp/builds/393930368?utm_source=github_status&utm_medium=notification)
<GitHub173> [smoltcp] m-labs-homu merged auto into master: https://github.com/m-labs/smoltcp/compare/7699265a0939...bf249333cdfb
<GitHub63> [smoltcp] m-labs-homu closed pull request #240: Add clippy to CI (master...add_clippy) https://github.com/m-labs/smoltcp/pull/240
<GitHub177> [smoltcp] dlrobertson opened issue #242: Fix clippy warnings https://github.com/m-labs/smoltcp/issues/242
<GitHub158> [smoltcp] dlrobertson closed issue #239: Add clippy to CI https://github.com/m-labs/smoltcp/issues/239
<travis-ci> m-labs/smoltcp#1042 (master - bf24933 : Dan Robertson): The build passed.
rohitksingh_work has joined #m-labs
<GitHub154> [smoltcp] pothos closed issue #224: Stuck at retransmitting https://github.com/m-labs/smoltcp/issues/224
rohitksingh_work has quit [Ping timeout: 276 seconds]
rohitksingh_work has joined #m-labs
rohitksingh_wor1 has joined #m-labs
rohitksingh_work has quit [Ping timeout: 245 seconds]
<GitHub177> [smoltcp] jD91mZM2 opened pull request #243: Fix impossible lifetime bounds (master...master) https://github.com/m-labs/smoltcp/pull/243
<GitHub78> [smoltcp] jD91mZM2 closed issue #226: Ethernet::poll infinite loop https://github.com/m-labs/smoltcp/issues/226
<GitHub156> [smoltcp] jD91mZM2 commented on issue #226: Fixed on the latest master. See #224. https://github.com/m-labs/smoltcp/issues/226#issuecomment-398289971
<GitHub-m-labs> [artiq] sbourdeauducq opened issue #1074: ".dirty" in firmware version does not appear reliably https://github.com/m-labs/artiq/issues/1074
sb0 has quit [Quit: Leaving]
hartytp has quit [Quit: Page closed]
sb0 has joined #m-labs
<GitHub-m-labs> [artiq] sbourdeauducq pushed 6 new commits to master: https://github.com/m-labs/artiq/compare/6f3ed816261b...574892a4e549
<GitHub-m-labs> artiq/master 476cfa0 Sebastien Bourdeauducq: si5324: improve lock messaging
<GitHub-m-labs> artiq/master 6403a0d Sebastien Bourdeauducq: sayma_amc: update without-sawg description
<GitHub-m-labs> artiq/master d29b3dd Sebastien Bourdeauducq: hmc830: compile-time configurable reference frequency
rohitksingh_work has joined #m-labs
rohitksingh_wor1 has quit [Ping timeout: 264 seconds]
rohitksingh_wor1 has joined #m-labs
rohitksingh_work has quit [Ping timeout: 240 seconds]
<bb-m-labs> build #1657 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1657
<bb-m-labs> build #2461 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2461 blamelist: Sebastien Bourdeauducq <sb@m-labs.hk>
rohitksingh_work has joined #m-labs
rohitksingh_wor1 has quit [Ping timeout: 245 seconds]
rohitksingh_wor1 has joined #m-labs
<sb0> of course, the vivado/ultrascale trash is giving me BS for clocking the JESD transceivers from the Si5324
<sb0> sigh
<sb0> _florent_, how do I clock the transceivers with one clock and the rest of the fabric with another (both from the same oscillator) in the JESD core?
rohitksingh_work has quit [Ping timeout: 256 seconds]
<sb0> _florent_, is it sufficient to give a refclk with one clock and a jesd clock domain with the other? do the elastic buffers take care of that?
<sb0> "Since the IBUFDS connects to the GT in intelligent pin mode..."
<sb0> they call it "intelligent pin mode"
<_florent_> sb0: yes, if the clock you are using before the EBs is from the same oscillator than the clocks used for the transceiver, the elastic buffer will handle that
<sb0> so I can provide any refclk and any jesd clock domains, as long as they're from the same oscillator?
<sb0> to be explicit, I want to keep the current refclk from the hmc7043 (since vivado breaks otherwise) to clock the transceiver, and use the existing rtio clock domain (coming from the si5234) to drive the jesd domain
<_florent_> in the current design, we use the clock from the hmc7043 as refclk to the transceiver, and also as clock for the core logic
<_florent_> so yes it should be fine if frequency are the same and derivated from the same oscillator
<sb0> okay, but I have to change this for drtio
<_florent_> frequencies
<sb0> what I am asking is whether refclk is used for anything other than the transceiver side of the elastic buffer
<sb0> (and the transceiver itself)
<larsc> just be aware that you'll probably have a random phase to the sysref signal if the clock is not sourced from the same clockchip
<sb0> yes, aligning that phase comes next
<_florent_> refclk is only use on the phy (so transceiver and transceiver side of the elastic buffer)
<_florent_> used
<sb0> ok, thanks
<GitHub-m-labs> [artiq] sbourdeauducq pushed 4 new commits to master: https://github.com/m-labs/artiq/compare/574892a4e549...75b6cea52ff6
<GitHub-m-labs> artiq/master eb3259b Sebastien Bourdeauducq: firmware: reduce number of DAC initialization attempts...
<GitHub-m-labs> artiq/master 158b5e3 Sebastien Bourdeauducq: satman: program Allaki
<GitHub-m-labs> artiq/master 1d594d0 Sebastien Bourdeauducq: firmware: make DAC initialization failures non-fatal...
<sb0> okay, we have DAC clocking and init from the DRTIO satellite now
<sb0> (without phase sync)
<sb0> let me try sawg-over-drtio...
sb0 has quit [Quit: Leaving]
<bb-m-labs> build #1658 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1658
<bb-m-labs> build #2462 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2462 blamelist: Sebastien Bourdeauducq <sb@m-labs.hk>
rohitksingh_wor1 has quit [Read error: Connection reset by peer]
<GitHub76> [smoltcp] dlrobertson commented on issue #243: @m-labs-homu r+ https://github.com/m-labs/smoltcp/pull/243#issuecomment-398394359
<GitHub83> [smoltcp] m-labs-homu commented on issue #243: :pushpin: Commit 659ff57 has been approved by `dlrobertson`
<GitHub14> [smoltcp] m-labs-homu pushed 1 new commit to auto: https://github.com/m-labs/smoltcp/commit/d23aee483f2dc78e8070d2ad53e400cd0dcb9ae1
<GitHub14> smoltcp/auto d23aee4 jD91mZM2: Fix impossible lifetime bounds...
<GitHub133> [smoltcp] m-labs-homu commented on issue #243: :hourglass: Testing commit 659ff57b03886ce763117585d9c163f4a3ec9a90 with merge d23aee483f2dc78e8070d2ad53e400cd0dcb9ae1... https://github.com/m-labs/smoltcp/pull/243#issuecomment-398394456
<GitHub70> [smoltcp] m-labs-homu merged auto into master: https://github.com/m-labs/smoltcp/compare/bf249333cdfb...d23aee483f2d
<GitHub163> [smoltcp] m-labs-homu commented on issue #243: :sunny: Test successful - [status-travis](https://travis-ci.org/m-labs/smoltcp/builds/394100120?utm_source=github_status&utm_medium=notification)
<GitHub100> [smoltcp] m-labs-homu closed pull request #243: Fix impossible lifetime bounds (master...master) https://github.com/m-labs/smoltcp/pull/243
<travis-ci> m-labs/smoltcp#1047 (master - d23aee4 : jD91mZM2): The build passed.
rohitksingh has joined #m-labs
sb0 has joined #m-labs
<GitHub106> [smoltcp] whitequark commented on issue #242: I'm not extremely enthusiastic about clippy and I believe that blindly following its suggestions can easily degrade code quality. r=me on every PR addressing its lints. https://github.com/m-labs/smoltcp/issues/242#issuecomment-398410312
<GitHub-m-labs> [migen] sbourdeauducq pushed 1 new commit to master: https://github.com/m-labs/migen/commit/07c46f55474b4a196382213d304890bd6ba2b941
<GitHub-m-labs> migen/master 07c46f5 Mikołaj Sowiński: Support for AFC 3v1
<bb-m-labs> build #286 of migen is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/migen/builds/286
<GitHub194> [smoltcp] jD91mZM2 opened pull request #244: Clean up routes module (master...master) https://github.com/m-labs/smoltcp/pull/244
<GitHub153> [smoltcp] whitequark commented on issue #244: The `update` method is there so that any invariants on the internal `ManagedMap` can be enforced. But maybe we should not expose it at all, rather have an `add_route` method and such. https://github.com/m-labs/smoltcp/pull/244#issuecomment-398418763
<GitHub198> [smoltcp] dlrobertson commented on issue #242: > believe that blindly following its suggestions can easily degrade code quality.... https://github.com/m-labs/smoltcp/issues/242#issuecomment-398419808
<GitHub121> [smoltcp] jD91mZM2 commented on issue #244: I think `update` is fine, but there's no reason to have it take a closure :) https://github.com/m-labs/smoltcp/pull/244#issuecomment-398423169
<GitHub63> [smoltcp] whitequark commented on issue #244: There is--if you just give away a `&mut` there is no way to maintain any internal invariants. There aren't any right now but that is likely wrong. https://github.com/m-labs/smoltcp/pull/244#issuecomment-398426745
<GitHub173> [smoltcp] jD91mZM2 commented on issue #244: What exactly do you mean by internal invariants? https://github.com/m-labs/smoltcp/pull/244#issuecomment-398428174
<GitHub17> [smoltcp] whitequark commented on issue #244: Not every route is legal. For example, no multicast addresses should be added to the route table. This isn't checked currently, but should be, and it can't be checked with `get_mut`. https://github.com/m-labs/smoltcp/pull/244#issuecomment-398429415
rohitksingh has quit [Read error: Connection reset by peer]
hartytp has joined #m-labs
<hartytp> marmelada: did you get a chance to look at Sayma PI with SAWG?
rohitksingh has joined #m-labs
<hartytp> sb0: did you see try commenting out the serwb + RTM init on Sayma?
<hartytp> if so, does that still crash?
<hartytp> it would be interesting to see if we can reproduce the crashes with the RTM physically disconnected (reduces the number of places for issues to hide)
<sb0> hartytp, no, feel free to try
<hartytp> I'm building gw to do that atm
<hartytp> just wanted to check that I wasn't missing something
<sb0> I don't think we can reproduce with the rtm disconnected, unfortunately; though if you want to try that, your best bet is the reboot loops
<hartytp> why not?
<sb0> the crash kernel needs a lot of things and resists attempts at minimizing it (the crashes disappear of you change the slightest thing)
<sb0> *if
<hartytp> hmmm o
<hartytp> ok
<hartytp> so, are you *sure* this is not a gw/fw bug?
<sb0> so, for the reboot loops, what you can try is jesd_unreset() and physically disconnect the rtm
<sb0> but first, see if you can reproduce my reboot loop results
<hartytp> based on how quickly rjo was able to find a number of issues with artiq I wonder if the review process has been sufficient
<sb0> there are also a lot of issues with the hardware
<hartytp> some
<hartytp> but there is no evidence so far that this is due to one
<sb0> well, things basically break when a lot of things are running in the FPGA, that looks like PI to me
<hartytp> or a missing timing constraint
<hartytp> or one of a few other things
<hartytp> I don't really understand the project management structure here
<sb0> the main CPU crashing when a clock buffer in an unrelated clock domain is enabled doesn't look like timing
<hartytp> which clock buffer was that again?
<sb0> the jesd_unreset()
<hartytp> are you sure you fixed the HMC7043 reset line?
<hartytp> i.e. does keeping it high in fw kill the DACs?
<sb0> no, and there can be other hardware problems, so this is why I asked for reproductions of those results on other boards
<hartytp> I did that
<hartytp> or, did I not do all the tests you wanted?
<sb0> how long did you run the reboot loops?
<hartytp> remind me exactly which test this was
<sb0> I know, but how long did you run it for?
<sb0> ah I missed the log ...
<sb0> well, i'd run that a bit longer.
<hartytp> that should have been plenty given the frequency of the symptoms you reported
<hartytp> I also didn't see the same behaviour with that Kernel
<sb0> yes, your board seems less crashy than mine
<hartytp> it did crash my system, but I only saw a single mem test issue afterwards
<sb0> the kernel also ran longer
<hartytp> right, which is why I want to know if you've verified that your HMC7043 is RESET properly
<sb0> and it does take a few minutes on mine to crash with the reboot loop
<sb0> does the hmc7043 still identify over SPI when in reset?
<sb0> we can simply add a check to the firmware
<hartytp> don't know
<hartytp> wouldn't have thought so, but needs to be checked
<sb0> also i reworked another board and brought it up today, but haven't had time to look into the corruption shitbug
<hartytp> sounds like a good idea though given the circs
<hartytp> it seems likely that there is more than one issue at play here
<hartytp> 1. The crash kernel, which I can reproduce
<hartytp> 2. You're saying that enabling the dac REFCLK and SYSREF inputs on the FPGA crashes your CPU
<hartytp> I haven't seen aything like 2 since I fixed the HMC7043 reset
<hartytp> which is why I want to be sure that you've fixed it on your board before I waste time trying to track it down further
<hartytp> and, the test is simple: just leave it in reset and see if the JESD is able to init
<sb0> reboot loop crashing on a reworked board would be very useful. the crash kernel is essentially impossible to debug.
<hartytp> ack, well, I'll look at that again once you've verified your hmc7043 rework
<sb0> hartytp, HMC ID reads 0 when in reset, and the correct value when not ...
<hartytp> okay, thanks, so it's not that then
<sb0> let me double check...
<hartytp> so, to be clear, you want me to add a jump(0) after the AD9154 init
<hartytp> then run for a while while gathering UART logs
<sb0> shouldn't we add a delay after releasing the reset?
<hartytp> on your board that causes a crash
<hartytp> every how often?
<sb0> few minutes
<hartytp> sb0: so, something like 1 time in 10?
<hartytp> sb0: no idea about need for delays
<sb0> the datasheet doesn't say afaict
<hartytp> data sheet?
<hartytp> 7043?
<hartytp> that clock starts up without glitches
<hartytp> because we simply unmute the output driver
<hartytp> (verified when we were using LVPECL, but I wouldn't have expected that to change)
<hartytp> also, did you check if rjo's commits fixed this issue?
<sb0> confirmed it's the hardware reset release that causes the ID register to assume its value.
<sb0> they don't fix the crash kernel issue, haven't tried reboot loops. those things suck time.
<GitHub-m-labs> [artiq] sbourdeauducq pushed 3 new commits to master: https://github.com/m-labs/artiq/compare/75b6cea52ff6...4803ca379940
<GitHub-m-labs> artiq/master 4803ca3 Sebastien Bourdeauducq: examples/sayma_drtio: add SAWG channels
<GitHub-m-labs> artiq/master 3d0e92a Sebastien Bourdeauducq: hmc7043: check that chip is disabled at startup
<GitHub-m-labs> artiq/master 740e686 Sebastien Bourdeauducq: hmc7043: add delay after releasing hardware reset
<hartytp> sb0: well, I'll double check the reboot loop
<hartytp> on my board
<sb0> ok, thanks
<sb0> I'll run them on the two boards I have, after the iostandard fixes...
<hartytp> then I'll see if I can get the kernel to crash on Sayma with some of the init removed
<hartytp> with the idea of working towards a reproduction of the kernel crash on the AMC only
<sb0> well, you need to get the RTIO clock running
<hartytp> (I can do the same for the reboot loop if I can reproduce that now)
<sb0> which is clocked by JESD
<sb0> so you need most of the stack running...
<hartytp> hmmm
<hartytp> okay
<sb0> that's not the case if you use the DRTIO master, though :)
<hartytp> does DRTIO master have the same crash
<hartytp> ?
<sb0> nope.
<hartytp> so, do you think there is some issue with the way you're clocking RTIO in the standalone variant?
<hartytp> what are the differences between the two variants?
<sb0> the DRTIO master so far does not load the FPGA very much
<sb0> it is also unaffected by this other unexplicable sayma-bug, https://github.com/m-labs/artiq/issues/998
<sb0> but I'm adding SAWG to the DRTIO master now ...
<hartytp> ok
<hartytp> well, let me know how that goes
<hartytp> I'll work on those tests
hartytp has quit [Quit: Page closed]
<GitHub-m-labs> [artiq] sbourdeauducq pushed 1 new commit to master: https://github.com/m-labs/artiq/commit/0c32d07e8b59e60ee304753e732bfa0c07e0daa6
<GitHub-m-labs> artiq/master 0c32d07 Sebastien Bourdeauducq: ad9154: new sysref scan...
rohitksingh has quit [Read error: Connection reset by peer]
rohitksingh has joined #m-labs
<bb-m-labs> build #1659 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1659
<bb-m-labs> build #2463 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2463 blamelist: Sebastien Bourdeauducq <sb@m-labs.hk>
rohitksingh has quit [Quit: Leaving.]
<bb-m-labs> build #1660 of artiq-board is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/1660
<bb-m-labs> build #2464 of artiq is complete: Failure [failed python_unittest_2] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2464 blamelist: Sebastien Bourdeauducq <sb@m-labs.hk>
<GitHub64> [smoltcp] podhrmic commented on issue #236: @pothos @whitequark any feedback here? https://github.com/m-labs/smoltcp/pull/236#issuecomment-398529379
mumptai has joined #m-labs
mumptai has quit [Quit: Verlassend]
Gurty has quit [Ping timeout: 245 seconds]
Gurty has joined #m-labs
little-dude has joined #m-labs