[Virtio-fs] [PATCH] virtiofsd: fix typo in error message in fv_queue_thread

Liu Bo bo.liu at linux.alibaba.com
Fri Jun 14 19:08:04 UTC 2019


On Fri, Jun 14, 2019 at 07:58:25PM +0100, Dr. David Alan Gilbert wrote:
> * Liu Bo (bo.liu at linux.alibaba.com) wrote:
> > It's misleading that pf[0].revents is printed when there is something
> > wrong with pf[1].revents.
> > 
> > Signed-off-by: Liu Bo <bo.liu at linux.alibaba.com>
> 
> Ah well spotted; that's a copy-paste problem on my half.
> 
> For this, I've just fixed up the original patch.

Sure, that's fine.

thanks,
-liubo

> 
> Dave
> 
> > ---
> >  contrib/virtiofsd/fuse_virtio.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/contrib/virtiofsd/fuse_virtio.c b/contrib/virtiofsd/fuse_virtio.c
> > index ed6ba57..bec4105 100644
> > --- a/contrib/virtiofsd/fuse_virtio.c
> > +++ b/contrib/virtiofsd/fuse_virtio.c
> > @@ -472,7 +472,7 @@ static void *fv_queue_thread(void *opaque)
> >                 }
> >                 if (pf[1].revents & (POLLERR | POLLHUP | POLLNVAL)) {
> >                         fprintf(stderr, "%s: Unexpected poll revents %x Queue %d killfd\n",
> > -                                __func__, pf[0].revents, qi->qidx);
> > +                                __func__, pf[1].revents, qi->qidx);
> >                         break;
> >                 }
> >                 if (pf[1].revents) {
> > -- 
> > 1.8.3.1
> > 
> --
> Dr. David Alan Gilbert / dgilbert at redhat.com / Manchester, UK




More information about the Virtio-fs mailing list