diff options
author | Eric Paris <eparis@redhat.com> | 2010-08-18 12:25:50 -0400 |
---|---|---|
committer | Eric Paris <eparis@redhat.com> | 2010-08-22 20:28:16 -0400 |
commit | 2eebf582c9b3106abb9c33f4fc0a347fb9391037 (patch) | |
tree | 0fab23d7e456713815890fc01a148132b9b857f8 /include/linux/fsnotify_backend.h | |
parent | 84e1ab4d875922c034db7f4f814ac445a20a14bd (diff) | |
download | linux-stable-2eebf582c9b3106abb9c33f4fc0a347fb9391037.tar.gz linux-stable-2eebf582c9b3106abb9c33f4fc0a347fb9391037.tar.bz2 linux-stable-2eebf582c9b3106abb9c33f4fc0a347fb9391037.zip |
fanotify: flush outstanding perm requests on group destroy
When an fanotify listener is closing it may cause a deadlock between the
listener and the original task doing an fs operation. If the original task
is waiting for a permissions response it will be holding the srcu lock. The
listener cannot clean up and exit until after that srcu lock is syncronized.
Thus deadlock. The fix introduced here is to stop accepting new permissions
events when a listener is shutting down and to grant permission for all
outstanding events. Thus the original task will eventually release the srcu
lock and the listener can complete shutdown.
Reported-by: Andreas Gruenbacher <agruen@suse.de>
Cc: Andreas Gruenbacher <agruen@suse.de>
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'include/linux/fsnotify_backend.h')
-rw-r--r-- | include/linux/fsnotify_backend.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/linux/fsnotify_backend.h b/include/linux/fsnotify_backend.h index ed36fb57c426..e40190d16878 100644 --- a/include/linux/fsnotify_backend.h +++ b/include/linux/fsnotify_backend.h @@ -156,6 +156,7 @@ struct fsnotify_group { struct mutex access_mutex; struct list_head access_list; wait_queue_head_t access_waitq; + bool bypass_perm; /* protected by access_mutex */ #endif /* CONFIG_FANOTIFY_ACCESS_PERMISSIONS */ int f_flags; } fanotify_data; |