From: Pratyush Brahma <quic_pbrahma@quicinc.com>
To: <danielwa@cisco.com>
Cc: <akpm@linux-foundation.org>, <christophe.leroy@c-s.fr>,
<dwalker@fifo99.com>, <linux-kernel@vger.kernel.org>,
<linuxppc-dev@lists.ozlabs.org>, <maksym.kokhan@globallogic.com>,
<mpe@ellerman.id.au>, <robh+dt@kernel.org>,
<tomas.mudrunka@gmail.com>, <xe-linux-external@cisco.com>,
Pavan Kondeti <quic_pkondeti@quicinc.com>,
<"quic_guptap@quicinc.com quic_ylal"@quicinc.com>,
<quic_vjitta@quicinc.com>
Subject: Re: [PATCH 1/4] add generic builtin command line
Date: Tue, 17 Oct 2023 16:10:42 +0530 [thread overview]
Message-ID: <4c420081-fe57-d036-ded7-2899c13738ee@quicinc.com> (raw)
In-Reply-To: <20230417162415.GF1391488@zorba>
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
next prev parent reply other threads:[~2023-10-17 10:41 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-19 23:24 [PATCH 1/4] add generic builtin command line Daniel Walker
2019-03-20 22:53 ` Andrew Morton
2019-03-20 23:23 ` Daniel Walker
2019-03-21 3:14 ` Andrew Morton
2019-03-21 15:13 ` Daniel Walker
2019-03-21 22:15 ` Andrew Morton
2021-02-15 19:32 ` Daniel Gimpelevich
2021-02-16 17:42 ` Christophe Leroy
2021-02-16 21:20 ` Daniel Gimpelevich
2021-02-16 19:02 ` Daniel Walker
2021-02-17 21:16 ` Andrew Morton
2023-04-17 16:18 ` Tomas Mudrunka
2023-04-17 16:24 ` Daniel Walker (danielwa)
2023-10-17 10:40 ` Pratyush Brahma [this message]
2023-10-17 14:21 ` Daniel Walker (danielwa)
2023-10-19 6:43 ` Pratyush Brahma
2023-11-08 11:33 ` Christophe Leroy
2023-11-08 16:23 ` Daniel Walker (danielwa)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4c420081-fe57-d036-ded7-2899c13738ee@quicinc.com \
--to=quic_pbrahma@quicinc.com \
--cc="quic_guptap@quicinc.com quic_ylal"@quicinc.com \
--cc=akpm@linux-foundation.org \
--cc=christophe.leroy@c-s.fr \
--cc=danielwa@cisco.com \
--cc=dwalker@fifo99.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=maksym.kokhan@globallogic.com \
--cc=mpe@ellerman.id.au \
--cc=quic_pkondeti@quicinc.com \
--cc=quic_vjitta@quicinc.com \
--cc=robh+dt@kernel.org \
--cc=tomas.mudrunka@gmail.com \
--cc=xe-linux-external@cisco.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).