Re: [SeaBIOS] [PATCH] coreboot: update memory table before handling execution to payload/system

[Kevin O'Connor]

On Mon, Nov 05, 2018 at 01:27:10PM +0100, Krystian Hebel wrote:
> SeaBIOS modifies its internal e820 structure, but does not propagate
> these changes back to coreboot tables. This resulted in multiple errors
> in MemTest86 when run on 2 GB platforms, probably because of some
> memory-mapped devices.
>
> This patch copies back modified e820 tables before booting an OS or
> a payload.

Thanks, but I'm not sure that at a "high level" this is a good idea.
I don't think SeaBIOS should be altering coreboot information.  (Doing
so leads to all sorts of painful debugging problems, for example.)  If
memtest86 is having problems with the coreboot tables I suspect it
would be best to alter memtest86 to not read the coreboot tables.

-Kevin