[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220506060310.7495-1-dmoulding@me.com>
Date: Fri, 6 May 2022 00:03:09 -0600
From: Dan Moulding <dmoulding@...com>
To: linux-kernel@...r.kernel.org
Cc: linux-doc@...r.kernel.org, tglx@...utronix.de,
akpm@...ux-foundation.org, corbet@....net,
Dan Moulding <dmoulding@...com>
Subject: [PATCH v2 0/1] Allow setting hostname before userspace starts
Some userspace processes may rely on gethostname to always return the
correct machine name. However, the only way that the hostname may be
set is by some other userspace process calling sethostname
first. During boot, if a process that depends on gethostname runs
before sethostname has been called, then the process that called
gethostname is going to get an incorrect result.
A real-world case where this comes up is with mdadm, which if
gethostname returns the wrong name, can cause local md-raid arrays to
appear to be foreign arrays. This can alter how mdadm assembles the
array, or can even cause array assembly to fail. I imagine there are
probbaly other real-world cases where undesirable behavior results when
the hostname is not set early enough.
I'm proposing adding the option to set the hostname from a kernel
parameter, so that the correct host name can be guaranteed to be set
before any userspace process can call gethostname.
I can imagine an even better way to do this would be to have the
hostname written to some non-volatile storage (like a firmware NVRAM
variable or such), which the kernel could read out during early
boot. But, alas, such designs require hardware support, standards, and
cooperation. This proposal is an alternative that can provide a simple
and immediate solution.
v2:
* Use strlcpy instead of strncpy to eliminate W=1 compiler warning
(assuaging it from its string truncation fears). Thanks to kernel
test robot for finding this.
* Move "hostname" after "hlt" in kernel-parameters.txt (I promise I
know my ABCs).
Dan Moulding (1):
init: Add "hostname" kernel parameter
Documentation/admin-guide/kernel-parameters.txt | 13 +++++++++++++
init/version.c | 17 +++++++++++++++++
2 files changed, 30 insertions(+)
--
2.35.1
Powered by blists - more mailing lists