From: Will Deacon <will@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Catalin Marinas <catalin.marinas@arm.com>,
linux-arm-kernel@lists.infradead.org,
Guillaume Tucker <guillaume.tucker@collabora.com>,
Kevin Hilman <khilman@baylibre.com>
Subject: Re: [PATCH v2] arm64: defconfig: Use DEBUG_INFO_REDUCED
Date: Thu, 4 Mar 2021 18:30:34 +0000 [thread overview]
Message-ID: <20210304183034.GA21795@willie-the-truck> (raw)
In-Reply-To: <20210304174407.17537-1-broonie@kernel.org>
On Thu, Mar 04, 2021 at 05:44:07PM +0000, Mark Brown wrote:
> We've had DEBUG_INFO enabled for arm64 defconfigs since the initial
> commit. This is probably not that frequently used but substantially
> inflates the size of the build tree and amount of I/O needed during the
> build. This was causing issues with storage usage in some automated CI
> environments which don't expect defconfigs to be quite this big, and
> generally increases the resource consumption for both them and people
> doing local builds. The main use case for the debug info is decoding
> things with scripts/faddr2line but that doesn't need the full
> DEBUG_INFO, DEBUG_INFO_REDUCED is enough for it, so enable that by
> default.
>
> Without this patch my build tree is 6.8G, with it the size drops to 2G
> (smaller than the 6.4G for allmodconfig!).
>
> Suggested-by: Catalin Marinas <catalin.marinas@arm.com>
> Reported-by: Guillaume Tucker <guillaume.tucker@collabora.com>
> Signed-off-by: Mark Brown <broonie@kernel.org>
> Acked-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index d612f633b771..8793a9cb9d4b 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -1156,6 +1156,7 @@ CONFIG_CRYPTO_DEV_HISI_TRNG=m
> CONFIG_CMA_SIZE_MBYTES=32
> CONFIG_PRINTK_TIME=y
> CONFIG_DEBUG_INFO=y
> +CONFIG_DEBUG_INFO_REDUCED=y
> CONFIG_MAGIC_SYSRQ=y
> CONFIG_DEBUG_FS=y
> CONFIG_DEBUG_KERNEL=y
I gave this a spin with clang and it looks alright! Are you targetting 5.12
with this or can it wait until 5.13? I can't tell how big a deal it is on
the CI side of things.
Will
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-03-04 18:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-04 17:44 [PATCH v2] arm64: defconfig: Use DEBUG_INFO_REDUCED Mark Brown
2021-03-04 18:30 ` Will Deacon [this message]
2021-03-04 18:33 ` Mark Brown
2021-03-04 18:49 ` Will Deacon
2021-03-15 19:13 ` Catalin Marinas
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=20210304183034.GA21795@willie-the-truck \
--to=will@kernel.org \
--cc=broonie@kernel.org \
--cc=catalin.marinas@arm.com \
--cc=guillaume.tucker@collabora.com \
--cc=khilman@baylibre.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.