Attention is currently required from: Aarya, Anastasia Klimchuk, Bill XIE.
Anastasia Klimchuk has uploaded a new patch set (#11) to the change originally created by Bill XIE. ( https://review.coreboot.org/c/flashrom/+/84721?usp=email )
The following approvals got outdated and were removed: Verified+1 by build bot (Jenkins)
Change subject: erasure_layout: Add an option to sacrifice unchanged blocks for speed ......................................................................
erasure_layout: Add an option to sacrifice unchanged blocks for speed
The patch adds command line option to handle the following situation:
There is a region which is requested to be erased (or written, because the write operation uses erase too). Some of the areas inside this region don't need to be erased, because the bytes already have expected value. Such areas can be skipped.
The logic selects eraseblocks that can cover the areas which need to be erased. Suppose there is a region which is partially covered by eraseblocks of size S (partially because remaining areas don't need to be erased). Now suppose we can cover the whole region with eraseblock of larger size, S+1, and erase it all at once. This will run faster: erase opcode will only be sent once instead of many smaller opcodes. However, this will run erase over some areas of the chip memory that didn't need to be erased. Which means, the chip, as a hardware, will wear faster.
New command line option sets the maximum % memory that is allowed for redundant erase. Default is 0, S+1 size block only selected if all the area needs to be erased in full. 50 means that if more than a half of the area needs to be erased, a S+1 size block can be selected to cover all area with one block.
The tradeoff is the speed of programming operation VS the longevity of the chip. Default is longevity.
Change-Id: I154e8a713f626c37dbbe118db700055b96d24803 Co-developed-by: persmule <persmule@hardenedlinux.org Co-developed-by: Anastasia Klimchuk aklm@flashrom.org Signed-off-by: persmule persmule@hardenedlinux.org Signed-off-by: Anastasia Klimchuk aklm@flashrom.org --- M cli_classic.c M doc/classic_cli_manpage.rst M doc/release_notes/devel.rst M erasure_layout.c M include/flash.h M tests/erase_func_algo.c 6 files changed, 152 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/21/84721/11