| *** tpb has joined #symbiflow | 00:00 | |
| *** kraiskil has quit IRC | 00:23 | |
| *** kraiskil has joined #symbiflow | 00:41 | |
| *** somlo has quit IRC | 01:25 | |
| *** somlo has joined #symbiflow | 01:28 | |
| *** kraiskil has quit IRC | 01:41 | |
| *** Bertl_oO is now known as Bertl_zZ | 02:26 | |
| *** OmniMancer has joined #symbiflow | 03:37 | |
| *** citypw has joined #symbiflow | 03:38 | |
| *** sorear has quit IRC | 04:39 | |
| *** litghost has quit IRC | 04:40 | |
| *** ovf has quit IRC | 04:40 | |
| *** litghost has joined #symbiflow | 04:40 | |
| *** sorear has joined #symbiflow | 04:42 | |
| *** ovf has joined #symbiflow | 04:44 | |
| *** _whitelogger has quit IRC | 05:22 | |
| *** _whitelogger has joined #symbiflow | 05:25 | |
| *** tmichalak has quit IRC | 05:41 | |
| *** tmichalak has joined #symbiflow | 06:55 | |
| *** Sarthak has joined #symbiflow | 07:40 | |
| *** Sarthak has quit IRC | 07:41 | |
| *** Sarthak has joined #symbiflow | 07:58 | |
| *** kraiskil has joined #symbiflow | 08:32 | |
| *** Sarthak has quit IRC | 08:35 | |
| sf-slack1 | <acomodi> upstream VTR merge update: murax is working on HW after merge (without round robin option) | 09:12 |
|---|---|---|
| *** citypw has quit IRC | 09:13 | |
| sf-slack1 | <mkurc> @acomodi I've stumbled on a similar issue as you did with routing the PicoSoC. The routing was taking a huge number of iterations and the number of overused connections were varying between 1 and 5. Don't remember which commit it was though... | 09:18 |
| sf-slack1 | <acomodi> @mkurc: I think it could be after the first merge done by @litghost. I will investigate, even though, given that Round Robin is the issue, it will not be pushed upstream as it was a workaround before the tile split. | 09:23 |
| sf-slack1 | <tmichalak> Hi everyone, a small update on the effort to remove the use of design.bit (https://github.com/SymbiFlow/symbiflow-arch-defs/issues/418). With the database that has the HCLK_CMT and CLK_HROW bits and the design.json generated with the small change that adds the bits outside of ROI to the required_features, I was able to generate a working counter bitstream from the frames with xc7frames2bit. So no harness bitstream | 09:27 |
| sf-slack1 | is needed anymore. | 09:27 |
| tpb | Title: Remove use of design.bit (ROI bitstream harness) · Issue #418 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 09:27 |
| sf-slack1 | <acomodi> That's great! | 09:29 |
| *** Bertl_zZ is now known as Bertl | 09:38 | |
| *** kraiskil has quit IRC | 10:25 | |
| *** kraiskil has joined #symbiflow | 10:46 | |
| *** kraiskil has quit IRC | 11:10 | |
| *** kc8apf_ has joined #symbiflow | 12:14 | |
| *** ants` has joined #symbiflow | 12:20 | |
| *** kc8apf has quit IRC | 12:22 | |
| *** galv[m] has quit IRC | 12:22 | |
| *** nats` has quit IRC | 12:22 | |
| *** kc8apf_ is now known as kc8apf | 12:22 | |
| *** galv[m] has joined #symbiflow | 12:29 | |
| sf-slack1 | <tmichalak> In parallel to adding the BRAM36 support to VTR I am looking at the stabilization of the CI fuzzer runs. Since it was mostly Vivado that failed I modifed the run_fuzzer.py script to be more verbose when a fuzzer fails and print out last 50 lines of the vivado.log. The build seems to be more stable, but not stable enough. | 13:25 |
| sf-slack1 | <tmichalak> The sad thing is that https://github.com/SymbiFlow/prjxray/issues/567 reappeared | 13:25 |
| tpb | Title: INT FAN_ALT[0-9].GFAN0 solutions are still unstable · Issue #567 · SymbiFlow/prjxray · GitHub (at github.com) | 13:26 |
| sf-slack1 | <tmichalak> @acomodi: you worked on a fix for this right? Can we improve your fix somehow? | 13:27 |
| sf-slack1 | <tmichalak> @mithro: btw, since I already mentioned the fuzzer stabilization, if I wanted to work on https://github.com/SymbiFlow/prjxray/issues/713 I would need some of my questions that I asked in the ticket answered | 13:30 |
| tpb | Title: Test output on CI cleanup needed · Issue #713 · SymbiFlow/prjxray · GitHub (at github.com) | 13:30 |
| sf-slack1 | <acomodi> @tmichalak: well, there is one option that could increase the success probabilities and it would be to increase the number of specimens, but as always it does not assure a 100% success. | 13:33 |
| sf-slack1 | <tmichalak> @acomodi: well, increase the number of specimen is not really the solution I would like to go with | 13:36 |
| sf-slack1 | <acomodi> @tmichalak: another thing that could be done is to have a fuzzer to get GFAN only related bits, and than FAN_ALT, that will depend on the GFAN fuzzer will not add the bits related to GFAN | 13:38 |
| sf-slack1 | <tmichalak> I will rerun the CI with the small stabilization changes I made a couple of times and see what is the success rate | 13:39 |
| sf-slack1 | <acomodi> I think that right now GFAN related bits are computed in the `056-rem` fuzzer, if I'm not wrong | 13:40 |
| sf-slack1 | <tmichalak> @acomodi: this sounds much better, I guess we might have to do this eventually | 13:42 |
| sf-slack1 | <tmichalak> @acomodi: you are right about the `056-rem` fuzzer and GFAN bits | 13:43 |
| sf-slack1 | <acomodi> Maybe it is worth extracting the GFAN in another fuzzer and add it to the dependencies of the FAN_ALT one | 13:47 |
| *** lethalbit has quit IRC | 13:58 | |
| *** lethalbit has joined #symbiflow | 13:58 | |
| litghost | tmichalak: I think a dedicated GFAN fuzzer is a good idea. Increases sample count has not stabilized it. One advantage of using a dedicated fuzzer is you can limit the solution size to 2 bits (e.g. "-c 2"), which I believe all the GFAN bits fall within. This is not generally true for all INT pips, but might | 15:13 |
| litghost | be true for the GFAN bits. | 15:14 |
| *** Bertl is now known as Bertl_oO | 15:18 | |
| sf-slack1 | <mkurc> Good time of day. I've created a test design that can be scaled in size for testing of VPR operation. https://github.com/SymbiFlow/symbiflow-arch-defs/pull/490 | 15:20 |
| tpb | Title: Scalable test design by mkurc-ant · Pull Request #490 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 15:20 |
| sf-slack1 | <mkurc> I wrote initial conclusions of testing in the PR's comment. | 15:21 |
| sf-slack1 | <acomodi> Hi everyone. Here there is a PR that changes the cmake build system to allow the creation of different devices/boards/parts for the same architecture: https://github.com/SymbiFlow/symbiflow-arch-defs/pull/462 | 15:26 |
| tpb | Title: WIP: Add arty support by acomodi · Pull Request #462 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 15:26 |
| *** ants` is now known as nats` | 15:30 | |
| sf-slack1 | <acomodi> Here I have started a doc used to think about the major VPR patches that have to be isolated and upstreamed: https://docs.google.com/document/d/17Rp4SEG0uSLG4gWUSPTTjb7We7-MRHtI0L4Mwb9YDPQ/edit?usp=sharing | 15:36 |
| tpb | Title: Google Docs - create and edit documents online, for free. (at docs.google.com) | 15:36 |
| *** futarisIRCcloud has quit IRC | 15:37 | |
| litghost | mkurc: Thanks, it's good to have something we can test with. It makes a good regression test too. I've found another case that breaks too. I suspect this may be caused by the MUX vs SHORT issue on the channel merges. I'll test if that helps in my case | 15:40 |
| sf-slack1 | <mkurc> I used the design also to test on the split grid based on SLICEs and observed that it fails for less complex designs than for the CLB based grid. | 15:42 |
| *** kraiskil has joined #symbiflow | 15:58 | |
| litghost | Huh | 15:59 |
| sf-slack1 | <acomodi> litghost, mithro: I have merged https://github.com/SymbiFlow/symbiflow-arch-defs/pull/462. Arty doesn't currently compile as a new harness is needed and placed in this directory `third_party/prjxray-db/artix7/harness/arty-a7/uart/` | 16:07 |
| tpb | Title: Sign in to GitHub · GitHub (at github.com) | 16:07 |
| litghost | acomodi: I assume prjxray can generate the arty harness right now? | 16:08 |
| sf-slack1 | <acomodi> yes indeed | 16:08 |
| litghost | acomodi: Because you've tested the ROI, can you make a PR on https://github.com/SymbiFlow/prjxray-db with the harness? | 16:09 |
| tpb | Title: GitHub - SymbiFlow/prjxray-db: Project X-Ray Database: XC7 Series (at github.com) | 16:09 |
| sf-slack1 | <acomodi> litghost: all right | 16:10 |
| sf-slack1 | <acomodi> litghost, mithro: done, PR opened to add working arty harness (UART and one LED are correctly working) | 16:30 |
| litghost | acomodi: Where are we with the testing with https://github.com/SymbiFlow/vtr-verilog-to-routing/pull/30 ? I believe you said the router was getting stuck? Still true? | 16:32 |
| tpb | Title: VTR: upstream merge by acomodi · Pull Request #30 · SymbiFlow/vtr-verilog-to-routing · GitHub (at github.com) | 16:32 |
| sf-slack1 | <acomodi> litghost: yes, it stucks only with round robin option enabled | 16:33 |
| sf-slack1 | <acomodi> I am still investigating why this happens, but I couldn't find the source of the issue yet | 16:33 |
| *** dhiraj24 has joined #symbiflow | 16:53 | |
| dhiraj24 | Hello | 16:53 |
| litghost | Hi | 16:54 |
| dhiraj24 | I am interested in https://github.com/SymbiFlow/ideas/issues/11 project for gsoc. I am Third year undergraduate student doing Bachelor of engineering in Electronics and Telecommunication in India and have good grasp over python.I have been contributing to opensource a lot.Please see my github profile https://github.com/Dhiraj240 and also in summers 2018 i had participated in an opensource contest where i built this project https | 16:57 |
| tpb | Title: Add a Verilog / SystemVerilog support to Sphinx · Issue #11 · SymbiFlow/ideas · GitHub (at github.com) | 16:57 |
| dhiraj24 | The link for sphinix document is not opening and please provide me a little advise over the above project.I had seen the approach given in the issue but basically i am confused as which of your repos will be used by doxygen-verilog . | 16:59 |
| litghost | symbiflow-arch-defs I believe is teh one we are targetting. mithro should pop in later and can answer more questions | 17:00 |
| litghost | https://github.com/SymbiFlow/symbiflow-arch-defs | 17:00 |
| tpb | Title: GitHub - SymbiFlow/symbiflow-arch-defs: FOSS architecture definitions of FPGA hardware useful for doing PnR device generation. (at github.com) | 17:00 |
| dhiraj24 | Alright so doxygen-verilog will use symbiflow-arch-defs to generate Doxygen output from the Verilog and i have to use either breathe or exhale to read it and output it to sphinx. | 17:04 |
| dhiraj24 | Am i right ? | 17:04 |
| litghost | That sounds correct. What we are intending for the architecture documentation process is to write some verilog, generate a schematic diagram in SVG from it, and then build sphnix documentation including both | 17:05 |
| litghost | mithro: Want to add to that explaination? | 17:05 |
| dhiraj24 | write some verilog ? can i get an example ? is verilog code not present in https://github.com/SymbiFlow/symbiflow-arch-defs | 17:08 |
| tpb | Title: GitHub - SymbiFlow/symbiflow-arch-defs: FOSS architecture definitions of FPGA hardware useful for doing PnR device generation. (at github.com) | 17:08 |
| *** kraiskil has quit IRC | 17:13 | |
| dhiraj24 | Asking again sorry for it. [22:30] <dhiraj24> write some verilog ? can i get an example ? is verilog code not present in https://github.com/SymbiFlow/symbiflow-arch-defs | 17:16 |
| tpb | Title: GitHub - SymbiFlow/symbiflow-arch-defs: FOSS architecture definitions of FPGA hardware useful for doing PnR device generation. (at github.com) | 17:16 |
| litghost | Please wait for @mithro to answer your question | 17:16 |
| dhiraj24 | alright, i will wait meanwhile should i add comments on the issue tracker for this project on github | 17:19 |
| litghost | Feel free | 17:19 |
| dhiraj24 | okay thanks, i will wait. | 17:19 |
| *** kraiskil has joined #symbiflow | 17:29 | |
| dhiraj24 | i guess he is not online | 17:41 |
| litghost | mkurc: I've filed https://github.com/SymbiFlow/symbiflow-arch-defs/issues/491 to track what is going with the designs failing on hardware | 17:48 |
| tpb | Title: [XC7] Some designs fail on hardware · Issue #491 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 17:48 |
| *** kraiskil has quit IRC | 17:56 | |
| *** OmniMancer has quit IRC | 18:02 | |
| dhiraj24 | @litghost can i tell me do i need to solve some issues in order to get selected for GSOC or should i directly work on proposal | 18:18 |
| mithro | dhiraj24: Hi! I think for GSoC adding Verilog support to Sphinx would be a better project | 18:31 |
| dhiraj24 | hello sir, alright but need some clarification | 18:32 |
| dhiraj24 | i had commented on https://github.com/SymbiFlow/ideas/issues/11 | 18:33 |
| tpb | Title: Add a Verilog / SystemVerilog support to Sphinx · Issue #11 · SymbiFlow/ideas · GitHub (at github.com) | 18:33 |
| *** acomodi has quit IRC | 18:37 | |
| mithro | dhiraj24: I've yet to get through my email this morning, so probably haven't seen your question | 18:46 |
| dhiraj24 | alright sir, in India there is 12:30 am and tomorrow is my college. can i know as when you will guide me for that ? | 18:49 |
| mithro | dhiraj24: I have replied to your comment | 18:55 |
| dhiraj24 | Okay sir, i had commented again there please check. | 19:06 |
| dhiraj24 | Also sir, should i directly start with my proposal and show the implementation of it ? or before that any contribution is necessary ? Meanwhile please check the updated comment on https://github.com/SymbiFlow/ideas/issues/11 | 19:19 |
| tpb | Title: Add a Verilog / SystemVerilog support to Sphinx · Issue #11 · SymbiFlow/ideas · GitHub (at github.com) | 19:19 |
| litghost | acomodi/mkurc: https://github.com/SymbiFlow/symbiflow-arch-defs/issues/491#issuecomment-475754051 Looks like round robin packing might be causing other issues | 19:45 |
| tpb | Title: [XC7] Some designs fail on hardware · Issue #491 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 19:45 |
| sf-slack1 | <acomodi> litghost: well, given that round robin is not going to be needed anymore with the clb tile being splitted we could disable it for good, right? | 19:55 |
| litghost | acomodi: Sure. Hopefully if mkurc tests with round robin disabled the tile split starts working as expect, then that is the path forward + equivilent tils | 19:56 |
| litghost | es | 19:56 |
| sf-slack1 | <acomodi> litghost: my question though is: why would the SLICEM be set to DRAMs mode instead of LUTs? | 19:57 |
| litghost | acomodi: Because the round robin packer is doing what it is supposed to, alturnate modes on packing | 19:57 |
| *** dhiraj24 has quit IRC | 20:02 | |
| sf-slack1 | <acomodi> litghost: All right. What is still needed to merge and start using @mkurc work on splitting tiles? (Provided that we test that there is no regression wrt the current status) | 20:02 |
| *** acomodi has joined #symbiflow | 20:21 | |
| *** kraiskil has joined #symbiflow | 21:06 | |
| litghost | acomodi: I believe there were regressions with it. See his comment on https://github.com/SymbiFlow/symbiflow-arch-defs/pull/490 | 21:51 |
| tpb | Title: Scalable test design by mkurc-ant · Pull Request #490 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 21:51 |
| litghost | acomodi: To be fair, that PR fails on both pre- and post- split | 21:51 |
| litghost | but the post-split seems worse | 21:51 |
| litghost | If round robin packing explains the failure, then maybe we are ready to merge the split. I would like the tile split to have the unified tile names (e.g. just SLICEL/SLICEM), but unclear how much more work is left for that | 21:52 |
| *** kraiskil has quit IRC | 21:52 | |
| mithro | litghost: Running a db rebuild now | 22:35 |
| litghost | mithro: Thanks | 22:35 |
| mithro | litghost: What is going on here -> | 22:40 |
| mithro | https://tinyurl.com/yy6qk49f CLK_HROW_BOT_R ? | 22:42 |
| tpb | Title: Diff to HTML by rtfpessoa (at tinyurl.com) | 22:42 |
| mithro | Looks like you renamed some stuff? | 22:42 |
| litghost | mithro: Which are you asking about? | 22:45 |
| litghost | mithro: Segbits for CLK_HROW? | 22:45 |
| mithro | litghost: Yeah | 22:46 |
| litghost | mithro: Couple things happened. Turns out the offset and word count was too small to include all the bits in the interconnect, so the offset was lowered, which resulted in all bits getting rebased | 22:49 |
| litghost | mithro: the other thing that happened is that we enabled fuzzing of the entire clk hrow interconnect switch box, which resulted in a bunch of new features | 22:51 |
| mithro | https://github.com/SymbiFlow/prjxray-db/compare/master...mithro:master | 22:52 |
| tpb | Title: Comparing SymbiFlow:master...mithro:master · SymbiFlow/prjxray-db · GitHub (at github.com) | 22:52 |
| mithro | litghost: How does that look? | 23:01 |
| litghost | mithro: Looks great | 23:02 |
| mithro | litghost: Pushed | 23:04 |
| litghost | mithro: Please merge https://github.com/SymbiFlow/prjxray-db/pull/6 | 23:05 |
| tpb | Title: arty-a7: add UART harness by acomodi · Pull Request #6 · SymbiFlow/prjxray-db · GitHub (at github.com) | 23:05 |
| litghost | mithro: acomodi has tested it, and there is now first class arty support in arch-defs | 23:05 |
| mithro | litghost: I don't quite understand this harness? | 23:15 |
| litghost | how so? | 23:15 |
| litghost | I believe it is one button, one LED and a UARt | 23:16 |
| litghost | mithro: How so? | 23:24 |
| mithro | litghost: Hold on a sec | 23:24 |
| mithro | litghost: https://gist.github.com/mithro/c41d7443dab3e6b33df17c9704c0cc57 | 23:34 |
| tpb | Title: gist:c41d7443dab3e6b33df17c9704c0cc57 · GitHub (at gist.github.com) | 23:34 |
| litghost | mithro: And? | 23:35 |
| mithro | Shouldn't we just add the UART to the ARTY-A7-SWBUT harness? Both the Zybo and Basys SWBUT harness have the UART available... -- then with that change, I'm not sure what the point of the Arty-A7-UART harness is? | 23:36 |
| litghost | mithro: https://github.com/SymbiFlow/prjxray/pull/724#issuecomment-472942015 | 23:36 |
| tpb | Title: roi_harness: Add ARTY-A7-UART configuration by acomodi · Pull Request #724 · SymbiFlow/prjxray · GitHub (at github.com) | 23:36 |
| mithro | litghost: BTW Have you landed the short switch change? | 23:39 |
| litghost | mithro: Not yet, I've been focused on a large refactor in the fasm2verilog work. Hopefully with a little more effort that will be ready to merge. | 23:40 |
| litghost | mithro: I don't believe we have evidence that the MUX vs SHORT has been causing a problem. There does appear to be an issue with the round robin packing interacting with the DRAM bits. | 23:41 |
| litghost | mithro: To be clear I do want to land the PR | 23:41 |
| mithro | litghost: without the short change vpr might end up trying to use the same wire for two signals? | 23:41 |
| litghost | mithro: But we don't have evidence that has actually happened | 23:41 |
| litghost | mithro: In fact I specifically added logic to the fasm2verilog code to look for such a problem | 23:42 |
| litghost | mithro: And did not see it | 23:42 |
| mithro | litghost: Oh, I guess the Xilinx parts really only have point-to-point wires... would be more a problem with the long wires which I don't think you are importing at the moment? | 23:53 |
| litghost | mithro: No I do believe that the MUX vs SHORT is a bug, and should be fixed. I just don't have evidence it is a problem this very minute. | 23:54 |
| *** futarisIRCcloud has joined #symbiflow | 23:58 | |
| mithro | litghost: I've committed a replacement for that pull request | 23:59 |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!