On Mon, Jun 10, 2024 at 2:21 PM Dragan Simic <dsimic@xxxxxxxxxxx> wrote: > > The compiled dtb files aren't executable, so install them with 0644 as their > permission mode, instead of defaulting to 0755 for the permission mode and > installing them with the executable bits set. > > Some Linux distributions, including Debian, [1][2][3] already include fixes > in their kernel package build recipes to change the dtb file permissions to > 0644 in their kernel packages. These changes, when additionally propagated > into the long-term kernel versions, will allow such distributions to remove > their downstream fixes. > > [1] https://salsa.debian.org/kernel-team/linux/-/merge_requests/642 > [2] https://salsa.debian.org/kernel-team/linux/-/merge_requests/749 > [3] https://salsa.debian.org/kernel-team/linux/-/blob/master/debian/rules.real?ref_type=heads#L193 The 'master' is a moving target. The line 193 in the future may not point to the correct position I changed it to this. [3] https://salsa.debian.org/kernel-team/linux/-/blob/debian/6.8.12-1/debian/rules.real#L193 This references the line 193 from the 'debian/6.8.12-1' tag. Applied to linux-kbuild. Thanks! > Cc: Diederik de Haas <didi.debian@xxxxxxxxx> > Cc: stable@xxxxxxxxxxxxxxx > Fixes: aefd80307a05 ("kbuild: refactor Makefile.dtbinst more") > Signed-off-by: Dragan Simic <dsimic@xxxxxxxxxxx> > --- > > Notes: > Changes in v2: > - Improved the patch description, to include additional details and > to address the patch submission issues pointed out by Greg K-H [4] > - No changes were made to the patch itself > > Link to v1: https://lore.kernel.org/linux-kbuild/ae087ef1715142f606ba6477ace3e4111972cf8b.1717961381.git.dsimic@xxxxxxxxxxx/T/#u > > [4] https://lore.kernel.org/linux-kbuild/2024061006-ladylike-paving-a36b@gregkh/ > > scripts/Makefile.dtbinst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/Makefile.dtbinst b/scripts/Makefile.dtbinst > index 67956f6496a5..9d920419a62c 100644 > --- a/scripts/Makefile.dtbinst > +++ b/scripts/Makefile.dtbinst > @@ -17,7 +17,7 @@ include $(srctree)/scripts/Kbuild.include > dst := $(INSTALL_DTBS_PATH) > > quiet_cmd_dtb_install = INSTALL $@ > - cmd_dtb_install = install -D $< $@ > + cmd_dtb_install = install -D -m 0644 $< $@ > > $(dst)/%: $(obj)/% > $(call cmd,dtb_install) -- Best Regards Masahiro Yamada
