Messages in this thread | | | From | Evgeniy Baskov <> | Subject | [PATCH v8 0/5] Parse CONFIG_CMDLINE in compressed kernel | Date | Thu, 10 Nov 2022 16:09:28 +0300 |
| |
CONFIG_CMDLINE_BOOL and CONFIG_CMDLINE_OVERRIDE were ignored during options lookup in compressed kernel, including earlyprintk option, so it was impossible to get earlyprintk messages from that stage of boot process via command line provided at compile time. Being able to enable earlyprintk via compile-time option might be desirable for booting on systems with broken UEFI command line arguments via EFISTUB.
Changes in v2:
* Compute resulting cmdline string once if needed and then reuse it. Store concatenation result in a static buffer. * Add strlcat() to compressed kernel to simplify the code.
Changes in v3:
v2 had a bug: cmd_line_ptr was set to a pointer to a buffer inside a kernel before kernel relocation, that makes this pointer invalid.
* Replace the pointer by a boolean variable to avoid storing a pointer, since it becomes invalid during kernel relocation.
Changes in v4:
* Use better wording for commit messages. * Add buffer overflow check to strlcat(). * Factor out common logic of cmdline resolving into helper function.
Changes in v5:
* Use strscpy() instead of strlcpy().
Changes in v6:
* Remove superfluous new line. * Rename resolve_cmdline() to cmdline_prepare(). * Move shared/setup-cmdline.h to shared/cmdline.h
Changes in v7:
* Replace #ifdef with IS_ENABLED() in cmdline_prepare() for consistency.
Changes in v8:
* Remove the need to use special initial value for command line result variable and put all logic in one function to improve the clarity of the code.
One #ifdef is still required for the initial value of builtin cmdline in case iti is not defined to enable the use of IS_ENABLED in cmdline_prapare(), since it is more readable.
Evgeniy Baskov (5): x86/boot: Add strlcat() and strscpy() to compressed kernel x86: Add cmdline_prepare() helper x86/setup: Use cmdline_prepare() in setup.c x86/boot: Use cmdline_prapare() in compressed kernel x86/boot: Remove no longer needed includes
arch/x86/boot/compressed/cmdline.c | 24 +++++++++++- arch/x86/boot/compressed/ident_map_64.c | 4 -- arch/x86/boot/compressed/kaslr.c | 4 -- arch/x86/boot/compressed/misc.h | 1 + arch/x86/boot/compressed/string.c | 50 +++++++++++++++++++++++++ arch/x86/include/asm/shared/cmdline.h | 35 +++++++++++++++++ arch/x86/kernel/setup.c | 20 ++-------- arch/x86/purgatory/purgatory.c | 1 + 8 files changed, 112 insertions(+), 27 deletions(-) create mode 100644 arch/x86/include/asm/shared/cmdline.h
-- 2.37.4
| |