[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 15/15] x86/hyperlaunch: add capabilities to boot domain
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>
- Date: Wed, 11 Dec 2024 14:56:19 -0500
- Arc-authentication-results: i=1; mx.zohomail.com; dkim=pass header.i=apertussolutions.com; spf=pass smtp.mailfrom=dpsmith@xxxxxxxxxxxxxxxxxxxx; dmarc=pass header.from=<dpsmith@xxxxxxxxxxxxxxxxxxxx>
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1733946983; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=yAsunI+vC/1qmV0Cgw5h2c+aVx89BeUOLRPKJtqyYDI=; b=QCzSTdxPlEqCc4Hz8c2baIerkP4J9KySTmttAgInV93IHA2/Ke4aA1B5TcSdp7Hhq2NoV+oVvoskLDquo+MEzqRZ6AUB2MWup7WNrkgafk42tIrlN0BIWMeHbLOpsn8XYk/s6X/VQ3CKonEzGIFb9ZlwnHUrA7PkqIVPzOJgqXk=
- Arc-seal: i=1; a=rsa-sha256; t=1733946983; cv=none; d=zohomail.com; s=zohoarc; b=CUuV3w9UlJAfvPEPk5RJUwPsuVrfWuw+sDEqQqAwPHXy5lrQMcvSOCq2GNk5COl4N77poakJ2WWpea6D49MFibsiEi6FI+4oPU7on2F8cH3jw7kRrL9UVykv3qBHrY9WjqhD3xtK40ZnyUNB2anPMpkYcDcEUw5NeRjgi+rIq6Y=
- Cc: jason.andryuk@xxxxxxx, christopher.w.clark@xxxxxxxxx, stefano.stabellini@xxxxxxx, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Wed, 11 Dec 2024 19:56:37 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 12/2/24 07:23, Jan Beulich wrote:
On 23.11.2024 19:20, Daniel P. Smith wrote:
--- a/xen/arch/x86/domain_builder/fdt.c
+++ b/xen/arch/x86/domain_builder/fdt.c
@@ -209,6 +209,19 @@ static int __init process_domain_node(
bd->max_vcpus = val;
printk(" max vcpus: %d\n", bd->max_vcpus);
}
+ if ( match_fdt_property(fdt, prop, "capabilities" ) )
+ {
+ if ( fdt_prop_as_u32(prop, &bd->capabilities) != 0 )
+ {
+ printk(" failed processing domain id for domain %s\n",
+ name == NULL ? "unknown" : name);
+ return -EINVAL;
+ }
+ printk(" caps: ");
+ if ( bd->capabilities & BUILD_CAPS_CONTROL )
+ printk("c");
+ printk("\n");
+ }
What if any of the other bits is set?
I'm not sure what you are getting at, but there is another cap added
later for HARDWARE and it will print an 'h' next to the 'c' if set.
--- a/xen/arch/x86/setup.c
+++ b/xen/arch/x86/setup.c
@@ -992,6 +992,7 @@ static size_t __init domain_cmdline_size(
static struct domain *__init create_dom0(struct boot_info *bi)
{
char *cmdline = NULL;
+ int create_flags = 0;
Once again unsigned int please.
ack.
@@ -1023,7 +1024,10 @@ static struct domain *__init create_dom0(struct
boot_info *bi)
/* Create initial domain. Not d0 for pvshim. */
if ( bd->domid == DOMID_INVALID )
bd->domid = get_initial_domain_id();
- d = domain_create(bd->domid, &dom0_cfg, pv_shim ? 0 : CDF_privileged);
+ if ( bd->capabilities & BUILD_CAPS_CONTROL )
+ create_flags |= CDF_privileged;
Nit: Indentation.
ack.
v/r,
dps
|
