Commit 77cd9d48 authored by Miklos Szeredi's avatar Miklos Szeredi

fuse: add req flag for private list

When an unlocked request is aborted, it is moved from fpq->io to a private
list.  Then, after unlocking fpq->lock, the private list is processed and
the requests are finished off.

To protect the private list, we need to mark the request with a flag, so if
in the meantime the request is unlocked the list is not corrupted.
Signed-off-by: default avatarMiklos Szeredi <mszeredi@suse.cz>
Reviewed-by: default avatarAshish Samant <ashish.samant@oracle.com>
parent 45a91cb1
...@@ -1328,7 +1328,8 @@ static ssize_t fuse_dev_do_read(struct fuse_conn *fc, struct file *file, ...@@ -1328,7 +1328,8 @@ static ssize_t fuse_dev_do_read(struct fuse_conn *fc, struct file *file,
return reqsize; return reqsize;
out_end: out_end:
list_del_init(&req->list); if (!test_bit(FR_PRIVATE, &req->flags))
list_del_init(&req->list);
spin_unlock(&fpq->lock); spin_unlock(&fpq->lock);
request_end(fc, req); request_end(fc, req);
return err; return err;
...@@ -1945,7 +1946,8 @@ static ssize_t fuse_dev_do_write(struct fuse_conn *fc, ...@@ -1945,7 +1946,8 @@ static ssize_t fuse_dev_do_write(struct fuse_conn *fc,
err = -ENOENT; err = -ENOENT;
else if (err) else if (err)
req->out.h.error = -EIO; req->out.h.error = -EIO;
list_del_init(&req->list); if (!test_bit(FR_PRIVATE, &req->flags))
list_del_init(&req->list);
spin_unlock(&fpq->lock); spin_unlock(&fpq->lock);
request_end(fc, req); request_end(fc, req);
...@@ -2149,8 +2151,10 @@ void fuse_abort_conn(struct fuse_conn *fc) ...@@ -2149,8 +2151,10 @@ void fuse_abort_conn(struct fuse_conn *fc)
req->out.h.error = -ECONNABORTED; req->out.h.error = -ECONNABORTED;
spin_lock(&req->waitq.lock); spin_lock(&req->waitq.lock);
set_bit(FR_ABORTED, &req->flags); set_bit(FR_ABORTED, &req->flags);
if (!test_bit(FR_LOCKED, &req->flags)) if (!test_bit(FR_LOCKED, &req->flags)) {
set_bit(FR_PRIVATE, &req->flags);
list_move(&req->list, &to_end1); list_move(&req->list, &to_end1);
}
spin_unlock(&req->waitq.lock); spin_unlock(&req->waitq.lock);
} }
list_splice_init(&fpq->processing, &to_end2); list_splice_init(&fpq->processing, &to_end2);
......
...@@ -269,6 +269,7 @@ struct fuse_io_priv { ...@@ -269,6 +269,7 @@ struct fuse_io_priv {
* FR_PENDING: request is not yet in userspace * FR_PENDING: request is not yet in userspace
* FR_SENT: request is in userspace, waiting for an answer * FR_SENT: request is in userspace, waiting for an answer
* FR_FINISHED: request is finished * FR_FINISHED: request is finished
* FR_PRIVATE: request is on private list
*/ */
enum fuse_req_flag { enum fuse_req_flag {
FR_ISREPLY, FR_ISREPLY,
...@@ -281,6 +282,7 @@ enum fuse_req_flag { ...@@ -281,6 +282,7 @@ enum fuse_req_flag {
FR_PENDING, FR_PENDING,
FR_SENT, FR_SENT,
FR_FINISHED, FR_FINISHED,
FR_PRIVATE,
}; };
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment