[PATCH] [fuse] Add lock annotations to request_end and fuse_read_interrupt

[PATCH] [fuse] Add lock annotations to request_end and fuse_read_interrupt

[Josh Triplett]

request_end and fuse_read_interrupt release fc->lock.  Add lock annotations to
these two functions so that sparse can check callers for lock pairing, and so
that sparse will not complain about these functions since they intentionally
use locks in this manner.

Signed-off-by: Josh Triplett <josh@freedesktop.org>
---
 fs/fuse/dev.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c
index 1e2006c..4fc557c 100644
--- a/fs/fuse/dev.c
+++ b/fs/fuse/dev.c
@@ -212,6 +212,7 @@ void fuse_put_request(struct fuse_conn *
  * Called with fc->lock, unlocks it
  */
 static void request_end(struct fuse_conn *fc, struct fuse_req *req)
+	__releases(fc->lock)
 {
 	void (*end) (struct fuse_conn *, struct fuse_req *) = req->end;
 	req->end = NULL;
@@ -640,6 +641,7 @@ static void request_wait(struct fuse_con
  */
 static int fuse_read_interrupt(struct fuse_conn *fc, struct fuse_req *req,
 			       const struct iovec *iov, unsigned long nr_segs)
+	__releases(fc->lock)
 {
 	struct fuse_copy_state cs;
 	struct fuse_in_header ih;

Re: [PATCH] [fuse] Add lock annotations to request_end and fuse_read_interrupt

[Miklos Szeredi]

> request_end and fuse_read_interrupt release fc->lock.  Add lock annotations to
> these two functions so that sparse can check callers for lock pairing, and so
> that sparse will not complain about these functions since they intentionally
> use locks in this manner.
> 
> Signed-off-by: Josh Triplett <josh@freedesktop.org>

Acked-by: Miklos Szeredi <miklos@szeredi.hu>

Thanks,
Miklos