linux-mips.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MIPS: Avoid macro redefinitions
@ 2021-07-18 13:07 Paul Cercueil
  2021-07-19 21:43 ` Nick Desaulniers
  0 siblings, 1 reply; 6+ messages in thread
From: Paul Cercueil @ 2021-07-18 13:07 UTC (permalink / raw)
  To: Thomas Bogendoerfer
  Cc: Nick Desaulniers, linux-mips, linux-kernel, list, Paul Cercueil

To be able to compile the kernel with LTO, the assembler macros cannot
be declared in the global scope, or the compiler will complain about
redefined macros.

Update the code so that macros are defined then undefined when they are
used.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
---
 arch/mips/include/asm/ginvt.h                 |  11 +-
 .../asm/mach-loongson64/loongson_regs.h       |  12 ++
 arch/mips/include/asm/mipsregs.h              | 192 +++++++++++-------
 arch/mips/include/asm/msa.h                   |  34 +++-
 4 files changed, 157 insertions(+), 92 deletions(-)

diff --git a/arch/mips/include/asm/ginvt.h b/arch/mips/include/asm/ginvt.h
index 6eb7c2b94dc7..87b2974ffc53 100644
--- a/arch/mips/include/asm/ginvt.h
+++ b/arch/mips/include/asm/ginvt.h
@@ -12,11 +12,13 @@ enum ginvt_type {
 
 #ifdef TOOLCHAIN_SUPPORTS_GINV
 # define _ASM_SET_GINV	".set	ginv\n"
+# define _ASM_UNSET_GINV
 #else
-_ASM_MACRO_1R1I(ginvt, rs, type,
-		_ASM_INSN_IF_MIPS(0x7c0000bd | (__rs << 21) | (\\type << 8))
-		_ASM_INSN32_IF_MM(0x0000717c | (__rs << 16) | (\\type << 9)));
-# define _ASM_SET_GINV
+# define _ASM_SET_GINV							\
+	_ASM_MACRO_1R1I(ginvt, rs, type,				\
+			_ASM_INSN_IF_MIPS(0x7c0000bd | (__rs << 21) | (\\type << 8))	\
+			_ASM_INSN32_IF_MM(0x0000717c | (__rs << 16) | (\\type << 9)))
+# define _ASM_UNSET_GINV ".purgem ginvt\n"
 #endif
 
 static __always_inline void ginvt(unsigned long addr, enum ginvt_type type)
@@ -25,6 +27,7 @@ static __always_inline void ginvt(unsigned long addr, enum ginvt_type type)
 		".set	push\n"
 		_ASM_SET_GINV
 		"	ginvt	%0, %1\n"
+		_ASM_UNSET_GINV
 		".set	pop"
 		: /* no outputs */
 		: "r"(addr), "i"(type)
diff --git a/arch/mips/include/asm/mach-loongson64/loongson_regs.h b/arch/mips/include/asm/mach-loongson64/loongson_regs.h
index 165993514762..b5be7511f6cd 100644
--- a/arch/mips/include/asm/mach-loongson64/loongson_regs.h
+++ b/arch/mips/include/asm/mach-loongson64/loongson_regs.h
@@ -21,8 +21,10 @@ static inline u32 read_cpucfg(u32 reg)
 	u32 __res;
 
 	__asm__ __volatile__(
+		_ASM_SET_PARSE_R
 		"parse_r __res,%0\n\t"
 		"parse_r reg,%1\n\t"
+		_ASM_UNSET_PARSE_R
 		".insn \n\t"
 		".word (0xc8080118 | (reg << 21) | (__res << 11))\n\t"
 		:"=r"(__res)
@@ -143,8 +145,10 @@ static inline u32 csr_readl(u32 reg)
 
 	/* RDCSR reg, val */
 	__asm__ __volatile__(
+		_ASM_SET_PARSE_R
 		"parse_r __res,%0\n\t"
 		"parse_r reg,%1\n\t"
+		_ASM_UNSET_PARSE_R
 		".insn \n\t"
 		".word (0xc8000118 | (reg << 21) | (__res << 11))\n\t"
 		:"=r"(__res)
@@ -160,8 +164,10 @@ static inline u64 csr_readq(u32 reg)
 
 	/* DRDCSR reg, val */
 	__asm__ __volatile__(
+		_ASM_SET_PARSE_R
 		"parse_r __res,%0\n\t"
 		"parse_r reg,%1\n\t"
+		_ASM_UNSET_PARSE_R
 		".insn \n\t"
 		".word (0xc8020118 | (reg << 21) | (__res << 11))\n\t"
 		:"=r"(__res)
@@ -175,8 +181,10 @@ static inline void csr_writel(u32 val, u32 reg)
 {
 	/* WRCSR reg, val */
 	__asm__ __volatile__(
+		_ASM_SET_PARSE_R
 		"parse_r reg,%0\n\t"
 		"parse_r val,%1\n\t"
+		_ASM_UNSET_PARSE_R
 		".insn \n\t"
 		".word (0xc8010118 | (reg << 21) | (val << 11))\n\t"
 		:
@@ -189,8 +197,10 @@ static inline void csr_writeq(u64 val, u32 reg)
 {
 	/* DWRCSR reg, val */
 	__asm__ __volatile__(
+		_ASM_SET_PARSE_R
 		"parse_r reg,%0\n\t"
 		"parse_r val,%1\n\t"
+		_ASM_UNSET_PARSE_R
 		".insn \n\t"
 		".word (0xc8030118 | (reg << 21) | (val << 11))\n\t"
 		:
@@ -243,8 +253,10 @@ static inline u64 drdtime(void)
 	u64 val = 0;
 
 	__asm__ __volatile__(
+		_ASM_SET_PARSE_R
 		"parse_r rID,%0\n\t"
 		"parse_r val,%1\n\t"
+		_ASM_UNSET_PARSE_R
 		".insn \n\t"
 		".word (0xc8090118 | (rID << 21) | (val << 11))\n\t"
 		:"=r"(rID),"=r"(val)
diff --git a/arch/mips/include/asm/mipsregs.h b/arch/mips/include/asm/mipsregs.h
index acdf8c69220b..f44f041e24de 100644
--- a/arch/mips/include/asm/mipsregs.h
+++ b/arch/mips/include/asm/mipsregs.h
@@ -1297,22 +1297,24 @@ static inline int mm_insn_16bit(u16 insn)
 	"\\var	= " #n "\n\t"			\
 	".endif\n\t"
 
-__asm__(".macro	parse_r var r\n\t"
-	"\\var	= -1\n\t"
-	_IFC_REG(0)  _IFC_REG(1)  _IFC_REG(2)  _IFC_REG(3)
-	_IFC_REG(4)  _IFC_REG(5)  _IFC_REG(6)  _IFC_REG(7)
-	_IFC_REG(8)  _IFC_REG(9)  _IFC_REG(10) _IFC_REG(11)
-	_IFC_REG(12) _IFC_REG(13) _IFC_REG(14) _IFC_REG(15)
-	_IFC_REG(16) _IFC_REG(17) _IFC_REG(18) _IFC_REG(19)
-	_IFC_REG(20) _IFC_REG(21) _IFC_REG(22) _IFC_REG(23)
-	_IFC_REG(24) _IFC_REG(25) _IFC_REG(26) _IFC_REG(27)
-	_IFC_REG(28) _IFC_REG(29) _IFC_REG(30) _IFC_REG(31)
-	".iflt	\\var\n\t"
-	".error	\"Unable to parse register name \\r\"\n\t"
-	".endif\n\t"
-	".endm");
-
-#undef _IFC_REG
+#define _ASM_SET_PARSE_R						\
+	".macro	parse_r var r\n\t"					\
+	"\\var	= -1\n\t"						\
+	_IFC_REG(0)  _IFC_REG(1)  _IFC_REG(2)  _IFC_REG(3)		\
+	_IFC_REG(4)  _IFC_REG(5)  _IFC_REG(6)  _IFC_REG(7)		\
+	_IFC_REG(8)  _IFC_REG(9)  _IFC_REG(10) _IFC_REG(11)		\
+	_IFC_REG(12) _IFC_REG(13) _IFC_REG(14) _IFC_REG(15)		\
+	_IFC_REG(16) _IFC_REG(17) _IFC_REG(18) _IFC_REG(19)		\
+	_IFC_REG(20) _IFC_REG(21) _IFC_REG(22) _IFC_REG(23)		\
+	_IFC_REG(24) _IFC_REG(25) _IFC_REG(26) _IFC_REG(27)		\
+	_IFC_REG(28) _IFC_REG(29) _IFC_REG(30) _IFC_REG(31)		\
+	".iflt	\\var\n\t"						\
+	".error	\"Unable to parse register name \\r\"\n\t"		\
+	".endif\n\t"							\
+	".endm\n\t"
+#define _ASM_UNSET_PARSE_R ".purgem parse_r\n\t"
+
+//#undef _IFC_REG
 
 /*
  * C macros for generating assembler macros for common instruction formats.
@@ -1322,43 +1324,45 @@ __asm__(".macro	parse_r var r\n\t"
  * the ENC encodings.
  */
 
-/* Instructions with no operands */
-#define _ASM_MACRO_0(OP, ENC)						\
-	__asm__(".macro	" #OP "\n\t"					\
-		ENC							\
-		".endm")
-
 /* Instructions with 1 register operand & 1 immediate operand */
 #define _ASM_MACRO_1R1I(OP, R1, I2, ENC)				\
-	__asm__(".macro	" #OP " " #R1 ", " #I2 "\n\t"			\
+		".macro	" #OP " " #R1 ", " #I2 "\n\t"			\
+		_ASM_SET_PARSE_R					\
 		"parse_r __" #R1 ", \\" #R1 "\n\t"			\
 		ENC							\
-		".endm")
+		_ASM_UNSET_PARSE_R					\
+		".endm\n\t"
 
 /* Instructions with 2 register operands */
 #define _ASM_MACRO_2R(OP, R1, R2, ENC)					\
-	__asm__(".macro	" #OP " " #R1 ", " #R2 "\n\t"			\
+		".macro	" #OP " " #R1 ", " #R2 "\n\t"			\
+		_ASM_SET_PARSE_R					\
 		"parse_r __" #R1 ", \\" #R1 "\n\t"			\
 		"parse_r __" #R2 ", \\" #R2 "\n\t"			\
 		ENC							\
-		".endm")
+		_ASM_UNSET_PARSE_R					\
+		".endm\n\t"
 
 /* Instructions with 3 register operands */
 #define _ASM_MACRO_3R(OP, R1, R2, R3, ENC)				\
-	__asm__(".macro	" #OP " " #R1 ", " #R2 ", " #R3 "\n\t"		\
+		".macro	" #OP " " #R1 ", " #R2 ", " #R3 "\n\t"		\
+		_ASM_SET_PARSE_R					\
 		"parse_r __" #R1 ", \\" #R1 "\n\t"			\
 		"parse_r __" #R2 ", \\" #R2 "\n\t"			\
 		"parse_r __" #R3 ", \\" #R3 "\n\t"			\
 		ENC							\
-		".endm")
+		_ASM_UNSET_PARSE_R					\
+		".endm\n\t"
 
 /* Instructions with 2 register operands and 1 optional select operand */
 #define _ASM_MACRO_2R_1S(OP, R1, R2, SEL3, ENC)				\
-	__asm__(".macro	" #OP " " #R1 ", " #R2 ", " #SEL3 " = 0\n\t"	\
+		".macro	" #OP " " #R1 ", " #R2 ", " #SEL3 " = 0\n\t"	\
+		_ASM_SET_PARSE_R					\
 		"parse_r __" #R1 ", \\" #R1 "\n\t"			\
 		"parse_r __" #R2 ", \\" #R2 "\n\t"			\
 		ENC							\
-		".endm")
+		_ASM_UNSET_PARSE_R					\
+		".endm\n\t"
 
 /*
  * TLB Invalidate Flush
@@ -1618,15 +1622,21 @@ do {									\
 } while (0)
 
 #ifndef TOOLCHAIN_SUPPORTS_XPA
-_ASM_MACRO_2R_1S(mfhc0, rt, rs, sel,
-	_ASM_INSN_IF_MIPS(0x40400000 | __rt << 16 | __rs << 11 | \\sel)
-	_ASM_INSN32_IF_MM(0x000000f4 | __rt << 21 | __rs << 16 | \\sel << 11));
-_ASM_MACRO_2R_1S(mthc0, rt, rd, sel,
-	_ASM_INSN_IF_MIPS(0x40c00000 | __rt << 16 | __rd << 11 | \\sel)
-	_ASM_INSN32_IF_MM(0x000002f4 | __rt << 21 | __rd << 16 | \\sel << 11));
-#define _ASM_SET_XPA ""
+#define _ASM_SET_MFHC0							\
+	_ASM_MACRO_2R_1S(mfhc0, rt, rs, sel,				\
+			 _ASM_INSN_IF_MIPS(0x40400000 | __rt << 16 | __rs << 11 | \\sel)	\
+			 _ASM_INSN32_IF_MM(0x000000f4 | __rt << 21 | __rs << 16 | \\sel << 11))
+#define _ASM_UNSET_MFHC0 ".purgem mfhc0\n\t"
+#define _ASM_SET_MTHC0							\
+	_ASM_MACRO_2R_1S(mthc0, rt, rd, sel,				\
+			 _ASM_INSN_IF_MIPS(0x40c00000 | __rt << 16 | __rd << 11 | \\sel)	\
+			 _ASM_INSN32_IF_MM(0x000002f4 | __rt << 21 | __rd << 16 | \\sel << 11))
+#define _ASM_UNSET_MTHC0 ".purgem mthc0\n\t"
 #else	/* !TOOLCHAIN_SUPPORTS_XPA */
-#define _ASM_SET_XPA ".set\txpa\n\t"
+#define _ASM_SET_MFHC0 ".set\txpa\n\t"
+#define _ASM_SET_MTHC0 ".set\txpa\n\t"
+#define _ASM_UNSET_MFHC0
+#define _ASM_UNSET_MTHC0
 #endif
 
 #define __readx_32bit_c0_register(source, sel)				\
@@ -1636,8 +1646,9 @@ _ASM_MACRO_2R_1S(mthc0, rt, rd, sel,
 	__asm__ __volatile__(						\
 	"	.set	push					\n"	\
 	"	.set	mips32r2				\n"	\
-	_ASM_SET_XPA							\
+	_ASM_SET_MFHC0							\
 	"	mfhc0	%0, " #source ", %1			\n"	\
+	_ASM_UNSET_MFHC0						\
 	"	.set	pop					\n"	\
 	: "=r" (__res)							\
 	: "i" (sel));							\
@@ -1649,8 +1660,9 @@ do {									\
 	__asm__ __volatile__(						\
 	"	.set	push					\n"	\
 	"	.set	mips32r2				\n"	\
-	_ASM_SET_XPA							\
+	_ASM_SET_MTHC0							\
 	"	mthc0	%z0, " #register ", %1			\n"	\
+	_ASM_UNSET_MTHC0						\
 	"	.set	pop					\n"	\
 	:								\
 	: "Jr" (value), "i" (sel));					\
@@ -2046,31 +2058,58 @@ do {									\
  */
 
 #ifndef TOOLCHAIN_SUPPORTS_VIRT
-_ASM_MACRO_2R_1S(mfgc0, rt, rs, sel,
-	_ASM_INSN_IF_MIPS(0x40600000 | __rt << 16 | __rs << 11 | \\sel)
-	_ASM_INSN32_IF_MM(0x000004fc | __rt << 21 | __rs << 16 | \\sel << 11));
-_ASM_MACRO_2R_1S(dmfgc0, rt, rs, sel,
-	_ASM_INSN_IF_MIPS(0x40600100 | __rt << 16 | __rs << 11 | \\sel)
-	_ASM_INSN32_IF_MM(0x580004fc | __rt << 21 | __rs << 16 | \\sel << 11));
-_ASM_MACRO_2R_1S(mtgc0, rt, rd, sel,
-	_ASM_INSN_IF_MIPS(0x40600200 | __rt << 16 | __rd << 11 | \\sel)
-	_ASM_INSN32_IF_MM(0x000006fc | __rt << 21 | __rd << 16 | \\sel << 11));
-_ASM_MACRO_2R_1S(dmtgc0, rt, rd, sel,
-	_ASM_INSN_IF_MIPS(0x40600300 | __rt << 16 | __rd << 11 | \\sel)
-	_ASM_INSN32_IF_MM(0x580006fc | __rt << 21 | __rd << 16 | \\sel << 11));
-_ASM_MACRO_0(tlbgp,    _ASM_INSN_IF_MIPS(0x42000010)
-		       _ASM_INSN32_IF_MM(0x0000017c));
-_ASM_MACRO_0(tlbgr,    _ASM_INSN_IF_MIPS(0x42000009)
-		       _ASM_INSN32_IF_MM(0x0000117c));
-_ASM_MACRO_0(tlbgwi,   _ASM_INSN_IF_MIPS(0x4200000a)
-		       _ASM_INSN32_IF_MM(0x0000217c));
-_ASM_MACRO_0(tlbgwr,   _ASM_INSN_IF_MIPS(0x4200000e)
-		       _ASM_INSN32_IF_MM(0x0000317c));
-_ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
-		       _ASM_INSN32_IF_MM(0x0000517c));
-#define _ASM_SET_VIRT ""
+#define _ASM_SET_MFGC0							\
+	_ASM_MACRO_2R_1S(mfgc0, rt, rs, sel,				\
+			 _ASM_INSN_IF_MIPS(0x40600000 | __rt << 16 | __rs << 11 | \\sel)	\
+			 _ASM_INSN32_IF_MM(0x000004fc | __rt << 21 | __rs << 16 | \\sel << 11))
+#define _ASM_UNSET_MFGC0 ".purgem mfgc0\n\t"
+#define _ASM_SET_DMFGC0							\
+	_ASM_MACRO_2R_1S(dmfgc0, rt, rs, sel,				\
+			 _ASM_INSN_IF_MIPS(0x40600100 | __rt << 16 | __rs << 11 | \\sel)	\
+			 _ASM_INSN32_IF_MM(0x580004fc | __rt << 21 | __rs << 16 | \\sel << 11))
+#define _ASM_UNSET_DMFGC0 ".purgem dmfgc0\n\t"
+#define _ASM_SET_MTGC0							\
+	_ASM_MACRO_2R_1S(mtgc0, rt, rd, sel,				\
+			 _ASM_INSN_IF_MIPS(0x40600200 | __rt << 16 | __rd << 11 | \\sel)	\
+			 _ASM_INSN32_IF_MM(0x000006fc | __rt << 21 | __rd << 16 | \\sel << 11))
+#define _ASM_UNSET_MTGC0 ".purgem mtgc0\n\t"
+#define _ASM_SET_DMTGC0							\
+	_ASM_MACRO_2R_1S(dmtgc0, rt, rd, sel,				\
+			 _ASM_INSN_IF_MIPS(0x40600300 | __rt << 16 | __rd << 11 | \\sel)	\
+			 _ASM_INSN32_IF_MM(0x580006fc | __rt << 21 | __rd << 16 | \\sel << 11))
+#define _ASM_UNSET_DMTGC0 ".purgem dmtgc0\n\t"
+
+#define __tlbgp()							\
+		_ASM_INSN_IF_MIPS(0x42000010)				\
+		_ASM_INSN32_IF_MM(0x0000017c)
+#define __tlbgr()							\
+		_ASM_INSN_IF_MIPS(0x42000009)				\
+		_ASM_INSN32_IF_MM(0x0000117c)
+#define __tlbgwi()							\
+		_ASM_INSN_IF_MIPS(0x4200000a)				\
+		_ASM_INSN32_IF_MM(0x0000217c)
+#define __tlbgwr()							\
+		_ASM_INSN_IF_MIPS(0x4200000e)				\
+		_ASM_INSN32_IF_MM(0x0000317c)
+#define __tlbginvf()							\
+		_ASM_INSN_IF_MIPS(0x4200000c)				\
+		_ASM_INSN32_IF_MM(0x0000517c)
 #else	/* !TOOLCHAIN_SUPPORTS_VIRT */
 #define _ASM_SET_VIRT ".set\tvirt\n\t"
+#define _ASM_SET_MFGC0	_ASM_SET_VIRT
+#define _ASM_SET_DMFGC0	_ASM_SET_VIRT
+#define _ASM_SET_MTGC0	_ASM_SET_VIRT
+#define _ASM_SET_DMTGC0	_ASM_SET_VIRT
+#define _ASM_UNSET_MFGC0
+#define _ASM_UNSET_DMFGC0
+#define _ASM_UNSET_MTGC0
+#define _ASM_UNSET_DMTGC0
+
+#define __tlbgp()	_ASM_SET_VIRT "tlbgp\n\t"
+#define __tlbgr()	_ASM_SET_VIRT "tlbgr\n\t"
+#define __tlbgwi()	_ASM_SET_VIRT "tlbgwi\n\t"
+#define __tlbgwr()	_ASM_SET_VIRT "tlbgwr\n\t"
+#define __tlbginvf()	_ASM_SET_VIRT "tlbginvf\n\t"
 #endif
 
 #define __read_32bit_gc0_register(source, sel)				\
@@ -2078,8 +2117,9 @@ _ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
 	__asm__ __volatile__(						\
 		".set\tpush\n\t"					\
 		".set\tmips32r5\n\t"					\
-		_ASM_SET_VIRT						\
+		_ASM_SET_MFGC0						\
 		"mfgc0\t%0, " #source ", %1\n\t"			\
+		_ASM_UNSET_MFGC0					\
 		".set\tpop"						\
 		: "=r" (__res)						\
 		: "i" (sel));						\
@@ -2091,8 +2131,9 @@ _ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
 	__asm__ __volatile__(						\
 		".set\tpush\n\t"					\
 		".set\tmips64r5\n\t"					\
-		_ASM_SET_VIRT						\
+		_ASM_SET_DMFGC0						\
 		"dmfgc0\t%0, " #source ", %1\n\t"			\
+		_ASM_UNSET_DMFGC0					\
 		".set\tpop"						\
 		: "=r" (__res)						\
 		: "i" (sel));						\
@@ -2104,8 +2145,9 @@ do {									\
 	__asm__ __volatile__(						\
 		".set\tpush\n\t"					\
 		".set\tmips32r5\n\t"					\
-		_ASM_SET_VIRT						\
+		_ASM_SET_MTGC0						\
 		"mtgc0\t%z0, " #register ", %1\n\t"			\
+		_ASM_UNSET_MTGC0					\
 		".set\tpop"						\
 		: : "Jr" ((unsigned int)(value)),			\
 		    "i" (sel));						\
@@ -2116,8 +2158,9 @@ do {									\
 	__asm__ __volatile__(						\
 		".set\tpush\n\t"					\
 		".set\tmips64r5\n\t"					\
-		_ASM_SET_VIRT						\
+		_ASM_SET_DMTGC0						\
 		"dmtgc0\t%z0, " #register ", %1\n\t"			\
+		_ASM_UNSET_DMTGC0					\
 		".set\tpop"						\
 		: : "Jr" (value),					\
 		    "i" (sel));						\
@@ -2788,8 +2831,7 @@ static inline void guest_tlb_probe(void)
 	__asm__ __volatile__(
 		".set push\n\t"
 		".set noreorder\n\t"
-		_ASM_SET_VIRT
-		"tlbgp\n\t"
+		__tlbgp()
 		".set pop");
 }
 
@@ -2798,8 +2840,7 @@ static inline void guest_tlb_read(void)
 	__asm__ __volatile__(
 		".set push\n\t"
 		".set noreorder\n\t"
-		_ASM_SET_VIRT
-		"tlbgr\n\t"
+		__tlbgr()
 		".set pop");
 }
 
@@ -2808,8 +2849,7 @@ static inline void guest_tlb_write_indexed(void)
 	__asm__ __volatile__(
 		".set push\n\t"
 		".set noreorder\n\t"
-		_ASM_SET_VIRT
-		"tlbgwi\n\t"
+		__tlbgwi()
 		".set pop");
 }
 
@@ -2818,8 +2858,7 @@ static inline void guest_tlb_write_random(void)
 	__asm__ __volatile__(
 		".set push\n\t"
 		".set noreorder\n\t"
-		_ASM_SET_VIRT
-		"tlbgwr\n\t"
+		__tlbgwr()
 		".set pop");
 }
 
@@ -2831,8 +2870,7 @@ static inline void guest_tlbinvf(void)
 	__asm__ __volatile__(
 		".set push\n\t"
 		".set noreorder\n\t"
-		_ASM_SET_VIRT
-		"tlbginvf\n\t"
+		__tlbginvf()
 		".set pop");
 }
 
diff --git a/arch/mips/include/asm/msa.h b/arch/mips/include/asm/msa.h
index e0a3dd52334d..236a49ee2e3e 100644
--- a/arch/mips/include/asm/msa.h
+++ b/arch/mips/include/asm/msa.h
@@ -162,16 +162,26 @@ static inline void init_msa_upper(void)
  * to allow compilation with toolchains that do not support MSA. Once all
  * toolchains in use support MSA these can be removed.
  */
-_ASM_MACRO_2R(cfcmsa, rd, cs,
-	_ASM_INSN_IF_MIPS(0x787e0019 | __cs << 11 | __rd << 6)
-	_ASM_INSN32_IF_MM(0x587e0016 | __cs << 11 | __rd << 6));
-_ASM_MACRO_2R(ctcmsa, cd, rs,
-	_ASM_INSN_IF_MIPS(0x783e0019 | __rs << 11 | __cd << 6)
-	_ASM_INSN32_IF_MM(0x583e0016 | __rs << 11 | __cd << 6));
-#define _ASM_SET_MSA ""
+
+#define _ASM_SET_CFCMSA							\
+	_ASM_MACRO_2R(cfcmsa, rd, cs,					\
+		      _ASM_INSN_IF_MIPS(0x787e0019 | __cs << 11 | __rd << 6)	\
+		      _ASM_INSN32_IF_MM(0x587e0016 | __cs << 11 | __rd << 6))
+#define _ASM_UNSET_CFCMSA ".purgem cfcmsa\n\t"
+#define _ASM_SET_CTCMSA							\
+	_ASM_MACRO_2R(ctcmsa, cd, rs,					\
+		      _ASM_INSN_IF_MIPS(0x783e0019 | __rs << 11 | __cd << 6)	\
+		      _ASM_INSN32_IF_MM(0x583e0016 | __rs << 11 | __cd << 6))
+#define _ASM_UNSET_CTCMSA ".purgem ctcmsa\n\t"
 #else /* TOOLCHAIN_SUPPORTS_MSA */
-#define _ASM_SET_MSA ".set\tfp=64\n\t"				\
-		     ".set\tmsa\n\t"
+#define _ASM_SET_CFCMSA						\
+		".set\tfp=64\n\t"				\
+		".set\tmsa\n\t"
+#define _ASM_UNSET_CFCMSA
+#define _ASM_SET_CTCMSA						\
+		".set\tfp=64\n\t"				\
+		".set\tmsa\n\t"
+#define _ASM_UNSET_CTCMSA
 #endif
 
 #define __BUILD_MSA_CTL_REG(name, cs)				\
@@ -180,8 +190,9 @@ static inline unsigned int read_msa_##name(void)		\
 	unsigned int reg;					\
 	__asm__ __volatile__(					\
 	"	.set	push\n"					\
-	_ASM_SET_MSA						\
+	_ASM_SET_CFCMSA						\
 	"	cfcmsa	%0, $" #cs "\n"				\
+	_ASM_UNSET_CFCMSA					\
 	"	.set	pop\n"					\
 	: "=r"(reg));						\
 	return reg;						\
@@ -191,8 +202,9 @@ static inline void write_msa_##name(unsigned int val)		\
 {								\
 	__asm__ __volatile__(					\
 	"	.set	push\n"					\
-	_ASM_SET_MSA						\
+	_ASM_SET_CTCMSA						\
 	"	ctcmsa	$" #cs ", %0\n"				\
+	_ASM_UNSET_CTCMSA					\
 	"	.set	pop\n"					\
 	: : "r"(val));						\
 }
-- 
2.30.2


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MIPS: Avoid macro redefinitions
  2021-07-18 13:07 [PATCH] MIPS: Avoid macro redefinitions Paul Cercueil
@ 2021-07-19 21:43 ` Nick Desaulniers
  2021-07-19 22:10   ` Paul Cercueil
  0 siblings, 1 reply; 6+ messages in thread
From: Nick Desaulniers @ 2021-07-19 21:43 UTC (permalink / raw)
  To: Paul Cercueil
  Cc: Thomas Bogendoerfer, linux-mips, linux-kernel, list, Kees Cook,
	clang-built-linux

On Sun, Jul 18, 2021 at 6:07 AM Paul Cercueil <paul@crapouillou.net> wrote:
>
> To be able to compile the kernel with LTO, the assembler macros cannot
> be declared in the global scope, or the compiler will complain about
> redefined macros.

Thanks for the patch! Minor comments below, so untested yet by me.
Please cc clang-built-linux@googlegroups.com for any follow ups.

If this is towards supporting LTO, consider sending as part of a
series enabling LTO for MIPS. That would be cool to see.

>
> Update the code so that macros are defined then undefined when they are
> used.
>
> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
> ---
>  arch/mips/include/asm/ginvt.h                 |  11 +-
>  .../asm/mach-loongson64/loongson_regs.h       |  12 ++
>  arch/mips/include/asm/mipsregs.h              | 192 +++++++++++-------
>  arch/mips/include/asm/msa.h                   |  34 +++-
>  4 files changed, 157 insertions(+), 92 deletions(-)
>
> diff --git a/arch/mips/include/asm/ginvt.h b/arch/mips/include/asm/ginvt.h
> index 6eb7c2b94dc7..87b2974ffc53 100644
> --- a/arch/mips/include/asm/ginvt.h
> +++ b/arch/mips/include/asm/ginvt.h
> @@ -12,11 +12,13 @@ enum ginvt_type {
>
>  #ifdef TOOLCHAIN_SUPPORTS_GINV
>  # define _ASM_SET_GINV ".set   ginv\n"
> +# define _ASM_UNSET_GINV
>  #else
> -_ASM_MACRO_1R1I(ginvt, rs, type,
> -               _ASM_INSN_IF_MIPS(0x7c0000bd | (__rs << 21) | (\\type << 8))
> -               _ASM_INSN32_IF_MM(0x0000717c | (__rs << 16) | (\\type << 9)));
> -# define _ASM_SET_GINV
> +# define _ASM_SET_GINV                                                 \
> +       _ASM_MACRO_1R1I(ginvt, rs, type,                                \
> +                       _ASM_INSN_IF_MIPS(0x7c0000bd | (__rs << 21) | (\\type << 8))    \
> +                       _ASM_INSN32_IF_MM(0x0000717c | (__rs << 16) | (\\type << 9)))
> +# define _ASM_UNSET_GINV ".purgem ginvt\n"
>  #endif
>
>  static __always_inline void ginvt(unsigned long addr, enum ginvt_type type)
> @@ -25,6 +27,7 @@ static __always_inline void ginvt(unsigned long addr, enum ginvt_type type)
>                 ".set   push\n"
>                 _ASM_SET_GINV
>                 "       ginvt   %0, %1\n"
> +               _ASM_UNSET_GINV
>                 ".set   pop"
>                 : /* no outputs */
>                 : "r"(addr), "i"(type)
> diff --git a/arch/mips/include/asm/mach-loongson64/loongson_regs.h b/arch/mips/include/asm/mach-loongson64/loongson_regs.h
> index 165993514762..b5be7511f6cd 100644
> --- a/arch/mips/include/asm/mach-loongson64/loongson_regs.h
> +++ b/arch/mips/include/asm/mach-loongson64/loongson_regs.h
> @@ -21,8 +21,10 @@ static inline u32 read_cpucfg(u32 reg)
>         u32 __res;
>
>         __asm__ __volatile__(
> +               _ASM_SET_PARSE_R
>                 "parse_r __res,%0\n\t"
>                 "parse_r reg,%1\n\t"
> +               _ASM_UNSET_PARSE_R
>                 ".insn \n\t"
>                 ".word (0xc8080118 | (reg << 21) | (__res << 11))\n\t"
>                 :"=r"(__res)
> @@ -143,8 +145,10 @@ static inline u32 csr_readl(u32 reg)
>
>         /* RDCSR reg, val */
>         __asm__ __volatile__(
> +               _ASM_SET_PARSE_R
>                 "parse_r __res,%0\n\t"
>                 "parse_r reg,%1\n\t"
> +               _ASM_UNSET_PARSE_R
>                 ".insn \n\t"
>                 ".word (0xc8000118 | (reg << 21) | (__res << 11))\n\t"
>                 :"=r"(__res)
> @@ -160,8 +164,10 @@ static inline u64 csr_readq(u32 reg)
>
>         /* DRDCSR reg, val */
>         __asm__ __volatile__(
> +               _ASM_SET_PARSE_R
>                 "parse_r __res,%0\n\t"
>                 "parse_r reg,%1\n\t"
> +               _ASM_UNSET_PARSE_R
>                 ".insn \n\t"
>                 ".word (0xc8020118 | (reg << 21) | (__res << 11))\n\t"
>                 :"=r"(__res)
> @@ -175,8 +181,10 @@ static inline void csr_writel(u32 val, u32 reg)
>  {
>         /* WRCSR reg, val */
>         __asm__ __volatile__(
> +               _ASM_SET_PARSE_R
>                 "parse_r reg,%0\n\t"
>                 "parse_r val,%1\n\t"
> +               _ASM_UNSET_PARSE_R
>                 ".insn \n\t"
>                 ".word (0xc8010118 | (reg << 21) | (val << 11))\n\t"
>                 :
> @@ -189,8 +197,10 @@ static inline void csr_writeq(u64 val, u32 reg)
>  {
>         /* DWRCSR reg, val */
>         __asm__ __volatile__(
> +               _ASM_SET_PARSE_R
>                 "parse_r reg,%0\n\t"
>                 "parse_r val,%1\n\t"
> +               _ASM_UNSET_PARSE_R
>                 ".insn \n\t"
>                 ".word (0xc8030118 | (reg << 21) | (val << 11))\n\t"
>                 :
> @@ -243,8 +253,10 @@ static inline u64 drdtime(void)
>         u64 val = 0;
>
>         __asm__ __volatile__(
> +               _ASM_SET_PARSE_R
>                 "parse_r rID,%0\n\t"
>                 "parse_r val,%1\n\t"
> +               _ASM_UNSET_PARSE_R
>                 ".insn \n\t"
>                 ".word (0xc8090118 | (rID << 21) | (val << 11))\n\t"
>                 :"=r"(rID),"=r"(val)
> diff --git a/arch/mips/include/asm/mipsregs.h b/arch/mips/include/asm/mipsregs.h
> index acdf8c69220b..f44f041e24de 100644
> --- a/arch/mips/include/asm/mipsregs.h
> +++ b/arch/mips/include/asm/mipsregs.h
> @@ -1297,22 +1297,24 @@ static inline int mm_insn_16bit(u16 insn)
>         "\\var  = " #n "\n\t"                   \
>         ".endif\n\t"
>
> -__asm__(".macro        parse_r var r\n\t"
> -       "\\var  = -1\n\t"
> -       _IFC_REG(0)  _IFC_REG(1)  _IFC_REG(2)  _IFC_REG(3)
> -       _IFC_REG(4)  _IFC_REG(5)  _IFC_REG(6)  _IFC_REG(7)
> -       _IFC_REG(8)  _IFC_REG(9)  _IFC_REG(10) _IFC_REG(11)
> -       _IFC_REG(12) _IFC_REG(13) _IFC_REG(14) _IFC_REG(15)
> -       _IFC_REG(16) _IFC_REG(17) _IFC_REG(18) _IFC_REG(19)
> -       _IFC_REG(20) _IFC_REG(21) _IFC_REG(22) _IFC_REG(23)
> -       _IFC_REG(24) _IFC_REG(25) _IFC_REG(26) _IFC_REG(27)
> -       _IFC_REG(28) _IFC_REG(29) _IFC_REG(30) _IFC_REG(31)
> -       ".iflt  \\var\n\t"
> -       ".error \"Unable to parse register name \\r\"\n\t"
> -       ".endif\n\t"
> -       ".endm");
> -
> -#undef _IFC_REG
> +#define _ASM_SET_PARSE_R                                               \
> +       ".macro parse_r var r\n\t"                                      \
> +       "\\var  = -1\n\t"                                               \
> +       _IFC_REG(0)  _IFC_REG(1)  _IFC_REG(2)  _IFC_REG(3)              \
> +       _IFC_REG(4)  _IFC_REG(5)  _IFC_REG(6)  _IFC_REG(7)              \
> +       _IFC_REG(8)  _IFC_REG(9)  _IFC_REG(10) _IFC_REG(11)             \
> +       _IFC_REG(12) _IFC_REG(13) _IFC_REG(14) _IFC_REG(15)             \
> +       _IFC_REG(16) _IFC_REG(17) _IFC_REG(18) _IFC_REG(19)             \
> +       _IFC_REG(20) _IFC_REG(21) _IFC_REG(22) _IFC_REG(23)             \
> +       _IFC_REG(24) _IFC_REG(25) _IFC_REG(26) _IFC_REG(27)             \
> +       _IFC_REG(28) _IFC_REG(29) _IFC_REG(30) _IFC_REG(31)             \
> +       ".iflt  \\var\n\t"                                              \
> +       ".error \"Unable to parse register name \\r\"\n\t"              \
> +       ".endif\n\t"                                                    \
> +       ".endm\n\t"
> +#define _ASM_UNSET_PARSE_R ".purgem parse_r\n\t"
> +
> +//#undef _IFC_REG

^ did you mean to commit that commented-out #undef?

>
>  /*
>   * C macros for generating assembler macros for common instruction formats.
> @@ -1322,43 +1324,45 @@ __asm__(".macro parse_r var r\n\t"
>   * the ENC encodings.
>   */
>
> -/* Instructions with no operands */
> -#define _ASM_MACRO_0(OP, ENC)                                          \
> -       __asm__(".macro " #OP "\n\t"                                    \
> -               ENC                                                     \
> -               ".endm")
> -
>  /* Instructions with 1 register operand & 1 immediate operand */
>  #define _ASM_MACRO_1R1I(OP, R1, I2, ENC)                               \
> -       __asm__(".macro " #OP " " #R1 ", " #I2 "\n\t"                   \
> +               ".macro " #OP " " #R1 ", " #I2 "\n\t"                   \
> +               _ASM_SET_PARSE_R                                        \
>                 "parse_r __" #R1 ", \\" #R1 "\n\t"                      \
>                 ENC                                                     \
> -               ".endm")
> +               _ASM_UNSET_PARSE_R                                      \
> +               ".endm\n\t"
>
>  /* Instructions with 2 register operands */
>  #define _ASM_MACRO_2R(OP, R1, R2, ENC)                                 \
> -       __asm__(".macro " #OP " " #R1 ", " #R2 "\n\t"                   \
> +               ".macro " #OP " " #R1 ", " #R2 "\n\t"                   \
> +               _ASM_SET_PARSE_R                                        \
>                 "parse_r __" #R1 ", \\" #R1 "\n\t"                      \
>                 "parse_r __" #R2 ", \\" #R2 "\n\t"                      \
>                 ENC                                                     \
> -               ".endm")
> +               _ASM_UNSET_PARSE_R                                      \
> +               ".endm\n\t"
>
>  /* Instructions with 3 register operands */
>  #define _ASM_MACRO_3R(OP, R1, R2, R3, ENC)                             \
> -       __asm__(".macro " #OP " " #R1 ", " #R2 ", " #R3 "\n\t"          \
> +               ".macro " #OP " " #R1 ", " #R2 ", " #R3 "\n\t"          \
> +               _ASM_SET_PARSE_R                                        \
>                 "parse_r __" #R1 ", \\" #R1 "\n\t"                      \
>                 "parse_r __" #R2 ", \\" #R2 "\n\t"                      \
>                 "parse_r __" #R3 ", \\" #R3 "\n\t"                      \
>                 ENC                                                     \
> -               ".endm")
> +               _ASM_UNSET_PARSE_R                                      \
> +               ".endm\n\t"
>
>  /* Instructions with 2 register operands and 1 optional select operand */
>  #define _ASM_MACRO_2R_1S(OP, R1, R2, SEL3, ENC)                                \
> -       __asm__(".macro " #OP " " #R1 ", " #R2 ", " #SEL3 " = 0\n\t"    \
> +               ".macro " #OP " " #R1 ", " #R2 ", " #SEL3 " = 0\n\t"    \
> +               _ASM_SET_PARSE_R                                        \
>                 "parse_r __" #R1 ", \\" #R1 "\n\t"                      \
>                 "parse_r __" #R2 ", \\" #R2 "\n\t"                      \
>                 ENC                                                     \
> -               ".endm")
> +               _ASM_UNSET_PARSE_R                                      \
> +               ".endm\n\t"
>
>  /*
>   * TLB Invalidate Flush
> @@ -1618,15 +1622,21 @@ do {                                                                    \
>  } while (0)
>
>  #ifndef TOOLCHAIN_SUPPORTS_XPA

When did GAS gain support for XPA? Do we still need to support
!TOOLCHAIN_SUPPORTS_XPA? (though that's orthogonal to this patch, such
a cleanup first might simplify this patch).

> -_ASM_MACRO_2R_1S(mfhc0, rt, rs, sel,
> -       _ASM_INSN_IF_MIPS(0x40400000 | __rt << 16 | __rs << 11 | \\sel)
> -       _ASM_INSN32_IF_MM(0x000000f4 | __rt << 21 | __rs << 16 | \\sel << 11));
> -_ASM_MACRO_2R_1S(mthc0, rt, rd, sel,
> -       _ASM_INSN_IF_MIPS(0x40c00000 | __rt << 16 | __rd << 11 | \\sel)
> -       _ASM_INSN32_IF_MM(0x000002f4 | __rt << 21 | __rd << 16 | \\sel << 11));
> -#define _ASM_SET_XPA ""
> +#define _ASM_SET_MFHC0                                                 \
> +       _ASM_MACRO_2R_1S(mfhc0, rt, rs, sel,                            \
> +                        _ASM_INSN_IF_MIPS(0x40400000 | __rt << 16 | __rs << 11 | \\sel)        \
> +                        _ASM_INSN32_IF_MM(0x000000f4 | __rt << 21 | __rs << 16 | \\sel << 11))
> +#define _ASM_UNSET_MFHC0 ".purgem mfhc0\n\t"
> +#define _ASM_SET_MTHC0                                                 \
> +       _ASM_MACRO_2R_1S(mthc0, rt, rd, sel,                            \
> +                        _ASM_INSN_IF_MIPS(0x40c00000 | __rt << 16 | __rd << 11 | \\sel)        \
> +                        _ASM_INSN32_IF_MM(0x000002f4 | __rt << 21 | __rd << 16 | \\sel << 11))
> +#define _ASM_UNSET_MTHC0 ".purgem mthc0\n\t"
>  #else  /* !TOOLCHAIN_SUPPORTS_XPA */
> -#define _ASM_SET_XPA ".set\txpa\n\t"
> +#define _ASM_SET_MFHC0 ".set\txpa\n\t"
> +#define _ASM_SET_MTHC0 ".set\txpa\n\t"
> +#define _ASM_UNSET_MFHC0
> +#define _ASM_UNSET_MTHC0
>  #endif
>
>  #define __readx_32bit_c0_register(source, sel)                         \
> @@ -1636,8 +1646,9 @@ _ASM_MACRO_2R_1S(mthc0, rt, rd, sel,
>         __asm__ __volatile__(                                           \
>         "       .set    push                                    \n"     \
>         "       .set    mips32r2                                \n"     \
> -       _ASM_SET_XPA                                                    \
> +       _ASM_SET_MFHC0                                                  \
>         "       mfhc0   %0, " #source ", %1                     \n"     \
> +       _ASM_UNSET_MFHC0                                                \
>         "       .set    pop                                     \n"     \
>         : "=r" (__res)                                                  \
>         : "i" (sel));                                                   \
> @@ -1649,8 +1660,9 @@ do {                                                                      \
>         __asm__ __volatile__(                                           \
>         "       .set    push                                    \n"     \
>         "       .set    mips32r2                                \n"     \
> -       _ASM_SET_XPA                                                    \
> +       _ASM_SET_MTHC0                                                  \
>         "       mthc0   %z0, " #register ", %1                  \n"     \
> +       _ASM_UNSET_MTHC0                                                \
>         "       .set    pop                                     \n"     \
>         :                                                               \
>         : "Jr" (value), "i" (sel));                                     \
> @@ -2046,31 +2058,58 @@ do {                                                                    \
>   */
>
>  #ifndef TOOLCHAIN_SUPPORTS_VIRT

Same question for GAS version support for virt?
Documentation/process/changes.rst
says GNU binutils 2.23+ is required for building the kernel.
If we still need to support, have you tested this change on such an
older version of GNU binutils?

> -_ASM_MACRO_2R_1S(mfgc0, rt, rs, sel,
> -       _ASM_INSN_IF_MIPS(0x40600000 | __rt << 16 | __rs << 11 | \\sel)
> -       _ASM_INSN32_IF_MM(0x000004fc | __rt << 21 | __rs << 16 | \\sel << 11));
> -_ASM_MACRO_2R_1S(dmfgc0, rt, rs, sel,
> -       _ASM_INSN_IF_MIPS(0x40600100 | __rt << 16 | __rs << 11 | \\sel)
> -       _ASM_INSN32_IF_MM(0x580004fc | __rt << 21 | __rs << 16 | \\sel << 11));
> -_ASM_MACRO_2R_1S(mtgc0, rt, rd, sel,
> -       _ASM_INSN_IF_MIPS(0x40600200 | __rt << 16 | __rd << 11 | \\sel)
> -       _ASM_INSN32_IF_MM(0x000006fc | __rt << 21 | __rd << 16 | \\sel << 11));
> -_ASM_MACRO_2R_1S(dmtgc0, rt, rd, sel,
> -       _ASM_INSN_IF_MIPS(0x40600300 | __rt << 16 | __rd << 11 | \\sel)
> -       _ASM_INSN32_IF_MM(0x580006fc | __rt << 21 | __rd << 16 | \\sel << 11));
> -_ASM_MACRO_0(tlbgp,    _ASM_INSN_IF_MIPS(0x42000010)
> -                      _ASM_INSN32_IF_MM(0x0000017c));
> -_ASM_MACRO_0(tlbgr,    _ASM_INSN_IF_MIPS(0x42000009)
> -                      _ASM_INSN32_IF_MM(0x0000117c));
> -_ASM_MACRO_0(tlbgwi,   _ASM_INSN_IF_MIPS(0x4200000a)
> -                      _ASM_INSN32_IF_MM(0x0000217c));
> -_ASM_MACRO_0(tlbgwr,   _ASM_INSN_IF_MIPS(0x4200000e)
> -                      _ASM_INSN32_IF_MM(0x0000317c));
> -_ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
> -                      _ASM_INSN32_IF_MM(0x0000517c));
> -#define _ASM_SET_VIRT ""
> +#define _ASM_SET_MFGC0                                                 \
> +       _ASM_MACRO_2R_1S(mfgc0, rt, rs, sel,                            \
> +                        _ASM_INSN_IF_MIPS(0x40600000 | __rt << 16 | __rs << 11 | \\sel)        \
> +                        _ASM_INSN32_IF_MM(0x000004fc | __rt << 21 | __rs << 16 | \\sel << 11))
> +#define _ASM_UNSET_MFGC0 ".purgem mfgc0\n\t"
> +#define _ASM_SET_DMFGC0                                                        \
> +       _ASM_MACRO_2R_1S(dmfgc0, rt, rs, sel,                           \
> +                        _ASM_INSN_IF_MIPS(0x40600100 | __rt << 16 | __rs << 11 | \\sel)        \
> +                        _ASM_INSN32_IF_MM(0x580004fc | __rt << 21 | __rs << 16 | \\sel << 11))
> +#define _ASM_UNSET_DMFGC0 ".purgem dmfgc0\n\t"
> +#define _ASM_SET_MTGC0                                                 \
> +       _ASM_MACRO_2R_1S(mtgc0, rt, rd, sel,                            \
> +                        _ASM_INSN_IF_MIPS(0x40600200 | __rt << 16 | __rd << 11 | \\sel)        \
> +                        _ASM_INSN32_IF_MM(0x000006fc | __rt << 21 | __rd << 16 | \\sel << 11))
> +#define _ASM_UNSET_MTGC0 ".purgem mtgc0\n\t"
> +#define _ASM_SET_DMTGC0                                                        \
> +       _ASM_MACRO_2R_1S(dmtgc0, rt, rd, sel,                           \
> +                        _ASM_INSN_IF_MIPS(0x40600300 | __rt << 16 | __rd << 11 | \\sel)        \
> +                        _ASM_INSN32_IF_MM(0x580006fc | __rt << 21 | __rd << 16 | \\sel << 11))
> +#define _ASM_UNSET_DMTGC0 ".purgem dmtgc0\n\t"
> +
> +#define __tlbgp()                                                      \
> +               _ASM_INSN_IF_MIPS(0x42000010)                           \
> +               _ASM_INSN32_IF_MM(0x0000017c)
> +#define __tlbgr()                                                      \
> +               _ASM_INSN_IF_MIPS(0x42000009)                           \
> +               _ASM_INSN32_IF_MM(0x0000117c)
> +#define __tlbgwi()                                                     \
> +               _ASM_INSN_IF_MIPS(0x4200000a)                           \
> +               _ASM_INSN32_IF_MM(0x0000217c)
> +#define __tlbgwr()                                                     \
> +               _ASM_INSN_IF_MIPS(0x4200000e)                           \
> +               _ASM_INSN32_IF_MM(0x0000317c)
> +#define __tlbginvf()                                                   \
> +               _ASM_INSN_IF_MIPS(0x4200000c)                           \
> +               _ASM_INSN32_IF_MM(0x0000517c)
>  #else  /* !TOOLCHAIN_SUPPORTS_VIRT */
>  #define _ASM_SET_VIRT ".set\tvirt\n\t"
> +#define _ASM_SET_MFGC0 _ASM_SET_VIRT
> +#define _ASM_SET_DMFGC0        _ASM_SET_VIRT
> +#define _ASM_SET_MTGC0 _ASM_SET_VIRT
> +#define _ASM_SET_DMTGC0        _ASM_SET_VIRT
> +#define _ASM_UNSET_MFGC0
> +#define _ASM_UNSET_DMFGC0
> +#define _ASM_UNSET_MTGC0
> +#define _ASM_UNSET_DMTGC0
> +
> +#define __tlbgp()      _ASM_SET_VIRT "tlbgp\n\t"
> +#define __tlbgr()      _ASM_SET_VIRT "tlbgr\n\t"
> +#define __tlbgwi()     _ASM_SET_VIRT "tlbgwi\n\t"
> +#define __tlbgwr()     _ASM_SET_VIRT "tlbgwr\n\t"
> +#define __tlbginvf()   _ASM_SET_VIRT "tlbginvf\n\t"
>  #endif
>
>  #define __read_32bit_gc0_register(source, sel)                         \
> @@ -2078,8 +2117,9 @@ _ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
>         __asm__ __volatile__(                                           \
>                 ".set\tpush\n\t"                                        \
>                 ".set\tmips32r5\n\t"                                    \
> -               _ASM_SET_VIRT                                           \
> +               _ASM_SET_MFGC0                                          \
>                 "mfgc0\t%0, " #source ", %1\n\t"                        \
> +               _ASM_UNSET_MFGC0                                        \
>                 ".set\tpop"                                             \
>                 : "=r" (__res)                                          \
>                 : "i" (sel));                                           \
> @@ -2091,8 +2131,9 @@ _ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
>         __asm__ __volatile__(                                           \
>                 ".set\tpush\n\t"                                        \
>                 ".set\tmips64r5\n\t"                                    \
> -               _ASM_SET_VIRT                                           \
> +               _ASM_SET_DMFGC0                                         \
>                 "dmfgc0\t%0, " #source ", %1\n\t"                       \
> +               _ASM_UNSET_DMFGC0                                       \
>                 ".set\tpop"                                             \
>                 : "=r" (__res)                                          \
>                 : "i" (sel));                                           \
> @@ -2104,8 +2145,9 @@ do {                                                                      \
>         __asm__ __volatile__(                                           \
>                 ".set\tpush\n\t"                                        \
>                 ".set\tmips32r5\n\t"                                    \
> -               _ASM_SET_VIRT                                           \
> +               _ASM_SET_MTGC0                                          \
>                 "mtgc0\t%z0, " #register ", %1\n\t"                     \
> +               _ASM_UNSET_MTGC0                                        \
>                 ".set\tpop"                                             \
>                 : : "Jr" ((unsigned int)(value)),                       \
>                     "i" (sel));                                         \
> @@ -2116,8 +2158,9 @@ do {                                                                      \
>         __asm__ __volatile__(                                           \
>                 ".set\tpush\n\t"                                        \
>                 ".set\tmips64r5\n\t"                                    \
> -               _ASM_SET_VIRT                                           \
> +               _ASM_SET_DMTGC0                                         \
>                 "dmtgc0\t%z0, " #register ", %1\n\t"                    \
> +               _ASM_UNSET_DMTGC0                                       \
>                 ".set\tpop"                                             \
>                 : : "Jr" (value),                                       \
>                     "i" (sel));                                         \
> @@ -2788,8 +2831,7 @@ static inline void guest_tlb_probe(void)
>         __asm__ __volatile__(
>                 ".set push\n\t"
>                 ".set noreorder\n\t"
> -               _ASM_SET_VIRT
> -               "tlbgp\n\t"
> +               __tlbgp()
>                 ".set pop");
>  }
>
> @@ -2798,8 +2840,7 @@ static inline void guest_tlb_read(void)
>         __asm__ __volatile__(
>                 ".set push\n\t"
>                 ".set noreorder\n\t"
> -               _ASM_SET_VIRT
> -               "tlbgr\n\t"
> +               __tlbgr()
>                 ".set pop");
>  }
>
> @@ -2808,8 +2849,7 @@ static inline void guest_tlb_write_indexed(void)
>         __asm__ __volatile__(
>                 ".set push\n\t"
>                 ".set noreorder\n\t"
> -               _ASM_SET_VIRT
> -               "tlbgwi\n\t"
> +               __tlbgwi()
>                 ".set pop");
>  }
>
> @@ -2818,8 +2858,7 @@ static inline void guest_tlb_write_random(void)
>         __asm__ __volatile__(
>                 ".set push\n\t"
>                 ".set noreorder\n\t"
> -               _ASM_SET_VIRT
> -               "tlbgwr\n\t"
> +               __tlbgwr()
>                 ".set pop");
>  }
>
> @@ -2831,8 +2870,7 @@ static inline void guest_tlbinvf(void)
>         __asm__ __volatile__(
>                 ".set push\n\t"
>                 ".set noreorder\n\t"
> -               _ASM_SET_VIRT
> -               "tlbginvf\n\t"
> +               __tlbginvf()
>                 ".set pop");
>  }
>
> diff --git a/arch/mips/include/asm/msa.h b/arch/mips/include/asm/msa.h
> index e0a3dd52334d..236a49ee2e3e 100644
> --- a/arch/mips/include/asm/msa.h
> +++ b/arch/mips/include/asm/msa.h
> @@ -162,16 +162,26 @@ static inline void init_msa_upper(void)
>   * to allow compilation with toolchains that do not support MSA. Once all
>   * toolchains in use support MSA these can be removed.
>   */
> -_ASM_MACRO_2R(cfcmsa, rd, cs,
> -       _ASM_INSN_IF_MIPS(0x787e0019 | __cs << 11 | __rd << 6)
> -       _ASM_INSN32_IF_MM(0x587e0016 | __cs << 11 | __rd << 6));
> -_ASM_MACRO_2R(ctcmsa, cd, rs,
> -       _ASM_INSN_IF_MIPS(0x783e0019 | __rs << 11 | __cd << 6)
> -       _ASM_INSN32_IF_MM(0x583e0016 | __rs << 11 | __cd << 6));
> -#define _ASM_SET_MSA ""
> +
> +#define _ASM_SET_CFCMSA                                                        \
> +       _ASM_MACRO_2R(cfcmsa, rd, cs,                                   \
> +                     _ASM_INSN_IF_MIPS(0x787e0019 | __cs << 11 | __rd << 6)    \
> +                     _ASM_INSN32_IF_MM(0x587e0016 | __cs << 11 | __rd << 6))
> +#define _ASM_UNSET_CFCMSA ".purgem cfcmsa\n\t"
> +#define _ASM_SET_CTCMSA                                                        \
> +       _ASM_MACRO_2R(ctcmsa, cd, rs,                                   \
> +                     _ASM_INSN_IF_MIPS(0x783e0019 | __rs << 11 | __cd << 6)    \
> +                     _ASM_INSN32_IF_MM(0x583e0016 | __rs << 11 | __cd << 6))
> +#define _ASM_UNSET_CTCMSA ".purgem ctcmsa\n\t"
>  #else /* TOOLCHAIN_SUPPORTS_MSA */
> -#define _ASM_SET_MSA ".set\tfp=64\n\t"                         \
> -                    ".set\tmsa\n\t"
> +#define _ASM_SET_CFCMSA                                                \
> +               ".set\tfp=64\n\t"                               \
> +               ".set\tmsa\n\t"
> +#define _ASM_UNSET_CFCMSA
> +#define _ASM_SET_CTCMSA                                                \
> +               ".set\tfp=64\n\t"                               \
> +               ".set\tmsa\n\t"
> +#define _ASM_UNSET_CTCMSA
>  #endif
>
>  #define __BUILD_MSA_CTL_REG(name, cs)                          \
> @@ -180,8 +190,9 @@ static inline unsigned int read_msa_##name(void)            \
>         unsigned int reg;                                       \
>         __asm__ __volatile__(                                   \
>         "       .set    push\n"                                 \
> -       _ASM_SET_MSA                                            \
> +       _ASM_SET_CFCMSA                                         \
>         "       cfcmsa  %0, $" #cs "\n"                         \
> +       _ASM_UNSET_CFCMSA                                       \
>         "       .set    pop\n"                                  \
>         : "=r"(reg));                                           \
>         return reg;                                             \
> @@ -191,8 +202,9 @@ static inline void write_msa_##name(unsigned int val)               \
>  {                                                              \
>         __asm__ __volatile__(                                   \
>         "       .set    push\n"                                 \
> -       _ASM_SET_MSA                                            \
> +       _ASM_SET_CTCMSA                                         \
>         "       ctcmsa  $" #cs ", %0\n"                         \
> +       _ASM_UNSET_CTCMSA                                       \
>         "       .set    pop\n"                                  \
>         : : "r"(val));                                          \
>  }
> --
> 2.30.2
>


-- 
Thanks,
~Nick Desaulniers

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MIPS: Avoid macro redefinitions
  2021-07-19 21:43 ` Nick Desaulniers
@ 2021-07-19 22:10   ` Paul Cercueil
  2021-07-19 22:23     ` Nick Desaulniers
  2021-07-21 16:17     ` Thomas Bogendoerfer
  0 siblings, 2 replies; 6+ messages in thread
From: Paul Cercueil @ 2021-07-19 22:10 UTC (permalink / raw)
  To: Nick Desaulniers
  Cc: Thomas Bogendoerfer, linux-mips, linux-kernel, list, Kees Cook,
	clang-built-linux

Hi Nick,

Le lun., juil. 19 2021 at 14:43:46 -0700, Nick Desaulniers 
<ndesaulniers@google.com> a écrit :
> On Sun, Jul 18, 2021 at 6:07 AM Paul Cercueil <paul@crapouillou.net> 
> wrote:
>> 
>>  To be able to compile the kernel with LTO, the assembler macros 
>> cannot
>>  be declared in the global scope, or the compiler will complain about
>>  redefined macros.
> 
> Thanks for the patch! Minor comments below, so untested yet by me.
> Please cc clang-built-linux@googlegroups.com for any follow ups.

Alright, noted.

> If this is towards supporting LTO, consider sending as part of a
> series enabling LTO for MIPS. That would be cool to see.

That would be nice to see, indeed. Unfortunately it's not quite there 
yet. With that patch applied, the linker will still complain that it's 
linking non-abicalls with abicalls, and hard-float with soft-float 
(while everything is supposedly compiled with non-abicalls and 
soft-float).

Even if that worked, regular clang-built kernels cannot boot on my 
system as the vmlinuz.bin extractor fails to run 
(https://github.com/ClangBuiltLinux/linux/issues/1333).

But this is one step closer to the goal.

>> 
>>  Update the code so that macros are defined then undefined when they 
>> are
>>  used.
>> 
>>  Signed-off-by: Paul Cercueil <paul@crapouillou.net>
>>  ---
>>   arch/mips/include/asm/ginvt.h                 |  11 +-
>>   .../asm/mach-loongson64/loongson_regs.h       |  12 ++
>>   arch/mips/include/asm/mipsregs.h              | 192 
>> +++++++++++-------
>>   arch/mips/include/asm/msa.h                   |  34 +++-
>>   4 files changed, 157 insertions(+), 92 deletions(-)
>> 
>>  diff --git a/arch/mips/include/asm/ginvt.h 
>> b/arch/mips/include/asm/ginvt.h
>>  index 6eb7c2b94dc7..87b2974ffc53 100644
>>  --- a/arch/mips/include/asm/ginvt.h
>>  +++ b/arch/mips/include/asm/ginvt.h
>>  @@ -12,11 +12,13 @@ enum ginvt_type {
>> 
>>   #ifdef TOOLCHAIN_SUPPORTS_GINV
>>   # define _ASM_SET_GINV ".set   ginv\n"
>>  +# define _ASM_UNSET_GINV
>>   #else
>>  -_ASM_MACRO_1R1I(ginvt, rs, type,
>>  -               _ASM_INSN_IF_MIPS(0x7c0000bd | (__rs << 21) | 
>> (\\type << 8))
>>  -               _ASM_INSN32_IF_MM(0x0000717c | (__rs << 16) | 
>> (\\type << 9)));
>>  -# define _ASM_SET_GINV
>>  +# define _ASM_SET_GINV                                             
>>     \
>>  +       _ASM_MACRO_1R1I(ginvt, rs, type,                            
>>     \
>>  +                       _ASM_INSN_IF_MIPS(0x7c0000bd | (__rs << 21) 
>> | (\\type << 8))    \
>>  +                       _ASM_INSN32_IF_MM(0x0000717c | (__rs << 16) 
>> | (\\type << 9)))
>>  +# define _ASM_UNSET_GINV ".purgem ginvt\n"
>>   #endif
>> 
>>   static __always_inline void ginvt(unsigned long addr, enum 
>> ginvt_type type)
>>  @@ -25,6 +27,7 @@ static __always_inline void ginvt(unsigned long 
>> addr, enum ginvt_type type)
>>                  ".set   push\n"
>>                  _ASM_SET_GINV
>>                  "       ginvt   %0, %1\n"
>>  +               _ASM_UNSET_GINV
>>                  ".set   pop"
>>                  : /* no outputs */
>>                  : "r"(addr), "i"(type)
>>  diff --git a/arch/mips/include/asm/mach-loongson64/loongson_regs.h 
>> b/arch/mips/include/asm/mach-loongson64/loongson_regs.h
>>  index 165993514762..b5be7511f6cd 100644
>>  --- a/arch/mips/include/asm/mach-loongson64/loongson_regs.h
>>  +++ b/arch/mips/include/asm/mach-loongson64/loongson_regs.h
>>  @@ -21,8 +21,10 @@ static inline u32 read_cpucfg(u32 reg)
>>          u32 __res;
>> 
>>          __asm__ __volatile__(
>>  +               _ASM_SET_PARSE_R
>>                  "parse_r __res,%0\n\t"
>>                  "parse_r reg,%1\n\t"
>>  +               _ASM_UNSET_PARSE_R
>>                  ".insn \n\t"
>>                  ".word (0xc8080118 | (reg << 21) | (__res << 
>> 11))\n\t"
>>                  :"=r"(__res)
>>  @@ -143,8 +145,10 @@ static inline u32 csr_readl(u32 reg)
>> 
>>          /* RDCSR reg, val */
>>          __asm__ __volatile__(
>>  +               _ASM_SET_PARSE_R
>>                  "parse_r __res,%0\n\t"
>>                  "parse_r reg,%1\n\t"
>>  +               _ASM_UNSET_PARSE_R
>>                  ".insn \n\t"
>>                  ".word (0xc8000118 | (reg << 21) | (__res << 
>> 11))\n\t"
>>                  :"=r"(__res)
>>  @@ -160,8 +164,10 @@ static inline u64 csr_readq(u32 reg)
>> 
>>          /* DRDCSR reg, val */
>>          __asm__ __volatile__(
>>  +               _ASM_SET_PARSE_R
>>                  "parse_r __res,%0\n\t"
>>                  "parse_r reg,%1\n\t"
>>  +               _ASM_UNSET_PARSE_R
>>                  ".insn \n\t"
>>                  ".word (0xc8020118 | (reg << 21) | (__res << 
>> 11))\n\t"
>>                  :"=r"(__res)
>>  @@ -175,8 +181,10 @@ static inline void csr_writel(u32 val, u32 reg)
>>   {
>>          /* WRCSR reg, val */
>>          __asm__ __volatile__(
>>  +               _ASM_SET_PARSE_R
>>                  "parse_r reg,%0\n\t"
>>                  "parse_r val,%1\n\t"
>>  +               _ASM_UNSET_PARSE_R
>>                  ".insn \n\t"
>>                  ".word (0xc8010118 | (reg << 21) | (val << 11))\n\t"
>>                  :
>>  @@ -189,8 +197,10 @@ static inline void csr_writeq(u64 val, u32 reg)
>>   {
>>          /* DWRCSR reg, val */
>>          __asm__ __volatile__(
>>  +               _ASM_SET_PARSE_R
>>                  "parse_r reg,%0\n\t"
>>                  "parse_r val,%1\n\t"
>>  +               _ASM_UNSET_PARSE_R
>>                  ".insn \n\t"
>>                  ".word (0xc8030118 | (reg << 21) | (val << 11))\n\t"
>>                  :
>>  @@ -243,8 +253,10 @@ static inline u64 drdtime(void)
>>          u64 val = 0;
>> 
>>          __asm__ __volatile__(
>>  +               _ASM_SET_PARSE_R
>>                  "parse_r rID,%0\n\t"
>>                  "parse_r val,%1\n\t"
>>  +               _ASM_UNSET_PARSE_R
>>                  ".insn \n\t"
>>                  ".word (0xc8090118 | (rID << 21) | (val << 11))\n\t"
>>                  :"=r"(rID),"=r"(val)
>>  diff --git a/arch/mips/include/asm/mipsregs.h 
>> b/arch/mips/include/asm/mipsregs.h
>>  index acdf8c69220b..f44f041e24de 100644
>>  --- a/arch/mips/include/asm/mipsregs.h
>>  +++ b/arch/mips/include/asm/mipsregs.h
>>  @@ -1297,22 +1297,24 @@ static inline int mm_insn_16bit(u16 insn)
>>          "\\var  = " #n "\n\t"                   \
>>          ".endif\n\t"
>> 
>>  -__asm__(".macro        parse_r var r\n\t"
>>  -       "\\var  = -1\n\t"
>>  -       _IFC_REG(0)  _IFC_REG(1)  _IFC_REG(2)  _IFC_REG(3)
>>  -       _IFC_REG(4)  _IFC_REG(5)  _IFC_REG(6)  _IFC_REG(7)
>>  -       _IFC_REG(8)  _IFC_REG(9)  _IFC_REG(10) _IFC_REG(11)
>>  -       _IFC_REG(12) _IFC_REG(13) _IFC_REG(14) _IFC_REG(15)
>>  -       _IFC_REG(16) _IFC_REG(17) _IFC_REG(18) _IFC_REG(19)
>>  -       _IFC_REG(20) _IFC_REG(21) _IFC_REG(22) _IFC_REG(23)
>>  -       _IFC_REG(24) _IFC_REG(25) _IFC_REG(26) _IFC_REG(27)
>>  -       _IFC_REG(28) _IFC_REG(29) _IFC_REG(30) _IFC_REG(31)
>>  -       ".iflt  \\var\n\t"
>>  -       ".error \"Unable to parse register name \\r\"\n\t"
>>  -       ".endif\n\t"
>>  -       ".endm");
>>  -
>>  -#undef _IFC_REG
>>  +#define _ASM_SET_PARSE_R                                           
>>     \
>>  +       ".macro parse_r var r\n\t"                                  
>>     \
>>  +       "\\var  = -1\n\t"                                           
>>     \
>>  +       _IFC_REG(0)  _IFC_REG(1)  _IFC_REG(2)  _IFC_REG(3)          
>>     \
>>  +       _IFC_REG(4)  _IFC_REG(5)  _IFC_REG(6)  _IFC_REG(7)          
>>     \
>>  +       _IFC_REG(8)  _IFC_REG(9)  _IFC_REG(10) _IFC_REG(11)         
>>     \
>>  +       _IFC_REG(12) _IFC_REG(13) _IFC_REG(14) _IFC_REG(15)         
>>     \
>>  +       _IFC_REG(16) _IFC_REG(17) _IFC_REG(18) _IFC_REG(19)         
>>     \
>>  +       _IFC_REG(20) _IFC_REG(21) _IFC_REG(22) _IFC_REG(23)         
>>     \
>>  +       _IFC_REG(24) _IFC_REG(25) _IFC_REG(26) _IFC_REG(27)         
>>     \
>>  +       _IFC_REG(28) _IFC_REG(29) _IFC_REG(30) _IFC_REG(31)         
>>     \
>>  +       ".iflt  \\var\n\t"                                          
>>     \
>>  +       ".error \"Unable to parse register name \\r\"\n\t"          
>>     \
>>  +       ".endif\n\t"                                                
>>     \
>>  +       ".endm\n\t"
>>  +#define _ASM_UNSET_PARSE_R ".purgem parse_r\n\t"
>>  +
>>  +//#undef _IFC_REG
> 
> ^ did you mean to commit that commented-out #undef?

IIRC it must be removed or commented, since the _IFC_REG() macro is 
evaluated at a later moment. But I should just remove the #undef then.

> 
>> 
>>   /*
>>    * C macros for generating assembler macros for common instruction 
>> formats.
>>  @@ -1322,43 +1324,45 @@ __asm__(".macro parse_r var r\n\t"
>>    * the ENC encodings.
>>    */
>> 
>>  -/* Instructions with no operands */
>>  -#define _ASM_MACRO_0(OP, ENC)                                      
>>     \
>>  -       __asm__(".macro " #OP "\n\t"                                
>>     \
>>  -               ENC                                                 
>>     \
>>  -               ".endm")
>>  -
>>   /* Instructions with 1 register operand & 1 immediate operand */
>>   #define _ASM_MACRO_1R1I(OP, R1, I2, ENC)                           
>>     \
>>  -       __asm__(".macro " #OP " " #R1 ", " #I2 "\n\t"               
>>     \
>>  +               ".macro " #OP " " #R1 ", " #I2 "\n\t"               
>>     \
>>  +               _ASM_SET_PARSE_R                                    
>>     \
>>                  "parse_r __" #R1 ", \\" #R1 "\n\t"                  
>>     \
>>                  ENC                                                 
>>     \
>>  -               ".endm")
>>  +               _ASM_UNSET_PARSE_R                                  
>>     \
>>  +               ".endm\n\t"
>> 
>>   /* Instructions with 2 register operands */
>>   #define _ASM_MACRO_2R(OP, R1, R2, ENC)                             
>>     \
>>  -       __asm__(".macro " #OP " " #R1 ", " #R2 "\n\t"               
>>     \
>>  +               ".macro " #OP " " #R1 ", " #R2 "\n\t"               
>>     \
>>  +               _ASM_SET_PARSE_R                                    
>>     \
>>                  "parse_r __" #R1 ", \\" #R1 "\n\t"                  
>>     \
>>                  "parse_r __" #R2 ", \\" #R2 "\n\t"                  
>>     \
>>                  ENC                                                 
>>     \
>>  -               ".endm")
>>  +               _ASM_UNSET_PARSE_R                                  
>>     \
>>  +               ".endm\n\t"
>> 
>>   /* Instructions with 3 register operands */
>>   #define _ASM_MACRO_3R(OP, R1, R2, R3, ENC)                         
>>     \
>>  -       __asm__(".macro " #OP " " #R1 ", " #R2 ", " #R3 "\n\t"      
>>     \
>>  +               ".macro " #OP " " #R1 ", " #R2 ", " #R3 "\n\t"      
>>     \
>>  +               _ASM_SET_PARSE_R                                    
>>     \
>>                  "parse_r __" #R1 ", \\" #R1 "\n\t"                  
>>     \
>>                  "parse_r __" #R2 ", \\" #R2 "\n\t"                  
>>     \
>>                  "parse_r __" #R3 ", \\" #R3 "\n\t"                  
>>     \
>>                  ENC                                                 
>>     \
>>  -               ".endm")
>>  +               _ASM_UNSET_PARSE_R                                  
>>     \
>>  +               ".endm\n\t"
>> 
>>   /* Instructions with 2 register operands and 1 optional select 
>> operand */
>>   #define _ASM_MACRO_2R_1S(OP, R1, R2, SEL3, ENC)                    
>>             \
>>  -       __asm__(".macro " #OP " " #R1 ", " #R2 ", " #SEL3 " = 
>> 0\n\t"    \
>>  +               ".macro " #OP " " #R1 ", " #R2 ", " #SEL3 " = 
>> 0\n\t"    \
>>  +               _ASM_SET_PARSE_R                                    
>>     \
>>                  "parse_r __" #R1 ", \\" #R1 "\n\t"                  
>>     \
>>                  "parse_r __" #R2 ", \\" #R2 "\n\t"                  
>>     \
>>                  ENC                                                 
>>     \
>>  -               ".endm")
>>  +               _ASM_UNSET_PARSE_R                                  
>>     \
>>  +               ".endm\n\t"
>> 
>>   /*
>>    * TLB Invalidate Flush
>>  @@ -1618,15 +1622,21 @@ do {                                        
>>                             \
>>   } while (0)
>> 
>>   #ifndef TOOLCHAIN_SUPPORTS_XPA
> 
> When did GAS gain support for XPA? Do we still need to support
> !TOOLCHAIN_SUPPORTS_XPA? (though that's orthogonal to this patch, such
> a cleanup first might simplify this patch).
> 
>>  -_ASM_MACRO_2R_1S(mfhc0, rt, rs, sel,
>>  -       _ASM_INSN_IF_MIPS(0x40400000 | __rt << 16 | __rs << 11 | 
>> \\sel)
>>  -       _ASM_INSN32_IF_MM(0x000000f4 | __rt << 21 | __rs << 16 | 
>> \\sel << 11));
>>  -_ASM_MACRO_2R_1S(mthc0, rt, rd, sel,
>>  -       _ASM_INSN_IF_MIPS(0x40c00000 | __rt << 16 | __rd << 11 | 
>> \\sel)
>>  -       _ASM_INSN32_IF_MM(0x000002f4 | __rt << 21 | __rd << 16 | 
>> \\sel << 11));
>>  -#define _ASM_SET_XPA ""
>>  +#define _ASM_SET_MFHC0                                             
>>     \
>>  +       _ASM_MACRO_2R_1S(mfhc0, rt, rs, sel,                        
>>     \
>>  +                        _ASM_INSN_IF_MIPS(0x40400000 | __rt << 16 
>> | __rs << 11 | \\sel)        \
>>  +                        _ASM_INSN32_IF_MM(0x000000f4 | __rt << 21 
>> | __rs << 16 | \\sel << 11))
>>  +#define _ASM_UNSET_MFHC0 ".purgem mfhc0\n\t"
>>  +#define _ASM_SET_MTHC0                                             
>>     \
>>  +       _ASM_MACRO_2R_1S(mthc0, rt, rd, sel,                        
>>     \
>>  +                        _ASM_INSN_IF_MIPS(0x40c00000 | __rt << 16 
>> | __rd << 11 | \\sel)        \
>>  +                        _ASM_INSN32_IF_MM(0x000002f4 | __rt << 21 
>> | __rd << 16 | \\sel << 11))
>>  +#define _ASM_UNSET_MTHC0 ".purgem mthc0\n\t"
>>   #else  /* !TOOLCHAIN_SUPPORTS_XPA */
>>  -#define _ASM_SET_XPA ".set\txpa\n\t"
>>  +#define _ASM_SET_MFHC0 ".set\txpa\n\t"
>>  +#define _ASM_SET_MTHC0 ".set\txpa\n\t"
>>  +#define _ASM_UNSET_MFHC0
>>  +#define _ASM_UNSET_MTHC0
>>   #endif
>> 
>>   #define __readx_32bit_c0_register(source, sel)                     
>>     \
>>  @@ -1636,8 +1646,9 @@ _ASM_MACRO_2R_1S(mthc0, rt, rd, sel,
>>          __asm__ __volatile__(                                       
>>     \
>>          "       .set    push                                    \n" 
>>     \
>>          "       .set    mips32r2                                \n" 
>>     \
>>  -       _ASM_SET_XPA                                                
>>     \
>>  +       _ASM_SET_MFHC0                                              
>>     \
>>          "       mfhc0   %0, " #source ", %1                     \n" 
>>     \
>>  +       _ASM_UNSET_MFHC0                                            
>>     \
>>          "       .set    pop                                     \n" 
>>     \
>>          : "=r" (__res)                                              
>>     \
>>          : "i" (sel));                                               
>>     \
>>  @@ -1649,8 +1660,9 @@ do {                                          
>>                             \
>>          __asm__ __volatile__(                                       
>>     \
>>          "       .set    push                                    \n" 
>>     \
>>          "       .set    mips32r2                                \n" 
>>     \
>>  -       _ASM_SET_XPA                                                
>>     \
>>  +       _ASM_SET_MTHC0                                              
>>     \
>>          "       mthc0   %z0, " #register ", %1                  \n" 
>>     \
>>  +       _ASM_UNSET_MTHC0                                            
>>     \
>>          "       .set    pop                                     \n" 
>>     \
>>          :                                                           
>>     \
>>          : "Jr" (value), "i" (sel));                                 
>>     \
>>  @@ -2046,31 +2058,58 @@ do {                                        
>>                             \
>>    */
>> 
>>   #ifndef TOOLCHAIN_SUPPORTS_VIRT
> 
> Same question for GAS version support for virt?
> Documentation/process/changes.rst
> says GNU binutils 2.23+ is required for building the kernel.
> If we still need to support, have you tested this change on such an
> older version of GNU binutils?

I have no idea about virt support - I hope Thomas can answer this.

I did build with binutils 2.35; 2.23 sounds very old.

Cheers,
-Paul

>>  -_ASM_MACRO_2R_1S(mfgc0, rt, rs, sel,
>>  -       _ASM_INSN_IF_MIPS(0x40600000 | __rt << 16 | __rs << 11 | 
>> \\sel)
>>  -       _ASM_INSN32_IF_MM(0x000004fc | __rt << 21 | __rs << 16 | 
>> \\sel << 11));
>>  -_ASM_MACRO_2R_1S(dmfgc0, rt, rs, sel,
>>  -       _ASM_INSN_IF_MIPS(0x40600100 | __rt << 16 | __rs << 11 | 
>> \\sel)
>>  -       _ASM_INSN32_IF_MM(0x580004fc | __rt << 21 | __rs << 16 | 
>> \\sel << 11));
>>  -_ASM_MACRO_2R_1S(mtgc0, rt, rd, sel,
>>  -       _ASM_INSN_IF_MIPS(0x40600200 | __rt << 16 | __rd << 11 | 
>> \\sel)
>>  -       _ASM_INSN32_IF_MM(0x000006fc | __rt << 21 | __rd << 16 | 
>> \\sel << 11));
>>  -_ASM_MACRO_2R_1S(dmtgc0, rt, rd, sel,
>>  -       _ASM_INSN_IF_MIPS(0x40600300 | __rt << 16 | __rd << 11 | 
>> \\sel)
>>  -       _ASM_INSN32_IF_MM(0x580006fc | __rt << 21 | __rd << 16 | 
>> \\sel << 11));
>>  -_ASM_MACRO_0(tlbgp,    _ASM_INSN_IF_MIPS(0x42000010)
>>  -                      _ASM_INSN32_IF_MM(0x0000017c));
>>  -_ASM_MACRO_0(tlbgr,    _ASM_INSN_IF_MIPS(0x42000009)
>>  -                      _ASM_INSN32_IF_MM(0x0000117c));
>>  -_ASM_MACRO_0(tlbgwi,   _ASM_INSN_IF_MIPS(0x4200000a)
>>  -                      _ASM_INSN32_IF_MM(0x0000217c));
>>  -_ASM_MACRO_0(tlbgwr,   _ASM_INSN_IF_MIPS(0x4200000e)
>>  -                      _ASM_INSN32_IF_MM(0x0000317c));
>>  -_ASM_MACRO_0(tlbginvf, _ASM_INSN_IF_MIPS(0x4200000c)
>>  -                      _ASM_INSN32_IF_MM(0x0000517c));
>>  -#define _ASM_SET_VIRT ""
>>  +#define _ASM_SET_MFGC0                                             
>>     \
>>  +       _ASM_MACRO_2R_1S(mfgc0, rt, rs, sel,                        
>>     \
>>  +                        _ASM_INSN_IF_MIPS(0x40600000 | __rt << 16 
>> | __rs << 11 | \\sel)        \
>>  +                        _ASM_INSN32_IF_MM(0x000004fc | __rt << 21 
>> | __rs << 16 | \\sel << 11))
>>  +#define _ASM_UNSET_MFGC0 ".purgem mfgc0\n\t"
>>  +#define _ASM_SET_DMFGC0                                            
>>             \
>>  +       _ASM_MACRO_2R_1S(dmfgc0, rt, rs, sel,                       
>>     \
>>  +                        _ASM_INSN_IF_MIPS(0x40600100 | __rt << 16 
>> | __rs << 11 | \\sel)        \
>>  +                        _ASM_INSN32_IF_MM(0x580004fc | __rt << 21 
>> | __rs << 16 | \\sel << 11))
>>  +#define _ASM_UNSET_DMFGC0 ".purgem dmfgc0\n\t"
>>  +#define _ASM_SET_MTGC0                                             
>>     \
>>  +       _ASM_MACRO_2R_1S(mtgc0, rt, rd, sel,                        
>>     \
>>  +                        _ASM_INSN_IF_MIPS(0x40600200 | __rt << 16 
>> | __rd << 11 | \\sel)        \
>>  +                        _ASM_INSN32_IF_MM(0x000006fc | __rt << 21 
>> | __rd << 16 | \\sel << 11))
>>  +#define _ASM_UNSET_MTGC0 ".purgem mtgc0\n\t"
>>  +#define _ASM_SET_DMTGC0                                            
>>             \
>>  +       _ASM_MACRO_2R_1S(dmtgc0, rt, rd, sel,                       
>>     \
>>  +                        _ASM_INSN_IF_MIPS(0x40600300 | __rt << 16 
>> | __rd << 11 | \\sel)        \
>>  +                        _ASM_INSN32_IF_MM(0x580006fc | __rt << 21 
>> | __rd << 16 | \\sel << 11))
>>  +#define _ASM_UNSET_DMTGC0 ".purgem dmtgc0\n\t"
>>  +
>>  +#define __tlbgp()                                                  
>>     \
>>  +               _ASM_INSN_IF_MIPS(0x42000010)                       
>>     \
>>  +               _ASM_INSN32_IF_MM(0x0000017c)
>>  +#define __tlbgr()                                                  
>>     \
>>  +               _ASM_INSN_IF_MIPS(0x42000009)                       
>>     \
>>  +               _ASM_INSN32_IF_MM(0x0000117c)
>>  +#define __tlbgwi()                                                 
>>     \
>>  +               _ASM_INSN_IF_MIPS(0x4200000a)                       
>>     \
>>  +               _ASM_INSN32_IF_MM(0x0000217c)
>>  +#define __tlbgwr()                                                 
>>     \
>>  +               _ASM_INSN_IF_MIPS(0x4200000e)                       
>>     \
>>  +               _ASM_INSN32_IF_MM(0x0000317c)
>>  +#define __tlbginvf()                                               
>>     \
>>  +               _ASM_INSN_IF_MIPS(0x4200000c)                       
>>     \
>>  +               _ASM_INSN32_IF_MM(0x0000517c)
>>   #else  /* !TOOLCHAIN_SUPPORTS_VIRT */
>>   #define _ASM_SET_VIRT ".set\tvirt\n\t"
>>  +#define _ASM_SET_MFGC0 _ASM_SET_VIRT
>>  +#define _ASM_SET_DMFGC0        _ASM_SET_VIRT
>>  +#define _ASM_SET_MTGC0 _ASM_SET_VIRT
>>  +#define _ASM_SET_DMTGC0        _ASM_SET_VIRT
>>  +#define _ASM_UNSET_MFGC0
>>  +#define _ASM_UNSET_DMFGC0
>>  +#define _ASM_UNSET_MTGC0
>>  +#define _ASM_UNSET_DMTGC0
>>  +
>>  +#define __tlbgp()      _ASM_SET_VIRT "tlbgp\n\t"
>>  +#define __tlbgr()      _ASM_SET_VIRT "tlbgr\n\t"
>>  +#define __tlbgwi()     _ASM_SET_VIRT "tlbgwi\n\t"
>>  +#define __tlbgwr()     _ASM_SET_VIRT "tlbgwr\n\t"
>>  +#define __tlbginvf()   _ASM_SET_VIRT "tlbginvf\n\t"
>>   #endif
>> 
>>   #define __read_32bit_gc0_register(source, sel)                     
>>     \
>>  @@ -2078,8 +2117,9 @@ _ASM_MACRO_0(tlbginvf, 
>> _ASM_INSN_IF_MIPS(0x4200000c)
>>          __asm__ __volatile__(                                       
>>     \
>>                  ".set\tpush\n\t"                                    
>>     \
>>                  ".set\tmips32r5\n\t"                                
>>     \
>>  -               _ASM_SET_VIRT                                       
>>     \
>>  +               _ASM_SET_MFGC0                                      
>>     \
>>                  "mfgc0\t%0, " #source ", %1\n\t"                    
>>     \
>>  +               _ASM_UNSET_MFGC0                                    
>>     \
>>                  ".set\tpop"                                         
>>     \
>>                  : "=r" (__res)                                      
>>     \
>>                  : "i" (sel));                                       
>>     \
>>  @@ -2091,8 +2131,9 @@ _ASM_MACRO_0(tlbginvf, 
>> _ASM_INSN_IF_MIPS(0x4200000c)
>>          __asm__ __volatile__(                                       
>>     \
>>                  ".set\tpush\n\t"                                    
>>     \
>>                  ".set\tmips64r5\n\t"                                
>>     \
>>  -               _ASM_SET_VIRT                                       
>>     \
>>  +               _ASM_SET_DMFGC0                                     
>>     \
>>                  "dmfgc0\t%0, " #source ", %1\n\t"                   
>>     \
>>  +               _ASM_UNSET_DMFGC0                                   
>>     \
>>                  ".set\tpop"                                         
>>     \
>>                  : "=r" (__res)                                      
>>     \
>>                  : "i" (sel));                                       
>>     \
>>  @@ -2104,8 +2145,9 @@ do {                                          
>>                             \
>>          __asm__ __volatile__(                                       
>>     \
>>                  ".set\tpush\n\t"                                    
>>     \
>>                  ".set\tmips32r5\n\t"                                
>>     \
>>  -               _ASM_SET_VIRT                                       
>>     \
>>  +               _ASM_SET_MTGC0                                      
>>     \
>>                  "mtgc0\t%z0, " #register ", %1\n\t"                 
>>     \
>>  +               _ASM_UNSET_MTGC0                                    
>>     \
>>                  ".set\tpop"                                         
>>     \
>>                  : : "Jr" ((unsigned int)(value)),                   
>>     \
>>                      "i" (sel));                                     
>>     \
>>  @@ -2116,8 +2158,9 @@ do {                                          
>>                             \
>>          __asm__ __volatile__(                                       
>>     \
>>                  ".set\tpush\n\t"                                    
>>     \
>>                  ".set\tmips64r5\n\t"                                
>>     \
>>  -               _ASM_SET_VIRT                                       
>>     \
>>  +               _ASM_SET_DMTGC0                                     
>>     \
>>                  "dmtgc0\t%z0, " #register ", %1\n\t"                
>>     \
>>  +               _ASM_UNSET_DMTGC0                                   
>>     \
>>                  ".set\tpop"                                         
>>     \
>>                  : : "Jr" (value),                                   
>>     \
>>                      "i" (sel));                                     
>>     \
>>  @@ -2788,8 +2831,7 @@ static inline void guest_tlb_probe(void)
>>          __asm__ __volatile__(
>>                  ".set push\n\t"
>>                  ".set noreorder\n\t"
>>  -               _ASM_SET_VIRT
>>  -               "tlbgp\n\t"
>>  +               __tlbgp()
>>                  ".set pop");
>>   }
>> 
>>  @@ -2798,8 +2840,7 @@ static inline void guest_tlb_read(void)
>>          __asm__ __volatile__(
>>                  ".set push\n\t"
>>                  ".set noreorder\n\t"
>>  -               _ASM_SET_VIRT
>>  -               "tlbgr\n\t"
>>  +               __tlbgr()
>>                  ".set pop");
>>   }
>> 
>>  @@ -2808,8 +2849,7 @@ static inline void 
>> guest_tlb_write_indexed(void)
>>          __asm__ __volatile__(
>>                  ".set push\n\t"
>>                  ".set noreorder\n\t"
>>  -               _ASM_SET_VIRT
>>  -               "tlbgwi\n\t"
>>  +               __tlbgwi()
>>                  ".set pop");
>>   }
>> 
>>  @@ -2818,8 +2858,7 @@ static inline void 
>> guest_tlb_write_random(void)
>>          __asm__ __volatile__(
>>                  ".set push\n\t"
>>                  ".set noreorder\n\t"
>>  -               _ASM_SET_VIRT
>>  -               "tlbgwr\n\t"
>>  +               __tlbgwr()
>>                  ".set pop");
>>   }
>> 
>>  @@ -2831,8 +2870,7 @@ static inline void guest_tlbinvf(void)
>>          __asm__ __volatile__(
>>                  ".set push\n\t"
>>                  ".set noreorder\n\t"
>>  -               _ASM_SET_VIRT
>>  -               "tlbginvf\n\t"
>>  +               __tlbginvf()
>>                  ".set pop");
>>   }
>> 
>>  diff --git a/arch/mips/include/asm/msa.h 
>> b/arch/mips/include/asm/msa.h
>>  index e0a3dd52334d..236a49ee2e3e 100644
>>  --- a/arch/mips/include/asm/msa.h
>>  +++ b/arch/mips/include/asm/msa.h
>>  @@ -162,16 +162,26 @@ static inline void init_msa_upper(void)
>>    * to allow compilation with toolchains that do not support MSA. 
>> Once all
>>    * toolchains in use support MSA these can be removed.
>>    */
>>  -_ASM_MACRO_2R(cfcmsa, rd, cs,
>>  -       _ASM_INSN_IF_MIPS(0x787e0019 | __cs << 11 | __rd << 6)
>>  -       _ASM_INSN32_IF_MM(0x587e0016 | __cs << 11 | __rd << 6));
>>  -_ASM_MACRO_2R(ctcmsa, cd, rs,
>>  -       _ASM_INSN_IF_MIPS(0x783e0019 | __rs << 11 | __cd << 6)
>>  -       _ASM_INSN32_IF_MM(0x583e0016 | __rs << 11 | __cd << 6));
>>  -#define _ASM_SET_MSA ""
>>  +
>>  +#define _ASM_SET_CFCMSA                                            
>>             \
>>  +       _ASM_MACRO_2R(cfcmsa, rd, cs,                               
>>     \
>>  +                     _ASM_INSN_IF_MIPS(0x787e0019 | __cs << 11 | 
>> __rd << 6)    \
>>  +                     _ASM_INSN32_IF_MM(0x587e0016 | __cs << 11 | 
>> __rd << 6))
>>  +#define _ASM_UNSET_CFCMSA ".purgem cfcmsa\n\t"
>>  +#define _ASM_SET_CTCMSA                                            
>>             \
>>  +       _ASM_MACRO_2R(ctcmsa, cd, rs,                               
>>     \
>>  +                     _ASM_INSN_IF_MIPS(0x783e0019 | __rs << 11 | 
>> __cd << 6)    \
>>  +                     _ASM_INSN32_IF_MM(0x583e0016 | __rs << 11 | 
>> __cd << 6))
>>  +#define _ASM_UNSET_CTCMSA ".purgem ctcmsa\n\t"
>>   #else /* TOOLCHAIN_SUPPORTS_MSA */
>>  -#define _ASM_SET_MSA ".set\tfp=64\n\t"                         \
>>  -                    ".set\tmsa\n\t"
>>  +#define _ASM_SET_CFCMSA                                            
>>     \
>>  +               ".set\tfp=64\n\t"                               \
>>  +               ".set\tmsa\n\t"
>>  +#define _ASM_UNSET_CFCMSA
>>  +#define _ASM_SET_CTCMSA                                            
>>     \
>>  +               ".set\tfp=64\n\t"                               \
>>  +               ".set\tmsa\n\t"
>>  +#define _ASM_UNSET_CTCMSA
>>   #endif
>> 
>>   #define __BUILD_MSA_CTL_REG(name, cs)                          \
>>  @@ -180,8 +190,9 @@ static inline unsigned int 
>> read_msa_##name(void)            \
>>          unsigned int reg;                                       \
>>          __asm__ __volatile__(                                   \
>>          "       .set    push\n"                                 \
>>  -       _ASM_SET_MSA                                            \
>>  +       _ASM_SET_CFCMSA                                         \
>>          "       cfcmsa  %0, $" #cs "\n"                         \
>>  +       _ASM_UNSET_CFCMSA                                       \
>>          "       .set    pop\n"                                  \
>>          : "=r"(reg));                                           \
>>          return reg;                                             \
>>  @@ -191,8 +202,9 @@ static inline void write_msa_##name(unsigned 
>> int val)               \
>>   {                                                              \
>>          __asm__ __volatile__(                                   \
>>          "       .set    push\n"                                 \
>>  -       _ASM_SET_MSA                                            \
>>  +       _ASM_SET_CTCMSA                                         \
>>          "       ctcmsa  $" #cs ", %0\n"                         \
>>  +       _ASM_UNSET_CTCMSA                                       \
>>          "       .set    pop\n"                                  \
>>          : : "r"(val));                                          \
>>   }
>>  --
>>  2.30.2
>> 
> 
> 
> --
> Thanks,
> ~Nick Desaulniers



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MIPS: Avoid macro redefinitions
  2021-07-19 22:10   ` Paul Cercueil
@ 2021-07-19 22:23     ` Nick Desaulniers
  2021-07-21 16:17     ` Thomas Bogendoerfer
  1 sibling, 0 replies; 6+ messages in thread
From: Nick Desaulniers @ 2021-07-19 22:23 UTC (permalink / raw)
  To: Paul Cercueil
  Cc: Thomas Bogendoerfer, linux-mips, linux-kernel, list, Kees Cook,
	clang-built-linux

On Mon, Jul 19, 2021 at 3:11 PM Paul Cercueil <paul@crapouillou.net> wrote:
>
> Hi Nick,
>
> Le lun., juil. 19 2021 at 14:43:46 -0700, Nick Desaulniers
> <ndesaulniers@google.com> a écrit :
> > Same question for GAS version support for virt?
> > Documentation/process/changes.rst
> > says GNU binutils 2.23+ is required for building the kernel.
> > If we still need to support, have you tested this change on such an
> > older version of GNU binutils?
>
> I have no idea about virt support - I hope Thomas can answer this.

Perhaps you can do some archeology on the commit messages of binutils-gdb?

binutils-gdb commit 9785fc2a4d22 ("MIPS: Fix XPA base and
Virtualization ASE instruction handling")
looks interesting.
commit b015e599c772e does too. Looks like git tags aren't used, but
checking out those sources might have a version hardcoded in a
Makefile.

>
> I did build with binutils 2.35; 2.23 sounds very old.
-- 
Thanks,
~Nick Desaulniers

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MIPS: Avoid macro redefinitions
  2021-07-19 22:10   ` Paul Cercueil
  2021-07-19 22:23     ` Nick Desaulniers
@ 2021-07-21 16:17     ` Thomas Bogendoerfer
  2021-07-27 20:25       ` Nick Desaulniers
  1 sibling, 1 reply; 6+ messages in thread
From: Thomas Bogendoerfer @ 2021-07-21 16:17 UTC (permalink / raw)
  To: Paul Cercueil
  Cc: Nick Desaulniers, linux-mips, linux-kernel, list, Kees Cook,
	clang-built-linux

On Mon, Jul 19, 2021 at 11:10:55PM +0100, Paul Cercueil wrote:
> > >   #ifndef TOOLCHAIN_SUPPORTS_VIRT
> > 
> > Same question for GAS version support for virt?
> > Documentation/process/changes.rst
> > says GNU binutils 2.23+ is required for building the kernel.
> > If we still need to support, have you tested this change on such an
> > older version of GNU binutils?
> 
> I have no idea about virt support - I hope Thomas can answer this.

virt support was added in 2.24 and xpa in 2.25. So we still need the
TOOLCHAIN defines for it.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MIPS: Avoid macro redefinitions
  2021-07-21 16:17     ` Thomas Bogendoerfer
@ 2021-07-27 20:25       ` Nick Desaulniers
  0 siblings, 0 replies; 6+ messages in thread
From: Nick Desaulniers @ 2021-07-27 20:25 UTC (permalink / raw)
  To: Thomas Bogendoerfer, Paul Cercueil
  Cc: linux-mips, linux-kernel, list, Kees Cook, clang-built-linux

On Wed, Jul 21, 2021 at 9:17 AM Thomas Bogendoerfer
<tsbogend@alpha.franken.de> wrote:
>
> On Mon, Jul 19, 2021 at 11:10:55PM +0100, Paul Cercueil wrote:
> > > >   #ifndef TOOLCHAIN_SUPPORTS_VIRT
> > >
> > > Same question for GAS version support for virt?
> > > Documentation/process/changes.rst
> > > says GNU binutils 2.23+ is required for building the kernel.
> > > If we still need to support, have you tested this change on such an
> > > older version of GNU binutils?
> >
> > I have no idea about virt support - I hope Thomas can answer this.
>
> virt support was added in 2.24 and xpa in 2.25. So we still need the
> TOOLCHAIN defines for it.

Ah, ok then. Thanks for finding that.  Perhaps Paul you can note that
in the v2 that removes the commented out line, then we should be good
to go?
-- 
Thanks,
~Nick Desaulniers

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-07-27 20:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-18 13:07 [PATCH] MIPS: Avoid macro redefinitions Paul Cercueil
2021-07-19 21:43 ` Nick Desaulniers
2021-07-19 22:10   ` Paul Cercueil
2021-07-19 22:23     ` Nick Desaulniers
2021-07-21 16:17     ` Thomas Bogendoerfer
2021-07-27 20:25       ` Nick Desaulniers

This is a public inbox, see mirroring instructions
on how to clone and mirror all data and code used for this inbox