[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH v6 01/11] xen: Conditionally compile out the boot_module.domU


  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>
  • Date: Tue, 8 Jul 2025 20:07:10 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=bPD/rKYLwoQh7EhPgtS74wicAf9RUsbK6gNUi7Q5AAI=; b=dRImN9qh8dYOsr8UveZhBhga66T3Y2k2nUj/USmtYzNerkEHN494StkNTb0OC/qi47Xr0yDAHC4qw1juPGpXIDGdx4luCswNMExEg104aRgVWUNFnKxa1ci0SJ/mgACAOb3ffvg8xOnFK6JDqTZwek3xY3Q5sR7uuFj8bnIwteBT4wcVzqDDM9QQ6OLZM1UaSzuhQYN7bRh8NmCbHu4Ha3pyG7ZFKqo1FtCiwnPH0iZnKP9oahQD0p9cbAI6eZRbDnNXLaoaRTWLRxID5l6o8pqo9WIN0oZiwRHLINmqxWQAvoNDHlVhAcr+DAv/OUwyS9lmKKtk9xHmIcpIuvWTxw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=HXdvDS96BA6aP+4nr6YFRIYeVczHrqZA3qCdv+8L/pJ188xaZq552I2oh+Rf3G9xR6GVaqIqv+z7lgiYjG9TJ54eWFHhqmaF5n7xZd3M3JHoEqyNlG5LEj/zhr/Oz/f/kN2CK6a+e+5wyoW4qtP9nYQifIRdAtOLPBfXbCOVBmYMYA3SEZk8WwJYv6PZWhgkXYhByOKjgWncSQbjMHF8/pcX8zAHTgCVwCFVJr0yO+tfDv4FoPUk3FyS21yfSVTxkgzraD/TDBQ48iUH9rfnspU1S3FGEp+DNCmrhpdhErbslsgGLtC90dQvktsnBZHmf/EE9UwaWdbTf1xbDgixQg==
  • Cc: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Tue, 08 Jul 2025 18:07:52 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

... based on CONFIG_X86. x86 won't depend on it, and it'll only be used
by the domain builder in common/device-tree/

Moving it to an arch-specific subfield (as introduced in a later patch)
is unwieldy, because all other arches depend on it.

Signed-off-by: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>
---
My expectation is to remove the field altogether after we reach full
integration in common code, by deducing this boolean from the domain
capabilities given in the "capabilities" prop.

That's not something I'm working at the moment though.
---
 xen/include/xen/bootfdt.h | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/xen/include/xen/bootfdt.h b/xen/include/xen/bootfdt.h
index 236b456dd2..7f49d0ccdd 100644
--- a/xen/include/xen/bootfdt.h
+++ b/xen/include/xen/bootfdt.h
@@ -96,16 +96,18 @@ struct shared_meminfo {
     struct shmem_membank_extra extra[NR_SHMEM_BANKS];
 };
 
-/*
- * The domU flag is set for kernels and ramdisks of "xen,domain" nodes.
- * The purpose of the domU flag is to avoid getting confused in
- * kernel_probe, where we try to guess which is the dom0 kernel and
- * initrd to be compatible with all versions of the multiboot spec.
- */
 #define BOOTMOD_MAX_CMDLINE 1024
 struct boot_module {
     boot_module_kind kind;
+#ifndef CONFIG_X86
+    /*
+     * The domU flag is set for kernels and ramdisks of "xen,domain" nodes.
+     * The purpose of the domU flag is to avoid getting confused in
+     * kernel_probe, where we try to guess which is the dom0 kernel and
+     * initrd to be compatible with all versions of the multiboot spec.
+     */
     bool domU;
+#endif
     paddr_t start;
     paddr_t size;
 };
-- 
2.43.0




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.