Opened 7 years ago

Closed 6 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 Keywords: testing
Cc: Dependencies:
Patch Status: there is no patch


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 7 years ago by hailfinger

Good idea.

comment:2 Changed 7 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 7 years ago by stepan

  • Milestone changed from flashrom v1.0 to flashrom 1.1

comment:4 Changed 7 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 7 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:

  1. 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.

  1. 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 6 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.

Add Comment

Modify Ticket

as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.