[flashrom-gerrit] [L] Change in flashrom[main]: erasure_layout: Fix get_flash_region bug

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Aarya, Alexander Goncharov, Nikolai Artemiev, Peter Marheine.

Anastasia Klimchuk has posted comments on this change by Peter Marheine. ( https://review.coreboot.org/c/flashrom/+/82393?usp=email )

Change subject: erasure_layout: Fix get_flash_region bug
......................................................................

Patch Set 14:

(1 comment)

File erasure_layout.c:

https://review.coreboot.org/c/flashrom/+/82393/comment/fecea642_135db866?usp... :
PS13, Line 368: for (unsigned int addr = region_start; addr <= region_end; addr += len) {
> I have changed tests, and looked into debugging messages, and now I have a piece of mind that the bu […]

I made CB:82723 to illustrate, the issue repro on head

Now the question is, how urgent the issue is. The memory is erased correctly, but not in an optimal way.
Peter what do you think?

(also for this latest patchset, Jenkins is not adding a comment, but the results are here https://qa.coreboot.org/job/flashrom_gerrit/11678/console)