tree 6c64c199583b5105977730589070654f025d55b2
parent e92e8dc5c08f9a7093d02b12f384716033c6b79f
author Ard Biesheuvel <ard.biesheuvel@linaro.org> 1543867085 +0100
committer SamarV-121 <samarvispute121@pm.me> 1667709733 +0530
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQGzBAABCAAdFiEEeLjQnpaA0nK4ePoG6k72I5JuL64FAmNnOyUACgkQ6k72I5Ju
 L65Wmwv/Z9eFGuH77C+KqHPh3HF+lxzmifgGuCKx0Lq/nA7R6mtaWrnU15CERgdC
 yY6jORgjmjc/K2r0e//k8gcK7Un2Y4qxlThVSNAqJ+Dq8oLf7kzbg64QpTdB3uVo
 IK3hFAGEDQvBWI6kLwclpHr0aZS9Z/xlWvKlT9dGG5Ab6jaCODYJhM+byW1LInhe
 UUkGqzUr3VLlAdpG5sZAZCZZOFMgfTH+Q7wRkTpjlo+Mwz/ChDV0hc9/8vsKDN/M
 /xgK1rommRPjMyy18CfrEfkXdOyT3o2nei4HrEQSXzGqse6cIw72h44XzAQcVhoh
 f//KPqO5WueYK4i+B3o8YjZAOxDldoHJ7Ux+RBvVQ0THMAqfyl0hk25tviXbQkx/
 /3e3v5p39EUsn1puoJwyQ1Uozgip6FIG1dPje3oi/gJgig0CW6qX643a9tqLeuUg
 KIGTDvdpN3qOB6JX9rI+MfqaVrQtzht1EIPCa0tDcZw8rOXFQuzrndwkI1oefIw8
 IHIxZt0R
 =H0gT
 -----END PGP SIGNATURE-----

arm64: relocatable: fix inconsistencies in linker script and options

commit 3bbd3db86470c701091fb1d67f1fab6621debf50 upstream.

readelf complains about the section layout of vmlinux when building
with CONFIG_RELOCATABLE=y (for KASLR):

  readelf: Warning: [21]: Link field (0) should index a symtab section.
  readelf: Warning: [21]: Info field (0) should index a relocatable section.

Also, it seems that our use of '-pie -shared' is contradictory, and
thus ambiguous. In general, the way KASLR is wired up at the moment
is highly tailored to how ld.bfd happens to implement (and conflate)
PIE executables and shared libraries, so given the current effort to
support other toolchains, let's fix some of these issues as well.

- Drop the -pie linker argument and just leave -shared. In ld.bfd,
  the differences between them are unclear (except for the ELF type
  of the produced image [0]) but lld chokes on seeing both at the
  same time.

- Rename the .rela output section to .rela.dyn, as is customary for
  shared libraries and PIE executables, so that it is not misidentified
  by readelf as a static relocation section (producing the warnings
  above).

- Pass the -z notext and -z norelro options to explicitly instruct the
  linker to permit text relocations, and to omit the RELRO program
  header (which requires a certain section layout that we don't adhere
  to in the kernel). These are the defaults for current versions of
  ld.bfd.

- Discard .eh_frame and .gnu.hash sections to avoid them from being
  emitted between .head.text and .text, screwing up the section layout.

These changes only affect the ELF image, and produce the same binary
image.

[0] b9dce7f1ba01 ("arm64: kernel: force ET_DYN ELF type for ...")

Cc: Nick Desaulniers <ndesaulniers@google.com>
Cc: Peter Smith <peter.smith@linaro.org>
Tested-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Change-Id: I9fe6c4b09993a97051ea856a5053b220f2722872