virtiofs: schedule blocking async replies in separate worker
authorVivek Goyal <vgoyal@redhat.com>
Mon, 20 Apr 2020 15:01:34 +0000 (17:01 +0200)
committerMiklos Szeredi <mszeredi@redhat.com>
Mon, 20 Apr 2020 15:01:34 +0000 (17:01 +0200)
commitbb737bbe48bea9854455cb61ea1dc06e92ce586c
tree71a1a272091c32f5f9a8e28a7bf3a280109f904a
parentae83d0b416db002fe95601e7f97f64b59514d936
virtiofs: schedule blocking async replies in separate worker

In virtiofs (unlike in regular fuse) processing of async replies is
serialized.  This can result in a deadlock in rare corner cases when
there's a circular dependency between the completion of two or more async
replies.

Such a deadlock can be reproduced with xfstests:generic/503 if TEST_DIR ==
SCRATCH_MNT (which is a misconfiguration):

 - Process A is waiting for page lock in worker thread context and blocked
   (virtio_fs_requests_done_work()).
 - Process B is holding page lock and waiting for pending writes to
   finish (fuse_wait_on_page_writeback()).
 - Write requests are waiting in virtqueue and can't complete because
   worker thread is blocked on page lock (process A).

Fix this by creating a unique work_struct for each async reply that can
block (O_DIRECT read).

Fixes: a62a8ef9d97d ("virtio-fs: add virtiofs filesystem")
Signed-off-by: Vivek Goyal <vgoyal@redhat.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
fs/fuse/file.c
fs/fuse/fuse_i.h
fs/fuse/virtio_fs.c