Re: [Qemu-devel] [PATCH for-3.1?] 9p: remove support for the "handle" backend

[Greg Kurz <groug@kaod.org>]

On Fri, 30 Nov 2018 17:00:29 +0000
Peter Maydell <peter.maydell@linaro.org> wrote:

> On Fri, 30 Nov 2018 at 16:45, Greg Kurz <groug@kaod.org> wrote:
> >
> > On Fri, 30 Nov 2018 09:52:18 -0600
> > Eric Blake <eblake@redhat.com> wrote:
> >  
> > > On 11/30/18 2:49 AM, Greg Kurz wrote:  
> > > > The "handle" fsdev backend was deprecated in QEMU 2.12.0 with:
> > > >
> > > > commit db3b3c7281ca82e2647e072a1f97db111313dd73
> > > > Author: Greg Kurz <groug@kaod.org>
> > > > Date:   Mon Jan 8 11:18:23 2018 +0100
> > > >
> > > >      9pfs: deprecate handle backend  
> > >  
> > > > It has passed the two release cooling period without any complaint.
> > > >
> > > > Remove it now.
> > > >
> > > > Signed-off-by: Greg Kurz <groug@kaod.org>
> > > > ---
> > > >
> > > > It could have been removed for 3.1.0, but I woke up too late, so moving
> > > > this to 4.0.
> > > >
> > > > ---
> > > >   fsdev/qemu-fsdev.c    |    3
> > > >   hw/9pfs/9p-handle.c   |  710 -------------------------------------------------
> > > >   hw/9pfs/Makefile.objs |    1
> > > >   qemu-deprecated.texi  |    8 -
> > > >   qemu-options.hx       |    8 -
> > > >   5 files changed, 4 insertions(+), 726 deletions(-)
> > > >   delete mode 100644 hw/9pfs/9p-handle.c  
> > >
> > > Well, it certainly doesn't drive -rc4 on its own, but as we may have an
> > > -rc4 for other reasons, maybe it could still be argued that this is 3.1
> > > material on the grounds of removing questionable code for less attack
> > > exposure?
> > >  
> >
> > Makes sense, as this is precisely the reason that led me to
> > deprecate this in the first place.  
> 
> I would prefer not to have a 700-line patch in rc4, even
> a negative-diffstat one. If it's not a definite problem
> then we shouldn't put the change in at this stage in the
> release process.
> 

I understand, and it's ok for me if this gets merged in 4.0.

Cheers,

--
Greg

> thanks
> -- PMM