lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <35fed0e0-abfd-42a3-856d-ce7c9b696964@ghiti.fr>
Date: Tue, 6 May 2025 10:05:22 +0200
From: Alexandre Ghiti <alex@...ti.fr>
To: Clément Léger <cleger@...osinc.com>,
 Alexandre Ghiti <alexghiti@...osinc.com>,
 Paul Walmsley <paul.walmsley@...ive.com>, Palmer Dabbelt
 <palmer@...belt.com>, Anup Patel <anup@...infault.org>,
 Atish Patra <atishp@...shpatra.org>, linux-riscv@...ts.infradead.org,
 linux-kernel@...r.kernel.org, kvm@...r.kernel.org,
 kvm-riscv@...ts.infradead.org
Subject: Re: [PATCH 3/3] riscv: Move all duplicate insn parsing macros into
 asm/insn.h

Hi Clément,

On 22/04/2025 11:36, Clément Léger wrote:
>
> On 22/04/2025 10:25, Alexandre Ghiti wrote:
>> kernel/traps_misaligned.c and kvm/vcpu_insn.c define the same macros to
>> extract information from the instructions.
>>
>> Let's move the definitions into asm/insn.h to avoid this duplication.
>>
>> Signed-off-by: Alexandre Ghiti <alexghiti@...osinc.com>
>> ---
>>   arch/riscv/include/asm/insn.h        | 164 +++++++++++++++++++++++++++
>>   arch/riscv/kernel/elf_kexec.c        |   1 +
>>   arch/riscv/kernel/traps_misaligned.c | 136 +---------------------
>>   arch/riscv/kvm/vcpu_insn.c           | 127 +--------------------
>>   4 files changed, 167 insertions(+), 261 deletions(-)
>>
>> diff --git a/arch/riscv/include/asm/insn.h b/arch/riscv/include/asm/insn.h
>> index 4063ca35be9b..35f316cdd699 100644
>> --- a/arch/riscv/include/asm/insn.h
>> +++ b/arch/riscv/include/asm/insn.h
>> @@ -286,9 +286,173 @@ static __always_inline bool riscv_insn_is_c_jalr(u32 code)
>>   	       (code & RVC_INSN_J_RS1_MASK) != 0;
>>   }
>>   
>> +#define INSN_MATCH_LB		0x3
>> +#define INSN_MASK_LB		0x707f
>> +#define INSN_MATCH_LH		0x1003
>> +#define INSN_MASK_LH		0x707f
>> +#define INSN_MATCH_LW		0x2003
>> +#define INSN_MASK_LW		0x707f
>> +#define INSN_MATCH_LD		0x3003
>> +#define INSN_MASK_LD		0x707f
>> +#define INSN_MATCH_LBU		0x4003
>> +#define INSN_MASK_LBU		0x707f
>> +#define INSN_MATCH_LHU		0x5003
>> +#define INSN_MASK_LHU		0x707f
>> +#define INSN_MATCH_LWU		0x6003
>> +#define INSN_MASK_LWU		0x707f
>> +#define INSN_MATCH_SB		0x23
>> +#define INSN_MASK_SB		0x707f
>> +#define INSN_MATCH_SH		0x1023
>> +#define INSN_MASK_SH		0x707f
>> +#define INSN_MATCH_SW		0x2023
>> +#define INSN_MASK_SW		0x707f
>> +#define INSN_MATCH_SD		0x3023
>> +#define INSN_MASK_SD		0x707f
>> +
>> +#define INSN_MATCH_C_LD		0x6000
>> +#define INSN_MASK_C_LD		0xe003
>> +#define INSN_MATCH_C_SD		0xe000
>> +#define INSN_MASK_C_SD		0xe003
>> +#define INSN_MATCH_C_LW		0x4000
>> +#define INSN_MASK_C_LW		0xe003
>> +#define INSN_MATCH_C_SW		0xc000
>> +#define INSN_MASK_C_SW		0xe003
>> +#define INSN_MATCH_C_LDSP	0x6002
>> +#define INSN_MASK_C_LDSP	0xe003
>> +#define INSN_MATCH_C_SDSP	0xe002
>> +#define INSN_MASK_C_SDSP	0xe003
>> +#define INSN_MATCH_C_LWSP	0x4002
>> +#define INSN_MASK_C_LWSP	0xe003
>> +#define INSN_MATCH_C_SWSP	0xc002
>> +#define INSN_MASK_C_SWSP	0xe003
>> +
>> +#define INSN_OPCODE_MASK	0x007c
>> +#define INSN_OPCODE_SHIFT	2
>> +#define INSN_OPCODE_SYSTEM	28
>> +
>> +#define INSN_MASK_WFI		0xffffffff
>> +#define INSN_MATCH_WFI		0x10500073
>> +
>> +#define INSN_MASK_WRS		0xffffffff
>> +#define INSN_MATCH_WRS		0x00d00073
>> +
>> +#define INSN_MATCH_CSRRW	0x1073
>> +#define INSN_MASK_CSRRW		0x707f
>> +#define INSN_MATCH_CSRRS	0x2073
>> +#define INSN_MASK_CSRRS		0x707f
>> +#define INSN_MATCH_CSRRC	0x3073
>> +#define INSN_MASK_CSRRC		0x707f
>> +#define INSN_MATCH_CSRRWI	0x5073
>> +#define INSN_MASK_CSRRWI	0x707f
>> +#define INSN_MATCH_CSRRSI	0x6073
>> +#define INSN_MASK_CSRRSI	0x707f
>> +#define INSN_MATCH_CSRRCI	0x7073
>> +#define INSN_MASK_CSRRCI	0x707f
>> +
>> +#define INSN_MATCH_FLW			0x2007
>> +#define INSN_MASK_FLW			0x707f
>> +#define INSN_MATCH_FLD			0x3007
>> +#define INSN_MASK_FLD			0x707f
>> +#define INSN_MATCH_FLQ			0x4007
>> +#define INSN_MASK_FLQ			0x707f
>> +#define INSN_MATCH_FSW			0x2027
>> +#define INSN_MASK_FSW			0x707f
>> +#define INSN_MATCH_FSD			0x3027
>> +#define INSN_MASK_FSD			0x707f
>> +#define INSN_MATCH_FSQ			0x4027
>> +#define INSN_MASK_FSQ			0x707f
>> +
>> +#define INSN_MATCH_C_FLD		0x2000
>> +#define INSN_MASK_C_FLD			0xe003
>> +#define INSN_MATCH_C_FLW		0x6000
>> +#define INSN_MASK_C_FLW			0xe003
>> +#define INSN_MATCH_C_FSD		0xa000
>> +#define INSN_MASK_C_FSD			0xe003
>> +#define INSN_MATCH_C_FSW		0xe000
>> +#define INSN_MASK_C_FSW			0xe003
>> +#define INSN_MATCH_C_FLDSP		0x2002
>> +#define INSN_MASK_C_FLDSP		0xe003
>> +#define INSN_MATCH_C_FSDSP		0xa002
>> +#define INSN_MASK_C_FSDSP		0xe003
>> +#define INSN_MATCH_C_FLWSP		0x6002
>> +#define INSN_MASK_C_FLWSP		0xe003
>> +#define INSN_MATCH_C_FSWSP		0xe002
>> +#define INSN_MASK_C_FSWSP		0xe003
>> +
>> +#define INSN_16BIT_MASK		0x3
>> +
>> +#define INSN_IS_16BIT(insn)	(((insn) & INSN_16BIT_MASK) != INSN_16BIT_MASK)
>> +
>> +#define INSN_LEN(insn)		(INSN_IS_16BIT(insn) ? 2 : 4)
>> +
>> +#define SHIFT_RIGHT(x, y)               \
>> +	((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
>> +
>> +#define REG_MASK			\
>> +	((1 << (5 + LOG_REGBYTES)) - (1 << LOG_REGBYTES))
>> +
>> +#define REG_OFFSET(insn, pos)		\
>> +	(SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
>> +
>> +#define REG_PTR(insn, pos, regs)	\
>> +	((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
>> +
>> +#define GET_RS1(insn, regs)	(*REG_PTR(insn, SH_RS1, regs))
>> +#define GET_RS2(insn, regs)	(*REG_PTR(insn, SH_RS2, regs))
>> +#define GET_RS1S(insn, regs)	(*REG_PTR(RVC_RS1S(insn), 0, regs))
>> +#define GET_RS2S(insn, regs)	(*REG_PTR(RVC_RS2S(insn), 0, regs))
>> +#define GET_RS2C(insn, regs)	(*REG_PTR(insn, SH_RS2C, regs))
>> +#define GET_SP(regs)		(*REG_PTR(2, 0, regs))
>> +#define SET_RD(insn, regs, val)	(*REG_PTR(insn, SH_RD, regs) = (val))
>> +#define IMM_I(insn)		((s32)(insn) >> 20)
>> +#define IMM_S(insn)		(((s32)(insn) >> 25 << 5) | \
>> +				 (s32)(((insn) >> 7) & 0x1f))
> Hi Alex,
>
>> +#define GET_PRECISION(insn) (((insn) >> 25) & 3)
>> +#define GET_RM(insn) (((insn) >> 12) & 7)
>> +#define PRECISION_S 0
>> +#define PRECISION_D 1
> These 4 defines seems unused.
>
>> +
>> +#define SH_RD			7
>> +#define SH_RS1			15
>> +#define SH_RS2			20
>> +#define SH_RS2C			2
>> +#define MASK_RX			0x1f
>> +
>> +#if defined(CONFIG_64BIT)
>> +#define LOG_REGBYTES			3
> There is already a definition for pointer log in asm.h (RISCV_LGPTR)
> although it's a string for !ASSEMBLY, maybe that could be reused rather
> than duplicating that ?


It does not work out of the box because of the string definition for 
!ASSEMBLY, I'll keep it that way then to avoid introducing a new define 
(I just move stuff here).


>
>> +#define XLEN				64
>> +#else
>> +#define LOG_REGBYTES			2
>> +#define XLEN				32
>> +#endif
>
>> +#define REGBYTES			(1 << LOG_REGBYTES)
>> +#define XLEN_MINUS_16			((XLEN) - 16)
> These 2 defines seems unused and can be removed (XLEN can be removed as
> well)
>
> Thanks,
>
> Clément


Thanks,

Alex


>
>> +
>> +#define MASK_FUNCT3			0x7000
>> +
>> +#define GET_FUNCT3(insn)	(((insn) >> 12) & 7)
>> +
>>   #define RV_IMM_SIGN(x) (-(((x) >> 31) & 1))
>>   #define RVC_IMM_SIGN(x) (-(((x) >> 12) & 1))
>>   #define RV_X(X, s, n) (((X) >> (s)) & ((1 << (n)) - 1))
>> +#define RVC_LW_IMM(x)	((RV_X(x, 6, 1) << 2) | \
>> +			 (RV_X(x, 10, 3) << 3) | \
>> +			 (RV_X(x, 5, 1) << 6))
>> +#define RVC_LD_IMM(x)	((RV_X(x, 10, 3) << 3) | \
>> +			 (RV_X(x, 5, 2) << 6))
>> +#define RVC_LWSP_IMM(x)	((RV_X(x, 4, 3) << 2) | \
>> +			 (RV_X(x, 12, 1) << 5) | \
>> +			 (RV_X(x, 2, 2) << 6))
>> +#define RVC_LDSP_IMM(x)	((RV_X(x, 5, 2) << 3) | \
>> +			 (RV_X(x, 12, 1) << 5) | \
>> +			 (RV_X(x, 2, 3) << 6))
>> +#define RVC_SWSP_IMM(x)	((RV_X(x, 9, 4) << 2) | \
>> +			 (RV_X(x, 7, 2) << 6))
>> +#define RVC_SDSP_IMM(x)	((RV_X(x, 10, 3) << 3) | \
>> +			 (RV_X(x, 7, 3) << 6))
>> +#define RVC_RS1S(insn)	(8 + RV_X(insn, SH_RD, 3))
>> +#define RVC_RS2S(insn)	(8 + RV_X(insn, SH_RS2C, 3))
>> +#define RVC_RS2(insn)	RV_X(insn, SH_RS2C, 5)
>>   #define RV_X_mask(X, s, mask)  (((X) >> (s)) & (mask))
>>   #define RVC_X(X, s, mask) RV_X_mask(X, s, mask)
>>   
>> diff --git a/arch/riscv/kernel/elf_kexec.c b/arch/riscv/kernel/elf_kexec.c
>> index 15e6a8f3d50b..1c3b76a67356 100644
>> --- a/arch/riscv/kernel/elf_kexec.c
>> +++ b/arch/riscv/kernel/elf_kexec.c
>> @@ -21,6 +21,7 @@
>>   #include <linux/memblock.h>
>>   #include <linux/vmalloc.h>
>>   #include <asm/setup.h>
>> +#include <asm/insn.h>
>>   
>>   int arch_kimage_file_post_load_cleanup(struct kimage *image)
>>   {
>> diff --git a/arch/riscv/kernel/traps_misaligned.c b/arch/riscv/kernel/traps_misaligned.c
>> index fb2599d62752..0151f670cd46 100644
>> --- a/arch/riscv/kernel/traps_misaligned.c
>> +++ b/arch/riscv/kernel/traps_misaligned.c
>> @@ -17,141 +17,7 @@
>>   #include <asm/hwprobe.h>
>>   #include <asm/cpufeature.h>
>>   #include <asm/vector.h>
>> -
>> -#define INSN_MATCH_LB			0x3
>> -#define INSN_MASK_LB			0x707f
>> -#define INSN_MATCH_LH			0x1003
>> -#define INSN_MASK_LH			0x707f
>> -#define INSN_MATCH_LW			0x2003
>> -#define INSN_MASK_LW			0x707f
>> -#define INSN_MATCH_LD			0x3003
>> -#define INSN_MASK_LD			0x707f
>> -#define INSN_MATCH_LBU			0x4003
>> -#define INSN_MASK_LBU			0x707f
>> -#define INSN_MATCH_LHU			0x5003
>> -#define INSN_MASK_LHU			0x707f
>> -#define INSN_MATCH_LWU			0x6003
>> -#define INSN_MASK_LWU			0x707f
>> -#define INSN_MATCH_SB			0x23
>> -#define INSN_MASK_SB			0x707f
>> -#define INSN_MATCH_SH			0x1023
>> -#define INSN_MASK_SH			0x707f
>> -#define INSN_MATCH_SW			0x2023
>> -#define INSN_MASK_SW			0x707f
>> -#define INSN_MATCH_SD			0x3023
>> -#define INSN_MASK_SD			0x707f
>> -
>> -#define INSN_MATCH_FLW			0x2007
>> -#define INSN_MASK_FLW			0x707f
>> -#define INSN_MATCH_FLD			0x3007
>> -#define INSN_MASK_FLD			0x707f
>> -#define INSN_MATCH_FLQ			0x4007
>> -#define INSN_MASK_FLQ			0x707f
>> -#define INSN_MATCH_FSW			0x2027
>> -#define INSN_MASK_FSW			0x707f
>> -#define INSN_MATCH_FSD			0x3027
>> -#define INSN_MASK_FSD			0x707f
>> -#define INSN_MATCH_FSQ			0x4027
>> -#define INSN_MASK_FSQ			0x707f
>> -
>> -#define INSN_MATCH_C_LD			0x6000
>> -#define INSN_MASK_C_LD			0xe003
>> -#define INSN_MATCH_C_SD			0xe000
>> -#define INSN_MASK_C_SD			0xe003
>> -#define INSN_MATCH_C_LW			0x4000
>> -#define INSN_MASK_C_LW			0xe003
>> -#define INSN_MATCH_C_SW			0xc000
>> -#define INSN_MASK_C_SW			0xe003
>> -#define INSN_MATCH_C_LDSP		0x6002
>> -#define INSN_MASK_C_LDSP		0xe003
>> -#define INSN_MATCH_C_SDSP		0xe002
>> -#define INSN_MASK_C_SDSP		0xe003
>> -#define INSN_MATCH_C_LWSP		0x4002
>> -#define INSN_MASK_C_LWSP		0xe003
>> -#define INSN_MATCH_C_SWSP		0xc002
>> -#define INSN_MASK_C_SWSP		0xe003
>> -
>> -#define INSN_MATCH_C_FLD		0x2000
>> -#define INSN_MASK_C_FLD			0xe003
>> -#define INSN_MATCH_C_FLW		0x6000
>> -#define INSN_MASK_C_FLW			0xe003
>> -#define INSN_MATCH_C_FSD		0xa000
>> -#define INSN_MASK_C_FSD			0xe003
>> -#define INSN_MATCH_C_FSW		0xe000
>> -#define INSN_MASK_C_FSW			0xe003
>> -#define INSN_MATCH_C_FLDSP		0x2002
>> -#define INSN_MASK_C_FLDSP		0xe003
>> -#define INSN_MATCH_C_FSDSP		0xa002
>> -#define INSN_MASK_C_FSDSP		0xe003
>> -#define INSN_MATCH_C_FLWSP		0x6002
>> -#define INSN_MASK_C_FLWSP		0xe003
>> -#define INSN_MATCH_C_FSWSP		0xe002
>> -#define INSN_MASK_C_FSWSP		0xe003
>> -
>> -#define INSN_LEN(insn)			((((insn) & 0x3) < 0x3) ? 2 : 4)
>> -
>> -#if defined(CONFIG_64BIT)
>> -#define LOG_REGBYTES			3
>> -#define XLEN				64
>> -#else
>> -#define LOG_REGBYTES			2
>> -#define XLEN				32
>> -#endif
>> -#define REGBYTES			(1 << LOG_REGBYTES)
>> -#define XLEN_MINUS_16			((XLEN) - 16)
>> -
>> -#define SH_RD				7
>> -#define SH_RS1				15
>> -#define SH_RS2				20
>> -#define SH_RS2C				2
>> -
>> -#define RVC_LW_IMM(x)			((RV_X(x, 6, 1) << 2) | \
>> -					 (RV_X(x, 10, 3) << 3) | \
>> -					 (RV_X(x, 5, 1) << 6))
>> -#define RVC_LD_IMM(x)			((RV_X(x, 10, 3) << 3) | \
>> -					 (RV_X(x, 5, 2) << 6))
>> -#define RVC_LWSP_IMM(x)			((RV_X(x, 4, 3) << 2) | \
>> -					 (RV_X(x, 12, 1) << 5) | \
>> -					 (RV_X(x, 2, 2) << 6))
>> -#define RVC_LDSP_IMM(x)			((RV_X(x, 5, 2) << 3) | \
>> -					 (RV_X(x, 12, 1) << 5) | \
>> -					 (RV_X(x, 2, 3) << 6))
>> -#define RVC_SWSP_IMM(x)			((RV_X(x, 9, 4) << 2) | \
>> -					 (RV_X(x, 7, 2) << 6))
>> -#define RVC_SDSP_IMM(x)			((RV_X(x, 10, 3) << 3) | \
>> -					 (RV_X(x, 7, 3) << 6))
>> -#define RVC_RS1S(insn)			(8 + RV_X(insn, SH_RD, 3))
>> -#define RVC_RS2S(insn)			(8 + RV_X(insn, SH_RS2C, 3))
>> -#define RVC_RS2(insn)			RV_X(insn, SH_RS2C, 5)
>> -
>> -#define SHIFT_RIGHT(x, y)		\
>> -	((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
>> -
>> -#define REG_MASK			\
>> -	((1 << (5 + LOG_REGBYTES)) - (1 << LOG_REGBYTES))
>> -
>> -#define REG_OFFSET(insn, pos)		\
>> -	(SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
>> -
>> -#define REG_PTR(insn, pos, regs)	\
>> -	(ulong *)((ulong)(regs) + REG_OFFSET(insn, pos))
>> -
>> -#define GET_RS1(insn, regs)		(*REG_PTR(insn, SH_RS1, regs))
>> -#define GET_RS2(insn, regs)		(*REG_PTR(insn, SH_RS2, regs))
>> -#define GET_RS1S(insn, regs)		(*REG_PTR(RVC_RS1S(insn), 0, regs))
>> -#define GET_RS2S(insn, regs)		(*REG_PTR(RVC_RS2S(insn), 0, regs))
>> -#define GET_RS2C(insn, regs)		(*REG_PTR(insn, SH_RS2C, regs))
>> -#define GET_SP(regs)			(*REG_PTR(2, 0, regs))
>> -#define SET_RD(insn, regs, val)		(*REG_PTR(insn, SH_RD, regs) = (val))
>> -#define IMM_I(insn)			((s32)(insn) >> 20)
>> -#define IMM_S(insn)			(((s32)(insn) >> 25 << 5) | \
>> -					 (s32)(((insn) >> 7) & 0x1f))
>> -#define MASK_FUNCT3			0x7000
>> -
>> -#define GET_PRECISION(insn) (((insn) >> 25) & 3)
>> -#define GET_RM(insn) (((insn) >> 12) & 7)
>> -#define PRECISION_S 0
>> -#define PRECISION_D 1
>> +#include <asm/insn.h>
>>   
>>   #ifdef CONFIG_FPU
>>   
>> diff --git a/arch/riscv/kvm/vcpu_insn.c b/arch/riscv/kvm/vcpu_insn.c
>> index ba4813673f95..de1f96ea6225 100644
>> --- a/arch/riscv/kvm/vcpu_insn.c
>> +++ b/arch/riscv/kvm/vcpu_insn.c
>> @@ -8,132 +8,7 @@
>>   #include <linux/kvm_host.h>
>>   
>>   #include <asm/cpufeature.h>
>> -
>> -#define INSN_OPCODE_MASK	0x007c
>> -#define INSN_OPCODE_SHIFT	2
>> -#define INSN_OPCODE_SYSTEM	28
>> -
>> -#define INSN_MASK_WFI		0xffffffff
>> -#define INSN_MATCH_WFI		0x10500073
>> -
>> -#define INSN_MASK_WRS		0xffffffff
>> -#define INSN_MATCH_WRS		0x00d00073
>> -
>> -#define INSN_MATCH_CSRRW	0x1073
>> -#define INSN_MASK_CSRRW		0x707f
>> -#define INSN_MATCH_CSRRS	0x2073
>> -#define INSN_MASK_CSRRS		0x707f
>> -#define INSN_MATCH_CSRRC	0x3073
>> -#define INSN_MASK_CSRRC		0x707f
>> -#define INSN_MATCH_CSRRWI	0x5073
>> -#define INSN_MASK_CSRRWI	0x707f
>> -#define INSN_MATCH_CSRRSI	0x6073
>> -#define INSN_MASK_CSRRSI	0x707f
>> -#define INSN_MATCH_CSRRCI	0x7073
>> -#define INSN_MASK_CSRRCI	0x707f
>> -
>> -#define INSN_MATCH_LB		0x3
>> -#define INSN_MASK_LB		0x707f
>> -#define INSN_MATCH_LH		0x1003
>> -#define INSN_MASK_LH		0x707f
>> -#define INSN_MATCH_LW		0x2003
>> -#define INSN_MASK_LW		0x707f
>> -#define INSN_MATCH_LD		0x3003
>> -#define INSN_MASK_LD		0x707f
>> -#define INSN_MATCH_LBU		0x4003
>> -#define INSN_MASK_LBU		0x707f
>> -#define INSN_MATCH_LHU		0x5003
>> -#define INSN_MASK_LHU		0x707f
>> -#define INSN_MATCH_LWU		0x6003
>> -#define INSN_MASK_LWU		0x707f
>> -#define INSN_MATCH_SB		0x23
>> -#define INSN_MASK_SB		0x707f
>> -#define INSN_MATCH_SH		0x1023
>> -#define INSN_MASK_SH		0x707f
>> -#define INSN_MATCH_SW		0x2023
>> -#define INSN_MASK_SW		0x707f
>> -#define INSN_MATCH_SD		0x3023
>> -#define INSN_MASK_SD		0x707f
>> -
>> -#define INSN_MATCH_C_LD		0x6000
>> -#define INSN_MASK_C_LD		0xe003
>> -#define INSN_MATCH_C_SD		0xe000
>> -#define INSN_MASK_C_SD		0xe003
>> -#define INSN_MATCH_C_LW		0x4000
>> -#define INSN_MASK_C_LW		0xe003
>> -#define INSN_MATCH_C_SW		0xc000
>> -#define INSN_MASK_C_SW		0xe003
>> -#define INSN_MATCH_C_LDSP	0x6002
>> -#define INSN_MASK_C_LDSP	0xe003
>> -#define INSN_MATCH_C_SDSP	0xe002
>> -#define INSN_MASK_C_SDSP	0xe003
>> -#define INSN_MATCH_C_LWSP	0x4002
>> -#define INSN_MASK_C_LWSP	0xe003
>> -#define INSN_MATCH_C_SWSP	0xc002
>> -#define INSN_MASK_C_SWSP	0xe003
>> -
>> -#define INSN_16BIT_MASK		0x3
>> -
>> -#define INSN_IS_16BIT(insn)	(((insn) & INSN_16BIT_MASK) != INSN_16BIT_MASK)
>> -
>> -#define INSN_LEN(insn)		(INSN_IS_16BIT(insn) ? 2 : 4)
>> -
>> -#ifdef CONFIG_64BIT
>> -#define LOG_REGBYTES		3
>> -#else
>> -#define LOG_REGBYTES		2
>> -#endif
>> -#define REGBYTES		(1 << LOG_REGBYTES)
>> -
>> -#define SH_RD			7
>> -#define SH_RS1			15
>> -#define SH_RS2			20
>> -#define SH_RS2C			2
>> -#define MASK_RX			0x1f
>> -
>> -#define RVC_LW_IMM(x)		((RV_X(x, 6, 1) << 2) | \
>> -				 (RV_X(x, 10, 3) << 3) | \
>> -				 (RV_X(x, 5, 1) << 6))
>> -#define RVC_LD_IMM(x)		((RV_X(x, 10, 3) << 3) | \
>> -				 (RV_X(x, 5, 2) << 6))
>> -#define RVC_LWSP_IMM(x)		((RV_X(x, 4, 3) << 2) | \
>> -				 (RV_X(x, 12, 1) << 5) | \
>> -				 (RV_X(x, 2, 2) << 6))
>> -#define RVC_LDSP_IMM(x)		((RV_X(x, 5, 2) << 3) | \
>> -				 (RV_X(x, 12, 1) << 5) | \
>> -				 (RV_X(x, 2, 3) << 6))
>> -#define RVC_SWSP_IMM(x)		((RV_X(x, 9, 4) << 2) | \
>> -				 (RV_X(x, 7, 2) << 6))
>> -#define RVC_SDSP_IMM(x)		((RV_X(x, 10, 3) << 3) | \
>> -				 (RV_X(x, 7, 3) << 6))
>> -#define RVC_RS1S(insn)		(8 + RV_X(insn, SH_RD, 3))
>> -#define RVC_RS2S(insn)		(8 + RV_X(insn, SH_RS2C, 3))
>> -#define RVC_RS2(insn)		RV_X(insn, SH_RS2C, 5)
>> -
>> -#define SHIFT_RIGHT(x, y)		\
>> -	((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
>> -
>> -#define REG_MASK			\
>> -	((1 << (5 + LOG_REGBYTES)) - (1 << LOG_REGBYTES))
>> -
>> -#define REG_OFFSET(insn, pos)		\
>> -	(SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
>> -
>> -#define REG_PTR(insn, pos, regs)	\
>> -	((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
>> -
>> -#define GET_FUNCT3(insn)	(((insn) >> 12) & 7)
>> -
>> -#define GET_RS1(insn, regs)	(*REG_PTR(insn, SH_RS1, regs))
>> -#define GET_RS2(insn, regs)	(*REG_PTR(insn, SH_RS2, regs))
>> -#define GET_RS1S(insn, regs)	(*REG_PTR(RVC_RS1S(insn), 0, regs))
>> -#define GET_RS2S(insn, regs)	(*REG_PTR(RVC_RS2S(insn), 0, regs))
>> -#define GET_RS2C(insn, regs)	(*REG_PTR(insn, SH_RS2C, regs))
>> -#define GET_SP(regs)		(*REG_PTR(2, 0, regs))
>> -#define SET_RD(insn, regs, val)	(*REG_PTR(insn, SH_RD, regs) = (val))
>> -#define IMM_I(insn)		((s32)(insn) >> 20)
>> -#define IMM_S(insn)		(((s32)(insn) >> 25 << 5) | \
>> -				 (s32)(((insn) >> 7) & 0x1f))
>> +#include <asm/insn.h>
>>   
>>   struct insn_func {
>>   	unsigned long mask;
>
> _______________________________________________
> linux-riscv mailing list
> linux-riscv@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ