Skip to content

Dev/alex/insn duplicate v2 manual#378

Closed
AlexGhiti wants to merge 4 commits into
masterfrom
dev/alex/insn_duplicate_v2_manual
Closed

Dev/alex/insn duplicate v2 manual#378
AlexGhiti wants to merge 4 commits into
masterfrom
dev/alex/insn_duplicate_v2_manual

Conversation

@AlexGhiti
Copy link
Copy Markdown

No description provided.

Alexandre Ghiti added 4 commits May 6, 2025 07:43
Simply fix a typo.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
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@rivosinc.com>
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.

Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants