diff options
author | Eric Paris <eparis@redhat.com> | 2009-08-16 21:51:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-08-17 16:37:37 -0400 |
commit | 08e53fcb0db34baca3db84a457b6d67faabee4c6 (patch) | |
tree | c9c112a0aa290895a416b5bfc3f58aa328bc74e8 /fs/notify | |
parent | cd94c8bbef8d4b796a7ed4c551355a334604fd36 (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.c | 2 |
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 | ||