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

[BUG] ASIC synthesis failed related to ariane_pkg.sv #1726

Closed
1 task done
andriami opened this issue Dec 19, 2023 · 8 comments
Closed
1 task done

[BUG] ASIC synthesis failed related to ariane_pkg.sv #1726

andriami opened this issue Dec 19, 2023 · 8 comments
Assignees
Labels
Type:Enhancement For feature requests and enhancements

Comments

@andriami
Copy link
Contributor

Is there an existing CVA6 bug for this?

  • I have searched the existing bug issues

Bug Description

Hi,

I am trying to implement the CVA6 on ASIC by using the proposed flow. I have errors related to ariane_pkg.sv synthesis.

Compiling source file ../..//core/include/ariane_pkg.sv
Error:  ../..//core/include/ariane_pkg.sv:169: Package 'ariane_pkg' has not been analyzed for import or content extraction. (VER-224) 

This is due to the use of ariane_pkg inside the ariane_pkg itself.

localparam int unsigned INSTR_PER_FETCH = RVC == 1'b1 ? (FETCH_WIDTH / 16) : 1;
localparam int unsigned LOG2_INSTR_PER_FETCH = RVC == 1'b1 ? $clog2(
   ariane_pkg::INSTR_PER_FETCH
) : 1;

To resolve this issue, I removed the ariane_pkg before INSTR_PER_FETCH.

localparam int unsigned INSTR_PER_FETCH = RVC == 1'b1 ? (FETCH_WIDTH / 16) : 1;
localparam int unsigned LOG2_INSTR_PER_FETCH = RVC == 1'b1 ? $clog2(
   INSTR_PER_FETCH
) : 1;

Do you know another way of solving this issue?

Thanks

@andriami andriami added the Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system label Dec 19, 2023
@JeanRochCoulon
Copy link
Contributor

Hello @andriami
The ASIC synthesis runs on CI. We run also regularly it in interactive way. I do not see why you get a bug due to RTL. What is your Dc_shell version ?

@andriami
Copy link
Contributor Author

Hi @JeanRochCoulon,

Thank you for your answer!
I am using version P-2019.03 of DC.

@JeanRochCoulon
Copy link
Contributor

JeanRochCoulon commented Dec 27, 2023

We use S-2021.09-SP1-1. Thsi can be the root cause.
As your suggested fix seems good to me. Can I ask you to submit a PR ?
As we did not succeed in reproducing the issue on CI and we proposed to modify the RTL to workaround the issue. I change the label from "bug" to "enhancement".

@JeanRochCoulon JeanRochCoulon self-assigned this Dec 27, 2023
@JeanRochCoulon JeanRochCoulon added Type:Enhancement For feature requests and enhancements and removed Type:Bug For bugs in the RTL, Documentation, Verification environment or Tool and Build system labels Dec 27, 2023
andriami pushed a commit to andriami/cva6 that referenced this issue Jan 8, 2024
… encountered during ASIC synthesis with DC version P-2019.03.
JeanRochCoulon pushed a commit that referenced this issue Jan 8, 2024
Modify ariane_pkg.sv following issue #1726.
rohan-10xe pushed a commit to 10x-Engineers/cva6 that referenced this issue Jan 23, 2024
@JeanRochCoulon
Copy link
Contributor

CLosed because fixed by #1750.

@hhhsiang
Copy link

hi @andriami , can you synthesize cva6 core successfully now?

@andriami
Copy link
Contributor Author

Hi @hhhsiang, I successfully synthesized it using the May 22nd, 2024 version, but I haven't tried it with the latest version yet.

@hhhsiang
Copy link

@andriami ,did you synthesize it with the makefile under pd/synth, or you create a script on your own? Which config pkg did you use to synthesize? Is that cv64a6_imafdc_sv39_config_pkg.sv?

I encounter error like this issue:#2544
Thanks in advance!

@andriami
Copy link
Contributor Author

Hi @hhhsiang, yes I used the makefile under pd/syn and the cv64a6_imafdc_sv39_config_pkg.sv config. I didn't encounter the issue #2544 you mentioned. For information, we have used an older version ( Q-2019.12) of DC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Enhancement For feature requests and enhancements
Projects
None yet
Development

No branches or pull requests

3 participants