diff mbox series

[meta-arago,master] arago: Pull in the Yocto space optimization settings

Message ID 20250811173948.19890-1-reatmon@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,master] arago: Pull in the Yocto space optimization settings | expand

Commit Message

Ryan Eatmon Aug. 11, 2025, 5:39 p.m. UTC
Turns out there is a .inc file in oe-core that sets some variables to
reduce the amount of space that a build takes up based on common
criteria.  Poky explicitly includes this file, but it is not included by
default.  So we need to include it as well.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-arago-distro/conf/distro/arago.conf | 2 ++
 1 file changed, 2 insertions(+)

Comments

PRC Automation Aug. 11, 2025, 5:45 p.m. UTC | #1
meta-arago / na / 20250811173948.19890-1-reatmon

PRC Results: PASS

=========================================================
  check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed



=========================================================
  apply-yocto-patch: PASS
=========================================================
master
=====================
Summary:
- Patch Series: [meta-arago][master][PATCH] arago: Pull in the Yocto space optimization settings
- Submitter: From: Ryan Eatmon <reatmon@ti.com>
- Date: Date: Mon, 11 Aug 2025 12:39:48 -0500
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: d33a6f12d48e8cba4b8bcaf0a7493b2caadb8220

Applied to:
- Repository: lcpd-prc-meta-arago
- Base Branch: master-next
- Commit Author: LCPD Automation Script <lcpdbld@list.ti.com>
- Commit Subject: CI/CD Auto-Merger: cicd.master.202508010100
- Commit SHA: 644f8b47f0eef26bda636717f5838871ec67d138

Patches
----------------------------------------
All patches applied



=========================================================
  check-yocto-repo: PASS
=========================================================
master
=====================
PASS
Denys Dmytriyenko Aug. 11, 2025, 6:37 p.m. UTC | #2
On Mon, Aug 11, 2025 at 12:39:48PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> Turns out there is a .inc file in oe-core that sets some variables to
> reduce the amount of space that a build takes up based on common
> criteria.  Poky explicitly includes this file, but it is not included by
> default.  So we need to include it as well.

Just a note:

It is a bit of a confusing name - it doesn't optimize all target binaries for 
space (vs. speed). What this .inc file does - it completely disables debugging 
output (which would otherwise be stripped into -dbg packages) during the 
build for a handful of components, such as all versions of clang (used to be 
llvm) and qemu. Those do generate a lot of debug data. The point is to save 
storage space and a bit of build time on autobuilder, assuming that nobody 
will be trying to debug those tools.


> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  meta-arago-distro/conf/distro/arago.conf | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
> index 8e45ca53..1ad4b4ce 100644
> --- a/meta-arago-distro/conf/distro/arago.conf
> +++ b/meta-arago-distro/conf/distro/arago.conf
> @@ -6,6 +6,8 @@ BUILD_VARIANT = "${@['prod','rt'][int(d.getVar('ARAGO_RT_ENABLE'))]}"
>  TISDK_VERSION ?= "live"
>  FEEDURIPREFIX ?= "builds/${DISTRO}-master/${BUILD_VARIANT}/${MACHINE}/latest/packages/"
>  
> +require conf/distro/include/yocto-space-optimize.inc
> +
>  OPKG_ARGS:remove = "--prefer-arch-to-version"
>  
>  TI_MIRROR = "http://software-dl.ti.com/processor-sdk-mirror/sources/"
> -- 
> 2.17.1
Jon Cormier Aug. 11, 2025, 6:48 p.m. UTC | #3
On Mon, Aug 11, 2025 at 2:37 PM Denys Dmytriyenko via lists.yoctoproject.org
<denis=denix.org@lists.yoctoproject.org> wrote:

> On Mon, Aug 11, 2025 at 12:39:48PM -0500, Ryan Eatmon via
> lists.yoctoproject.org wrote:
> > Turns out there is a .inc file in oe-core that sets some variables to
> > reduce the amount of space that a build takes up based on common
> > criteria.  Poky explicitly includes this file, but it is not included by
> > default.  So we need to include it as well.
>
> Just a note:
>
> It is a bit of a confusing name - it doesn't optimize all target binaries
> for
> space (vs. speed). What this .inc file does - it completely disables
> debugging
> output (which would otherwise be stripped into -dbg packages) during the
> build for a handful of components, such as all versions of clang (used to
> be
> llvm) and qemu. Those do generate a lot of debug data. The point is to
> save
> storage space and a bit of build time on autobuilder, assuming that nobody
> will be trying to debug those tools.
>
Thanks, this is really useful additional info.

On a slight tangent, any idea why arago builds don't enable rm_work by
default?  In our testings, it speeds up builds slightly, greatly reduces
space usage, and more importantly makes deleting the work dir way way
faster. With the only downside being occasionally you need to add
RM_WORK_EXCLUDES when debugging a build failure.

>
>
> > Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> > ---
> >  meta-arago-distro/conf/distro/arago.conf | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/meta-arago-distro/conf/distro/arago.conf
> b/meta-arago-distro/conf/distro/arago.conf
> > index 8e45ca53..1ad4b4ce 100644
> > --- a/meta-arago-distro/conf/distro/arago.conf
> > +++ b/meta-arago-distro/conf/distro/arago.conf
> > @@ -6,6 +6,8 @@ BUILD_VARIANT =
> "${@['prod','rt'][int(d.getVar('ARAGO_RT_ENABLE'))]}"
> >  TISDK_VERSION ?= "live"
> >  FEEDURIPREFIX ?=
> "builds/${DISTRO}-master/${BUILD_VARIANT}/${MACHINE}/latest/packages/"
> >
> > +require conf/distro/include/yocto-space-optimize.inc
> > +
> >  OPKG_ARGS:remove = "--prefer-arch-to-version"
> >
> >  TI_MIRROR = "http://software-dl.ti.com/processor-sdk-mirror/sources/"
> > --
> > 2.17.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#16359):
> https://lists.yoctoproject.org/g/meta-arago/message/16359
> Mute This Topic: https://lists.yoctoproject.org/mt/114651955/7902621
> Group Owner: meta-arago+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [
> jcormier@criticallink.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
>
diff mbox series

Patch

diff --git a/meta-arago-distro/conf/distro/arago.conf b/meta-arago-distro/conf/distro/arago.conf
index 8e45ca53..1ad4b4ce 100644
--- a/meta-arago-distro/conf/distro/arago.conf
+++ b/meta-arago-distro/conf/distro/arago.conf
@@ -6,6 +6,8 @@  BUILD_VARIANT = "${@['prod','rt'][int(d.getVar('ARAGO_RT_ENABLE'))]}"
 TISDK_VERSION ?= "live"
 FEEDURIPREFIX ?= "builds/${DISTRO}-master/${BUILD_VARIANT}/${MACHINE}/latest/packages/"
 
+require conf/distro/include/yocto-space-optimize.inc
+
 OPKG_ARGS:remove = "--prefer-arch-to-version"
 
 TI_MIRROR = "http://software-dl.ti.com/processor-sdk-mirror/sources/"