Attention is currently required from: Sam McNally, Anastasia Klimchuk.
Hello Sam McNally, build bot (Jenkins), Anastasia Klimchuk,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/71659
to look at the new patch set (#2).
Change subject: tests/: Add subregion alignment unit-test ......................................................................
tests/: Add subregion alignment unit-test
A written region that is sized below that of the erasure granularity can result in a incorrectly read region that does not include prior content within the region before the write op. This was dealt with in ChromeOS downstream by expanding out the read to match the erase granularity however does not seem to impact upstream. Add a unit-test to avoid regression as this is important behaviour to cover.
Change-Id: Id3ce5cd1936f0f348d34a6c77cee15e27a5c353f Signed-off-by: Edward O'Callaghan quasisec@google.com --- M tests/chip.c M tests/tests.c M tests/tests.h 3 files changed, 120 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/59/71659/2