diff mbox series

default-distrovars: Enable minidebuginfo DISTRO_FEATURE by default

Message ID 20260723134101.864910-1-richard.purdie@linuxfoundation.org
State New
Headers show
Series default-distrovars: Enable minidebuginfo DISTRO_FEATURE by default | expand

Commit Message

Richard Purdie July 23, 2026, 1:41 p.m. UTC
Enable the "minidebuginfo" DISTRO_FEATURE in builds so that all
binaries have just enough debuginfo in them to get a useful stack trace.

This aids with debugging occasional failures that hit the autobuilder,
at minimal cost.  For example, core-image-sato with systemd:

   core-image-sato: IMAGESIZE changed from 481532 to 486044 (+1%)
   glibc: PKGSIZE changed from 3715743 to 3805871 (+2%)
   sqlite3: PKGSIZE changed from 462840 to 469056 (+1%)

[Idea taken from a poky patch from Ross Burton]
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/conf/distro/include/default-distrovars.inc | 1 +
 1 file changed, 1 insertion(+)

Comments

Khem Raj July 23, 2026, 2:48 p.m. UTC | #1
Hi Richard,

Since default policies are generally used by derived distros, changing this
default may have broader impacts. I wonder if the Yocto autobuilder issues
you mentioned are commonly seen by OE users, as this change will embed
symbols into binaries that some users may not expect.

Additionally, there are alternatives that people might have already
deployed. I would suggest seeking user feedback before enabling this by
default in OE-core.

Best regards,

Khem Raj

On Thu, Jul 23, 2026, 6:41 AM Richard Purdie via lists.openembedded.org
<richard.purdie=linuxfoundation.org@lists.openembedded.org> wrote:

> Enable the "minidebuginfo" DISTRO_FEATURE in builds so that all
> binaries have just enough debuginfo in them to get a useful stack trace.
>
> This aids with debugging occasional failures that hit the autobuilder,
> at minimal cost.  For example, core-image-sato with systemd:
>
>    core-image-sato: IMAGESIZE changed from 481532 to 486044 (+1%)
>    glibc: PKGSIZE changed from 3715743 to 3805871 (+2%)
>    sqlite3: PKGSIZE changed from 462840 to 469056 (+1%)
>
> [Idea taken from a poky patch from Ross Burton]
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  meta/conf/distro/include/default-distrovars.inc | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/conf/distro/include/default-distrovars.inc
> b/meta/conf/distro/include/default-distrovars.inc
> index 69c6db589b7..aea458d28c4 100644
> --- a/meta/conf/distro/include/default-distrovars.inc
> +++ b/meta/conf/distro/include/default-distrovars.inc
> @@ -29,6 +29,7 @@ DISTRO_FEATURES_DEFAULTS ?= " \
>      acl alsa bluetooth debuginfod ext2 ipv4 ipv6 \
>      wifi xattr nfs zeroconf pci 3g nfc x11 vfat seccomp pulseaudio \
>      gobject-introspection-data ldconfig opengl ptest multiarch wayland
> vulkan \
> +    minidebuginfo \
>      "
>  IMAGE_FEATURES ?= ""
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#241840):
> https://lists.openembedded.org/g/openembedded-core/message/241840
> Mute This Topic: https://lists.openembedded.org/mt/120410504/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Richard Purdie July 23, 2026, 8:12 p.m. UTC | #2
Hi Khem,

On Thu, 2026-07-23 at 07:48 -0700, Khem Raj wrote:
> Since default policies are generally used by derived distros,
> changing this default may have broader impacts. I wonder if the Yocto
> autobuilder issues you mentioned are commonly seen by OE users, as
> this change will embed symbols into binaries that some users may not
> expect.
> 
> Additionally, there are alternatives that people might have already
> deployed. I would suggest seeking user feedback before enabling this
> by default in OE-core.

Our defaults tend to reflect developer usage rather than fully hardened
and given that, I still would lean to make this the default, allowing
easier debugging of issues. I suspect if you have issues with the debug
symbols, you already set your own distro features...

The original proposal was to enable it in poky, however that would
"fork" nodistro and poky sstate and I'm not sure we want to do that? We
had decided to try and keep them more compatible for better sstate
reuse and faster builds "out the box".

I'm open to opinions on what we should do if anyone else has opinions.

Cheers,

Richard
diff mbox series

Patch

diff --git a/meta/conf/distro/include/default-distrovars.inc b/meta/conf/distro/include/default-distrovars.inc
index 69c6db589b7..aea458d28c4 100644
--- a/meta/conf/distro/include/default-distrovars.inc
+++ b/meta/conf/distro/include/default-distrovars.inc
@@ -29,6 +29,7 @@  DISTRO_FEATURES_DEFAULTS ?= " \
     acl alsa bluetooth debuginfod ext2 ipv4 ipv6 \
     wifi xattr nfs zeroconf pci 3g nfc x11 vfat seccomp pulseaudio \
     gobject-introspection-data ldconfig opengl ptest multiarch wayland vulkan \
+    minidebuginfo \
     "
 IMAGE_FEATURES ?= ""