openhwgroup / cv32e40p

CV32E40P is an in-order 4-stage RISC-V RV32IMFCXpulp CPU based on RI5CY from PULP-Platform
https://docs.openhwgroup.org/projects/cv32e40p-user-manual/en/latest
Other
900 stars 399 forks source link

RTL Code Coverage Hole in cv32e40p_EX_stage module line 387 for FPU configuration #1017

Open YoannPruvost opened 3 days ago

YoannPruvost commented 3 days ago

Component

Component:RTL

Issue Description

One of the possible combination in the if statement line 387of cv32e40p_EX_stage was not covered during all the simulation non-regressions.

After analysis we suspected that this scenario was in fact unreachable. Siemens Questa Static formal tool was used to prove that this scenario was unreachable. For this scenario a dedicated assertion was written, named assert_unreachable_ex_387.

All information necessary to reproduce and analyze our work with formal can be found in the ReadMe in the cv32e40p/scripts/formal folder

As it was too late to implement a fix in the RTL due to long RISC-V ISA Formal Verification runs and requiring to update all waivers files as well, it has been decided to waive this scenario hole in v2.

387-396 387