]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
Merge pull request #31526 from poettering/proc-cmdline-underscorify
authorLuca Boccassi <bluca@debian.org>
Wed, 28 Feb 2024 17:59:46 +0000 (17:59 +0000)
committerGitHub <noreply@github.com>
Wed, 28 Feb 2024 17:59:46 +0000 (17:59 +0000)
make sure we use underscores for kernel cmdline option names, not dashes

18 files changed:
NEWS
docs/ENVIRONMENT.md
man/kernel-command-line.xml
man/systemd-battery-check.service.xml
man/systemd-network-generator.service.xml
man/systemd-udevd.service.xml
man/systemd-update-done.service.xml
man/systemd.net-naming-scheme.xml
man/systemd.unit.xml
meson.build
meson_options.txt
src/battery-check/battery-check.c
src/core/main.c
src/network/generator/network-generator.c
src/shared/condition.c
src/shared/netif-naming-scheme.c
src/tpm2-setup/tpm2-generator.c
units/systemd-battery-check.service.in

diff --git a/NEWS b/NEWS
index e7b673d650037da4d49593a3ef0e557b40f9d6a2..cd9965aaded523b5666fd849b753b44e54b9c06b 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1390,7 +1390,7 @@ CHANGES WITH 254:
           user is notified (graphically via Plymouth – if available – as well
           as in text form on the console), and the system is turned off after a
           10s delay. The feature can be disabled by passing
-          systemd.battery-check=0 through the kernel command line.
+          systemd.battery_check=0 through the kernel command line.
 
         * The 'passwdqc' library is now supported as an alternative to the
           'pwquality' library and can be selected at build time.
@@ -3718,7 +3718,7 @@ CHANGES WITH 250:
           interfaces. The Kind= setting in .netdev file accepts "ipoib". And
           systemd.netdev files gained the [IPoIB] section.
 
-        * systemd-networkd and systemd-udevd now support net.ifname-policy=
+        * systemd-networkd and systemd-udevd now support net.ifname_policy=
           option on the kernel command-line. This is implemented through the
           systemd-network-generator service that automatically generates
           appropriate .link, .network, and .netdev files.
@@ -4290,7 +4290,7 @@ CHANGES WITH 249:
           - Invalid characters in interface names are replaced with "_".
 
           The new version of the net naming scheme is "v249". The previous
-          scheme can be selected via the "net.naming-scheme=v247" kernel
+          scheme can be selected via the "net.naming_scheme=v247" kernel
           command line parameter.
 
         * sd-bus' sd_bus_is_ready() and sd_bus_is_open() calls now accept a
@@ -5821,7 +5821,7 @@ CHANGES WITH 247:
           bridge. Since this is a naming scheme incompatibility on systems that
           possess hardware like this it has been introduced as new naming
           scheme "v247". The previous scheme can be selected via the
-          "net.naming-scheme=v245" kernel command line parameter.
+          "net.naming_scheme=v245" kernel command line parameter.
 
         * ConditionFirstBoot= semantics have been modified to be safe towards
           abnormal system power-off during first boot. Specifically, the
@@ -6034,12 +6034,12 @@ CHANGES WITH 246:
           added, which may be used to turn off automatic activation of swap
           devices listed in /etc/fstab.
 
-        * New kernel command line options systemd.condition-needs-update= and
-          systemd.condition-first-boot= have been added, which override the
+        * New kernel command line options systemd.condition_needs_update= and
+          systemd.condition_first_boot= have been added, which override the
           result of the ConditionNeedsUpdate= and ConditionFirstBoot=
           conditions.
 
-        * A new kernel command line option systemd.clock-usec= has been added
+        * A new kernel command line option systemd.clock_usec= has been added
           that allows setting the system clock to the specified time in µs
           since Jan 1st, 1970 early during boot. This is in particular useful
           in order to make test cases more reliable.
@@ -6726,7 +6726,7 @@ CHANGES WITH 245:
           multiple containers whose names all begin with the same prefix. Since
           this changes the primary interface names pointing to containers if
           truncation happens, the old scheme may still be requested by
-          selecting an older naming scheme, via the net.naming-scheme= kernel
+          selecting an older naming scheme, via the net.naming_scheme= kernel
           command line option.
 
         * PrivateUsers= in service files now works in services run by the
@@ -7256,9 +7256,9 @@ CHANGES WITH 243:
           Moreover, the "en" prefix was dropped from the ID_NET_NAME_ONBOARD
           udev property.
 
-          Those two changes form a new net.naming-policy-scheme= entry.
-          Distributions which want to preserve naming stability may want to set
-          the -Ddefault-net-naming-scheme= configuration option.
+          Those two changes form a new net.naming_scheme= entry.  Distributions
+          which want to preserve naming stability may want to set the
+          -Ddefault-net-naming-scheme= configuration option.
 
         * systemd-networkd now supports MACsec, nlmon, IPVTAP and Xfrm
           interfaces natively.
@@ -8333,7 +8333,7 @@ CHANGES WITH 240:
           name as argument.
 
         * systemd-udevd's network naming logic now understands a new
-          net.naming-scheme= kernel command line switch, which may be used to
+          net.naming_scheme= kernel command line switch, which may be used to
           pick a specific version of the naming scheme. This helps stabilizing
           interface names even as systemd/udev are updated and the naming logic
           is improved.
index eab1ce23e46a2f692b82dc3ade2a877794748817..d55bcaf802f03092823f4c2f60b79528357b2bf8 100644 (file)
@@ -252,7 +252,7 @@ All tools:
   when determining stable network interface names. This may be used to revert
   to naming schemes of older udev versions, in order to provide more stable
   naming across updates. This environment variable takes precedence over the
-  kernel command line option `net.naming-scheme=`, except if the value is
+  kernel command line option `net.naming_scheme=`, except if the value is
   prefixed with `:` in which case the kernel command line option takes
   precedence, if it is specified as well.
 
index d4b005f8769f581f458ceb8e6b634f066e2a4172..1629ab2f2b33be5f91ad8aea42f9a27d7f311a94 100644 (file)
         <term><varname>udev.blockdev_read_only</varname></term>
         <term><varname>rd.udev.blockdev_read_only</varname></term>
         <term><varname>net.ifnames=</varname></term>
-        <term><varname>net.naming-scheme=</varname></term>
+        <term><varname>net.naming_scheme=</varname></term>
 
         <listitem>
           <para>Parameters understood by the device event managing
         <citerefentry><refentrytitle>systemd-homed-firstboot.service</refentrytitle><manvolnum>1</manvolnum></citerefentry>
         will not query the user for basic system settings, even if the system boots up for the first time and
         the relevant settings are not initialized yet. Not to be confused with
-        <varname>systemd.condition-first-boot=</varname> (see below), which overrides the result of the
+        <varname>systemd.condition_first_boot=</varname> (see below), which overrides the result of the
         <varname>ConditionFirstBoot=</varname> unit file condition, and thus controls more than just
         <filename>systemd-firstboot.service</filename> behaviour.</para>
 
       </varlistentry>
 
       <varlistentry>
-        <term><varname>systemd.condition-needs-update=</varname></term>
+        <term><varname>systemd.condition_needs_update=</varname></term>
 
         <listitem><para>Takes a boolean argument. If specified, overrides the result of
         <varname>ConditionNeedsUpdate=</varname> unit condition checks. See
       </varlistentry>
 
       <varlistentry>
-        <term><varname>systemd.condition-first-boot=</varname></term>
+        <term><varname>systemd.condition_first_boot=</varname></term>
 
         <listitem><para>Takes a boolean argument. If specified, overrides the result of
         <varname>ConditionFirstBoot=</varname> unit condition checks. See
       </varlistentry>
 
       <varlistentry>
-        <term><varname>systemd.clock-usec=</varname></term>
+        <term><varname>systemd.clock_usec=</varname></term>
 
         <listitem><para>Takes a decimal, numeric timestamp in μs since January 1st 1970, 00:00am, to set the
         system clock to. The system time is set to the specified timestamp early during boot. It is not
       </varlistentry>
 
       <varlistentry>
-        <term><varname>systemd.random-seed=</varname></term>
+        <term><varname>systemd.random_seed=</varname></term>
 
         <listitem><para>Takes a base64 encoded random seed value to credit with full entropy to the kernel's
         random pool during early service manager initialization. This option is useful in testing
 
         <xi:include href="version-info.xml" xpointer="v254"/></listitem>
       </varlistentry>
+
+      <varlistentry>
+        <term><varname>systemd.battery_check=</varname></term>
+
+        <listitem><para>Accepts a boolean argument. If false the boot-time battery charge check implemented
+        by
+        <citerefentry><refentrytitle>systemd-battery-check.service</refentrytitle><manvolnum>8</manvolnum></citerefentry>
+        is disabled.</para>
+
+        <xi:include href="version-info.xml" xpointer="v254"/></listitem>
+      </varlistentry>
+
+      <varlistentry>
+        <term><varname>ifname=</varname></term>
+        <term><varname>net.ifname_policy=</varname></term>
+
+        <listitem><para>Controls interface naming policies, implemented by
+        <citerefentry><refentrytitle>systemd-network-generator.service</refentrytitle><manvolnum>8</manvolnum></citerefentry>.</para>
+
+        <xi:include href="version-info.xml" xpointer="v245"/></listitem>
+      </varlistentry>
+
+      <varlistentry>
+        <term><varname>systemd.tpm2_wait=</varname></term>
+
+        <listitem><para>Controls whether to wait for a TPM2 device at boot, implemented by
+        <citerefentry><refentrytitle>systemd-tpm2-generator</refentrytitle><manvolnum>8</manvolnum></citerefentry>.</para>
+
+        <xi:include href="version-info.xml" xpointer="v256"/></listitem>
+      </varlistentry>
+
     </variablelist>
   </refsect1>
 
index 698e9d7c3f4af66c14523823f0f05e6b04d8c166..a5f564b888f1e0c9944027c356ae411103e53a7e 100644 (file)
@@ -66,7 +66,7 @@
 
     <variablelist class='kernel-commandline-options'>
       <varlistentry>
-        <term><varname>systemd.battery-check=<replaceable>BOOL</replaceable></varname></term>
+        <term><varname>systemd.battery_check=<replaceable>BOOL</replaceable></varname></term>
 
         <listitem>
           <para>Takes a boolean. If specified with false, <command>systemd-battery-check</command> command
index 1d498cbf1dd1e80013340d035ca6f9baea45466f..59d608028a70a0659859103356ea2f2ca6558a12 100644 (file)
@@ -76,7 +76,7 @@
 
       <varlistentry>
         <term><varname>ifname=</varname></term>
-        <term><varname>net.ifname-policy=</varname></term>
+        <term><varname>net.ifname_policy=</varname></term>
         <listitem>
           <para>Translated into
           <citerefentry><refentrytitle>systemd.link</refentrytitle><manvolnum>5</manvolnum></citerefentry>
index b63ee4bb7715fbe24d41104e12aa02d28cbd2462..c781765e1cf80f73e3923519373cf38f94269d1a 100644 (file)
         </listitem>
       </varlistentry>
       <varlistentry>
-        <term><varname>net.naming-scheme=</varname></term>
+        <term><varname>net.naming_scheme=</varname></term>
         <listitem>
           <para>Network interfaces are renamed to give them predictable names when possible (unless
           <varname>net.ifnames=0</varname> is specified, see above). With this kernel command line option it
         </listitem>
       </varlistentry>
       <varlistentry>
-        <term><varname>net.ifname-policy=<replaceable>policy1</replaceable>[,<replaceable>policy2</replaceable>,…][,<replaceable>MAC</replaceable>]</varname></term>
+        <term><varname>net.ifname_policy=<replaceable>policy1</replaceable>[,<replaceable>policy2</replaceable>,…][,<replaceable>MAC</replaceable>]</varname></term>
         <listitem>
           <para>Specifies naming policies applied when renaming network interfaces. Takes a list of
           policies and an optional MAC address separated with comma. Each policy value must be one of
           enabled.</para>
 
           <para>Example:
-          <programlisting>net.ifname-policy=keep,kernel,path,slot,onboard,01:23:45:67:89:ab
-net.ifname-policy=keep,kernel,path,slot,onboard,mac</programlisting>
+          <programlisting>net.ifname_policy=keep,kernel,path,slot,onboard,01:23:45:67:89:ab
+net.ifname_policy=keep,kernel,path,slot,onboard,mac</programlisting>
           This is mostly equivalent to creating the following .link files:
           <programlisting># 91-name-policy-with-mac.link
 [Match]
index 46bb2f41c886a076dfc88ccd7f3abb13443be139..6b863ecff3a871722ec0b6aa9cec5f4f12d6acc3 100644 (file)
@@ -58,7 +58,7 @@
     <citerefentry project='man-pages'><refentrytitle>touch</refentrytitle><manvolnum>1</manvolnum></citerefentry>
     on it.</para>
 
-    <para>Note that if the <varname>systemd.condition-needs-update=</varname> kernel command line option is
+    <para>Note that if the <varname>systemd.condition_needs_update=</varname> kernel command line option is
     used it overrides the <varname>ConditionNeedsUpdate=</varname> unit condition checks. In that case
     <filename>systemd-update-done.service</filename> will not reset the condition state until a follow-up
     reboot where the kernel switch is not specified anymore.</para>
index 09df42aea27808be0993fe6626df67ae6ac91a2c..2fec1055b7017a0253b23710ef9cbfaef1546d0d 100644 (file)
@@ -38,7 +38,7 @@
     implemented version, but it is also possible to set one of the older versions to preserve
     compatibility. This may be useful for example for distributions, which may introduce new versions of
     systemd in stable releases without changing the naming scheme. The naming scheme may also be overridden
-    using the <varname>net.naming-scheme=</varname> kernel command line switch, see
+    using the <varname>net.naming_scheme=</varname> kernel command line switch, see
     <citerefentry><refentrytitle>systemd-udevd.service</refentrytitle><manvolnum>8</manvolnum></citerefentry>.
     Available naming schemes are described below.</para>
 
     <title>History</title>
 
     <para>The following "naming schemes" have been defined (which may be chosen at system boot-up time via
-    the <varname>net.naming-scheme=</varname> kernel command line switch, see above):</para>
+    the <varname>net.naming_scheme=</varname> kernel command line switch, see above):</para>
 
     <variablelist>
         <varlistentry>
index 46184f5b7e76e38867aa5634c9ff70cd2c594d5b..7c11598a941528a7d7aa12708ea917d69e182292 100644 (file)
           to make sure they run before the stamp file's modification time gets reset indicating a completed
           update.</para>
 
-          <para>If the <varname>systemd.condition-needs-update=</varname> option is specified on the kernel
+          <para>If the <varname>systemd.condition_needs_update=</varname> option is specified on the kernel
           command line (taking a boolean), it will override the result of this condition check, taking
           precedence over any file modification time checks. If the kernel command line option is used,
           <filename>systemd-update-done.service</filename> will not have immediate effect on any following
           <varname>Wants=</varname>. This ensures that in a case of an aborted first boot, these units will
           be re-run during the next system startup.</para>
 
-          <para>If the <varname>systemd.condition-first-boot=</varname> option is specified on the kernel
+          <para>If the <varname>systemd.condition_first_boot=</varname> option is specified on the kernel
           command line (taking a boolean), it will override the result of this condition check, taking
           precedence over <filename>/etc/machine-id</filename> existence checks.</para>
 
index 10404cebd60728f6ba1e04f1352f6c2575b9988f..e27e944caff7cf063f816cf64e6f78673c62a4da 100644 (file)
@@ -2757,8 +2757,8 @@ summary({
         'default LLMNR mode' :              default_llmnr,
         'default DNS servers' :             dns_servers.split(' '),
         'default NTP servers' :             ntp_servers.split(' '),
-        'default net.naming-scheme value' : default_net_naming_scheme,
-        'default KillUserProcesses value' : kill_user_processes,
+        'default net.naming_scheme= value': default_net_naming_scheme,
+        'default KillUserProcesses= value': kill_user_processes,
         'default locale' :                  default_locale,
         'default nspawn locale' :           nspawn_locale,
         'default status unit format' :      status_unit_format_default,
index 51434f74b3bbbd1889c0c6f6403adb710e7653d2..e9a5f18cd63f10758b22a503a52a68a1bf413e0b 100644 (file)
@@ -230,9 +230,9 @@ option('default-hierarchy', type : 'combo',
        choices : ['legacy', 'hybrid', 'unified'], deprecated : true,
        description : '''This option is deprecated and will be removed in a future release''')
 option('extra-net-naming-schemes', type : 'string',
-       description : 'comma-separated list of extra net.naming-scheme= definitions')
+       description : 'comma-separated list of extra net.naming_scheme= definitions')
 option('default-net-naming-scheme', type : 'string', value : 'latest',
-       description : 'default net.naming-scheme= value')
+       description : 'default net.naming_scheme= value')
 option('status-unit-format-default', type : 'combo',
        choices : ['auto', 'description', 'name', 'combined'],
        description : 'use unit name or description in messages by default')
index 03628c8b921494db79ab083fea66b5dc0d37490b..1563147b7b673c101c33121e4b8ddd2521bdf23a 100644 (file)
@@ -123,9 +123,9 @@ static int run(int argc, char *argv[]) {
 
         log_setup();
 
-        r = proc_cmdline_get_bool("systemd.battery-check", PROC_CMDLINE_STRIP_RD_PREFIX|PROC_CMDLINE_TRUE_WHEN_MISSING, &arg_doit);
+        r = proc_cmdline_get_bool("systemd.battery_check", PROC_CMDLINE_STRIP_RD_PREFIX|PROC_CMDLINE_TRUE_WHEN_MISSING, &arg_doit);
         if (r < 0)
-                log_warning_errno(r, "Failed to parse systemd.battery-check= kernel command line option, ignoring: %m");
+                log_warning_errno(r, "Failed to parse systemd.battery_check= kernel command line option, ignoring: %m");
 
         r = parse_argv(argc, argv);
         if (r <= 0)
index be6d07b333ca429259ad3dd210891930dd49913d..fa0dc3191acb0fbde6c9b2209e147cf884aa52cb 100644 (file)
@@ -2202,9 +2202,9 @@ static void log_execution_mode(bool *ret_first_boot) {
                         /* Let's check whether we are in first boot. First, check if an override was
                          * specified on the kernel command line. If yes, we honour that. */
 
-                        r = proc_cmdline_get_bool("systemd.condition-first-boot", /* flags = */ 0, &first_boot);
+                        r = proc_cmdline_get_bool("systemd.condition_first_boot", /* flags = */ 0, &first_boot);
                         if (r < 0)
-                                log_debug_errno(r, "Failed to parse systemd.condition-first-boot= kernel command line argument, ignoring: %m");
+                                log_debug_errno(r, "Failed to parse systemd.condition_first_boot= kernel command line argument, ignoring: %m");
 
                         if (r > 0)
                                 log_full(first_boot ? LOG_INFO : LOG_DEBUG,
index 48527a2c73dfc7dbad1e01abdd315699edafaf7e..106e10a5752543c4e26b19ed13478343b3f23fb1 100644 (file)
@@ -27,7 +27,7 @@
 
   # .link
   ifname=<interface>:<MAC>
-  net.ifname-policy=policy1[,policy2,...][,<MAC>] # This is an original rule, not supported by other tools.
+  net.ifname_policy=policy1[,policy2,...][,<MAC>] # This is an original rule, not supported by other tools.
 
   # .netdev
   vlan=<vlanname>:<phydevice>
@@ -1114,7 +1114,7 @@ static int parse_cmdline_ifname_policy(Context *context, const char *key, const
         assert(context);
         assert(key);
 
-        /* net.ifname-policy=policy1[,policy2,...][,<MAC>] */
+        /* net.ifname_policy=policy1[,policy2,...][,<MAC>] */
 
         if (proc_cmdline_value_missing(key, value))
                 return -EINVAL;
@@ -1172,23 +1172,23 @@ int parse_cmdline_item(const char *key, const char *value, void *data) {
 
         assert(key);
 
-        if (streq(key, "ip"))
+        if (proc_cmdline_key_streq(key, "ip"))
                 return parse_cmdline_ip(context, key, value);
-        if (streq(key, "rd.route"))
+        if (proc_cmdline_key_streq(key, "rd.route"))
                 return parse_cmdline_rd_route(context, key, value);
-        if (streq(key, "nameserver"))
+        if (proc_cmdline_key_streq(key, "nameserver"))
                 return parse_cmdline_nameserver(context, key, value);
-        if (streq(key, "rd.peerdns"))
+        if (proc_cmdline_key_streq(key, "rd.peerdns"))
                 return parse_cmdline_rd_peerdns(context, key, value);
-        if (streq(key, "vlan"))
+        if (proc_cmdline_key_streq(key, "vlan"))
                 return parse_cmdline_vlan(context, key, value);
-        if (streq(key, "bridge"))
+        if (proc_cmdline_key_streq(key, "bridge"))
                 return parse_cmdline_bridge(context, key, value);
-        if (streq(key, "bond"))
+        if (proc_cmdline_key_streq(key, "bond"))
                 return parse_cmdline_bond(context, key, value);
-        if (streq(key, "ifname"))
+        if (proc_cmdline_key_streq(key, "ifname"))
                 return parse_cmdline_ifname(context, key, value);
-        if (streq(key, "net.ifname-policy"))
+        if (proc_cmdline_key_streq(key, "net.ifname_policy"))
                 return parse_cmdline_ifname_policy(context, key, value);
 
         return 0;
index 385ceee332b44e92300c7268bc9b2a45938dae8d..b7d2248b94be0744d315b680fc55f6933c0094e2 100644 (file)
@@ -751,9 +751,9 @@ static int condition_test_needs_update(Condition *c, char **env) {
         assert(c->parameter);
         assert(c->type == CONDITION_NEEDS_UPDATE);
 
-        r = proc_cmdline_get_bool("systemd.condition-needs-update", /* flags = */ 0, &b);
+        r = proc_cmdline_get_bool("systemd.condition_needs_update", /* flags = */ 0, &b);
         if (r < 0)
-                log_debug_errno(r, "Failed to parse systemd.condition-needs-update= kernel command line argument, ignoring: %m");
+                log_debug_errno(r, "Failed to parse systemd.condition_needs_update= kernel command line argument, ignoring: %m");
         if (r > 0)
                 return b;
 
index 38c24760f0a9e60f1807f10fd27376a473a2470b..2955b6e8d57757db2fad01ee1a4d5c659bdaa2b2 100644 (file)
@@ -53,7 +53,7 @@ const NamingScheme* naming_scheme(void) {
                 return cache;
 
         /* Acquire setting from the kernel command line */
-        (void) proc_cmdline_get_key("net.naming-scheme", 0, &buffer);
+        (void) proc_cmdline_get_key("net.naming_scheme", 0, &buffer);
 
         /* Also acquire it from an env var */
         e = getenv("NET_NAMING_SCHEME");
index 723b31898a923e066e6089527ab5ffc5434b999e..f1d903ee81ad4b0b7ef36d574a833784d95fee15 100644 (file)
@@ -20,10 +20,10 @@ static int parse_proc_cmdline_item(const char *key, const char *value, void *dat
 
         assert(key);
 
-        if (proc_cmdline_key_streq(key, "systemd.tpm2-wait")) {
+        if (proc_cmdline_key_streq(key, "systemd.tpm2_wait")) {
                 r = value ? parse_boolean(value) : 1;
                 if (r < 0)
-                        log_warning_errno(r, "Failed to parse 'systemd.tpm2-wait' kernel command line argument, ignoring: %s", value);
+                        log_warning_errno(r, "Failed to parse 'systemd.tpm2_wait=' kernel command line argument, ignoring: %s", value);
                 else
                         arg_tpm2_wait = r;
         }
index a5f532da707dce1da8271b1e95d27d4861a10e93..0f826041078a8735550dc8c98382c1e6f34b65e4 100644 (file)
@@ -12,7 +12,7 @@ Description=Check battery level during early boot
 Documentation=man:systemd-battery-check.service(8)
 ConditionVirtualization=no
 ConditionDirectoryNotEmpty=/sys/class/power_supply/
-ConditionKernelCommandLine=!systemd.battery-check=0
+ConditionKernelCommandLine=!systemd.battery_check=0
 AssertPathExists=/etc/initrd-release
 DefaultDependencies=no
 After=plymouth-start.service