diff mbox series

[for-4.15,v2] SUPPORT.MD: Mark LiveUpdate of C XenStored daemon as Tech Preview

Message ID 20210317112745.15207-1-julien@xen.org (mailing list archive)
State New, archived
Headers show
Series [for-4.15,v2] SUPPORT.MD: Mark LiveUpdate of C XenStored daemon as Tech Preview | expand

Commit Message

Julien Grall March 17, 2021, 11:27 a.m. UTC 
From: Julien Grall <jgrall@amazon.com>

Support to liveupdate C XenStored daemon was added during the 4.15
development cycle. Add a section in SUPPORT.MD to explain what is the
support state.

For now, it is a tech preview.

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

---

CC: Juergen Gross <jgross@suse.com>

    Changes in v2:
        - Clarify this is only supported for the daemon version
        - Fix typoes
---
 SUPPORT.md | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Jürgen Groß March 17, 2021, 11:43 a.m. UTC | #1 
On 17.03.21 12:27, Julien Grall wrote:
> From: Julien Grall <jgrall@amazon.com>
> 
> Support to liveupdate C XenStored daemon was added during the 4.15
> development cycle. Add a section in SUPPORT.MD to explain what is the
> support state.
> 
> For now, it is a tech preview.
> 
> Signed-off-by: Julien Grall <jgrall@amazon.com>

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen
Andrew Cooper March 17, 2021, 11:49 a.m. UTC | #2 
On 17/03/2021 11:27, Julien Grall wrote:
> From: Julien Grall <jgrall@amazon.com>
>
> Support to liveupdate C XenStored daemon was added during the 4.15
> development cycle. Add a section in SUPPORT.MD to explain what is the
> support state.
>
> For now, it is a tech preview.
>
> Signed-off-by: Julien Grall <jgrall@amazon.com>

What about oxenstored ?

~Andrew
Julien Grall March 17, 2021, 11:54 a.m. UTC | #3 
On 17/03/2021 11:49, Andrew Cooper wrote:
> On 17/03/2021 11:27, Julien Grall wrote:
>> From: Julien Grall <jgrall@amazon.com>
>>
>> Support to liveupdate C XenStored daemon was added during the 4.15
>> development cycle. Add a section in SUPPORT.MD to explain what is the
>> support state.
>>
>> For now, it is a tech preview.
>>
>> Signed-off-by: Julien Grall <jgrall@amazon.com>
> 
> What about oxenstored ?

Oh, I read your answer as there is no support. I can resend a patch with 
the following title:

"LiveUpdate of C/Ocaml XenStored daemon"

Cheers,
Andrew Cooper March 17, 2021, 11:59 a.m. UTC | #4 
On 17/03/2021 11:54, Julien Grall wrote:
> On 17/03/2021 11:49, Andrew Cooper wrote:
>> On 17/03/2021 11:27, Julien Grall wrote:
>>> From: Julien Grall <jgrall@amazon.com>
>>>
>>> Support to liveupdate C XenStored daemon was added during the 4.15
>>> development cycle. Add a section in SUPPORT.MD to explain what is the
>>> support state.
>>>
>>> For now, it is a tech preview.
>>>
>>> Signed-off-by: Julien Grall <jgrall@amazon.com>
>>
>> What about oxenstored ?
>
> Oh, I read your answer as there is no support. I can resend a patch
> with the following title:
>
> "LiveUpdate of C/Ocaml XenStored daemon"

I'd put it at tech preview, just like Cxenstored.  In particular, it has
the same issues concerning outstanding transactions.

It might however be worth having separate line items in SUPPORT.md to
start with, just in case the different implementations progress to
supported at different times.  (More likely, as there isn't an embargo
adding urgency the work now.)

~Andrew
Julien Grall March 17, 2021, 12:06 p.m. UTC | #5 
Hi Andrew,

On 17/03/2021 11:59, Andrew Cooper wrote:
> On 17/03/2021 11:54, Julien Grall wrote:
>> On 17/03/2021 11:49, Andrew Cooper wrote:
>>> On 17/03/2021 11:27, Julien Grall wrote:
>>>> From: Julien Grall <jgrall@amazon.com>
>>>>
>>>> Support to liveupdate C XenStored daemon was added during the 4.15
>>>> development cycle. Add a section in SUPPORT.MD to explain what is the
>>>> support state.
>>>>
>>>> For now, it is a tech preview.
>>>>
>>>> Signed-off-by: Julien Grall <jgrall@amazon.com>
>>>
>>> What about oxenstored ?
>>
>> Oh, I read your answer as there is no support. I can resend a patch
>> with the following title:
>>
>> "LiveUpdate of C/Ocaml XenStored daemon"
> 
> I'd put it at tech preview, just like Cxenstored.  In particular, it has
> the same issues concerning outstanding transactions.

I have some work planned for C XenStored to mitigate that as Paul 
recently discovered that some Windows PV drivers may leak transactions 
(see [1]).

> It might however be worth having separate line items in SUPPORT.md to
> start with, just in case the different implementations progress to
> supported at different times.  (More likely, as there isn't an embargo
> adding urgency the work now.)

Sure. Let me respin the patch.

Cheers,


[1] 
https://xenbits.xen.org/gitweb/?p=pvdrivers/win/xenbus.git;a=commit;h=4d3c233a51aef91d464db4d295a1d76ef774a27d
Ian Jackson March 20, 2021, 1:58 p.m. UTC | #6 
Julien Grall writes ("[PATCH for-4.15 v2] SUPPORT.MD: Mark LiveUpdate of C XenStored daemon as Tech Preview"):
> From: Julien Grall <jgrall@amazon.com>
> 
> Support to liveupdate C XenStored daemon was added during the 4.15
> development cycle. Add a section in SUPPORT.MD to explain what is the
> support state.
> 
> For now, it is a tech preview.

Release-Acked-by: Ian Jackson <iwj@xenproject.org>
diff mbox series

Patch

diff --git a/SUPPORT.md b/SUPPORT.md
index 7db4568f1a0f..4c93afa3613b 100644
--- a/SUPPORT.md
+++ b/SUPPORT.md
@@ -177,6 +177,10 @@  Support for running qemu-xen device model in a linux stubdomain.
 
     Status: Tech Preview
 
+## Liveupdate of C XenStored daemon
+
+    Status: Tech Preview
+
 ## Toolstack/3rd party
 
 ### libvirt driver for xl