From: Nick Desaulniers <ndesaulniers@google.com>
To: Paul Cercueil <paul@crapouillou.net>
Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Huacai Chen <chenhuacai@kernel.org>,
Jiaxun Yang <jiaxun.yang@flygoat.com>,
Tiezhu Yang <yangtiezhu@loongson.cn>,
linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org,
list@opendingux.net, clang-built-linux@googlegroups.com
Subject: Re: [PATCH v2] MIPS: Avoid macro redefinitions
Date: Tue, 14 Sep 2021 09:24:54 -0700 [thread overview]
Message-ID: <CAKwvOd=w0=i2GPiBaXHizaMP=b0VD9Lw_o+GuvvWp-brMx2fRw@mail.gmail.com> (raw)
In-Reply-To: <HMMFZQ.4BHOG14DQZPA3@crapouillou.net>
On Tue, Sep 14, 2021 at 9:18 AM Paul Cercueil <paul@crapouillou.net> wrote:
>
> Hi Nick,
>
> Le mar., sept. 14 2021 at 08:56:59 -0700, Nick Desaulniers
> <ndesaulniers@google.com> a écrit :
> > On Mon, Sep 13, 2021 at 2:34 PM 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.
> >>
> >> Update the code so that macros are defined then undefined when they
> >> are
> >> used.
> >>
> >> Note that virt support was added in 2.24 and xpa in 2.25. So we
> >> still
> >> need the TOOLCHAIN defines for them.
> >>
> >> Signed-off-by: Paul Cercueil <paul@crapouillou.net>
> >> ---
> >>
> >> Notes:
> >> v2: Removed commented undef line
> >
> > Hi Paul, thanks for the v2. I think the point of the #undef was that
> > the _IFC_REG doesn't make sense outside of this header, so they
> > probably did not want to pollute the namespace for preprocessor
> > defines. As such, I think it should actually stay, not be removed (or
> > commented out).
>
> I can test it again later, but IIRC the #undef must be removed. Since
> we use the new _ASM_SET_PARSE_R macro, which uses _IFC_REG, if we
> #undef it early then it won't compile.
Ok, it's not a big deal anyways.
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
>
> Cheers,
> -Paul
>
> >>
> >> arch/mips/include/asm/ginvt.h | 11 +-
> >> .../asm/mach-loongson64/loongson_regs.h | 12 ++
> >> arch/mips/include/asm/mipsregs.h | 190
> >> +++++++++++-------
> >> arch/mips/include/asm/msa.h | 34 +++-
> >> 4 files changed, 155 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..2616353b940c 100644
> >> --- a/arch/mips/include/asm/mipsregs.h
> >> +++ b/arch/mips/include/asm/mipsregs.h
> >> @@ -1297,22 +1297,22 @@ 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
> >
> > ^ ie. I don't think we want to remove this undef.
> >
> >> +#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"
> >>
> >> /*
> >> * C macros for generating assembler macros for common instruction
> >> formats.
> >> @@ -1322,43 +1322,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 +1620,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 +1644,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 +1658,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 +2056,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 +2115,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 +2129,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 +2143,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 +2156,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 +2829,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 +2838,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 +2847,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 +2856,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 +2868,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.33.0
> >>
> >
> >
> > --
> > Thanks,
> > ~Nick Desaulniers
>
>
--
Thanks,
~Nick Desaulniers
next prev parent reply other threads:[~2021-09-14 16:25 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-13 21:34 [PATCH v2] MIPS: Avoid macro redefinitions Paul Cercueil
2021-09-14 15:56 ` Nick Desaulniers
2021-09-14 16:18 ` Paul Cercueil
2021-09-14 16:24 ` Nick Desaulniers [this message]
2021-09-23 9:36 ` Thomas Bogendoerfer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAKwvOd=w0=i2GPiBaXHizaMP=b0VD9Lw_o+GuvvWp-brMx2fRw@mail.gmail.com' \
--to=ndesaulniers@google.com \
--cc=chenhuacai@kernel.org \
--cc=clang-built-linux@googlegroups.com \
--cc=jiaxun.yang@flygoat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=list@opendingux.net \
--cc=paul@crapouillou.net \
--cc=tsbogend@alpha.franken.de \
--cc=yangtiezhu@loongson.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.