Messages in this thread | | | From | Ard Biesheuvel <> | Date | Sun, 11 Mar 2018 20:35:58 +0000 | Subject | Re: [PATCH v8 1/6] arch: enable relative relocations for arm64, power and x86 |
| |
On 11 March 2018 at 20:20, Linus Torvalds <torvalds@linux-foundation.org> wrote: > On Sun, Mar 11, 2018 at 5:38 AM, Ard Biesheuvel > <ard.biesheuvel@linaro.org> wrote: >> Before updating certain subsystems to use place relative 32-bit >> relocations in special sections, to save space and reduce the >> number of absolute relocations that need to be processed at runtime >> by relocatable kernels, introduce the Kconfig symbol and define it >> for some architectures that should be able to support and benefit >> from it. > > Are there actually architectures hat _don't_ support those relative > 32-bit relocations? That really feels pretty fundamental. >
I started out doing all of them, and I don't remember the exact list, but there are quite a few architectures that don't support these relocations in their module loaders, and in fact, not even ARM does (as one of the 'big' architectures).
This is not really surprising, given that the C compiler never emits such relative references.
> I would have expected all of them to do it - is your selection of > arm64/powerpc/x86 perhaps more about "I have tested these" than about > anything else? > > Because I'd almost prefer to just do the switch-over unconditionally > if that is at all possible? >
arm64, powerpc and x86 implement CONFIG_RELOCATABLE, and so they benefit more than other architectures, because of the fact that the relocation metadata for these data structures can be dropped entirely. Other 64-bit architectures only have the 50% size reduction benefit, and 32-bit architectures have no benefit at all.
I'm sure all of these architectures define some kind of 32-bit place relative relocation in their ELF psABI, and I see how it would be cleaner to change everything at once, but I anticipate a long tail of issues with toolchains for niche architectures that I have no way to test.
| |