| Message ID | 20220209165040.56062-1-dgilbert@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Deprecate C virtiofsd | expand |
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 >
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
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
* 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 >
* 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 :| >
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
* 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 :| >
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. +