1. Patchew
  2. Xen
  3. [PATCH v3 0/9] Enable Xenstore-stubdom Live Update
  4. View patch

[PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update

Juergen Gross posted 9 patches 3 months ago
[PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Juergen Gross 3 months ago 
Live update is now working with the PVH variant of xenstore-stubdom.

Signed-off-by: Juergen Gross <jgross@suse.com>
---
V2:
- new patch
---
 SUPPORT.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/SUPPORT.md b/SUPPORT.md
index 6a82a92189..eb44ee85fd 100644
--- a/SUPPORT.md
+++ b/SUPPORT.md
@@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
 ### C xenstore stubdom PVH
 
     Status: Supported
-    Status, Liveupdate: Not implemented
+    Status, Liveupdate: Supported
 
 ### OCaml xenstored daemon
 
-- 
2.43.0
Re: [PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Julien Grall 1 month, 2 weeks ago 
Hi Juergen,

Sorry for the late answer.

On 30/07/2025 13:23, Juergen Gross wrote:
> Live update is now working with the PVH variant of xenstore-stubdom.
 > > Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
> V2:
> - new patch
> ---
>   SUPPORT.md | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/SUPPORT.md b/SUPPORT.md
> index 6a82a92189..eb44ee85fd 100644
> --- a/SUPPORT.md
> +++ b/SUPPORT.md
> @@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
>   ### C xenstore stubdom PVH
>   
>       Status: Supported
> -    Status, Liveupdate: Not implemented
> +    Status, Liveupdate: Supported

I went through the changes in Xenstored for the stubdom and I have one 
question regarding [1]. Does this mean the event channel will be closed 
during Live-Update? Asking because I vaguely remember we need to keep 
them open to avoid losing events. I am wondering how this would work in 
stubdom?

Cheers,

[1] https://lore.kernel.org/all/20250730122305.4050-7-jgross@suse.com/

>   
>   ### OCaml xenstored daemon
>   

-- 
Julien Grall
Re: [PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Jürgen Groß 1 month, 2 weeks ago 
On 12.09.25 23:09, Julien Grall wrote:
> Hi Juergen,
> 
> Sorry for the late answer.
> 
> On 30/07/2025 13:23, Juergen Gross wrote:
>> Live update is now working with the PVH variant of xenstore-stubdom.
>  > > Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>> V2:
>> - new patch
>> ---
>>   SUPPORT.md | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/SUPPORT.md b/SUPPORT.md
>> index 6a82a92189..eb44ee85fd 100644
>> --- a/SUPPORT.md
>> +++ b/SUPPORT.md
>> @@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
>>   ### C xenstore stubdom PVH
>>       Status: Supported
>> -    Status, Liveupdate: Not implemented
>> +    Status, Liveupdate: Supported
> 
> I went through the changes in Xenstored for the stubdom and I have one question 
> regarding [1]. Does this mean the event channel will be closed during Live- 
> Update? Asking because I vaguely remember we need to keep them open to avoid 
> losing events. I am wondering how this would work in stubdom?

The event channel is still open, its just the Mini-OS internals that don't
know about this. This is the reason for calling xenevtchn_bind() to bring
hypervisor and Mini-OS view in sync again, without changing the hypervisor
side.


Juergen
Re: [PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Julien Grall 1 month, 2 weeks ago 
Hi Juergen,

On 13/09/2025 08:16, Jürgen Groß wrote:
> On 12.09.25 23:09, Julien Grall wrote:
>> Hi Juergen,
>>
>> Sorry for the late answer.
>>
>> On 30/07/2025 13:23, Juergen Gross wrote:
>>> Live update is now working with the PVH variant of xenstore-stubdom.
>>  > > Signed-off-by: Juergen Gross <jgross@suse.com>
>>> ---
>>> V2:
>>> - new patch
>>> ---
>>>   SUPPORT.md | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/SUPPORT.md b/SUPPORT.md
>>> index 6a82a92189..eb44ee85fd 100644
>>> --- a/SUPPORT.md
>>> +++ b/SUPPORT.md
>>> @@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
>>>   ### C xenstore stubdom PVH
>>>       Status: Supported
>>> -    Status, Liveupdate: Not implemented
>>> +    Status, Liveupdate: Supported
>>
>> I went through the changes in Xenstored for the stubdom and I have one 
>> question regarding [1]. Does this mean the event channel will be 
>> closed during Live- Update? Asking because I vaguely remember we need 
>> to keep them open to avoid losing events. I am wondering how this 
>> would work in stubdom?
> 
> The event channel is still open, its just the Mini-OS internals that don't
> know about this. This is the reason for calling xenevtchn_bind() to bring
> hypervisor and Mini-OS view in sync again, without changing the hypervisor
> side.

Thanks for the clarification.

Acked-by: Julien Grall <jgrall@amazon.com>

Cheers,
-- 
Julien Grall
Re: [PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Julien Grall 1 month, 2 weeks ago 

On 12/09/2025 22:09, Julien Grall wrote:
> Hi Juergen,
> 
> Sorry for the late answer.
> 
> On 30/07/2025 13:23, Juergen Gross wrote:
>> Live update is now working with the PVH variant of xenstore-stubdom.
>  > > Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>> V2:
>> - new patch
>> ---
>>   SUPPORT.md | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/SUPPORT.md b/SUPPORT.md
>> index 6a82a92189..eb44ee85fd 100644
>> --- a/SUPPORT.md
>> +++ b/SUPPORT.md
>> @@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
>>   ### C xenstore stubdom PVH
>>       Status: Supported
>> -    Status, Liveupdate: Not implemented
>> +    Status, Liveupdate: Supported
> 
> I went through the changes in Xenstored for the stubdom and I have one 
> question regarding [1]. Does this mean the event channel will be closed 
> during Live-Update? Asking because I vaguely remember we need to keep 
> them open to avoid losing events. I am wondering how this would work in 
> stubdom?

I forgot to mention, this is the only thing blocking my ack for this patch.

Cheers,

-- 
Julien Grall
Re: [PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Juergen Gross 2 months, 1 week ago 
Ping?

On 30.07.25 14:23, Juergen Gross wrote:
> Live update is now working with the PVH variant of xenstore-stubdom.
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
> V2:
> - new patch
> ---
>   SUPPORT.md | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/SUPPORT.md b/SUPPORT.md
> index 6a82a92189..eb44ee85fd 100644
> --- a/SUPPORT.md
> +++ b/SUPPORT.md
> @@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
>   ### C xenstore stubdom PVH
>   
>       Status: Supported
> -    Status, Liveupdate: Not implemented
> +    Status, Liveupdate: Supported
>   
>   ### OCaml xenstored daemon
>
Re: [PATCH v3 8/9] SUPPORT.md: add xenstorepvh-stubdom live update
Posted by Juergen Gross 1 month, 3 weeks ago 
On 8/20/25 08:05, Juergen Gross wrote:
> Ping?
> 
> On 30.07.25 14:23, Juergen Gross wrote:
>> Live update is now working with the PVH variant of xenstore-stubdom.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>> V2:
>> - new patch
>> ---
>>   SUPPORT.md | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/SUPPORT.md b/SUPPORT.md
>> index 6a82a92189..eb44ee85fd 100644
>> --- a/SUPPORT.md
>> +++ b/SUPPORT.md
>> @@ -280,7 +280,7 @@ or itself will not be regarded a security issue.
>>   ### C xenstore stubdom PVH
>>       Status: Supported
>> -    Status, Liveupdate: Not implemented
>> +    Status, Liveupdate: Supported
>>   ### OCaml xenstored daemon
> 

And 40 days after sending out: PING?


Juergen

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.