Hello Anton,
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.