Re: [PATCH v3 2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()

From: Andrew Jones
Date: Fri May 09 2025 - 07:30:09 EST


On Thu, May 08, 2025 at 02:52:01PM +0200, Alexandre Ghiti wrote:
> RV_X() macro is defined in two different ways which is error prone.
>
> So harmonize its first definition and add another macro RV_X_mask() for
> the second one.
>
> Signed-off-by: Alexandre Ghiti <alexghiti@xxxxxxxxxxxx>
> ---
> arch/riscv/include/asm/insn.h | 39 ++++++++++++++--------------
> arch/riscv/kernel/elf_kexec.c | 2 +-
> arch/riscv/kernel/traps_misaligned.c | 2 +-
> arch/riscv/kvm/vcpu_insn.c | 2 +-
> 4 files changed, 23 insertions(+), 22 deletions(-)
>

Reviewed-by: Andrew Jones <ajones@xxxxxxxxxxxxxxxx>