[RHEL7,COMMIT] ms/fuse: Kill fasync only if interrupt is queued in queue_interrupt()

Submitted by Konstantin Khorenko on April 3, 2019, 9:49 a.m.

Details

Message ID 201904030949.x339nQZg006853@finist-ce7.sw.ru
State New
Series "fuse: Backport of ms locking patches part 1"
Headers show

Commit Message

Konstantin Khorenko April 3, 2019, 9:49 a.m.
The commit is pushed to "branch-rh7-3.10.0-957.10.1.vz7.94.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-957.10.1.vz7.85.10
------>
commit 40a3ff3f708050a4c859f2d1550f60677da401c3
Author: Kirill Tkhai <ktkhai@virtuozzo.com>
Date:   Wed Apr 3 12:49:25 2019 +0300

    ms/fuse: Kill fasync only if interrupt is queued in queue_interrupt()
    
    ms commit 8da6e9183275
    
    We should sent signal only in case of interrupt is really queued.  Not a
    real problem, but this makes the code clearer and intuitive.
    
    Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
    Signed-off-by: Kirill Tkhai <ktkhai@virtuozzo.com>
---
 fs/fuse/dev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
index dc21886ee55d..eb5c62e3170f 100644
--- a/fs/fuse/dev.c
+++ b/fs/fuse/dev.c
@@ -479,9 +479,9 @@  static void queue_interrupt(struct fuse_iqueue *fiq, struct fuse_req *req)
 	if (list_empty(&req->intr_entry)) {
 		list_add_tail(&req->intr_entry, &fiq->interrupts);
 		wake_up_locked(&fiq->waitq);
+		kill_fasync(&fiq->fasync, SIGIO, POLL_IN);
 	}
 	spin_unlock(&fiq->waitq.lock);
-	kill_fasync(&fiq->fasync, SIGIO, POLL_IN);
 }
 
 static void request_wait_answer(struct fuse_conn *fc, struct fuse_req *req)