lkml.org 
[lkml]   [2019]   [Sep]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v6] virtio-fs: add virtiofs filesystem
On Wed, Sep 18, 2019 at 2:57 PM Michael S. Tsirkin <mst@redhat.com> wrote:

> Overall this looks ok to me. Handling full vq by a timer is really gross
> but it's correct - just terribly inefficient.

Yes. The reason this didn't get too much attention is that it's
relatively difficult to make the queues actually fill up (which is a
good thing).

But this should definitely be one of the first things to be cleaned up...

> I think you should add a
> MAINTAINERS entry though, we want
> virtualization@lists.linux-foundation.org Cc'd on patches.

Already done in the documentation patch (which I didn't resend, but
you should find the v5 version in your mailbox).

> With that corrected:
>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
>
> Who's going to merge this? Miklos do you want to merge it yourself?

I'll merge this since it depends on the rest of the fuse queue.

> > + /**
> > + * Cleanup up when fuse_iqueue is destroyed
>
> Cleanup up -> Clean up

Fixed.

> > + * virtio-fs: Virtio Filesystem
> > + * Copyright (C) 2018 Red Hat, Inc.
>
>
> 2019 at this point?

I'd consider the creation year okay for this. These copyright notices
don't seem to be kept up to date, and don't need to, since the SOB
procedure allows for fine-grained tracking of authorship.

> > + /* After holding mutex, make sure virtiofs device is still there.
> > + * Though we are holding a refernce to it, drive ->remove might
>
> refernce -> reference

Fixed.

Changes, with your ack pushed to:

git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/fuse.git#for-next

Thanks,
Miklos

\
 
 \ /
  Last update: 2019-09-18 15:40    [W:0.073 / U:1.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site