[v1] Allow CONFIG_DEBUG_INFO_DWARF5=y + CONFIG_DEBUG_INFO_BTF=y

[PATCH 0/5] Allow CONFIG_DEBUG_INFO_DWARF5=y + CONFIG_DEBUG_INFO_BTF=y

Posted by maennich@google.com 2 years, 6 months ago

From: Matthias Maennich <maennich@google.com>

Can we please pick this series up for 5.15? I am particularly interested
in the last patch to enable BTF + DWARF5, but the cleanup patches before
are a very reasonable choice for stable@ as well as they simplify the
pahole version calculation and allow future BTF/pahole related patches
to apply cleanly as well. I intentionally kept the config
PAHOLE_HAS_BTF_TAG and hence its patch complete, even though there is no
user for it.

Cheers,
Matthias

Cc: <stable@vger.kernel.org> # v5.15+
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Nathan Chancellor <nathan@kernel.org>
Cc: Andrii Nakryiko <andrii@kernel.org>
Signed-off-by: Matthias Maennich <maennich@google.com>

Nathan Chancellor (5):
  MAINTAINERS: Add scripts/pahole-flags.sh to BPF section
  kbuild: Add CONFIG_PAHOLE_VERSION
  scripts/pahole-flags.sh: Use pahole-version.sh
  lib/Kconfig.debug: Use CONFIG_PAHOLE_VERSION
  lib/Kconfig.debug: Allow BTF + DWARF5 with pahole 1.21+

 MAINTAINERS               |  2 ++
 init/Kconfig              |  4 ++++
 lib/Kconfig.debug         | 12 ++++++++++--
 scripts/pahole-flags.sh   |  2 +-
 scripts/pahole-version.sh | 13 +++++++++++++
 5 files changed, 30 insertions(+), 3 deletions(-)
 create mode 100755 scripts/pahole-version.sh

-- 
2.39.2.637.g21b0678d19-goog

Re: [PATCH 0/5] Allow CONFIG_DEBUG_INFO_DWARF5=y + CONFIG_DEBUG_INFO_BTF=y

Posted by Greg Kroah-Hartman 2 years, 6 months ago

On Wed, Feb 22, 2023 at 11:21:37AM +0000, maennich@google.com wrote:
> From: Matthias Maennich <maennich@google.com>
> 
> Can we please pick this series up for 5.15? I am particularly interested
> in the last patch to enable BTF + DWARF5, but the cleanup patches before
> are a very reasonable choice for stable@ as well as they simplify the
> pahole version calculation and allow future BTF/pahole related patches
> to apply cleanly as well. I intentionally kept the config
> PAHOLE_HAS_BTF_TAG and hence its patch complete, even though there is no
> user for it.

What are the upstream git commit ids for these changes?

thanks,

greg k-h

Re: [PATCH 0/5] Allow CONFIG_DEBUG_INFO_DWARF5=y + CONFIG_DEBUG_INFO_BTF=y

Posted by Matthias Männich 2 years, 6 months ago

On Wed, Feb 22, 2023 at 11:26 AM Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> On Wed, Feb 22, 2023 at 11:21:37AM +0000, maennich@google.com wrote:
> > From: Matthias Maennich <maennich@google.com>
> >
> > Can we please pick this series up for 5.15? I am particularly interested
> > in the last patch to enable BTF + DWARF5, but the cleanup patches before
> > are a very reasonable choice for stable@ as well as they simplify the
> > pahole version calculation and allow future BTF/pahole related patches
> > to apply cleanly as well. I intentionally kept the config
> > PAHOLE_HAS_BTF_TAG and hence its patch complete, even though there is no
> > user for it.
>
> What are the upstream git commit ids for these changes?

Sorry, they should have been part of the commit message ... here they are

f67644b4f282d42acf5ad9b0175ef5671314ab12 ("MAINTAINERS: Add
scripts/pahole-flags.sh to BPF section")
613fe169237785a4bb1d06397b52606b2967da53 ("kbuild: Add CONFIG_PAHOLE_VERSION")
2d6c9810eb8915c4ddede707b8e167a1d919e1ca ("scripts/pahole-flags.sh:
Use pahole-version.sh")
6323c81350b73a4569cf52df85f80273faa64071 ("lib/Kconfig.debug: Use
CONFIG_PAHOLE_VERSION")
42d9b379e3e1790eafb87c799c9edfd0b37a37c7 ("lib/Kconfig.debug: Allow
BTF + DWARF5 with pahole 1.21+")

Cheers,
Matthias

>
> thanks,
>
> greg k-h