chiark / gitweb /
path: don't use IN_ATTRIB on parent dirs when watching a file, since those cannot...
authorLennart Poettering <lennart@poettering.net>
Thu, 3 Mar 2011 23:22:00 +0000 (00:22 +0100)
committerLennart Poettering <lennart@poettering.net>
Thu, 3 Mar 2011 23:22:00 +0000 (00:22 +0100)
TODO
src/path.c

diff --git a/TODO b/TODO
index 820391d..36c738e 100644 (file)
--- a/TODO
+++ b/TODO
@@ -11,9 +11,6 @@ F15:
 * load EnvironmentFile= when starting services, not when reloading configuration
   https://bugzilla.redhat.com/show_bug.cgi?id=661282
 
-* drop IN_ATTRIB from inotify watches for .path units where possible to avoid
-  lots of wakeups due to /dev changing when we watch a subdir of /dev.
-
 * NFS, networkmanager ordering issue
 
 * add fstab fields to add wait timeouts, change Wants to Requires by local-fs.target
index bd40ab6..1455b47 100644 (file)
@@ -223,9 +223,9 @@ static int path_watch_one(Path *p, PathSpec *s) {
                 /* Trim the path at the last slash. Keep the slash if it's the root dir. */
                 slash[slash == k] = 0;
 
-                flags = IN_DELETE_SELF|IN_MOVE_SELF|IN_ATTRIB;
+                flags = IN_MOVE_SELF;
                 if (!exists)
-                        flags |= IN_CREATE | IN_MOVED_TO;
+                        flags |= IN_DELETE_SELF | IN_ATTRIB | IN_CREATE | IN_MOVED_TO;
 
                 if (inotify_add_watch(s->inotify_fd, k, flags) >= 0)
                         exists = true;