cva6 icon indicating copy to clipboard operation
cva6 copied to clipboard

Bump Verilator to v4.224

Open vighneshiyer opened this issue 3 years ago • 3 comments

We're working on bumping Verilator to a modern release in Chipyard (https://github.com/ucb-bar/chipyard/pull/1205) and as part of that process we ran into a segfault when trying to simulate a Chipyard SoC with an Ariane core (which might be a Verilator bug).

To isolate the issue, I bumped Verilator in cva6 and verified that the simulation using your infrastructure works fine. This PR contains the minor changes required (see) if you all are interested. Feel free to close this otherwise.

Seemingly resolves (https://github.com/openhwgroup/cva6/issues/931) (vcd dumping works with the latest Verilator) and (https://github.com/openhwgroup/cva6/issues/934)

vighneshiyer avatar Jul 29 '22 00:07 vighneshiyer

@zchamski The Thales CI became FAIL with bumped Verilator, do you confirm on your side ?!?

JeanRochCoulon avatar Sep 05 '22 14:09 JeanRochCoulon

Does anyone know how to resolve the Eclipse ECA error? I've created an account, signed the ECA, and verified that the email on the ECA matches the one used as the email for this commit. But, the Eclipse webpage still shows a 'failure' (revalidating doesn't do anything either, as expected). Is there something else I need to sign?

2022-09-07-183622_1530x1198_scrot 2022-09-07-183640_2060x411_scrot

vighneshiyer avatar Sep 12 '22 17:09 vighneshiyer

@MikeOpenHWGroup or @duncan, do you have an idea ?

JeanRochCoulon avatar Sep 12 '22 21:09 JeanRochCoulon

Hi folks, @Jbalkind helped us resolve the eca issue. I believe it's now good to go

tianrui-wei avatar Oct 09 '22 04:10 tianrui-wei

Feel free to merge this now that the ECA issue is resolved, or update the PR to bump to an even more recent Verilator version or close this PR, thanks

vighneshiyer avatar Oct 17 '22 23:10 vighneshiyer

Thanks for your contribution but Verilator 4.224 does not pass the regression tests. See https://riscv-ci.pages.thales-invia.fr/dashboard/dashboard_cva6.html As Verilator 5 passes the tests. A plan could be to switch to this up-to-date version.

JeanRochCoulon avatar Oct 18 '22 04:10 JeanRochCoulon

Hello @vighneshiyer Thank you for your PR. Your PR make us understand that Verilator 5 is better. Our plan is to switch on Verilator 5 soon. I close your PR. See you soon (I hope).

JeanRochCoulon avatar Feb 02 '23 21:02 JeanRochCoulon