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

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

[Julien Grall]

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(+)

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
-- 
2.17.1

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

[Jürgen Groß]

[-- Attachment #1.1.1: Type: text/plain, Size: 401 bytes --]

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

[-- Attachment #1.1.2: OpenPGP_0xB0DE9DD628BF132F.asc --]
[-- Type: application/pgp-keys, Size: 3135 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

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

[Andrew Cooper]

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

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

[Julien Grall]

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,

-- 
Julien Grall

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

[Andrew Cooper]

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

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

[Julien Grall]

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

-- 
Julien Grall

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

[Ian Jackson]

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>