diff options
author | Lennart Poettering <lennart@poettering.net> | 2014-02-26 04:27:50 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2014-02-26 04:32:03 +0100 |
commit | ff5f34d08c191c326c41a083745522383ac86cae (patch) | |
tree | c3016d96b35bf204412a37f5dd903f751f58d816 | |
parent | dd7a22a990023f083ef488177fb46c9c4667009b (diff) |
mount: don't send out PropertiesChanged message if actually nothing got changed
-rw-r--r-- | src/core/mount.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/src/core/mount.c b/src/core/mount.c index b35e507878..98812c9c08 100644 --- a/src/core/mount.c +++ b/src/core/mount.c @@ -1390,7 +1390,7 @@ static int mount_add_one( _cleanup_free_ char *e = NULL, *w = NULL, *o = NULL, *f = NULL; bool load_extras = false; MountParameters *p; - bool delete; + bool delete, changed = false; Unit *u; int r; @@ -1458,6 +1458,7 @@ static int mount_add_one( } unit_add_to_load_queue(u); + changed = true; } else { delete = false; @@ -1476,6 +1477,7 @@ static int mount_add_one( /* Load in the extras later on, after we * finished initialization of the unit */ load_extras = true; + changed = true; } } @@ -1488,10 +1490,16 @@ static int mount_add_one( } p = &MOUNT(u)->parameters_proc_self_mountinfo; + + changed = changed || + !streq_ptr(p->options, options) || + !streq_ptr(p->what, what) || + !streq_ptr(p->fstype, fstype); + if (set_flags) { MOUNT(u)->is_mounted = true; MOUNT(u)->just_mounted = !MOUNT(u)->from_proc_self_mountinfo; - MOUNT(u)->just_changed = !streq_ptr(p->options, o); + MOUNT(u)->just_changed = changed; } MOUNT(u)->from_proc_self_mountinfo = true; @@ -1514,7 +1522,8 @@ static int mount_add_one( goto fail; } - unit_add_to_dbus_queue(u); + if (changed) + unit_add_to_dbus_queue(u); return 0; |