chipsalliance / fpga-tool-perf

FPGA tool performance profiling
https://chipsalliance.github.io/fpga-tool-perf
Apache License 2.0
102 stars 30 forks source link

Enable VPR toolchain for baselitex-nexys-video #262

Open wtatarski opened 4 years ago

wtatarski commented 4 years ago

Below error needs to be fixed for successful launch of baselitex-nexys-video with VPR:

  File "/fpga-tool-perf/env/conda/envs/symbiflow-env/lib/python3.8/site-packages/prjxray/fasm_assembler.py", line 190, in parse_fasm_filename
    raise FasmLookupError('\n'.join(missing_features))
prjxray.fasm_assembler.FasmLookupError: Segment DB RIOI3, key RIOI3.IOI_OCLKM_0.IOI_IMUX31_1 not found from line 'RIOI3_X105Y145.IOI_OCLKM_0.IOI_IMUX31_1'
Makefile:43: recipe for target 'top.bit' failed
litghost commented 4 years ago

It is a little suspicious that the clock for that RIOI3 is coming from the interconnect, though I do agree the bits are missing. We should likely investigate adding the bits and figuring out why the router is routing the clock via the interconnect.

Is this a case where an inverter needs to be absorbed into the OLOGIC tile, but hasn't been? So the routing looks like CLK -> LUT1/INV -> OLOGIC, instead of CLK -> OLOGIC?

acomodi commented 4 years ago

Is this a case where an inverter needs to be absorbed into the OLOGIC tile

Looking at the design, there is no OSERDES that has an inverted clock, but we need to further investigate this issue.

mithro commented 4 years ago

What does https://github.com/SymbiFlow/fpga-tool-perf/pull/257 do?

acomodi commented 4 years ago

@mithro it adds the test design, for now built with vivado and yosys-vivado, until we find a solution to this issue.

HackerFoo commented 4 years ago

Could I get a statement whether we can currently generate a working bitstream using SymbiFlow & VPR for baselitex on nexys-video? If so, is there a working test somewhere? Thanks.

wtatarski commented 4 years ago

@HackerFoo At this moment generating bitstream using VPR is not possible due to issue which I mentioned in this issue.