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

Eliminate CVA6 dependencies within the PAU #2

Closed
RaulMurillo opened this issue Sep 12, 2023 · 6 comments
Closed

Eliminate CVA6 dependencies within the PAU #2

RaulMurillo opened this issue Sep 12, 2023 · 6 comments

Comments

@RaulMurillo
Copy link

To make a more flexible code (so the PAU could be extended with new features, or adapted to other cores), there are some dependencies that should be removed. More precisely, all the params within the pau_top.sv referencing ariane_pkg and riscv_pkg should be replaced.
This would make the PAU to be core-agnostic, in a similar way as FPnew works. Then, a specialization layer should be included to adapt the PAU to the different cores (in this case, the CVA6), similarly as fpu_wrap.sv.

P.S.: Ideally, the PAU could be an independent repo, but I know this implies much extra work.

@RaulMurillo
Copy link
Author

I did some modifications in a branch to eliminate the aforementioned dependencies, but I do not have all the infrastructure to check if the changes are correct.

Can you check if everything works the same?

@davidmallasen
Copy link
Collaborator

I have added a "Getting Started" section in the README when trying to test your branch.
When executing your branch I get the following error:

#############  Autofindloop Analysis  ###############
#############  Loop found at time 11390 ns ###############
#   Active process: /ariane_tb/dut/i_dm_top/i_dm_mem/#ASSIGN#114 @ sub-iteration 0
#     Source: /home/david/Documents/repos/RaulMurillo/PERCIVAL/corev_apu/riscv-dbg/src/dm_top.sv:191
#   Active process: /ariane_tb/dut/i_dm_top/i_dm_mem/#IMPLICIT-WIRE(cmdbusy_o)#49 @ sub-iteration 1
#     Source: /home/david/Documents/repos/RaulMurillo/PERCIVAL/corev_apu/riscv-dbg/src/dm_top.sv:191
#     Assigning reg to (val=0)
#   Active process: /ariane_tb/dut/i_dm_top/i_dm_mem/#ASSIGN#114 @ sub-iteration 2
#     Source: /home/david/Documents/repos/RaulMurillo/PERCIVAL/corev_apu/riscv-dbg/src/dm_top.sv:191
################# END OF LOOP #################
# ** Error (suppressible): (vsim-3601) Iteration limit 10000000 reached at time 11390 ns.

@davidmallasen
Copy link
Collaborator

@wwwuxy
Copy link

wwwuxy commented Nov 5, 2024

I have added a "Getting Started" section in the README when trying to test your branch. When executing your branch I get the following error:

#############  Autofindloop Analysis  ###############
#############  Loop found at time 11390 ns ###############
#   Active process: /ariane_tb/dut/i_dm_top/i_dm_mem/#ASSIGN#114 @ sub-iteration 0
#     Source: /home/david/Documents/repos/RaulMurillo/PERCIVAL/corev_apu/riscv-dbg/src/dm_top.sv:191
#   Active process: /ariane_tb/dut/i_dm_top/i_dm_mem/#IMPLICIT-WIRE(cmdbusy_o)#49 @ sub-iteration 1
#     Source: /home/david/Documents/repos/RaulMurillo/PERCIVAL/corev_apu/riscv-dbg/src/dm_top.sv:191
#     Assigning reg to (val=0)
#   Active process: /ariane_tb/dut/i_dm_top/i_dm_mem/#ASSIGN#114 @ sub-iteration 2
#     Source: /home/david/Documents/repos/RaulMurillo/PERCIVAL/corev_apu/riscv-dbg/src/dm_top.sv:191
################# END OF LOOP #################
# ** Error (suppressible): (vsim-3601) Iteration limit 10000000 reached at time 11390 ns.

my error is same as you,which we talk in issue #4

@davidmallasen
Copy link
Collaborator

I didn't look into that back in the day. @RaulMurillo do you remember?

@RaulMurillo
Copy link
Author

RaulMurillo commented Nov 5, 2024

Uff, a lot has happened since then.
I made no additional changes, so I don't know where the problem can be.

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

No branches or pull requests

3 participants