Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Symbiflow not working for Xilinx #808

Closed
navan93 opened this issue Nov 23, 2023 · 6 comments · Fixed by #809
Closed

Symbiflow not working for Xilinx #808

navan93 opened this issue Nov 23, 2023 · 6 comments · Fixed by #809

Comments

@navan93
Copy link

navan93 commented Nov 23, 2023

Similar to the issue #722 I am seeing that synthesis is failing

Project status:
    [S] bitstream:  bitstream -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.bit
    [N] build_dir:  /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware
    [S] eblif:  synth -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.eblif
    [S] fasm:  fasm -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.fasm
    [S] fasm_extra:  synth -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty_fasm_extra.fasm
    [S] io_place:  ioplace -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.ioplace
    [S] net:  pack -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.net
    [X] pcf:  MISSING
    [S] place:  place -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.place
    [S] place_constraints:  place_constraints -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.preplace
    [S] route:  route -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.route
    [S] sdc:  synth -> /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.sdc
    [N] sources:  ['/home/navaneeth/Projects/openmpw/CFU-Playground/proj/proj_template/cfu.v', '/home/navaneeth/Projects/openmpw/CFU-Playground/third_party/python/pythondata_cpu_vexriscv/pythondata_cpu_vexriscv/verilog/VexRiscv_FullCfu.v', '/home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.v']
    [N] xdc:  /home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.xdc

Executing module `synthesize`:
    [1/3] : Synthesizing sources: ['/home/navaneeth/Projects/openmpw/CFU-Playground/proj/proj_template/cfu.v', '/home/navaneeth/Projects/openmpw/CFU-Playground/third_party/python/pythondata_cpu_vexriscv/pythondata_cpu_vexriscv/verilog/VexRiscv_FullCfu.v', '/home/navaneeth/Projects/openmpw/CFU-Playground/soc/build/digilent_arty.proj_template/gateware/digilent_arty.v']...
[ERROR]: yosys non-zero return code.
stderr:
ERROR: TCL interpreter returned an error: Yosys command produced an error



make[5]: *** [/home/navaneeth/Projects/openmpw/CFU-Playground/soc/common_soc.mk:115: build/digilent_arty.proj_template/gateware/digilent_arty.bit] Error 1
make[5]: Leaving directory '/home/navaneeth/Projects/openmpw/CFU-Playground/soc'
make[4]: *** [../proj.mk:319: prog] Error 2
make[4]: Leaving directory '/home/navaneeth/Projects/openmpw/CFU-Playground/proj/proj_template'

In digilent_arty_synth.log

-- Running command `tcl /home/navaneeth/Projects/openmpw/CFU-Playground/env/symbiflow/xc7/install/share/f4pga/scripts/xc7/synth.tcl' --
[TCL: yosys -import] Command name collision: found pre-existing command `cd' -> skip.
[TCL: yosys -import] Command name collision: found pre-existing command `eval' -> skip.
[TCL: yosys -import] Command name collision: found pre-existing command `exec' -> skip.
[TCL: yosys -import] Command name collision: found pre-existing command `read' -> skip.
[TCL: yosys -import] Command name collision: found pre-existing command `trace' -> skip.
ERROR: Can't load module `./xdc': /home/navaneeth/Projects/openmpw/CFU-Playground/env/conda/envs/cfu-symbiflow/bin/../share/yosys/plugins/xdc.so: undefined symbol: _ZN5Yosys7stringfB5cxx11EPKcz
ERROR: TCL interpreter returned an error: Yosys command produced an error

Commands I have used are

make install-sf
make enter-sf
make prog TARGET=digilent_arty USE_SYMBIFLOW=1
@tcal-x
Copy link
Collaborator

tcal-x commented Nov 29, 2023

Thanks for the report @navan93 ! I haven't been able to reproduce it yet because I hit a different issue trying to do a fresh Symbiflow installation.

In case it matters, what OS are you using?

@navan93
Copy link
Author

navan93 commented Nov 29, 2023

I am using Ubuntu 22.04 desktop.

@tcal-x
Copy link
Collaborator

tcal-x commented Dec 4, 2023

Hi @navan93 , I have reproduced the error. That's all I have for now.

tcal-x added a commit to tcal-x/CFU-Playground that referenced this issue Dec 12, 2023
…ugins.

This addresses google#808.

When the plugins package updates, we can unpin Yosys.

Also, comment out the download of the xc7a200t database
to make the installation of F4PGA faster.

Signed-off-by: Tim Callahan <[email protected]>
@tcal-x
Copy link
Collaborator

tcal-x commented Dec 12, 2023

This may be related to chipsalliance/yosys-f4pga-plugins#542. In any case, I find that pinning Yosys to an older version resolves the issue: #809.

@tcal-x
Copy link
Collaborator

tcal-x commented Dec 15, 2023

I hit the same issue when targeting Nexus FPGAs since that flow also uses the yosys-f4pga-plugins:

ERROR: Can't load module `./dsp-ff': /home/tim/CFU-Playground/env/conda/envs/cfu-common/bin/../share/yosys/plugins/dsp-ff.so: undefined symbol: _ZN5Yosys7stringfB5cxx11EPKcz

I'll update the PR to pin Yosys in the place used for that flow.

@navan93
Copy link
Author

navan93 commented Dec 19, 2023

Thanks @tcal-x for looking into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants