Hello Anton,
Thanks for the fixes. The patch looks good, provides a complete solution,
and addresses all issues without requiring any additional changes
downstream layers. I tested the patch with various images that use
DTB_FILES, and it worked fine with all builds.
Regards,
Badrikesh
On Wednesday, October 8, 2025 at 3:15:56 PM UTC+5:30 Anton Mikanovich wrote:
> 08/10/2025 09:20, 'Badrikesh Prusty' via isar-users wrote:
> > Hello all,
> >
> > Apologies for the delayed response. I somehow missed the message.
> >
> > > This sounds familiar, and I think we discussed that multiple times
> > > already. Did you study previous threads/patches, e.g.
> > > https://patchwork.isar-build.org/project/isar/list/?series=1241?
> > >
> > > Yes, there were few patches that tried to solve the issue.
> > >
> > > Using subdirectory (variant similar to TS's)
> > > - https://patchwork.isar-build.org/project/isar/list/?series=1149
> > > Allows to overlap files
> > > - https://patchwork.isar-build.org/project/isar/list/?series=1235
> > > Make kernel deploy the files, instead of image recipes
> > > - https://patchwork.isar-build.org/project/isar/list/?series=1241
> > >
> >
> > I reviewed the previously shared patch, but not in full — thanks for
> > sharing it here.
> >
> > The patch I sent is very similar to @nicusor
> > patch: https://patchwork.isar-build.org/project/isar/list/?series=1149
> >
> > I just reviewed the other patches and believe that many may encounter
> > the same issue. For instance, when attempting to build multiple
> > kernels for a single image—such as `linux-image-amd64` and
> > `linux-image-amd64-rt` kernels—this overwrite problem is likely to occur.
> >
> > This patch also resolves the issue with image builds that include
> > multiple kernels by storing the DTB files in the directory
> > DEPLOYDIR/IMAGE_FULLNAME/KERNEL_NAME. This will also fix issues when
> > building with different DISTROs as IMAGE_FULLNAME includes target
> > DISTRO. Additionally, it fixes problems that arise when building with
> > different DISTROs, since IMAGE_FULLNAME includes the target DISTRO.
> >
> > > This would affect downstream as well. You are missing a
> > > RECIPE-API-CHANGELOG.md entry.
> > >
> >
> > Yes, it will affect downstream layers. E.g. I checked with the
> > isar-cip-core repo, changes will be required in `efibootguard-boot`
> > wic plugin. I will work on fixing it, once this is approved.
> >
> > Thanks and Regards,
> > Badrikesh
> Hello Badrikesh,
>
> Files like scripts/lib/wic/plugins/source/bootimg-efi.py are borrowed
> from the
> upstream and supposed to have as minimal changes as possible. Even having
> changes in custom WIC plugins can results in additional side-effects.
>
> Also, in general, DTBs should be part of a kernel, but not an image.
> I've just released the new version of DTB deployment fix proposal which
> I hope
> will cover all the cases. Please have a look:
>
>
> https://patchwork.isar-build.org/project/isar/cover/20251008085507....@ilbers.de/
>
>
>
> There are also some observations regarding other implementations of the
> same
> fix were shared on the list previously:
>
>
> https://patchwork.isar-build.org/project/isar/patch/20240416100746.568...@mentor.com/
>
>
> - We don't want to modify WIC plugins if solving outside of them is
> possible.
>
>
> https://patchwork.isar-build.org/project/isar/patch/20240705071856.904-...@siemens.com/
>
>
> - Manipulating of sstate sounds tricky and errors might be costly.
> - Allowing overlapping will cause deploy file loosing on multiple target
> setup.
>
>
--
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/f80e75b1-28aa-47ec-a0f6-1f41f3cc3d51n%40googlegroups.com.