Background: https://review.coreboot.org/c/flashrom/+/80807

A long time ago, in the pre-git times [1], flashrom added a 1 second
delay to all verification, and claimed that some chips "need some time
to calm down." The commit message claims it "fixes a few reports where
verify directly after erase had unpleasant side effects like
corrupting flash or at least getting incorrect verify results." It
provides no details of what systems, chips, or programmers were
involved.

This delay remains in the --verify path today, and IMO, it's a big
waste of time. If there are truly chips that are, say, deasserting the
BUSY line before they're truly finished programming ... well, we
should track those chips down and add targeted quirk flags for them.
We shouldn't penalize all flashrom users in all cases for all time.

Personally, I highly doubt that this delay is still relevant today --
there may have been a bug in some programmer that has since been
fixed; there may have been some malfunctioning system that is no
longer in use; ... or it's possible this is still hiding a real buggy
chip somewhere out there.

I propose: we still remove the delay. There's plenty of description in
the above Gerrit link about why, and how we can handle regressions.
(For one, it's relatively simple to split up a --verify operation into
its constituent --write/sleep/--read operations, for debugging.)

The request:
1. Tests: I've tested a few Chromebooks, but imagine this had to have
been some more esoteric system. Extra testing is welcome.
2. Thoughts: does my proposal make sense? Am I missing something obvious?
3. Awareness: if nothing else, this email may serve to highlight in
case we land the above patch and later hear back that there are some
sketchy --verify reports from users.

Thanks,
Brian

[1] The svn->git commit in question:

  commit 8ab49e72af8465d4527de2ec37b22cd44f7a1169
  Date:   Wed Aug 19 13:55:34 2009 +0000
  Disallow erase/write for known bad chips so people won't try without
a clear understanding
_______________________________________________
flashrom mailing list -- flashrom@flashrom.org
To unsubscribe send an email to flashrom-le...@flashrom.org

Reply via email to