From: Jason Andryuk <jason.andryuk@amd.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: "Christian Lindig" <christian.lindig@citrix.com>,
"David Scott" <dave@recoil.org>,
"Anthony PERARD" <anthony.perard@vates.tech>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"Michal Orzel" <michal.orzel@amd.com>,
"Julien Grall" <julien@xen.org>,
"Roger Pau Monné" <roger.pau@citrix.com>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Bertrand Marquis" <bertrand.marquis@arm.com>,
"Volodymyr Babchuk" <Volodymyr_Babchuk@epam.com>,
"Daniel P. Smith" <dpsmith@apertussolutions.com>,
xen-devel@lists.xenproject.org
Subject: Re: [PATCH 3/4] xen: Add DOMAIN_CAPS_DEVICE_MODEL & XEN_DOMCTL_CDF_device_model
Date: Wed, 11 Jun 2025 00:35:45 -0400 [thread overview]
Message-ID: <9819e041-3b4a-4e5f-b391-149e0900f286@amd.com> (raw)
In-Reply-To: <af247ba8-150f-4c19-b332-2bf5f53a81a5@suse.com>
On 2025-06-11 09:24, Jan Beulich wrote:
> On 11.06.2025 00:57, Jason Andryuk wrote:
>> To add more flexibility in system configuration add the new
>> DOMAIN_CAPS_DEVICE_MODEL flag and XEN_DOMCTL_CDF_device_model.
>>
>> Thie new flag corresponds to allowing XSM_DM_PRIV for the domain. This
>> will enable running device model emulators (QEMU) from the assigne
>> domain for multiple target domains.
>>
>> Stubdoms assign target allowing the stubdom to serve as the device
>> model for a single domain. This new flag allows the single domain to
>> provide emulators for multiple guests.
>>
>> The specific scenario is a disaggregated system with the hardware domain
>> providing device models for muitple guest domains.
>
> Why the hardware domain? Unless a DM also needs access to some of the
> physical hardware, it ought to run in a separate domain. Conceivably
> such a domain could service multiply guests, so maybe the "single
> target" concept presently used for stubdom simply needed extending?
One configuration is the hardware domain running QEMU for the
virtio-gpu. In an earlier iteration, I allowed XSM_DM_PRIV for
is_hardware_domain(). Rightfully, there was some questioning of that
hardcoding. Adding a new flag allows it to be configurable.
Maybe target could be extended. I was thinking that could be left for
the stubdom case as it is today. i.e. a 1-1 device model. But a 1-N
case could be handled this way.
Today dom0 XSM_DM_PRIV falls through to is_control_domain(). The idea
was place a new check directly corresponding to XSM_DM_PRIV.
Regards,
Jason
next prev parent reply other threads:[~2025-06-11 17:15 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-10 22:57 [PATCH 0/4] XSM changes for split hardware / control domain Jason Andryuk
2025-06-10 22:57 ` [PATCH 1/4] xen/xsm: Add XSM_HW_PRIV Jason Andryuk
2025-06-11 13:02 ` Jan Beulich
2025-06-11 3:13 ` Jason Andryuk
2025-06-12 7:36 ` Jan Beulich
2025-06-12 17:31 ` Jason Andryuk
2025-06-10 22:57 ` [PATCH 2/4] xsm/silo: Support hwdom/control domains Jason Andryuk
2025-06-11 13:17 ` Jan Beulich
2025-06-11 4:20 ` Jason Andryuk
2025-06-12 7:52 ` Jan Beulich
2025-06-12 16:56 ` Jason Andryuk
2025-06-12 20:30 ` Jason Andryuk
2025-06-13 6:20 ` Jan Beulich
2025-06-10 22:57 ` [PATCH 3/4] xen: Add DOMAIN_CAPS_DEVICE_MODEL & XEN_DOMCTL_CDF_device_model Jason Andryuk
2025-06-11 8:25 ` Christian Lindig
2025-06-11 13:24 ` Jan Beulich
2025-06-11 4:35 ` Jason Andryuk [this message]
2025-06-13 22:47 ` Stefano Stabellini
2025-06-13 23:44 ` Demi Marie Obenour
2025-06-14 0:15 ` Stefano Stabellini
2025-06-16 5:58 ` Jan Beulich
2025-06-17 0:21 ` Stefano Stabellini
2025-06-10 22:57 ` [PATCH 4/4] xsm/dummy: Allow hwdom SYSCTL_readconsole/physinfo Jason Andryuk
2025-06-11 13:27 ` Jan Beulich
2025-06-11 4:48 ` Jason Andryuk
2025-06-13 22:51 ` Stefano Stabellini
2025-06-16 6:36 ` Jan Beulich
2025-06-17 0:10 ` Stefano Stabellini
2025-06-17 5:23 ` Jan Beulich
2025-06-19 0:36 ` Stefano Stabellini
2025-06-20 6:05 ` Jan Beulich
2025-07-07 21:52 ` Stefano Stabellini
2025-06-11 13:28 ` [PATCH 0/4] XSM changes for split hardware / control domain Jan Beulich
2025-06-11 5:08 ` Jason Andryuk
2025-06-12 7:33 ` Jan Beulich
2025-06-13 22:59 ` Stefano Stabellini
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=9819e041-3b4a-4e5f-b391-149e0900f286@amd.com \
--to=jason.andryuk@amd.com \
--cc=Volodymyr_Babchuk@epam.com \
--cc=andrew.cooper3@citrix.com \
--cc=anthony.perard@vates.tech \
--cc=bertrand.marquis@arm.com \
--cc=christian.lindig@citrix.com \
--cc=dave@recoil.org \
--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.