<div dir="ltr">Hi,<div><br></div><div>I thought mounts coming from mountinfo are not getting default dependencies anyways. mount_add_one() never sets "load_extras" to true for new mount units. "load_extras" is the control to call mount_add_extras which will eventually call mount_add_default_dependencies(). Or I have replied to quickly? :)</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 8, 2013 at 3:13 PM, Michal Sekletar <span dir="ltr"><<a href="mailto:msekleta@redhat.com" target="_blank">msekleta@redhat.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We should not try to get information about mount unit from fragment<br>
if the unit was created because of /proc/self/mountinfo event.<br>
---<br>
 src/core/mount.c | 6 +++++-<br>
 1 file changed, 5 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/core/mount.c b/src/core/mount.c<br>
index 419cf27..d3333f9 100644<br>
--- a/src/core/mount.c<br>
+++ b/src/core/mount.c<br>
@@ -447,7 +447,11 @@ static int mount_add_default_dependencies(Mount *m) {<br>
         if (UNIT(m)->manager->running_as != SYSTEMD_SYSTEM)<br>
                 return 0;<br>
<br>
-        p = get_mount_parameters_fragment(m);<br>
+        if (m->from_fragment)<br>
+                p = get_mount_parameters_fragment(m);<br>
+        else<br>
+                p = get_mount_parameters(m);<br>
+<br>
         if (!p)<br>
                 return 0;<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.8.1.4<br>
<br>
_______________________________________________<br>
systemd-devel mailing list<br>
<a href="mailto:systemd-devel@lists.freedesktop.org">systemd-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</font></span></blockquote></div><br></div>