linux-kernel - Re: [PATCH 1/4] add generic builtin command line

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4c420081-fe57-d036-ded7-2899c13738ee@quicinc.com>
Date:   Tue, 17 Oct 2023 16:10:42 +0530
From:   Pratyush Brahma <quic_pbrahma@...cinc.com>
To:     <danielwa@...co.com>
CC:     <akpm@...ux-foundation.org>, <christophe.leroy@....fr>,
        <dwalker@...o99.com>, <linux-kernel@...r.kernel.org>,
        <linuxppc-dev@...ts.ozlabs.org>, <maksym.kokhan@...ballogic.com>,
        <mpe@...erman.id.au>, <robh+dt@...nel.org>,
        <tomas.mudrunka@...il.com>, <xe-linux-external@...co.com>,
        Pavan Kondeti <quic_pkondeti@...cinc.com>,
        <"quic_guptap@...cinc.com quic_ylal"@quicinc.com>,
        <quic_vjitta@...cinc.com>
Subject: Re: [PATCH 1/4] add generic builtin command line

Hi Daniel

We have a usecase which requires this patch necessarily. For android 
usecases, we have two different build variants
differentiated by defconfigs - production and debug. However, we only 
have a single dts for both these variants.

We want to enable certain features like page owner and slub debug which 
require cmdline params in addition to
their respective configs to be enabled. Enabling page_owner and 
slub_debug options in dts file enables it for both
production and debug variants. These features have significant memory 
overhead which are undesirable for
our production environment. However, these are necessary for debug 
environment to enable internal testing and debug.
Currently, android uses out-of-tree configs like CONFIG_CMDLINE_EXTEND 
to do so in gki_defconfig [1].
One option is to use CMDLINE_FORCE option which would enable these 
cmdline params but this disables the bootloader to add
any additional cmdline params which may be necessary.

For such a usecase, the CONFIG_CMDLINE_PREPEND seems to be quite useful 
as it would help to stitch bootloader
and the desired build variant's configs together. Can you please help to 
merge this patch?

[1] 
https://android.googlesource.com/kernel/common/+/refs/heads/android14-6.1-lts/arch/arm64/configs/gki_defconfig#62