Thread (22 messages) 22 messages, 3 authors, 2023-02-27

Re: [PATCH RFC bootconfig] Allow forcing unconditional bootconfig processing

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2023-02-24 08:32:11
Also in: lkml

Hi Paul,

On Sat, Jan 7, 2023 at 5:33 PM Paul E. McKenney [off-list ref] wrote:
quoted hunk ↗ jump to hunk
On Sun, Jan 08, 2023 at 12:22:15AM +0900, Masami Hiramatsu wrote:
quoted
BTW, maybe CONFIG_BOOT_CONFIG_EMBED is better to select this.
(or at least recommend to enable this)
Like this?

                                                        Thanx, Paul

------------------------------------------------------------------------

commit d09a1505c51a70da38b34ac38062977299aef742
Author: Paul E. McKenney [off-list ref]
Date:   Sat Jan 7 08:09:22 2023 -0800

    bootconfig: Default BOOT_CONFIG_FORCE to y if BOOT_CONFIG_EMBED

    When a kernel is built with CONFIG_BOOT_CONFIG_EMBED=y, the intention
    will normally be to unconditionally provide the specified kernel-boot
    arguments to the kernel, as opposed to requiring a separately provided
    bootconfig parameter.  Therefore, make the BOOT_CONFIG_FORCE Kconfig
    option default to y in kernels built with CONFIG_BOOT_CONFIG_EMBED=y.

    The old semantics may be obtained by manually overriding this default.

    Suggested-by: Masami Hiramatsu [off-list ref]
    Signed-off-by: Paul E. McKenney [off-list ref]
diff --git a/init/Kconfig b/init/Kconfig
index 0fb19fa0edba9..97a0f14d9020d 100644
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -1379,6 +1379,7 @@ config BOOT_CONFIG
 config BOOT_CONFIG_FORCE
        bool "Force unconditional bootconfig processing"
        depends on BOOT_CONFIG
+       default y if BOOT_CONFIG_EMBED
        help
          With this Kconfig option set, BOOT_CONFIG processing is carried
          out even when the "bootconfig" kernel-boot parameter is omitted.
Thanks for your patch, which is now commit 6ded8a28ed80e4cc
("bootconfig: Default BOOT_CONFIG_FORCE to y if BOOT_CONFIG_EMBED").

After this change, an all{mod,yes}config kernel has:

    CONFIG_BOOT_CONFIG_FORCE=y
    CONFIG_BOOT_CONFIG_EMBED=y
    CONFIG_BOOT_CONFIG_EMBED_FILE=""

Will this actually work? I haven't tried booting such a kernel yet.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help