ARM: kprobes-test: Workaround GAS .align bug
authorTaras Kondratiuk <taras.kondratiuk@linaro.org>
Fri, 8 Nov 2013 18:29:29 +0000 (18:29 +0000)
committerTaras Kondratiuk <taras@ti.com>
Tue, 1 Apr 2014 13:48:27 +0000 (16:48 +0300)
commit41b5368f31f86a1c2be2bb16a1cf18d23f1223d4
tree19bb9fd519dba304852611c94aa2aedb288309ba
parent3b86ee7a691def5dafef46a21ef4fc581f18d74c
ARM: kprobes-test: Workaround GAS .align bug

By default if no fill symbol is given to .align directive in a code
section it fills gap with NOPs. If previous fragment is not
instruction-aligned, additional pre-alignment is done by zero bytes
before NOPs. These zero bytes are marked as data by special symbol $d in
symbol table. Unfortunately GAS assumes that there is only code in the
code section so it "puts back" code symbol $a at the end of this
pre-alignment. So if there is some data after alignment it will be
interpreted as code and will be swapped back to LE for BE8 system during
a final linking.

If explicit fill value is given to .align, the NOP-padding code is
skipped and symbol table does not get messed-up.

So the workaround for this issue:
Use explicit fill value if data should be aligned in the code section.

Acked-by: Ben Dooks <ben.dooks@codethink.co.uk>
Acked-by: Jon Medhurst <tixy@linaro.org>
Signed-off-by: Taras Kondratiuk <taras.kondratiuk@linaro.org>
arch/arm/kernel/kprobes-test.c
arch/arm/kernel/kprobes-test.h