search

openhwgroup / cva6

The CORE-V CVA6 is an Application class 6-stage RISC-V CPU capable of booting Linux
https://docs.openhwgroup.org/projects/cva6-user-manual/
Other
2.23k stars 675 forks source link

Update submodule core/cache_subsystem/hpdcache #2265

Closed cfuguet closed 3 months ago

cfuguet commented 3 months ago

This new version of the HPDcache includes multiple improvements in the write buffer microarchitecture to improve both area and performance

github-actions[bot] commented 3 months ago

:x: failed run, report available here.

github-actions[bot] commented 3 months ago

:x: failed run, report available here.

JeanRochCoulon commented 3 months ago

The Coremark result was slightly improved (good!). I took the liberty to stack a commit to fix it.

github-actions[bot] commented 3 months ago

:x: failed run, report available here.

cfuguet commented 3 months ago

We still have this issue of Spyglass. I guess it is the same. I have Spyglass in CEA, but I do not have the aipk tool that you use to run the Spyglass tool in the CI.

The issue you sent me was with an assertion using the final keyword in a new component. Either I can comment out that line, either we modify the Spyglass configuration to authorize the final keyword in assertions (which is only useful in simulation, or waive that specific line.

JeanRochCoulon commented 3 months ago

The short term solution can be to comment the line. This would allow us to merge the PR. Then the Spyglass configuration could be discussed later on.

github-actions[bot] commented 3 months ago

:x: failed run, report available here.

cfuguet commented 3 months ago

Thank you @JeanRochCoulon !

What was the issue with Linux ? I saw a failed test.

JeanRochCoulon commented 3 months ago

An issue of connection between PC and FPGA board. I relaunched the job and it worked... Nice surprise !