[flash_ctrl] Improve ICV
Open
@matutem is already working on this.
Since Sep 14, 2022.
Component:Security
Earlgrey-PROD Triaged
IP:flash_ctrl
Type:FutureRelease
- Dominant language
- SystemVerilog
- Stars
- 3.6k
- Forks
- 1.1k
- Avg merge
- 2d 22h
- Merged PRs (30d)
- 141
Description
We currently use truncated Hamming SECDED for flash integrity, but that is not necessarily good for integrity, since it can fail to detect multiple bit flips. We may want to use a different scheme with better guarantees. @jadephilipoom suggested we may consider https://sites.bu.edu/mark/files/2018/02/214.pdf since they claim better detection, and there may be other such schemes.
I am not suggesting we change the current implementation at this time, but this could be useful for future implementations.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.