On 2024-08-23 09:10, Andrew Cooper wrote:
From: Frediano Ziglio <frediano.zig...@cloud.com>

Right now, Xen clobbers the value at 0xffc when performing it's load-base
calculation.  We've got plenty of free registers at this point, so the value
can be preserved easily.

This fixes a real bug booting under Coreboot+SeaBIOS, where 0xffc happens to
be the cbmem pointer (e.g. Coreboot's dmesg ring, among other things).

However, there's also a better choice of memory location to use than 0xffc, as
all our supported boot protocols have a pointer to an info structure in %ebx.

Update the documentation to match.

Fixes: 1695e53851e5 ("x86/boot: Fix the boot time relocation calculations")
Fixes: d96bb172e8c9 ("x86/entry: Early PVH boot code")
Signed-off-by: Frediano Ziglio <frediano.zig...@cloud.com>
Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>

Reviewed-by: Jason Andryuk <jason.andr...@amd.com>

Reply via email to