Thread (38 messages) 38 messages, 4 authors, 2020-12-02

Re: [PATCHv4 08/17] arm64: uaccess: move uao_* alternatives to asm-uaccess.h

flat view

From: Mark Rutland <mark.rutland@arm.com>
Date: 2020-11-16 12:47:12

On Sat, Nov 14, 2020 at 10:22:15AM +0100, Christoph Hellwig wrote:
quoted
+#ifdef CONFIG_ARM64_UAO
+	.macro uao_ldp l, reg1, reg2, addr, post_inc
+		alternative_if_not ARM64_HAS_UAO
+8888:			ldp	\reg1, \reg2, [\addr], \post_inc;
+8889:			nop;
No actually new in this patch, but that is the reason for these horrible
label names?
It's a historic way of having non-conflicting labels. I also hate it,
and intend to clean it up subsequently.

The new/preferred way for labels in asm macros is to use a descriptive
local label with a '\@' suffix to ensure true uniqueness, e.g.

| 	.macro foo
| 	cbz	x0, .Lskip\@
| 	mov	x0, x1
| .Lskip\@:
| 	.endm

... which I assume is what you'd also like here.

Thanks,
Mark.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help