From: Alejandro Vallejo <agarciav@amd.com>
To: Julien Grall <julien@xen.org>, <xen-devel@lists.xenproject.org>
Cc: "Jan Beulich" <jbeulich@suse.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"Roger Pau Monné" <roger.pau@citrix.com>,
"Anthony PERARD" <anthony.perard@vates.tech>,
"Michal Orzel" <michal.orzel@amd.com>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Bertrand Marquis" <bertrand.marquis@arm.com>,
"Daniel P. Smith" <dpsmith@apertussolutions.com>
Subject: Re: [PATCH v3 14/14] kconfig: Allow x86 to pick CONFIG_DOM0LESS_BOOT
Date: Mon, 16 Jun 2025 16:39:59 +0200 [thread overview]
Message-ID: <DAO1A9SJ9I2G.2L00II6AC0JDZ@amd.com> (raw)
In-Reply-To: <2a54e49b-c1ad-4654-9291-4373e92580ff@xen.org>
On Mon Jun 16, 2025 at 10:00 AM CEST, Julien Grall wrote:
> Hi,
>
> On 13/06/2025 16:13, Alejandro Vallejo wrote:
>> Without picking CONFIG_HAS_DEVICE_TREE.
>>
>> In order to do that. Allow CONFIG_DOM0LESS_BOOT to enable a subset
>> of the common/device-tree/ directory.
> > x86 doesn't want dom0less-build.c,> as that's tightly integrated
> still to the ARM way of building domains.
>
> I don't understand this argument. dom0less-build.c was moved to common
> and it will soon be used by RISC-V. This raises the question what's so
> special with x86?
That's 2 separate matters:
1. dom0less-build.c not being compiled in.
2. CONFIG_DOM0LESS_BOOT enabling use of DT code without CONFIG_HAS_DEVICE_TREE.
(1) is a matter of not wanting to boil the ocean upfront. The way x86 and
everyone else build domains is just different and duplicated in non-trivially
consolidable ways. The goal here is to enable the domain builders in any arch
to use the same backbone. I don't want to go the extra mile just yet to unify
domain construction (though in time I will want to).
(2) has to do with compiling OUT things I really cannot have around. Anything
involving devices described in a DT must not exist on x86, because it has no
concept of a "struct device".
My intent is/was to repurpose CONFIG_HAS_DEVICE_TREE to mean "this hypervisor
goes on a platform that gives a platform-describing DT". On x86 that's given by
DSDT/SSDTs with ACPI.
>
> Note I don't particularly care if you don't want to use it on x86.
> However, the argument provided is lacking some details... This will be
> useful in the future if someone thinks about consolidating the two.
I very definitely will want it all unified, but I'm working one elephant at
a time.
Cheers,
Alejandro
next prev parent reply other threads:[~2025-06-16 14:40 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-13 15:13 [PATCH v3 00/15] Allow x86 to unflatten DTs Alejandro Vallejo
2025-06-13 15:13 ` [PATCH v3 01/14] arm/gnttab: Remove xen/grant_table.h cyclic include Alejandro Vallejo
2025-06-14 0:30 ` Stefano Stabellini
2025-06-13 15:13 ` [PATCH v3 02/14] x86: Preinitialise all modules to be of kind BOOTMOD_UNKNOWN Alejandro Vallejo
2025-06-16 6:46 ` Jan Beulich
2025-06-13 15:13 ` [PATCH v3 03/14] CODING_STYLE: Custom type names must be snake-cased by word Alejandro Vallejo
2025-06-14 0:32 ` Stefano Stabellini
2025-06-16 11:08 ` Alejandro Vallejo
2025-06-16 6:48 ` Jan Beulich
2025-06-16 11:07 ` Alejandro Vallejo
2025-06-13 15:13 ` [PATCH v3 04/14] xen: Rename bootmodule{,s} to boot_module{,s} Alejandro Vallejo
2025-06-14 0:41 ` Stefano Stabellini
2025-06-17 1:12 ` Daniel P. Smith
2025-06-13 15:13 ` [PATCH v3 05/14] x86: Replace arch-specific boot_module with common one Alejandro Vallejo
2025-06-14 0:54 ` Stefano Stabellini
2025-06-16 11:35 ` Alejandro Vallejo
2025-06-17 11:36 ` Alejandro Vallejo
2025-06-13 15:13 ` [PATCH v3 06/14] xen: Refactor kernel_info to have a header like boot_domain Alejandro Vallejo
2025-06-14 0:58 ` Stefano Stabellini
2025-06-17 1:22 ` Daniel P. Smith
2025-06-17 10:05 ` Alejandro Vallejo
2025-06-13 15:13 ` [PATCH v3 07/14] x86: Replace arch-specific boot_domain common one bootdomain Alejandro Vallejo
2025-06-14 1:08 ` Stefano Stabellini
2025-06-13 15:13 ` [PATCH v3 08/14] xen/dt: Move bootfdt functions to xen/bootfdt.h Alejandro Vallejo
2025-06-14 1:16 ` Stefano Stabellini
2025-06-16 13:57 ` Alejandro Vallejo
2025-06-19 14:24 ` Alejandro Vallejo
2025-06-19 19:50 ` Stefano Stabellini
2025-06-13 15:13 ` [PATCH v3 09/14] xen/dt: Move bootinfo functions to a new bootinfo.h Alejandro Vallejo
2025-06-14 1:22 ` Stefano Stabellini
2025-06-13 15:13 ` [PATCH v3 10/14] xen/dt: Rename bootfdt.c -> bootinfo-fdt.c Alejandro Vallejo
2025-06-13 15:13 ` [PATCH v3 11/14] xen/dt: Extract helper to map nodes to module kinds Alejandro Vallejo
2025-06-13 15:13 ` [PATCH v3 12/14] xen/dt: ifdef out DEV_DT-related bits from device_tree.{c,h} Alejandro Vallejo
2025-06-14 1:24 ` Stefano Stabellini
2025-06-16 7:01 ` Jan Beulich
2025-06-16 7:57 ` Julien Grall
2025-06-13 15:13 ` [PATCH v3 13/14] xen/dt: Allow CONFIG_DOM0LESS_BOOT to include device-tree/ Alejandro Vallejo
2025-06-14 1:25 ` Stefano Stabellini
2025-06-16 6:55 ` Jan Beulich
2025-06-16 14:08 ` Alejandro Vallejo
2025-06-16 23:41 ` Stefano Stabellini
2025-06-13 15:13 ` [PATCH v3 14/14] kconfig: Allow x86 to pick CONFIG_DOM0LESS_BOOT Alejandro Vallejo
2025-06-14 1:26 ` Stefano Stabellini
2025-06-16 6:59 ` Jan Beulich
2025-06-16 14:12 ` Alejandro Vallejo
2025-06-16 14:19 ` Jan Beulich
2025-06-16 8:00 ` Julien Grall
2025-06-16 14:39 ` Alejandro Vallejo [this message]
2025-06-16 23:39 ` Stefano Stabellini
2025-06-17 1:01 ` dmkhn
2025-06-17 6:17 ` Jan Beulich
2025-06-13 15:28 ` [PATCH v3 00/15] Allow x86 to unflatten DTs Alejandro Vallejo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DAO1A9SJ9I2G.2L00II6AC0JDZ@amd.com \
--to=agarciav@amd.com \
--cc=andrew.cooper3@citrix.com \
--cc=anthony.perard@vates.tech \
--cc=bertrand.marquis@arm.com \
--cc=dpsmith@apertussolutions.com \
--cc=jbeulich@suse.com \
--cc=julien@xen.org \
--cc=michal.orzel@amd.com \
--cc=roger.pau@citrix.com \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.