diff options
| author | Marc Zyngier <marc.zyngier@arm.com> | 2015-06-01 10:47:42 +0100 |
|---|---|---|
| committer | Catalin Marinas <catalin.marinas@arm.com> | 2015-06-05 10:38:54 +0100 |
| commit | eb7c11ee3c5ce6c45ac28a5015a8e60ed458b412 (patch) | |
| tree | 62b2a7c0b2639a364c09ad3d3631495ca07871ae /arch/arm64/kernel | |
| parent | 8d883b23aed73cad844ba48051c7e96eddf0f51c (diff) | |
| download | op-kernel-dev-eb7c11ee3c5ce6c45ac28a5015a8e60ed458b412.zip op-kernel-dev-eb7c11ee3c5ce6c45ac28a5015a8e60ed458b412.tar.gz | |
arm64: alternative: Work around .inst assembler bugs
AArch64 toolchains suffer from the following bug:
$ cat blah.S
1:
.inst 0x01020304
.if ((. - 1b) != 4)
.error "blah"
.endif
$ aarch64-linux-gnu-gcc -c blah.S
blah.S: Assembler messages:
blah.S:3: Error: non-constant expression in ".if" statement
which precludes the use of msr_s and co as part of alternatives.
We workaround this issue by not directly testing the labels
themselves, but by moving the current output pointer by a value
that should always be zero. If this value is not null, then
we will trigger a backward move, which is expclicitely forbidden.
This triggers the error we're after:
AS arch/arm64/kvm/hyp.o
arch/arm64/kvm/hyp.S: Assembler messages:
arch/arm64/kvm/hyp.S:1377: Error: attempt to move .org backwards
scripts/Makefile.build:294: recipe for target 'arch/arm64/kvm/hyp.o' failed
make[1]: *** [arch/arm64/kvm/hyp.o] Error 1
Makefile:946: recipe for target 'arch/arm64/kvm' failed
Not pretty, but at least works on the current toolchains.
Acked-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Diffstat (limited to 'arch/arm64/kernel')
0 files changed, 0 insertions, 0 deletions
