[PATCH v2 1/5] fuse: Kill fasync only if interrupt is queued in queue_interrupt()
We should sent signal only in case of interrupt is really queued.
Not a real problem, but this makes the code clearer and intuitive.
v2: Move kill_fasync() under fiq->waitq.lock
Signed-off-by: Kirill Tkhai
---
fs/fuse/dev.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
index ae813e609932..1ecec7fcb841 100644
--- a/fs/fuse/dev.c
+++ b/fs/fuse/dev.c
@@ -476,9 +476,9 @@ static void queue_interrupt(struct fuse_iqueue *fiq, struct
fuse_req *req)
if (list_empty(>intr_entry)) {
list_add_tail(>intr_entry, >interrupts);
wake_up_locked(>waitq);
+ kill_fasync(>fasync, SIGIO, POLL_IN);
}
spin_unlock(>waitq.lock);
- kill_fasync(>fasync, SIGIO, POLL_IN);
}
static void request_wait_answer(struct fuse_conn *fc, struct fuse_req *req)
[PATCH v2 1/5] fuse: Kill fasync only if interrupt is queued in queue_interrupt()
We should sent signal only in case of interrupt is really queued.
Not a real problem, but this makes the code clearer and intuitive.
v2: Move kill_fasync() under fiq->waitq.lock
Signed-off-by: Kirill Tkhai
---
fs/fuse/dev.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
index ae813e609932..1ecec7fcb841 100644
--- a/fs/fuse/dev.c
+++ b/fs/fuse/dev.c
@@ -476,9 +476,9 @@ static void queue_interrupt(struct fuse_iqueue *fiq, struct
fuse_req *req)
if (list_empty(>intr_entry)) {
list_add_tail(>intr_entry, >interrupts);
wake_up_locked(>waitq);
+ kill_fasync(>fasync, SIGIO, POLL_IN);
}
spin_unlock(>waitq.lock);
- kill_fasync(>fasync, SIGIO, POLL_IN);
}
static void request_wait_answer(struct fuse_conn *fc, struct fuse_req *req)
