Wednesday, 2019-10-30

*** tpb has joined #symbiflow00:00
*** proteusguy has quit IRC00:58
*** _whitelogger has quit IRC01:06
*** _whitelogger has joined #symbiflow01:09
*** proteusguy has joined #symbiflow01:12
*** proteusguy has quit IRC01:28
*** proteusguy has joined #symbiflow01:41
*** craigo has joined #symbiflow02:01
*** davidw has joined #symbiflow03:11
*** davidw is now known as Guest5488503:12
*** davidthings has quit IRC03:12
*** Guest54885 has quit IRC03:55
*** Guest54885 has joined #symbiflow03:56
*** Guest54885 has quit IRC03:59
*** davidthings has joined #symbiflow04:00
*** davidthings has quit IRC05:50
*** kraiskil has joined #symbiflow06:19
*** OmniMancer has joined #symbiflow06:20
*** Bertl_oO is now known as Bertl_zZ06:33
*** freemint has joined #symbiflow07:34
-_whitenotifier- [symbiflow-arch-defs] mkurc-ant opened issue #1119: Add a simulation model for the PLLE2_ADV - https://git.io/JezUw09:25
*** celadon has quit IRC09:41
*** celadon has joined #symbiflow09:43
*** kraiskil has quit IRC10:05
*** proteusguy has quit IRC10:34
*** proteusguy has joined #symbiflow10:34
*** proteus-guy has joined #symbiflow10:35
*** proteus-guy has quit IRC10:37
*** proteusguy has quit IRC10:37
*** proteusguy has joined #symbiflow10:37
*** zeigren has quit IRC10:42
*** hzeller[m] has quit IRC10:42
*** mrhat2010[m] has quit IRC10:42
*** nrossi has quit IRC10:42
*** galv[m] has quit IRC10:43
*** alexhw[m] has quit IRC10:43
*** synaption[m] has quit IRC10:43
*** xobs has quit IRC10:43
*** lromor[m] has quit IRC10:43
*** proteus-guy has joined #symbiflow10:43
*** clacktronics has quit IRC11:04
*** synaption[m] has joined #symbiflow11:07
*** clacktronics has joined #symbiflow11:12
*** clacktronics has joined #symbiflow11:14
*** clacktronics has quit IRC11:31
*** clacktronics has joined #symbiflow11:32
*** kraiskil has joined #symbiflow11:48
*** zeigren has joined #symbiflow11:49
*** hzeller[m] has joined #symbiflow11:49
*** galv[m] has joined #symbiflow11:49
*** nrossi has joined #symbiflow11:49
*** xobs has joined #symbiflow11:49
*** alexhw[m] has joined #symbiflow11:49
*** mrhat2010[m] has joined #symbiflow11:49
*** lromor[m] has joined #symbiflow11:49
*** citypw has joined #symbiflow12:08
*** freemint has quit IRC13:30
*** davidthings has joined #symbiflow13:33
*** freemint has joined #symbiflow13:45
*** Bertl_zZ is now known as Bertl14:38
*** OmniMancer has quit IRC15:44
*** freemint has quit IRC15:53
hackerfoolitghost: I hadn't tested FILL_LIMIT yet; commenting that out at the top avoids the routing error.16:06
litghostAh, sure16:07
hackerfooBut then it ran all night, so I need to diagnose that.16:07
*** freemint has joined #symbiflow16:08
hackerfooI'm going to test without BREAK_ON_MISS, but that has been tested and worked before the rebase.16:09
hackerfooI changed the hash algorithm too, but bad hashes shouldn't affect the result.16:12
*** citypw has quit IRC16:33
*** david__ has joined #symbiflow16:33
*** davidthings has quit IRC16:34
*** david__ has quit IRC16:51
*** Bertl is now known as Bertl_oO17:13
*** freemint has quit IRC17:31
hackerfoolitghost: Were there any recent changes that might affect routing speed?18:40
hackerfoo(upstream)18:40
litghosthackerfoo: Nothing obvious, why?18:40
hackerfooIt seems much slower, and I don't know why yet.18:41
litghostSlower with the same heap push count?18:42
litghostOr are you getting more heap push's?18:42
litghostAnd by what factor are we talking about?18:42
litghost10% or 2x/3x/4x?18:42
hackerfooLast night it took 10x per iteration without any major changes to the lookahead.18:44
litghostDo you have 10x heap push's? Or just 10x slower with same # of heap pushs?18:44
hackerfooI think it was about 1.5x heap pushes.18:45
litghostThat's pretty odd.  If you recompile VTR at the previous git commit and use the same rrgraph and lookahead, does the behavior revert?18:46
litghostAt the end of the day, the route time should only be affected by:18:47
litghostthe circuit and it's pack/place18:47
litghostthe lookahead18:47
litghostthe rrgraph18:47
litghostVTR itself18:47
litghostYou can fix all but one and retest (though you might need to supply "--verify_file_digests off"18:48
litghostThe arch.xml is in there too, but I doubt it's changing at all18:49
hackerfooOkay, thanks. I'll look into it more.18:49
*** proteus-guy has quit IRC19:32
lromor[m]Hi, my makefile is not working anymore when running:20:26
lromor[m]```20:26
lromor[m]```20:26
* lromor[m] sent a long message: < https://matrix.org/_matrix/media/r0/download/matrix.org/FsVMjjQPcOYeaPlTHrdSzxNB >20:27
litghostThere is a defect in the CMake dependencies around python libraries, solution is to remove <build_dir>/xc7/archs/artix7/channels.db20:33
litghostSorry for the confusion20:33
litghostOf course maybe I'm misunderstanding what you are trying to do20:33
litghostThat particular error would arise if the arch.xml and rrgraph.xml got out of sync20:34
lromor[m]I'20:35
litghostThat message was truncated, I don't follow what you were asking for20:35
lromor[m] * When generating the packed netlist (.net) I'm using those the arch timings and the rr graph. I guess there's a mismatch.20:36
litghostThe mismatch inidicated in that error is a mismatch between the arch.xml and rrgraph.xml20:36
litghostI recommend re-running the following CMake target "file_xc7_archs_artix7_devices_rr_graph_xc7a50t-arty-swbut_test.lookahead.bin"20:37
lromor[m]I'm not sure at which point I should remove the channels.db as it's not a dependency of my vpr command.20:37
litghostI would remove channels.db and then rebuild up to "file_xc7_archs_artix7_devices_rr_graph_xc7a50t-arty-swbut_test.lookahead.bin"20:37
litghostThen take the new arch.xml/rrgraph.xml/lookaheads20:38
lromor[m]ok20:38
litghostDoes that make sense?20:38
lromor[m]Yup! Thanks!20:38
*** craigo has quit IRC20:39
litghostOne thing we will likely be adding the in next couple weeks once the ROI breakout is done is some kind of "make install" that exports the files needed to run VPR against various circuits20:39
litghostOutside of the symbiflow CMake system20:39
litghostI imagine that will make your flow easier20:40
lromor[m]That would ideal! Thank you for your work!20:46
*** kraiskil has quit IRC21:55
lromor[m]litghost: That fixed the issue! Thanks21:56
litghostGreat!21:56
*** kraiskil has joined #symbiflow22:24
*** freemint has joined #symbiflow22:27
*** kraiskil has quit IRC22:28
*** Bertl_oO is now known as Bertl_zZ22:34
hackerfoolitghost: Routing works now, but it's still a little under 2x slower than before for some reason. I'm going to try to get the PR out today.23:00
litghostSounds good23:01
litghostI have my debugging tools ready to go and tommorow hopefully I'll be able to see problems23:02
mithrolitghost: Does this look good? https://github.com/SymbiFlow/prjxray-db/compare/master...mithro:master23:20
tpbTitle: Comparing SymbiFlow:master...mithro:master · SymbiFlow/prjxray-db · GitHub (at github.com)23:20
hackerfoolitghost: The PR is ready for review. There are some things you can tweak at the top of connection_box_lookahead_map.cpp. The things we decide to keep can be made into flags.23:30
litghostmithro: https://github.com/SymbiFlow/prjxray-db/compare/master...mithro:master#diff-55623f36146a8aa2a84d52b0106f499aR1287 is wrong23:30
tpbTitle: Comparing SymbiFlow:master...mithro:master · SymbiFlow/prjxray-db · GitHub (at github.com)23:30
hackerfooThere's also some nice ASCII-art of the cost maps that I've left on for now.23:30
litghostmithro: Everything else looks okay23:31
litghosthackerfoo: How does murax first iteration look with https://github.com/SymbiFlow/vtr-verilog-to-routing/pull/310 ?23:35
tpbTitle: Parallel connection box lookahead by HackerFoo · Pull Request #310 · SymbiFlow/vtr-verilog-to-routing · GitHub (at github.com)23:35
mithrolitghost: The INT_L.IMUX_L33.LOGIC_OUTS_L18  change in that pull request doesn't look right?23:37
litghostmithro: Ya, there was no change that should have caused that23:38
litghostmithro: So it's some form of instability23:38
hackerfoolitghost: I'm currently running it, but last time it ran for 1188s with 2.4e9 heap pushes and a CPD of 11.809. This was with BREAK_ON_MISS = true, so I'm running without that since I don't trust that optimization yet.23:39
hackerfooIt'll probably be the same, though.23:39
litghosthackerfoo: Okay, that matches about what I was getting with your wip-testing @ 0477b874ad1be65626c99ae13f707586a7dae11623:41
hackerfooThe fill now follows a diamond pattern to match the Manhattan distance, and penalizes based on the distance. The max_fill's (sizes of largest hole) are still surprizingly high.23:41
hackerfooSo increasing sampling from 3x3 might help, because there is a lot of guessing right now.23:42
litghosthackerfoo: I don't expect increasing the sample grid will change much actually, as the holes are likely a function of the graph structure23:43
litghosthackerfoo: I do think a 2D interpolation may be better suited to hole filling, but I'm waiting to get data to support that23:43
mithrolitghost: Should I just revert those changes?23:44
hackerfooTurning on FILL_LIMIT limits the amount of filling, but then I get an error routing GND to a LUT.23:44
litghostmithro: Just the ones to INT_L23:44
hackerfooEven setting it at 30.23:44
litghosthackerfoo: It's interesting, I didn't see the same issue on the prevoius search, but I cannot explain why yet23:45
litghosthackerfoo: I was able to use a fill limit of 3 and it worked23:45
hackerfooMaybe I'll try it again, because I fixed some things.23:46
hackerfooThis might be useful to someone else: https://github.com/HackerFoo/hackerfoo-go-functions/blob/master/functions/symbiflow#L734-L74523:54
tpbTitle: hackerfoo-go-functions/symbiflow at master · HackerFoo/hackerfoo-go-functions · GitHub (at github.com)23:54
hackerfooIt lists all the wip branches that will and will not merge cleanly.23:55
hackerfooIt doesn't detect conflicts between them, though.23:56

Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!