aboutsummaryrefslogtreecommitdiffstats
path: root/fs/notify
diff options
context:
space:
mode:
authorEric Paris <eparis@redhat.com>2009-08-16 21:51:55 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2009-08-17 16:37:37 -0400
commit08e53fcb0db34baca3db84a457b6d67faabee4c6 (patch)
treec9c112a0aa290895a416b5bfc3f58aa328bc74e8 /fs/notify
parentcd94c8bbef8d4b796a7ed4c551355a334604fd36 (diff)
inotify: start watch descriptor count at 1
The inotify_add_watch man page specifies that inotify_add_watch() will return a non-negative integer. However, historically the inotify watches started at 1, not at 0. Turns out that the inotifywait program provided by the inotify-tools package doesn't properly handle a 0 watch descriptor. In 7e790dd5 we changed from starting at 1 to starting at 0. This patch starts at 1, just like in previous kernels, but also just like in previous kernels it's possible for it to wrap back to 0. This preserves the kernel functionality exactly like it was before the patch (neither method broke the spec) Signed-off-by: Eric Paris <eparis@redhat.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/notify')
-rw-r--r--fs/notify/inotify/inotify_user.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/notify/inotify/inotify_user.c b/fs/notify/inotify/inotify_user.c
index c172a7a17b17..dc32ed8323ba 100644
--- a/fs/notify/inotify/inotify_user.c
+++ b/fs/notify/inotify/inotify_user.c
@@ -567,7 +567,7 @@ static struct fsnotify_group *inotify_new_group(struct user_struct *user, unsign
567 567
568 spin_lock_init(&group->inotify_data.idr_lock); 568 spin_lock_init(&group->inotify_data.idr_lock);
569 idr_init(&group->inotify_data.idr); 569 idr_init(&group->inotify_data.idr);
570 group->inotify_data.last_wd = 0; 570 group->inotify_data.last_wd = 1;
571 group->inotify_data.user = user; 571 group->inotify_data.user = user;
572 group->inotify_data.fa = NULL; 572 group->inotify_data.fa = NULL;
573 573