]> git.baikalelectronics.ru Git - kernel.git/commit
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)
commit616c0c0d75e97b509133b44bdffda1cecb5870cd
tree71a1a272091c32f5f9a8e28a7bf3a280109f904a
parente4e79bc94ba2381f2a677c8e345f3bbe0674f2c2
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: cbe1d0133e15 ("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