diff options
author | Robert Love <rml@novell.com> | 2005-07-12 17:06:03 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-07-12 23:38:38 -0400 |
commit | 0eeca28300df110bd6ed54b31193c83b87921443 (patch) | |
tree | 7db42d8a18d80eca538f5b7d25e0532b8fa38b85 /fs/sysfs/file.c | |
parent | bd4c625c061c2a38568d0add3478f59172455159 (diff) |
[PATCH] inotify
inotify is intended to correct the deficiencies of dnotify, particularly
its inability to scale and its terrible user interface:
* dnotify requires the opening of one fd per each directory
that you intend to watch. This quickly results in too many
open files and pins removable media, preventing unmount.
* dnotify is directory-based. You only learn about changes to
directories. Sure, a change to a file in a directory affects
the directory, but you are then forced to keep a cache of
stat structures.
* dnotify's interface to user-space is awful. Signals?
inotify provides a more usable, simple, powerful solution to file change
notification:
* inotify's interface is a system call that returns a fd, not SIGIO.
You get a single fd, which is select()-able.
* inotify has an event that says "the filesystem that the item
you were watching is on was unmounted."
* inotify can watch directories or files.
Inotify is currently used by Beagle (a desktop search infrastructure),
Gamin (a FAM replacement), and other projects.
See Documentation/filesystems/inotify.txt.
Signed-off-by: Robert Love <rml@novell.com>
Cc: John McCutchan <ttb@tentacle.dhs.org>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/sysfs/file.c')
-rw-r--r-- | fs/sysfs/file.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/fs/sysfs/file.c b/fs/sysfs/file.c index d72c1ce48559..335288b9be0f 100644 --- a/fs/sysfs/file.c +++ b/fs/sysfs/file.c | |||
@@ -3,7 +3,7 @@ | |||
3 | */ | 3 | */ |
4 | 4 | ||
5 | #include <linux/module.h> | 5 | #include <linux/module.h> |
6 | #include <linux/dnotify.h> | 6 | #include <linux/fsnotify.h> |
7 | #include <linux/kobject.h> | 7 | #include <linux/kobject.h> |
8 | #include <linux/namei.h> | 8 | #include <linux/namei.h> |
9 | #include <asm/uaccess.h> | 9 | #include <asm/uaccess.h> |
@@ -391,9 +391,6 @@ int sysfs_create_file(struct kobject * kobj, const struct attribute * attr) | |||
391 | * sysfs_update_file - update the modified timestamp on an object attribute. | 391 | * sysfs_update_file - update the modified timestamp on an object attribute. |
392 | * @kobj: object we're acting for. | 392 | * @kobj: object we're acting for. |
393 | * @attr: attribute descriptor. | 393 | * @attr: attribute descriptor. |
394 | * | ||
395 | * Also call dnotify for the dentry, which lots of userspace programs | ||
396 | * use. | ||
397 | */ | 394 | */ |
398 | int sysfs_update_file(struct kobject * kobj, const struct attribute * attr) | 395 | int sysfs_update_file(struct kobject * kobj, const struct attribute * attr) |
399 | { | 396 | { |
@@ -408,7 +405,7 @@ int sysfs_update_file(struct kobject * kobj, const struct attribute * attr) | |||
408 | if (victim->d_inode && | 405 | if (victim->d_inode && |
409 | (victim->d_parent->d_inode == dir->d_inode)) { | 406 | (victim->d_parent->d_inode == dir->d_inode)) { |
410 | victim->d_inode->i_mtime = CURRENT_TIME; | 407 | victim->d_inode->i_mtime = CURRENT_TIME; |
411 | dnotify_parent(victim, DN_MODIFY); | 408 | fsnotify_modify(victim); |
412 | 409 | ||
413 | /** | 410 | /** |
414 | * Drop reference from initial sysfs_get_dentry(). | 411 | * Drop reference from initial sysfs_get_dentry(). |