[prim] DV infra for additional standardized countermeasures

lowRISC / opentitan

OpenTitan: Open source silicon root of trust

https://www.opentitan.org

Apache License 2.0

2.57k stars 771 forks source link

[prim] DV infra for additional standardized countermeasures #12301

Open msfschaffner opened 2 years ago

msfschaffner commented 2 years ago

We just recently added two additional standardized countermeasure primitives. Also, I believe we have not yet added common tests for the double LFSR (similar to double counter). Hence, we should consider adding common tests for these:

[x] Spurious WE checks in regfiles (prim_reg_we_check), see #12113
[ ] Duplicated RR arbiter (prim_arbiter_tree_dup)
[x] Duplicated LFSR (prim_double_lfsr)
[ ] Add countermeasure ID label CONFIG.GLITCH_DETECT to all IPs in the RTL hjson (#13726)

msfschaffner commented 2 years ago

CC @eunchan @tjaychen @jeoongp

msfschaffner commented 2 years ago

We should probably add the countermeasure ID label CONFIG.GLITCH_DETECT to all IPs once the common test for spurious WE checks is available (similar to BUS.INTEGRITY).

msfschaffner commented 2 years ago

@weicaiyang we added common tests for the WE checker, right?

weicaiyang commented 2 years ago

@weicaiyang we added common tests for the WE checker, right?

yes, just marked it done.

vogelpi commented 7 months ago

I've discussed this offline with @rswarbrick and our DV team is going to look at this. For now, I am assigning @rswarbrick . We need this for M3.

vogelpi commented 7 months ago

@moidx Changing the labels could be done after TO. @vogelpi DV should be done for M3/4.

andreaskurth commented 7 months ago

Deprioritized from Earlgrey-PROD