On 9/3/24 19:40, Andrew Cooper wrote:
On 30/08/2024 10:46 pm, Daniel P. Smith wrote:
diff --git a/xen/arch/x86/include/asm/bootinfo.h 
b/xen/arch/x86/include/asm/bootinfo.h
index 844262495962..3e0e36df096b 100644
--- a/xen/arch/x86/include/asm/bootinfo.h
+++ b/xen/arch/x86/include/asm/bootinfo.h
@@ -13,6 +13,7 @@
struct boot_module {
      module_t *early_mod;
+    unsigned long headroom;

This needs a comment explaining what it's for.  Perhaps crib from ...

Ack.

diff --git a/xen/arch/x86/setup.c b/xen/arch/x86/setup.c
index 8912956ee7f1..fd6cc7fac907 100644
--- a/xen/arch/x86/setup.c
+++ b/xen/arch/x86/setup.c
@@ -1475,8 +1477,10 @@ void asmlinkage __init noreturn __start_xen(unsigned 
long mbi_p)
               * decompressor overheads of mod[0] (the dom0 kernel).  When we
               * move mod[0], we incorporate this as extra space at the start.
               */

... here, while also editing to to prevent it going stale.

It is this patch which stops modules_headroom being strictly the dom0
kernel.

Correct, because once multi-domain construction is introduced, there will be more than one kernel needing headroom for extraction.

v/r,
dps

Reply via email to