]> git.baikalelectronics.ru Git - kernel.git/commit
bootconfig: Only load bootconfig if "bootconfig" is on the kernel cmdline
authorSteven Rostedt (VMware) <rostedt@goodmis.org>
Tue, 4 Feb 2020 12:33:53 +0000 (07:33 -0500)
committerSteven Rostedt (VMware) <rostedt@goodmis.org>
Wed, 5 Feb 2020 09:22:43 +0000 (04:22 -0500)
commitaffc66bb08fcb62aa1dd06b5524d360eaddcbeff
tree4c3fb757786ce82fe314227f128d673080d55cc3
parent3d45c380a5c30a7bc660d23a3c473091f11573f6
bootconfig: Only load bootconfig if "bootconfig" is on the kernel cmdline

As the bootconfig is appended to the initrd it is not as easy to modify as
the kernel command line. If there's some issue with the kernel, and the
developer wants to boot a pristine kernel, it should not be needed to modify
the initrd to remove the bootconfig for a single boot.

As bootconfig is silently added (if the admin does not know where to look
they may not know it's being loaded). It should be explicitly added to the
kernel cmdline. The loading of the bootconfig is only done if "bootconfig"
is on the kernel command line. This will let admins know that the kernel
command line is extended.

Note, after adding printk()s for when the size is too great or the checksum
is wrong, exposed that the current method always looked for the boot config,
and if this size and checksum matched, it would parse it (as if either is
wrong a printk has been added to show this). It's better to only check this
if the boot config is asked to be looked for.

Link: https://lore.kernel.org/r/CAHk-=wjfjO+h6bQzrTf=YCZA53Y3EDyAs3Z4gEsT7icA3u_Psw@mail.gmail.com
Acked-by: Masami Hiramatsu <mhiramat@kernel.org>
Suggested-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Documentation/admin-guide/bootconfig.rst
Documentation/admin-guide/kernel-parameters.txt
init/main.c