virtiofs: do not use fuse_fill_super_common() for device installation
authorVivek Goyal <vgoyal@redhat.com>
Mon, 4 May 2020 18:33:15 +0000 (14:33 -0400)
committerMiklos Szeredi <mszeredi@redhat.com>
Tue, 19 May 2020 12:50:37 +0000 (14:50 +0200)
commit7fd3abfa8dd7c08ecacd25b2f9f9e1d3fb642440
tree211d8f1b487e5be4eedd94c98bc6f67b9134b8d5
parent5157da2ca42cbeca01709e29ce7b797cffed2432
virtiofs: do not use fuse_fill_super_common() for device installation

fuse_fill_super_common() allocates and installs one fuse_device.  Hence
virtiofs allocates and install all fuse devices by itself except one.

This makes logic little twisted.  There does not seem to be any real need
that why virtiofs can't allocate and install all fuse devices itself.

So opt out of fuse device allocation and installation while calling
fuse_fill_super_common().

Regular fuse still wants fuse_fill_super_common() to install fuse_device.
It needs to prevent against races where two mounters are trying to mount
fuse using same fd.  In that case one will succeed while other will get
-EINVAL.

virtiofs does not have this issue because sget_fc() resolves the race
w.r.t multiple mounters and only one instance of virtio_fs_fill_super()
should be in progress for same filesystem.

Signed-off-by: Vivek Goyal <vgoyal@redhat.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
fs/fuse/inode.c
fs/fuse/virtio_fs.c