[PATCH 1/4] xen/public: introduce DOMID_ANY

Juergen Gross posted 4 patches 3 months, 3 weeks ago
There is a newer version of this series
[PATCH 1/4] xen/public: introduce DOMID_ANY
Posted by Juergen Gross 3 months, 3 weeks ago
Add DOMID_ANY to xen/include/public/xen.h meant to be a wildcard for
domids.

Signed-off-by: Denis Mukhin <dmukhin@ford.com>
Signed-off-by: Juergen Gross <jgross@suse.com>
---
This is based on Denis Mukhin's patch "xen/domain: introduce DOMID_ANY".
As my series is another use case for DOMID_ANY and it is a backport
candidate, I've split out the definition of DOMID_ANY from Denis'
patch in order to make progress for my series.
---
 xen/include/public/xen.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h
index b12fd10e63..f35a6f21f0 100644
--- a/xen/include/public/xen.h
+++ b/xen/include/public/xen.h
@@ -608,6 +608,13 @@ DEFINE_XEN_GUEST_HANDLE(mmuext_op_t);
 /* DOMID_INVALID is used to identify pages with unknown owner. */
 #define DOMID_INVALID        xen_mk_uint(0x7FF4)
 
+/*
+ * DOMID_ANY is used to signal no specific domain ID requested.
+ * Handler should pick a valid ID, or handle it as a broadcast value
+ * depending on the context.
+ */
+#define DOMID_ANY            xen_mk_uint(0x7FF5)
+
 /* Idle domain. */
 #define DOMID_IDLE           xen_mk_uint(0x7FFF)
 
-- 
2.53.0
Re: [PATCH 1/4] xen/public: introduce DOMID_ANY
Posted by Jason Andryuk 3 months, 2 weeks ago
On 2026-04-23 04:08, Juergen Gross wrote:
> Add DOMID_ANY to xen/include/public/xen.h meant to be a wildcard for
> domids.
> 
> Signed-off-by: Denis Mukhin <dmukhin@ford.com>
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
> This is based on Denis Mukhin's patch "xen/domain: introduce DOMID_ANY".
> As my series is another use case for DOMID_ANY and it is a backport
> candidate, I've split out the definition of DOMID_ANY from Denis'
> patch in order to make progress for my series.

I think we probably want Denis as the Author, since this is an 
unmodified piece of his patch.

> ---
>   xen/include/public/xen.h | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h
> index b12fd10e63..f35a6f21f0 100644
> --- a/xen/include/public/xen.h
> +++ b/xen/include/public/xen.h
> @@ -608,6 +608,13 @@ DEFINE_XEN_GUEST_HANDLE(mmuext_op_t);
>   /* DOMID_INVALID is used to identify pages with unknown owner. */
>   #define DOMID_INVALID        xen_mk_uint(0x7FF4)
>   
> +/*
> + * DOMID_ANY is used to signal no specific domain ID requested.
> + * Handler should pick a valid ID, or handle it as a broadcast value

Maybe s/broadcast/wildcard/?

Either way:

Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>
Re: [PATCH 1/4] xen/public: introduce DOMID_ANY
Posted by Jürgen Groß 3 months, 2 weeks ago
On 28.04.26 00:00, Jason Andryuk wrote:
> On 2026-04-23 04:08, Juergen Gross wrote:
>> Add DOMID_ANY to xen/include/public/xen.h meant to be a wildcard for
>> domids.
>>
>> Signed-off-by: Denis Mukhin <dmukhin@ford.com>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>> This is based on Denis Mukhin's patch "xen/domain: introduce DOMID_ANY".
>> As my series is another use case for DOMID_ANY and it is a backport
>> candidate, I've split out the definition of DOMID_ANY from Denis'
>> patch in order to make progress for my series.
> 
> I think we probably want Denis as the Author, since this is an unmodified piece 
> of his patch.

Apart from the commit message, yes.

Keeping Denis as the author is fine with me.

>> ---
>>   xen/include/public/xen.h | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h
>> index b12fd10e63..f35a6f21f0 100644
>> --- a/xen/include/public/xen.h
>> +++ b/xen/include/public/xen.h
>> @@ -608,6 +608,13 @@ DEFINE_XEN_GUEST_HANDLE(mmuext_op_t);
>>   /* DOMID_INVALID is used to identify pages with unknown owner. */
>>   #define DOMID_INVALID        xen_mk_uint(0x7FF4)
>> +/*
>> + * DOMID_ANY is used to signal no specific domain ID requested.
>> + * Handler should pick a valid ID, or handle it as a broadcast value
> 
> Maybe s/broadcast/wildcard/?

Yes, sounds better.

> 
> Either way:
> 
> Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>

Thanks,


Juergen
Re: [PATCH 1/4] xen/public: introduce DOMID_ANY
Posted by dmukhin@ford.com 3 months, 1 week ago
On Tue, Apr 28, 2026 at 03:36:20PM +0200, Jürgen Groß wrote:
> On 28.04.26 00:00, Jason Andryuk wrote:
> > On 2026-04-23 04:08, Juergen Gross wrote:
> > > Add DOMID_ANY to xen/include/public/xen.h meant to be a wildcard for
> > > domids.
> > > 
> > > Signed-off-by: Denis Mukhin <dmukhin@ford.com>
> > > Signed-off-by: Juergen Gross <jgross@suse.com>
> > > ---
> > > This is based on Denis Mukhin's patch "xen/domain: introduce DOMID_ANY".
> > > As my series is another use case for DOMID_ANY and it is a backport
> > > candidate, I've split out the definition of DOMID_ANY from Denis'
> > > patch in order to make progress for my series.
> > 
> > I think we probably want Denis as the Author, since this is an
> > unmodified piece of his patch.
> 
> Apart from the commit message, yes.
> 
> Keeping Denis as the author is fine with me.

Thanks for taking care of this!

--
Denis