[coreboot] [PATCH]Fix selfboot bounce buffer handling

Patrick Georgi patrick at georgi-clan.de
Wed May 13 12:29:15 CEST 2009


Hi,

attached patches do:
* dynamic-bounce-buffer-size
Eliminate the assumption that the bouncebuffer is twice the size of coreboot.

* flexible-self-bounce-buffer
Bouncebuffer handling with compressed segments was changed so that the entire 
segment ends up in the buffer and isn't split into "overlaps coreboot" and 
"doesn't overlap coreboot".
That can lead to problems if the segment is larger than coreboot, incl. 
corruption of the high memory tables, or crashes in extreme cases.

This patch builds on dynamic-bounce-buffer-size and allocates the bouncebuffer 
so that all segments that overlap coreboot fit into it entirely.

* update-comments
Update comments accordingly

The patches have some different line numbers than upstream, but apply 
correctly. The remaining changes are just debug prints.

Tested on Kontron. Together with the other patch by me this morning and the 
commited FILO change, ACPI and e820 tables are seen correctly by Linux.

Signed-off-by: Patrick Georgi <patrick.georgi at coresystems.de>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dynamic-bounce-buffer-size.diff
Type: text/x-patch
Size: 3819 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090513/cd1c3baf/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: flexible-self-bounce-buffer.diff
Type: text/x-patch
Size: 5225 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090513/cd1c3baf/attachment-0001.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: update-comments.diff
Type: text/x-patch
Size: 1646 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090513/cd1c3baf/attachment-0002.diff>


More information about the coreboot mailing list