[coreboot] [PATCH] put coreboot table in high tables area
Kevin O'Connor
kevin at koconnor.net
Tue Mar 17 15:43:40 CET 2009
Hi Stefan,
On Mon, Mar 16, 2009 at 05:23:35PM +0100, Stefan Reinauer wrote:
> See patches
>
> I'll gladly send another fix for libpayload if the two patches are accepted
[...]
> --- src/include/boot/coreboot_tables.h (revision 1304)
> +++ src/include/boot/coreboot_tables.h (working copy)
> @@ -155,6 +155,13 @@
> uint16_t type;
> };
>
> +#define LB_TAG_FORWARD 0x0011
> +struct lb_forward {
> + uint32_t tag;
> + uint32_t size;
> + uint64_t forward;
> +};
One suggestion - instead of making the "floating table" be a sub-table
of a coreboot-table, make it a stand alone data structure. As it
stands, I think existing code might get confused when it finds a
coreboot table without any real data in it.
-Kevin
More information about the coreboot
mailing list