From: Lennart Poettering <lennart@poettering.net>
Date: Wed, 21 Sep 2011 01:16:00 +0000 (+0200)
Subject: units: drop /sys/kernel/config automount unit
X-Git-Tag: v36~41
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0286cc1551927ea93e22b2d451a8bedddfc9d4ea;p=systemd

units: drop /sys/kernel/config automount unit

The mount point directory /sys/kernel/config is only created after the
module is loaded, hence there's little value in having this an automount
unit: the runtime penalty for mounting an autofs here should be the same
as for a real mount.
---

diff --git a/Makefile.am b/Makefile.am
index 60fb3ff7..de6325d4 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -361,7 +361,6 @@ dist_systemunit_DATA = \
 	units/dev-hugepages.mount \
 	units/dev-mqueue.automount \
 	units/dev-mqueue.mount \
-	units/sys-kernel-config.automount \
 	units/sys-kernel-config.mount \
 	units/sys-kernel-debug.automount \
 	units/sys-kernel-debug.mount \
@@ -1842,7 +1841,7 @@ endif
 	( cd $(DESTDIR)$(systemunitdir)/sysinit.target.wants && \
 		rm -f dev-hugepages.automount \
 			dev-mqueue.automount \
-			sys-kernel-config.automount \
+                        sys-kernel-config.mount \
 			sys-kernel-debug.automount \
 			sys-kernel-security.automount \
 			systemd-vconsole-setup.service \
@@ -1855,7 +1854,7 @@ endif
 			cryptsetup.target && \
 		$(LN_S) ../dev-hugepages.automount dev-hugepages.automount && \
 		$(LN_S) ../dev-mqueue.automount dev-mqueue.automount && \
-		$(LN_S) ../sys-kernel-config.automount sys-kernel-config.automount && \
+                $(LN_S) ../sys-kernel-config.mount sys-kernel-config.mount && \
 		$(LN_S) ../sys-kernel-debug.automount sys-kernel-debug.automount && \
 		$(LN_S) ../sys-kernel-security.automount sys-kernel-security.automount && \
 		$(LN_S) ../systemd-vconsole-setup.service systemd-vconsole-setup.service && \
diff --git a/TODO b/TODO
index fcd76cb0..123e9619 100644
--- a/TODO
+++ b/TODO
@@ -13,6 +13,16 @@ Bugfixes:
 
 Features:
 
+* File systems:
+
+  mqueue         -       boot
+  configfs       module  boot (order after module load)
+  securityfs     -       boot
+  binfmt_misc    module  autofs
+  fuse           module  udev
+  debug          -       boot
+  hugepages      -       boot
+
 * introduce sd_log_open() to create a connection to the syslog bridge via sd-daemon.h
 
 * when a bus name of a service disappears from the bus make sure to queue further activation requests
diff --git a/units/sys-kernel-config.automount b/units/sys-kernel-config.automount
deleted file mode 100644
index 1b4e8111..00000000
--- a/units/sys-kernel-config.automount
+++ /dev/null
@@ -1,15 +0,0 @@
-#  This file is part of systemd.
-#
-#  systemd is free software; you can redistribute it and/or modify it
-#  under the terms of the GNU General Public License as published by
-#  the Free Software Foundation; either version 2 of the License, or
-#  (at your option) any later version.
-
-[Unit]
-Description=Config File System Automount Point
-DefaultDependencies=no
-Before=sysinit.target
-ConditionPathExists=/sys/kernel/config
-
-[Automount]
-Where=/sys/kernel/config
diff --git a/units/sys-kernel-config.mount b/units/sys-kernel-config.mount
index a80d0525..f64b1ac2 100644
--- a/units/sys-kernel-config.mount
+++ b/units/sys-kernel-config.mount
@@ -6,8 +6,10 @@
 #  (at your option) any later version.
 
 [Unit]
-Description=Config File System
+Description=Configuration File System
 DefaultDependencies=no
+ConditionPathExists=/sys/kernel/config
+After=systemd-modules-load.service
 
 [Mount]
 What=configfs