Re: [PATCH v3] kbuild: Strip runtime const RELA sections correctly

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Feb 1, 2025 at 4:02 AM Ard Biesheuvel <ardb@xxxxxxxxxx> wrote:
>
> On Fri, 31 Jan 2025 at 19:58, Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
> >
> > On Sat, Feb 1, 2025 at 3:40 AM Ard Biesheuvel <ardb@xxxxxxxxxx> wrote:
> > >
> > > On Fri, 31 Jan 2025 at 19:35, Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
> > > >
> > > > On Tue, Jan 14, 2025 at 12:53 AM Ard Biesheuvel <ardb+git@xxxxxxxxxx> wrote:
> > > > >
> > > > > From: Ard Biesheuvel <ardb@xxxxxxxxxx>
> > > > >
> > > > > Due to the fact that runtime const ELF sections are named without a
> > > > > leading period or double underscore, the RSTRIP logic that removes the
> > > > > static RELA sections from vmlinux fails to identify them.
> > > >
> > > >
> > > > Dumb question - why can't you rename the section 'runtime*' to '.runtime*' ?
> > > >
> > > > .rel* matches to sections that just happen to start with '.rel',
> > > > which may not be REL or RELA.
> > > >
> > >
> > > That is what I proposed originally [0] but it was shot down by Linus.
> > >
> > > [0] https://lore.kernel.org/all/20241014125703.2287936-4-ardb+git@xxxxxxxxxx/T/#u
> >
> >
> > Does this work ?
> >
> > $(OBJCOPY) --remove-relocations='*' $@
> >
>
> Only with binutils, not with llvm-objcopy


Applied to linux-kbuild.
Thanks.


-- 
Best Regards
Masahiro Yamada





[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux