Opened 5 years ago
Closed 3 years ago
#106 closed enhancement (duplicate)
flashrom: Add -T to automatically test all flash chip operations
| Reported by: | stuge | Owned by: | somebody |
|---|---|---|---|
| Priority: | major | Milestone: | flashrom v1.0 |
| Component: | flashrom (please use trac on flashrom.org) | Keywords: | testing |
| Cc: | Dependencies: | ||
| Patch Status: | there is no patch |
Description
Maybe require -f to actually perform the test, since it will potentially fail to restore the flash chip to it's original state.
Attachments (0)
Change History (6)
comment:1 Changed 5 years ago by hailfinger
comment:2 Changed 5 years ago by hailfinger
For -T to work, you have to remove the mutual exclusion between -r and -w in a meaningful way. This may require a rewrite of the main loop.
comment:3 Changed 5 years ago by stepan
- Milestone changed from flashrom v1.0 to flashrom 1.1
comment:4 Changed 4 years ago by stuge
- Dependencies set to #103
- Milestone changed from flashrom v1.1 to flashrom v1.0
- Resolution set to duplicate
- Status changed from new to closed
dup #103
comment:5 Changed 4 years ago by stuge
- Dependencies #103 deleted
- Resolution duplicate deleted
- Status changed from closed to reopened
Carl-Daniel Hailfinger wrote:
The self-test mode has to execute:
- read (and store on disk)
I think the point is that it goes to disk to allow out-of-band restoration in case writing back with flashrom fails?
I would prefer no file. I don't want to have to remove files after testing, and not having a file would encourage testing strictly with flash chips without valuable contents. I think that is a very good thing™!
There's always -r to do a manual backup first, but I like the idea of making the difficult path more labor intense, to discourage testing with critical flash chips.
- write (old contents)
I think it has to write all 0 to the chip rather than the original contents. If the chip is blank to begin with there would be a false positive when erase or write does not work, and flashrom can't tell.
comment:6 Changed 3 years ago by hailfinger
- Resolution set to duplicate
- Status changed from reopened to closed
Flashrom now has a partial test infrastructure (among others, it can generate test patterns for write). The rest of this ticket is indeed a dup of #103.

Good idea.