public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: "'MOESSBAUER, Felix' via isar-users" <isar-users@googlegroups.com>
To: Zhihang Wei <wzh@ilbers.de>,
	"isar-users@googlegroups.com" <isar-users@googlegroups.com>,
	"Kiszka, Jan" <jan.kiszka@siemens.com>
Cc: "amikan@ilbers.de" <amikan@ilbers.de>
Subject: Re: [PATCH v11 0/3] Move kernel artifacts deployment from image recipe
Date: Mon, 29 Jun 2026 15:54:36 +0000	[thread overview]
Message-ID: <59dc65a158628c529c37f5f1c93337a7b33610e4.camel@siemens.com> (raw)
In-Reply-To: <51b0dfa9-9de6-4b03-86fe-c0acad5780a1@ilbers.de>

On Mon, 2026-06-29 at 17:21 +0200, Zhihang Wei wrote:
> On 6/9/26 10:44, Anton Mikanovich wrote:
> > Image task "do_copy_boot_files" is used to deploy kernel, initrd and
> > devicetree binaries to deploy directory.

Technically, only the kernel and device tree is deployed. The initrd is
deployed from the rootfs_generate_initramfs task (which currently
breaks reproducibility under sstate cache due to a bug I will fix once
rootless is merged).

I'll review the other patches as well now.

Felix

> > 
> > When different images for the same target are built (e.g., "-base" and
> > "-debug") in parallel, this causes issues with DTB deployment since
> > they have no any image_name-specific stuff in the path, unlike kernel
> > and initrd.
> > 
> > Here we move DTB deployment to the kernel recipe (for both distro and
> > custom kernel recipes), where DTBs and kernel image are extracted from
> > the kernel package.
> > For the distro kernels it means we install its deb into the schroot to
> > obtain the real artifacts from the filesystem.
> > 
> > To keep the same artifacts path as before "do_copy_boot_files" task is
> > now searching for the artifacts and create per-image symlinks.
> > In case any naming conflicts (like dtbs with the same name from
> > different kernels) the latest built dtb will be pointing by the
> > symlink, while both will be deployed to different subdirs without
> > conflicting and stay untouched.
> > This symlink creation is done in the same way as artifacts conficts are
> > handled by Yocto.
> > 
> > The case when different distros (e.g., bullseye/bookworm) are using the
> > same name for DTB binaries is covered by the patch 2. It also covers
> > the case when image uses different kernel suffix (e.g., "realtime"
> > option for the kernel in some downstreams).
> > 
> > Related topics on maillist:
> > 
> > https://groups.google.com/g/isar-users/c/ht3YmKZcJlM
> > https://groups.google.com/g/isar-users/c/4BRDM02xC40
> > https://groups.google.com/g/isar-users/c/qAnjahjjfsw
> > https://groups.google.com/g/isar-users/c/ZMD4XY4dKWQ
> > https://groups.google.com/g/isar-users/c/PSGU_AcdPZ8
> > https://groups.google.com/g/isar-users/c/Va0Ue-ISYeA
> > https://groups.google.com/g/isar-users/c/ixzytHYnTmI
> > 
> > Changes since v10:
> > - Rebase on next.
> > - Use the same deploy dir with per-kernel subdirs.
> > - Allow using different kernels for one machine.
> > - Restore paths for downstreams in Yocto way (symlinks).
> > 
> > Changes since v9:
> > - Base on v6.
> > - Rebase on next.
> > - Move also kernel deployment from image recipe.
> > 
> > Changes since v8:
> > - Rebase on next.
> > - Document how this approach differs from OpenEmbedded.
> > 
> > Changes since v7:
> > - Fix SOB.
> > 
> > Changes since v6:
> > - Fix isoimage-isohybrid-isar WIC plugin.
> > - Split testcases.
> > - Update wording.
> > 
> > Changes since v4:
> > - Rebase on next.
> > - Squash latest patch to first one.
> > - Fix deploy paths in initrd tests.
> > 
> > Changes since v3:
> > - Rebase on next.
> > - Fix target deploy dir path for installer.
> > 
> > Changes since v2:
> > - Patch 1: Consider also kernel suffix.
> > - Patch 2: Use separate testcase for possible dtb clashes.
> > - Minor cosmetic fixes.
> > 
> > Changes since v1:
> > - Separate `dtb-files` recipe used instead of linux one
> > - Included older "[PATCH] Fix do_copy_boot_files error" into the
> > series ( https://groups.google.com/g/isar-users/c/Va0Ue-ISYeA )
> > 
> > Anton Mikanovich (3):
> >    meta: Fix do_copy_boot_files error for different distros of same
> >      machine
> >    meta: Move kernel artifacts deployment from image recipe
> >    CI: Check kernel artifacts deployment
> > 
> >   RECIPE-API-CHANGELOG.md                       | 28 +++++++++++
> >   .../installer-add-rootfs.bbclass              |  2 +-
> >   meta/classes-recipe/image.bbclass             | 39 ++++++++--------
> >   meta/classes-recipe/linux-deploy.bbclass      | 46 +++++++++++++++++++
> >   meta/classes-recipe/linux-kernel.bbclass      |  3 ++
> >   meta/conf/bitbake.conf                        |  2 +-
> >   meta/recipes-kernel/linux/files/getkernel.sh  | 40 ++++++++++++++++
> >   meta/recipes-kernel/linux/files/rules.tmpl    | 12 +++++
> >   meta/recipes-kernel/linux/linux-distro.bb     | 27 +++++++++++
> >   testsuite/citest.py                           | 30 ++++++++----
> >   10 files changed, 199 insertions(+), 30 deletions(-)
> >   create mode 100644 meta/classes-recipe/linux-deploy.bbclass
> >   create mode 100755 meta/recipes-kernel/linux/files/getkernel.sh
> >   create mode 100644 meta/recipes-kernel/linux/files/rules.tmpl
> > 
> Hi all,
> 
> Any feedback or downstream testing results for this version? Thanks.
> 
> Zhihang
> 
> -- 
> You received this message because you are subscribed to the Google Groups "isar-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
> To view this discussion visit https://groups.google.com/d/msgid/isar-users/51b0dfa9-9de6-4b03-86fe-c0acad5780a1%40ilbers.de.

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/59dc65a158628c529c37f5f1c93337a7b33610e4.camel%40siemens.com.

  reply	other threads:[~2026-06-29 15:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-09  8:44 Anton Mikanovich
2026-06-09  8:44 ` [PATCH v11 1/3] meta: Fix do_copy_boot_files error for different distros of same machine Anton Mikanovich
2026-06-09  8:44 ` [PATCH v11 2/3] meta: Move kernel artifacts deployment from image recipe Anton Mikanovich
2026-06-29 16:04   ` 'MOESSBAUER, Felix' via isar-users
2026-06-30 12:21     ` Anton Mikanovich
2026-06-30 14:28       ` 'MOESSBAUER, Felix' via isar-users
2026-06-09  8:44 ` [PATCH v11 3/3] CI: Check kernel artifacts deployment Anton Mikanovich
2026-06-29 16:06   ` 'MOESSBAUER, Felix' via isar-users
2026-06-29 15:21 ` [PATCH v11 0/3] Move kernel artifacts deployment from image recipe Zhihang Wei
2026-06-29 15:54   ` 'MOESSBAUER, Felix' via isar-users [this message]
2026-06-30 12:21     ` Anton Mikanovich

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=59dc65a158628c529c37f5f1c93337a7b33610e4.camel@siemens.com \
    --to=isar-users@googlegroups.com \
    --cc=amikan@ilbers.de \
    --cc=felix.moessbauer@siemens.com \
    --cc=jan.kiszka@siemens.com \
    --cc=wzh@ilbers.de \
    /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