@@ -383,21 +383,23 @@ alternative_endif
sub \tmp2, \tmp1, #1
bic \kaddr, \kaddr, \tmp2
9998:
- .if (\op == cvau || \op == cvac)
+ .ifnc \op, civac
+ .ifnc \op, cvap
alternative_if_not ARM64_WORKAROUND_CLEAN_CACHE
dc \op, \kaddr
alternative_else
dc civac, \kaddr
alternative_endif
- .elseif (\op == cvap)
+ .else
alternative_cb arm64_handle_dc_cvap
dc civac, \kaddr
alternative_cb_alt
sys 3, c7, c12, 1, \kaddr // dc cvap
dc cvac, \kaddr
alternative_cb_end
+ .endif
.else
- dc \op, \kaddr
+ dc civac, \kaddr
.endif
add \kaddr, \kaddr, \tmp1
cmp \kaddr, \size
The GAS directives that are currently being used in dcache_by_line_op rely on assembler behavior that is not documented, and probably not guaranteed to produce the correct behavior going forward. Currently, we end up with some undefined symbols in cache.o: $ nm arch/arm64/mm/cache.o ... U civac ... U cvac U cvap U cvau This is due to the fact that the comparisons used to select the operation type in the dcache_by_line_op macro are comparing symbols not strings, and even though it seems that GAS is doing the right thing here (undefined symbols by the same name are equal to each other), it seems unwise to rely on this. So let's switch to conditional directives that are documented as operating on strings. Since these cannot be combined like ordinary arithmetic expressions, invert the comparison logic. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> --- arch/arm64/include/asm/assembler.h | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)