* [PATCH] Deprecate C virtiofsd
@ 2022-02-09 16:50 ` Dr. David Alan Gilbert (git)
0 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert (git) @ 2022-02-09 16:50 UTC (permalink / raw)
To: qemu-devel, virtio-fs; +Cc: stefanha, slp, vgoyal
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
There's a nice new Rust implementation out there; recommend people
do new work on that.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
---
docs/about/deprecated.rst | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index 47a594a3b6..3a0e15f8f5 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -454,3 +454,17 @@ nanoMIPS ISA
The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
As it is hard to generate binaries for it, declare it deprecated.
+
+Tools
+-----
+
+virtiofsd
+'''''''''
+
+There is a new Rust implementation of ``virtiofs`` at
+``https://gitlab.com/virtio-fs/virtiofsd``;
+since this is now marked stable, new development should be done on that
+rather than the existing C version in the QEMU tree.
+The C version will still accept fixes and patches that
+are already in development for the moment.
+
--
2.34.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-09 16:50 ` Dr. David Alan Gilbert (git)
@ 2022-02-09 20:02 ` Vivek Goyal
-1 siblings, 0 replies; 16+ messages in thread
From: Vivek Goyal @ 2022-02-09 20:02 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git); +Cc: virtio-fs, qemu-devel
On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> There's a nice new Rust implementation out there; recommend people
> do new work on that.
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Acked-by: Vivek Goyal <vgoyal@redhat.com>
Vivek
> ---
> docs/about/deprecated.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 47a594a3b6..3a0e15f8f5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -454,3 +454,17 @@ nanoMIPS ISA
>
> The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> As it is hard to generate binaries for it, declare it deprecated.
> +
> +Tools
> +-----
> +
> +virtiofsd
> +'''''''''
> +
> +There is a new Rust implementation of ``virtiofs`` at
> +``https://gitlab.com/virtio-fs/virtiofsd``;
> +since this is now marked stable, new development should be done on that
> +rather than the existing C version in the QEMU tree.
> +The C version will still accept fixes and patches that
> +are already in development for the moment.
> +
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-09 20:02 ` Vivek Goyal
0 siblings, 0 replies; 16+ messages in thread
From: Vivek Goyal @ 2022-02-09 20:02 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git); +Cc: virtio-fs, stefanha, qemu-devel, slp
On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> There's a nice new Rust implementation out there; recommend people
> do new work on that.
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Acked-by: Vivek Goyal <vgoyal@redhat.com>
Vivek
> ---
> docs/about/deprecated.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 47a594a3b6..3a0e15f8f5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -454,3 +454,17 @@ nanoMIPS ISA
>
> The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> As it is hard to generate binaries for it, declare it deprecated.
> +
> +Tools
> +-----
> +
> +virtiofsd
> +'''''''''
> +
> +There is a new Rust implementation of ``virtiofs`` at
> +``https://gitlab.com/virtio-fs/virtiofsd``;
> +since this is now marked stable, new development should be done on that
> +rather than the existing C version in the QEMU tree.
> +The C version will still accept fixes and patches that
> +are already in development for the moment.
> +
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-09 16:50 ` Dr. David Alan Gilbert (git)
@ 2022-02-09 20:47 ` Peter Maydell
-1 siblings, 0 replies; 16+ messages in thread
From: Peter Maydell @ 2022-02-09 20:47 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git); +Cc: virtio-fs, qemu-devel, vgoyal
On Wed, 9 Feb 2022 at 16:57, Dr. David Alan Gilbert (git)
<dgilbert@redhat.com> wrote:
>
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> There's a nice new Rust implementation out there; recommend people
> do new work on that.
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> docs/about/deprecated.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 47a594a3b6..3a0e15f8f5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -454,3 +454,17 @@ nanoMIPS ISA
>
> The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> As it is hard to generate binaries for it, declare it deprecated.
> +
> +Tools
> +-----
> +
> +virtiofsd
> +'''''''''
> +
> +There is a new Rust implementation of ``virtiofs`` at
I guess there's a missing "d" here ?
> +``https://gitlab.com/virtio-fs/virtiofsd``;
> +since this is now marked stable, new development should be done on that
> +rather than the existing C version in the QEMU tree.
> +The C version will still accept fixes and patches that
> +are already in development for the moment.
thanks
-- PMM
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-09 20:47 ` Peter Maydell
0 siblings, 0 replies; 16+ messages in thread
From: Peter Maydell @ 2022-02-09 20:47 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git); +Cc: virtio-fs, slp, qemu-devel, stefanha, vgoyal
On Wed, 9 Feb 2022 at 16:57, Dr. David Alan Gilbert (git)
<dgilbert@redhat.com> wrote:
>
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> There's a nice new Rust implementation out there; recommend people
> do new work on that.
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> docs/about/deprecated.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 47a594a3b6..3a0e15f8f5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -454,3 +454,17 @@ nanoMIPS ISA
>
> The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> As it is hard to generate binaries for it, declare it deprecated.
> +
> +Tools
> +-----
> +
> +virtiofsd
> +'''''''''
> +
> +There is a new Rust implementation of ``virtiofs`` at
I guess there's a missing "d" here ?
> +``https://gitlab.com/virtio-fs/virtiofsd``;
> +since this is now marked stable, new development should be done on that
> +rather than the existing C version in the QEMU tree.
> +The C version will still accept fixes and patches that
> +are already in development for the moment.
thanks
-- PMM
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-09 20:47 ` Peter Maydell
@ 2022-02-10 17:07 ` Dr. David Alan Gilbert
-1 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert @ 2022-02-10 17:07 UTC (permalink / raw)
To: Peter Maydell; +Cc: virtio-fs, qemu-devel, vgoyal
* Peter Maydell (peter.maydell@linaro.org) wrote:
> On Wed, 9 Feb 2022 at 16:57, Dr. David Alan Gilbert (git)
> <dgilbert@redhat.com> wrote:
> >
> > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> >
> > There's a nice new Rust implementation out there; recommend people
> > do new work on that.
> >
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> > docs/about/deprecated.rst | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > index 47a594a3b6..3a0e15f8f5 100644
> > --- a/docs/about/deprecated.rst
> > +++ b/docs/about/deprecated.rst
> > @@ -454,3 +454,17 @@ nanoMIPS ISA
> >
> > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > As it is hard to generate binaries for it, declare it deprecated.
> > +
> > +Tools
> > +-----
> > +
> > +virtiofsd
> > +'''''''''
> > +
> > +There is a new Rust implementation of ``virtiofs`` at
>
> I guess there's a missing "d" here ?
Oops yes, can add that.
Dave
> > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > +since this is now marked stable, new development should be done on that
> > +rather than the existing C version in the QEMU tree.
> > +The C version will still accept fixes and patches that
> > +are already in development for the moment.
>
> thanks
> -- PMM
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-10 17:07 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert @ 2022-02-10 17:07 UTC (permalink / raw)
To: Peter Maydell; +Cc: virtio-fs, slp, qemu-devel, stefanha, vgoyal
* Peter Maydell (peter.maydell@linaro.org) wrote:
> On Wed, 9 Feb 2022 at 16:57, Dr. David Alan Gilbert (git)
> <dgilbert@redhat.com> wrote:
> >
> > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> >
> > There's a nice new Rust implementation out there; recommend people
> > do new work on that.
> >
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> > docs/about/deprecated.rst | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > index 47a594a3b6..3a0e15f8f5 100644
> > --- a/docs/about/deprecated.rst
> > +++ b/docs/about/deprecated.rst
> > @@ -454,3 +454,17 @@ nanoMIPS ISA
> >
> > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > As it is hard to generate binaries for it, declare it deprecated.
> > +
> > +Tools
> > +-----
> > +
> > +virtiofsd
> > +'''''''''
> > +
> > +There is a new Rust implementation of ``virtiofs`` at
>
> I guess there's a missing "d" here ?
Oops yes, can add that.
Dave
> > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > +since this is now marked stable, new development should be done on that
> > +rather than the existing C version in the QEMU tree.
> > +The C version will still accept fixes and patches that
> > +are already in development for the moment.
>
> thanks
> -- PMM
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-09 16:50 ` Dr. David Alan Gilbert (git)
@ 2022-02-10 8:57 ` Daniel P. Berrangé
-1 siblings, 0 replies; 16+ messages in thread
From: Daniel P. Berrangé @ 2022-02-10 8:57 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git); +Cc: virtio-fs, qemu-devel, vgoyal
On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> There's a nice new Rust implementation out there; recommend people
> do new work on that.
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> docs/about/deprecated.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 47a594a3b6..3a0e15f8f5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -454,3 +454,17 @@ nanoMIPS ISA
>
> The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> As it is hard to generate binaries for it, declare it deprecated.
> +
> +Tools
> +-----
> +
> +virtiofsd
> +'''''''''
> +
> +There is a new Rust implementation of ``virtiofs`` at
> +``https://gitlab.com/virtio-fs/virtiofsd``;
> +since this is now marked stable, new development should be done on that
> +rather than the existing C version in the QEMU tree.
> +The C version will still accept fixes and patches that
> +are already in development for the moment.
Just to be clear, 'deprecation' is usually expected to turn into
'deletion' a minimum of 2 releases later. We are targetting the
C virtiofsd to be deleted, right ? The last sentance here gives
vibes that we're expecting it to stick around despite deprecation.
Assuming we're deleting it after deprecation, then I think thue
message should be more direct in telling people they shold be
proactively switching to deployment of the rust version. Something
along lines of
"The C implementation is in maintenance mode only and will
be deleted once the deprecation period is complete. New
deployments of virtiofs are strongly recommended to switch
to use of the Rust implementation of virtiofsd, which is
a drop in replacement will compatible command line and
featureset."
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-10 8:57 ` Daniel P. Berrangé
0 siblings, 0 replies; 16+ messages in thread
From: Daniel P. Berrangé @ 2022-02-10 8:57 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git); +Cc: virtio-fs, slp, qemu-devel, stefanha, vgoyal
On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
>
> There's a nice new Rust implementation out there; recommend people
> do new work on that.
>
> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> docs/about/deprecated.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> index 47a594a3b6..3a0e15f8f5 100644
> --- a/docs/about/deprecated.rst
> +++ b/docs/about/deprecated.rst
> @@ -454,3 +454,17 @@ nanoMIPS ISA
>
> The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> As it is hard to generate binaries for it, declare it deprecated.
> +
> +Tools
> +-----
> +
> +virtiofsd
> +'''''''''
> +
> +There is a new Rust implementation of ``virtiofs`` at
> +``https://gitlab.com/virtio-fs/virtiofsd``;
> +since this is now marked stable, new development should be done on that
> +rather than the existing C version in the QEMU tree.
> +The C version will still accept fixes and patches that
> +are already in development for the moment.
Just to be clear, 'deprecation' is usually expected to turn into
'deletion' a minimum of 2 releases later. We are targetting the
C virtiofsd to be deleted, right ? The last sentance here gives
vibes that we're expecting it to stick around despite deprecation.
Assuming we're deleting it after deprecation, then I think thue
message should be more direct in telling people they shold be
proactively switching to deployment of the rust version. Something
along lines of
"The C implementation is in maintenance mode only and will
be deleted once the deprecation period is complete. New
deployments of virtiofs are strongly recommended to switch
to use of the Rust implementation of virtiofsd, which is
a drop in replacement will compatible command line and
featureset."
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-10 8:57 ` Daniel P. Berrangé
@ 2022-02-10 17:13 ` Dr. David Alan Gilbert
-1 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert @ 2022-02-10 17:13 UTC (permalink / raw)
To: Daniel P. Berrangé; +Cc: virtio-fs, qemu-devel, vgoyal
* Daniel P. Berrangé (berrange@redhat.com) wrote:
> On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> >
> > There's a nice new Rust implementation out there; recommend people
> > do new work on that.
> >
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> > docs/about/deprecated.rst | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > index 47a594a3b6..3a0e15f8f5 100644
> > --- a/docs/about/deprecated.rst
> > +++ b/docs/about/deprecated.rst
> > @@ -454,3 +454,17 @@ nanoMIPS ISA
> >
> > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > As it is hard to generate binaries for it, declare it deprecated.
> > +
> > +Tools
> > +-----
> > +
> > +virtiofsd
> > +'''''''''
> > +
> > +There is a new Rust implementation of ``virtiofs`` at
> > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > +since this is now marked stable, new development should be done on that
> > +rather than the existing C version in the QEMU tree.
> > +The C version will still accept fixes and patches that
> > +are already in development for the moment.
>
> Just to be clear, 'deprecation' is usually expected to turn into
> 'deletion' a minimum of 2 releases later. We are targetting the
> C virtiofsd to be deleted, right ? The last sentance here gives
> vibes that we're expecting it to stick around despite deprecation.
I am targeting it to be deleted, but I'm expecting to give it a bit
more time than 2 releases; how do I do that?
> Assuming we're deleting it after deprecation, then I think thue
> message should be more direct in telling people they shold be
> proactively switching to deployment of the rust version. Something
> along lines of
>
> "The C implementation is in maintenance mode only and will
> be deleted once the deprecation period is complete. New
> deployments of virtiofs are strongly recommended to switch
> to use of the Rust implementation of virtiofsd, which is
> a drop in replacement will compatible command line and
> featureset."
Yeh, I'll rework it in that direction; I was trying to be gentle at
first.
Dave
>
> Regards,
> Daniel
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-10 17:13 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert @ 2022-02-10 17:13 UTC (permalink / raw)
To: Daniel P. Berrangé; +Cc: virtio-fs, slp, qemu-devel, stefanha, vgoyal
* Daniel P. Berrangé (berrange@redhat.com) wrote:
> On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> >
> > There's a nice new Rust implementation out there; recommend people
> > do new work on that.
> >
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> > docs/about/deprecated.rst | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > index 47a594a3b6..3a0e15f8f5 100644
> > --- a/docs/about/deprecated.rst
> > +++ b/docs/about/deprecated.rst
> > @@ -454,3 +454,17 @@ nanoMIPS ISA
> >
> > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > As it is hard to generate binaries for it, declare it deprecated.
> > +
> > +Tools
> > +-----
> > +
> > +virtiofsd
> > +'''''''''
> > +
> > +There is a new Rust implementation of ``virtiofs`` at
> > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > +since this is now marked stable, new development should be done on that
> > +rather than the existing C version in the QEMU tree.
> > +The C version will still accept fixes and patches that
> > +are already in development for the moment.
>
> Just to be clear, 'deprecation' is usually expected to turn into
> 'deletion' a minimum of 2 releases later. We are targetting the
> C virtiofsd to be deleted, right ? The last sentance here gives
> vibes that we're expecting it to stick around despite deprecation.
I am targeting it to be deleted, but I'm expecting to give it a bit
more time than 2 releases; how do I do that?
> Assuming we're deleting it after deprecation, then I think thue
> message should be more direct in telling people they shold be
> proactively switching to deployment of the rust version. Something
> along lines of
>
> "The C implementation is in maintenance mode only and will
> be deleted once the deprecation period is complete. New
> deployments of virtiofs are strongly recommended to switch
> to use of the Rust implementation of virtiofsd, which is
> a drop in replacement will compatible command line and
> featureset."
Yeh, I'll rework it in that direction; I was trying to be gentle at
first.
Dave
>
> Regards,
> Daniel
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-10 17:13 ` Dr. David Alan Gilbert
@ 2022-02-10 17:17 ` Daniel P. Berrangé
-1 siblings, 0 replies; 16+ messages in thread
From: Daniel P. Berrangé @ 2022-02-10 17:17 UTC (permalink / raw)
To: Dr. David Alan Gilbert; +Cc: virtio-fs, qemu-devel, vgoyal
On Thu, Feb 10, 2022 at 05:13:19PM +0000, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrangé (berrange@redhat.com) wrote:
> > On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> > >
> > > There's a nice new Rust implementation out there; recommend people
> > > do new work on that.
> > >
> > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > > ---
> > > docs/about/deprecated.rst | 14 ++++++++++++++
> > > 1 file changed, 14 insertions(+)
> > >
> > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > > index 47a594a3b6..3a0e15f8f5 100644
> > > --- a/docs/about/deprecated.rst
> > > +++ b/docs/about/deprecated.rst
> > > @@ -454,3 +454,17 @@ nanoMIPS ISA
> > >
> > > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > > As it is hard to generate binaries for it, declare it deprecated.
> > > +
> > > +Tools
> > > +-----
> > > +
> > > +virtiofsd
> > > +'''''''''
> > > +
> > > +There is a new Rust implementation of ``virtiofs`` at
> > > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > > +since this is now marked stable, new development should be done on that
> > > +rather than the existing C version in the QEMU tree.
> > > +The C version will still accept fixes and patches that
> > > +are already in development for the moment.
> >
> > Just to be clear, 'deprecation' is usually expected to turn into
> > 'deletion' a minimum of 2 releases later. We are targetting the
> > C virtiofsd to be deleted, right ? The last sentance here gives
> > vibes that we're expecting it to stick around despite deprecation.
>
> I am targeting it to be deleted, but I'm expecting to give it a bit
> more time than 2 releases; how do I do that?
You can forget to remove it after 2 releases, which is what
happens with many deprecations :-) '2' is really just the
minimum bar in practice.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-10 17:17 ` Daniel P. Berrangé
0 siblings, 0 replies; 16+ messages in thread
From: Daniel P. Berrangé @ 2022-02-10 17:17 UTC (permalink / raw)
To: Dr. David Alan Gilbert; +Cc: virtio-fs, slp, qemu-devel, stefanha, vgoyal
On Thu, Feb 10, 2022 at 05:13:19PM +0000, Dr. David Alan Gilbert wrote:
> * Daniel P. Berrangé (berrange@redhat.com) wrote:
> > On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> > >
> > > There's a nice new Rust implementation out there; recommend people
> > > do new work on that.
> > >
> > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > > ---
> > > docs/about/deprecated.rst | 14 ++++++++++++++
> > > 1 file changed, 14 insertions(+)
> > >
> > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > > index 47a594a3b6..3a0e15f8f5 100644
> > > --- a/docs/about/deprecated.rst
> > > +++ b/docs/about/deprecated.rst
> > > @@ -454,3 +454,17 @@ nanoMIPS ISA
> > >
> > > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > > As it is hard to generate binaries for it, declare it deprecated.
> > > +
> > > +Tools
> > > +-----
> > > +
> > > +virtiofsd
> > > +'''''''''
> > > +
> > > +There is a new Rust implementation of ``virtiofs`` at
> > > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > > +since this is now marked stable, new development should be done on that
> > > +rather than the existing C version in the QEMU tree.
> > > +The C version will still accept fixes and patches that
> > > +are already in development for the moment.
> >
> > Just to be clear, 'deprecation' is usually expected to turn into
> > 'deletion' a minimum of 2 releases later. We are targetting the
> > C virtiofsd to be deleted, right ? The last sentance here gives
> > vibes that we're expecting it to stick around despite deprecation.
>
> I am targeting it to be deleted, but I'm expecting to give it a bit
> more time than 2 releases; how do I do that?
You can forget to remove it after 2 releases, which is what
happens with many deprecations :-) '2' is really just the
minimum bar in practice.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Virtio-fs] [PATCH] Deprecate C virtiofsd
2022-02-10 17:17 ` Daniel P. Berrangé
@ 2022-02-10 17:27 ` Dr. David Alan Gilbert
-1 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert @ 2022-02-10 17:27 UTC (permalink / raw)
To: Daniel P. Berrangé; +Cc: virtio-fs, qemu-devel, vgoyal
* Daniel P. Berrangé (berrange@redhat.com) wrote:
> On Thu, Feb 10, 2022 at 05:13:19PM +0000, Dr. David Alan Gilbert wrote:
> > * Daniel P. Berrangé (berrange@redhat.com) wrote:
> > > On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> > > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> > > >
> > > > There's a nice new Rust implementation out there; recommend people
> > > > do new work on that.
> > > >
> > > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > > > ---
> > > > docs/about/deprecated.rst | 14 ++++++++++++++
> > > > 1 file changed, 14 insertions(+)
> > > >
> > > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > > > index 47a594a3b6..3a0e15f8f5 100644
> > > > --- a/docs/about/deprecated.rst
> > > > +++ b/docs/about/deprecated.rst
> > > > @@ -454,3 +454,17 @@ nanoMIPS ISA
> > > >
> > > > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > > > As it is hard to generate binaries for it, declare it deprecated.
> > > > +
> > > > +Tools
> > > > +-----
> > > > +
> > > > +virtiofsd
> > > > +'''''''''
> > > > +
> > > > +There is a new Rust implementation of ``virtiofs`` at
> > > > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > > > +since this is now marked stable, new development should be done on that
> > > > +rather than the existing C version in the QEMU tree.
> > > > +The C version will still accept fixes and patches that
> > > > +are already in development for the moment.
> > >
> > > Just to be clear, 'deprecation' is usually expected to turn into
> > > 'deletion' a minimum of 2 releases later. We are targetting the
> > > C virtiofsd to be deleted, right ? The last sentance here gives
> > > vibes that we're expecting it to stick around despite deprecation.
> >
> > I am targeting it to be deleted, but I'm expecting to give it a bit
> > more time than 2 releases; how do I do that?
>
> You can forget to remove it after 2 releases, which is what
> happens with many deprecations :-) '2' is really just the
> minimum bar in practice.
OK, I can remember to forget.
Dave
> Regards,
> Daniel
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] Deprecate C virtiofsd
@ 2022-02-10 17:27 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 16+ messages in thread
From: Dr. David Alan Gilbert @ 2022-02-10 17:27 UTC (permalink / raw)
To: Daniel P. Berrangé; +Cc: virtio-fs, slp, qemu-devel, stefanha, vgoyal
* Daniel P. Berrangé (berrange@redhat.com) wrote:
> On Thu, Feb 10, 2022 at 05:13:19PM +0000, Dr. David Alan Gilbert wrote:
> > * Daniel P. Berrangé (berrange@redhat.com) wrote:
> > > On Wed, Feb 09, 2022 at 04:50:40PM +0000, Dr. David Alan Gilbert (git) wrote:
> > > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
> > > >
> > > > There's a nice new Rust implementation out there; recommend people
> > > > do new work on that.
> > > >
> > > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > > > ---
> > > > docs/about/deprecated.rst | 14 ++++++++++++++
> > > > 1 file changed, 14 insertions(+)
> > > >
> > > > diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
> > > > index 47a594a3b6..3a0e15f8f5 100644
> > > > --- a/docs/about/deprecated.rst
> > > > +++ b/docs/about/deprecated.rst
> > > > @@ -454,3 +454,17 @@ nanoMIPS ISA
> > > >
> > > > The ``nanoMIPS`` ISA has never been upstreamed to any compiler toolchain.
> > > > As it is hard to generate binaries for it, declare it deprecated.
> > > > +
> > > > +Tools
> > > > +-----
> > > > +
> > > > +virtiofsd
> > > > +'''''''''
> > > > +
> > > > +There is a new Rust implementation of ``virtiofs`` at
> > > > +``https://gitlab.com/virtio-fs/virtiofsd``;
> > > > +since this is now marked stable, new development should be done on that
> > > > +rather than the existing C version in the QEMU tree.
> > > > +The C version will still accept fixes and patches that
> > > > +are already in development for the moment.
> > >
> > > Just to be clear, 'deprecation' is usually expected to turn into
> > > 'deletion' a minimum of 2 releases later. We are targetting the
> > > C virtiofsd to be deleted, right ? The last sentance here gives
> > > vibes that we're expecting it to stick around despite deprecation.
> >
> > I am targeting it to be deleted, but I'm expecting to give it a bit
> > more time than 2 releases; how do I do that?
>
> You can forget to remove it after 2 releases, which is what
> happens with many deprecations :-) '2' is really just the
> minimum bar in practice.
OK, I can remember to forget.
Dave
> Regards,
> Daniel
> --
> |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o- https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 16+ messages in thread