| *** tpb has joined #symbiflow | 00:00 | |
| *** proteusguy has quit IRC | 02:50 | |
| *** citypw has joined #symbiflow | 03:21 | |
| *** asd123_ has joined #symbiflow | 05:19 | |
| *** asd123_ has quit IRC | 05:28 | |
| *** xvilka has joined #symbiflow | 05:37 | |
| *** OmniMancer has joined #symbiflow | 06:08 | |
| *** citypw has quit IRC | 10:13 | |
| sf-slack | <acomodi> So, regarding the BRKH_INT issue, I think that there should be a fuzzer as there are some pips included in them | 11:27 |
|---|---|---|
| *** proteusguy has joined #symbiflow | 11:37 | |
| sf-slack | <kgugala> From what I see there are 14 PIPs there and they are used to route the signals between clock regions | 11:56 |
| sf-slack | <kgugala> so if those are used in our case we're running outside the ROI | 11:57 |
| *** asdf1234 has joined #symbiflow | 12:15 | |
| *** asdf1234 has left #symbiflow | 12:21 | |
| *** apurvanandan has joined #symbiflow | 12:22 | |
| apurvanandan | I would like to know that can I contribute to your orgnisation in GSoC, if I have experience with Verilog/VHDL but little experience of working of EDA tools? | 12:24 |
| sf-slack | <mgielda> Hi! Great! Please take a look at the ideas list and look for topics that fit your skill set and interests. Doing what you like is key to a successful application | 12:44 |
| sf-slack | <mgielda> You actually just need to know how to program and understand the concepts to realise many of them, but we don't expect people to have lots of prior experience with exactly this | 12:50 |
| sf-slack | <mgielda> https://github.com/SymbiFlow/ideas/issues the ideas list | 12:50 |
| tpb | Title: Issues · SymbiFlow/ideas · GitHub (at github.com) | 12:50 |
| sf-slack | <acomodi> litghost, @kgugala: I have checked the routing graph with the graphics tool. There are quite a lot wires going out of the ROI for what regards picosoc. Therefore, the BRKH issue popped up | 14:36 |
| *** OmniMancer has quit IRC | 14:45 | |
| *** kraiskil has joined #symbiflow | 14:55 | |
| *** citypw has joined #symbiflow | 14:56 | |
| *** apurvanandan has quit IRC | 15:37 | |
| *** kraiskil has quit IRC | 15:43 | |
| litghost | acomodi: Two options. One is to the explicitly remove all pips out side of the ROI | 15:47 |
| litghost | acomodi: Other is to check whether the BRKH pips are real or not | 15:47 |
| litghost | via a minitest or fuzzer | 15:47 |
| litghost | If the pips are real, then we need a fuzzer. If the pips are ppips, then add BRKH to the ppips fuzzer | 15:49 |
| litghost | FYI, the ppips fuzzer is for pips that are either always one or implicit. In generally we've seen most pips that are inline line the pips in the BRKH are ppips not real ones | 15:50 |
| sf-slack | <acomodi> @litghost: I think that the first one is the fastest and can be used for the short term. The second has to be done for sure because it will be needed when we go out of the ROI | 15:50 |
| * litghost acomodi: https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/utils/prjxray_routing_import.py | 15:50 | |
| sf-slack | <acomodi> Yeah, I agree, I have seen those BRKH PIPs and they should be ppips | 15:51 |
| sf-slack | <mkurc> Good time of day. I've been thinking about the mithro's concept about adding a `<tile_type>` instead of a top-level `<pb_type>` to the VPR. I've been through the VPR code and narrowed down the problem down to making the clustering step operate at second level pb_types (childs of the root pb_type). Can anybody provide me with some insights is it the right way to do it? | 16:14 |
| sf-slack | <kgugala> this will be hard to do | 16:14 |
| sf-slack | <kgugala> if you look at the way the packer searches for the root block you'll see it does that recursively | 16:15 |
| sf-slack | <kgugala> so you never know how many steps are left | 16:16 |
| sf-slack | <kgugala> moreover it always compares the root type of the cluster with a root type of a chain root pin | 16:16 |
| sf-slack | <kgugala> and because of the pin you'll never know which slice you're considering | 16:17 |
| litghost | mkurc: The proposal I heard the other day was to leave the clusterer alone | 16:17 |
| litghost | mkurc: And change the placer to understand that a tile can map to one or more pb_types | 16:17 |
| sf-slack | <kgugala> @litghost: I agree | 16:18 |
| sf-slack | <kgugala> this is the way to go | 16:18 |
| litghost | mkurc: So the slice_m pb_type is a super set of the slice_l pb_type, so the placer would be able to take a slice_l cluster and put it on a slice_m tile | 16:18 |
| litghost | mkurc: This is assuming we've done the tile split | 16:18 |
| litghost | mkurc: I also chatter with the VPR devs, and they indicated your idea of splitting the tile at the grid level and leaving the routing graph (e.g. channel definitions) would work | 16:19 |
| sf-slack | <mkurc> Ok then, I'll look into the placer starting from tomorrow. | 16:32 |
| litghost | mkurc: Before that, we probably need to split the tiles? | 16:39 |
| litghost | mkurc: That way once the placer is updated, we have a good test case? It would also be find to construct a test architecture to test the feature | 16:39 |
| litghost | mkurc: Split the CLB tiles into two slices | 16:40 |
| *** kraiskil has joined #symbiflow | 16:41 | |
| sf-slack | <acomodi> litghost: would it be ok just to reduce the Y_MAX from the ROI from 52 (which includes the BRKH tiles) to 51? | 16:58 |
| *** citypw has quit IRC | 17:00 | |
| litghost | acomodi: Should be fine. | 17:04 |
| *** celadon has quit IRC | 17:12 | |
| elms | litghost: have you had the case that you needed to emit a feature if for a counter case? eg emit feature IO.Disable if that IO pad is not used | 18:19 |
| litghost | elms: To date, no. That is something that likely need to be handled in a post VPR fixup phase | 18:20 |
| litghost | elms: If we add active IO FASM annotations, it should be a straight forward fixup | 18:20 |
| litghost | elms: I'd recommend handling it in the FASM to asc phase | 18:21 |
| litghost | mkurc: Regression in murax was due to enabling RAM32X1D, https://github.com/SymbiFlow/symbiflow-arch-defs/issues/409 | 18:22 |
| tpb | Title: RAM32X1D not working · Issue #409 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 18:22 |
| litghost | mkurc: I'm gonna disable yosys RAM32X1D synthesis for now. I never tested RAM32X1D, so it is not super uprising there was a bug. | 18:23 |
| litghost | *suprising | 18:23 |
| sf-slack | <acomodi> litghost: murax is built til the end and does not work on HW or it does not reach the end of the compilation? | 18:25 |
| sf-slack | <acomodi> Because I had tested all the dram_tests and there were two of them that did present a weird (probably wrong) behavior on HW | 18:26 |
| litghost | acomodi: Fails on hardware. Like I said, I never tested RAM32X1D. I only tested RAM64X1D. | 18:26 |
| litghost | acomodi: This wasn't a problem before, because yosys would never emit a RAM32X1D until https://github.com/SymbiFlow/yosys/pull/8 | 18:28 |
| tpb | Title: Moved techmaps from Yosys to SymbiFlow by mkurc-ant · Pull Request #8 · SymbiFlow/yosys · GitHub (at github.com) | 18:28 |
| sf-slack | <acomodi> Ok, got it, I had previously seen that 32X1D was not working and there is also another configuration which is not working on HW. It is reported in this PR https://github.com/SymbiFlow/symbiflow-arch-defs/pull/402 | 18:30 |
| tpb | Title: d_dram.pb_type: update xml definition to solve slicem issue by acomodi · Pull Request #402 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 18:30 |
| litghost | acomodi: The other mode you reported not working is 32X1 DRAM mode. So I suspect there is some bug in either the FASM or pb_type for the 32 wide DRAMs. There is actually never a reason to use a RAM32X1D over a RAM64X1D, as the consume the same number of resources | 18:32 |
| litghost | acomodi: 32X2S or 32X2D is a useful mode, so we should eventually debug it, but I don't think it's a priority. | 18:33 |
| sf-slack | <acomodi> litghost: right, I guess for now it's ok just to disable it from Yosys as you said. We need to file a ticket and get back to it later on | 18:35 |
| litghost | acomodi: Already filed: https://github.com/SymbiFlow/symbiflow-arch-defs/issues/409 | 18:36 |
| tpb | Title: RAM32X1D not working · Issue #409 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 18:36 |
| litghost | and PR is https://github.com/SymbiFlow/yosys/pull/11, just waiting for green from CI | 18:36 |
| sf-slack | <acomodi> Great, BTW the picosoc run I have launched earlier is about to finish, probably it could present the same issue with the DRAM32x1d, but other than that it should produce a bit file | 18:39 |
| sf-slack | <acomodi> If that happens I'll test it on HW tomorrow | 18:40 |
| litghost | acomodi: It will almost certainly fail, unless there are no RAM32X1D, which is possible | 18:40 |
| sf-slack | <acomodi> Yep, tomorrow I'll test again. Anyways I got a bitstream for picosoc finally, just finished | 18:45 |
| litghost | Yay! | 18:50 |
| elms | litghost: when we want to add more complex IO modes for ice40, there will be other complicated cases. As of now the other IO are just driven to constant 0 output. Are you also thinking a post stage is also when hard IP blocks would be handled? | 18:56 |
| litghost | elms: Are you implying that unused ice40 IO drive output? That doesn't make sense. They should be tristated? | 18:58 |
| elms | Is no bits in the IO tile are set I think so. | 18:58 |
| litghost | elms: Not sure I understand. If no bits are set in the ice40 bitstream, are the outputs tristated? | 18:59 |
| elms | I haven't confirmed, but it does seem like the default high-z there should be some bits set | 18:59 |
| litghost | elms: So part of the FASM specification is the idea of a "default" bitstream, which is the set of bits if no features are enabled. I believe tristating outputs should be part of that "default" set. If a feature wants to enable an IO, it would clear the high-z bit. | 19:00 |
| litghost | elms: Can that work? | 19:00 |
| elms | I think it would require diverging from icestorm nomenclature which I want to be as close to as possible. | 19:03 |
| elms | litghost: Are you saying the feature would be affirmative as in IO.Enable which would clear a bit, eg "!1_20" or something? | 19:04 |
| litghost | elms: Yes. | 19:06 |
| litghost | elms: IMO, an empty FASM file should generate a bitstream that does "nothing", e.g. is safe to load on any design. | 19:07 |
| litghost | elms: Having an empty FASM file drive outputs on every output pin of an ice40 is a recipe for disaster | 19:08 |
| elms | The advanced cases are also going to cause some issues. Features are mutually exclusive right? Also I haven't found the specifics, but apparently there are times that IO bits are in other IO tiles | 19:08 |
| litghost | elms: Example? Features do not have to be mutually exclusive, but they cannot request a bit be cleared and set | 19:09 |
| elms | litghost: agreed. But I think there is a sensible way to handle that in the fasm to bitstream | 19:09 |
| litghost | elms: This is ignoring bits set in the "default" bitstream | 19:09 |
| litghost | elms: I am also saying handle it in the fasm to bitstream layer. I'm saying logically handling it in the fasm to bitstream layer is equivilant to what I am describing above, e.g. setting a feature will clear a bit. | 19:12 |
| elms | litghost: re mutual exclusive, two features can't be set that set the state of a specific bit. Can both of these be enabled? | 19:15 |
| elms | feature1 0_0 0_2 | 19:15 |
| elms | feature2 0_0 0_3 | 19:15 |
| litghost | elms: That's perfect fine. Setting a bit in multiple features is legal. Disagreement about the bit is an error. | 19:15 |
| elms | ok | 19:16 |
| elms | Do you have an existing example of a default bitstream for existing fasm? | 19:17 |
| litghost | elms: No, because to date 7-series a zero bitstream is acceptable. However it shouldn't be too hard. Just have a set of default bits, that can be cleared by the FASM bits. | 19:18 |
| litghost | elms: Because compile the FASM feature to bits, check for conflicts within the FASM bits, and then have the FASM bits override the default bits | 19:19 |
| litghost | elms: You should be able to declare the "default" bitstream using FASM features (e.g. tristate), but you'll need to book keep the default bits from the FASM bits to not raise a conflict | 19:20 |
| litghost | elms: Should be straight forward | 19:20 |
| elms | litghost: I would like to keep the features using icestorm nomenclature if possible. That said the default stream would have some bits set which would correspond to features being set. It doesn't seem reconcilable. So maybe I should have a method to denote the contra case in a somewhat standard way. | 19:21 |
| litghost | elms: By contra case, you mean a feature that is "not tristate"? | 19:22 |
| elms | litghost: Hmm so I can keep the same features but add some negated features as well. | 19:22 |
| litghost | elms: Of course | 19:23 |
| litghost | elms: We do that with zero bit groups in prjxray | 19:23 |
| elms | litghost: can you point me to that? | 19:23 |
| litghost | elms: To be clear, negated features are no-ops. But negated bits are doable | 19:23 |
| litghost | elms: Do you want an example of the zero bit group, or zero bit features? | 19:24 |
| litghost | https://github.com/SymbiFlow/prjxray-db/blob/master/artix7/segbits_bram_l.db#L3 is a zero bit features (e.g. this feature requires those 3 bits to be cleared) | 19:24 |
| tpb | Title: prjxray-db/segbits_bram_l.db at master · SymbiFlow/prjxray-db · GitHub (at github.com) | 19:24 |
| elms | I guess I was thinking "enable 0_0" "disable !0_0" | 19:24 |
| litghost | elms: That is fine in FASM parlance. As long as the input FASM file doesn't specify both features | 19:25 |
| elms | litghost: ok. I'll do that and see if that gives a safe empty fasm bitstream. | 19:27 |
| elms | litghost: I've also been back and forth about where the fasm2asc code should live. If I follow prjxray it should live in icestorm, but depending on the fasm python package may require some more justification. The other option is to have it live in arch-defs and we still generate an asc file for icestorm. | 19:28 |
| litghost | elms: I think having it in arch-defs is fine for now. icestorm itself doesn't use it. prjxray actually uses the FASM stuff as part of ROI generation. | 19:29 |
| elms | litghost: sounds good. that's what I was leaning toward. | 19:30 |
| *** abhyuday has joined #symbiflow | 20:28 | |
| *** abhyuday has quit IRC | 20:31 | |
| elms | litghost: I thought I needed to wait to merge https://github.com/SymbiFlow/symbiflow-arch-defs/pull/407 until I saw a new vpr https://anaconda.org/SymbiFlow/vtr/files . Is that not where the new build would show up? | 20:37 |
| tpb | Title: Remove tile level fasm_prefix by elmsfu · Pull Request #407 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 20:37 |
| litghost | elms: I tested murax with https://github.com/SymbiFlow/symbiflow-arch-defs/pull/407 and it was fine | 20:41 |
| tpb | Title: Remove tile level fasm_prefix by elmsfu · Pull Request #407 · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 20:42 |
| litghost | elms: So I went ahead and merged it | 20:42 |
| litghost | elms: Actually I guess I do have the latest VPR :/ | 20:42 |
| elms | litghost: was that a local build of vpr? I'm not sure the anaconda build of vpr has | 20:42 |
| *** kraiskil has quit IRC | 20:42 | |
| litghost | elms: Ya, local | 20:42 |
| elms | I just looked and saw there is a bump file in the anaconda repo | 20:43 |
| elms | what's up with that https://github.com/SymbiFlow/conda-packages/blob/master/.bump | 20:44 |
| tpb | Title: conda-packages/.bump at master · SymbiFlow/conda-packages · GitHub (at github.com) | 20:44 |
| litghost | elms: Writing to the .bump file causes travis to rebuild and deploy the packages | 20:44 |
| elms | hmm ok | 20:45 |
| elms | litghost: not sure why PR 407 didn't generate a travis build for arch-defs. That should have failed. | 20:46 |
| litghost | Good question, it was green when I merged | 20:46 |
| litghost | Ah, no 7-series stuff is being tested on travis, only on kokoro | 20:46 |
| litghost | And I don't think the kokoro CI is wired in? | 20:46 |
| litghost | Unclear | 20:46 |
| elms | litghost: another fasm/ice40 corner case. a parameter is integer type, but only 2 bits are used. Should I add ability to map eblif param bit range in fasm_params ? or is there a better solution? | 20:51 |
| litghost | elms: Bit ranges are supports? Just specify the slice you are interested in? | 20:51 |
| elms | on the eblif side? | 20:51 |
| elms | eg fasm_INIT[1:0] = eblif_init[3:2] | 20:52 |
| litghost | elms: In general if parameter munging is required, I've been doing it in synthesis as a techmap | 20:52 |
| elms | litghost: example? | 20:52 |
| litghost | elms: Yosys is fairly powerful, and I'd rather not put too much into genfasm if we can avoid it | 20:52 |
| litghost | inversion - > https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/techmap/cells_map.v#L6 | 20:53 |
| tpb | Title: symbiflow-arch-defs/cells_map.v at master · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 20:53 |
| litghost | string to bit -> https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/techmap/cells_map.v#L891 | 20:53 |
| tpb | Title: symbiflow-arch-defs/cells_map.v at master · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 20:53 |
| litghost | enum mapping -> https://github.com/SymbiFlow/symbiflow-arch-defs/blob/master/xc7/techmap/cells_map.v#L880 | 20:53 |
| tpb | Title: symbiflow-arch-defs/cells_map.v at master · SymbiFlow/symbiflow-arch-defs · GitHub (at github.com) | 20:53 |
| litghost | elms: In this case I'm open to expanding the genfasm ability, but I think I prefer to keep as much of the munging in a yosys techmap rather than code in VPR | 20:54 |
| elms | litghost: exactly why I'm asking :) | 20:55 |
| litghost | elms: If you don't mind doing it via techmap, I think that is the cleaner solution | 20:57 |
| litghost | elms: I think Yosys can do any parameter munging we could need, where-as genfasm would need to eventually gain more and more capabilities if we added eblif side manipulation | 21:00 |
| elms | sounds good. I'd like to be consistent across archs as well. There are going to be a lot of permutations, test coverage for all this will be interesting (thinks of future self) | 21:02 |
| litghost | elms: What do you mean by "lots of permutations"? I lack context for this issue | 21:03 |
| elms | litghost: maybe permutations isn't right, but there are a lot of modules to make sure are behaving correctly eventually on hardware | 21:06 |
| litghost | elms: Sure. I hope there are test fixtures for various tiles. I've been building that up for 7-series as I've gone along | 21:07 |
| litghost | elms: E.g a FF test, a LUT test, a DRAM test, BRAM tests, etc | 21:08 |
| elms | litghost: there are some tests, but not great coverage for all the combinations of latching and inverted clocks, etc. | 21:27 |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!