<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - use_smack have corrupt cached result if no /sys/fs/smackfs systemd statup"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=77646#c3">Comment # 3</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - use_smack have corrupt cached result if no /sys/fs/smackfs systemd statup"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=77646">bug 77646</a>
              from <span class="vcard"><a class="email" href="mailto:ronan@fridu.net" title="Ronan22 <ronan@fridu.net>"> <span class="fn">Ronan22</span></a>
</span></b>
        <pre>When systemd do is first "use_smack" ?

-At the begining of systemd (main at src/core/main.c), mount_setup_early is
execute.

-mount_setup_early mount each entry of the "mount_table" thow function
mount_one.

-mount_one use mkdir_p_label.

-mkdir_p_label use label_mkdir.

-label_mkdir use use_smack.

So At the first mount of the first entry of "mount_table" ("/proc"), the return
value of use_smack is cached.

Add a funct "retest_smack()" seems to be heavy to me.

-------------------------------------------------------------------
For me, a cleaner way is to mount "/sys" first (before "/proc").

static const MountPoint mount_table[] = {
        { "sysfs",      "/sys",                      "sysfs",      NULL,
MS_NOSUID|MS_NOEXEC|MS_NODEV,
          NULL,       MNT_FATAL|MNT_IN_CONTAINER },
        { "proc",       "/proc",                     "proc",       NULL,
MS_NOSUID|MS_NOEXEC|MS_NODEV,
          NULL,       MNT_FATAL|MNT_IN_CONTAINER },
...</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>