base-filesystem: mention why we don't carry an entry for /tmp/ for now

diff --git a/src/shared/base-filesystem.c b/src/shared/base-filesystem.c
index 2d91e80ad603de1d96684c4539785f7031349c17..0244c92c79889277a18570da6abf679706abcfae 100644 (file)
--- a/src/shared/base-filesystem.c
+++ b/src/shared/base-filesystem.c
@@ -41,6 +41,10 @@ static const BaseFilesystem table[] = {
         { "sys",   0555, NULL,                         NULL, true },
         { "dev",   0555, NULL,                         NULL, true },
         { "run",   0555, NULL,                         NULL, true },
+        /* We don't add /tmp/ here for now (even though it's necessary for regular operation), because we
+         * want to support both cases where /tmp/ is a mount of its own (in which case we probably should set
+         * the mode to 1555, to indicate that noone should write to it, not even root) and when it's part of
+         * the rootfs (in which case we should set mode 1777), and we simply don't know what's right. */
 
         /* Various architecture ABIs define the path to the dynamic loader via the /lib64/ subdirectory of
          * the root directory. When booting from an otherwise empty root file system (where only /usr/ has