[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 1/7] target/mips: Fix emulation of nanoMips MTHLIP instruction
From: |
Stefan Pejic |
Subject: |
[PATCH 1/7] target/mips: Fix emulation of nanoMips MTHLIP instruction |
Date: |
Wed, 4 May 2022 13:03:57 +0200 |
The field ac in nanoMips instruction MTHLIP rs, ac is specified in
nanoMips documentation as opcode[15..14] (2 bits). However, in the
current QEMU code, the corresponding argument passed to the helper
gen_helper_mthlip() has the value of opcode[15..11] (5 bits). Right
shift the value of this argument by three bits to fix this.
Signed-off-by: Stefan Pejic <stefan.pejic@syrmia.com>
---
target/mips/tcg/nanomips_translate.c.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/mips/tcg/nanomips_translate.c.inc
b/target/mips/tcg/nanomips_translate.c.inc
index 916cece4d2..58ae35a156 100644
--- a/target/mips/tcg/nanomips_translate.c.inc
+++ b/target/mips/tcg/nanomips_translate.c.inc
@@ -1597,7 +1597,7 @@ static void gen_pool32axf_1_nanomips_insn(DisasContext
*ctx, uint32_t opc,
check_dsp(ctx);
switch (extract32(ctx->opcode, 12, 2)) {
case NM_MTHLIP:
- tcg_gen_movi_tl(t0, v2);
+ tcg_gen_movi_tl(t0, v2 >> 3);
gen_helper_mthlip(t0, v0_t, cpu_env);
break;
case NM_SHILOV:
--
2.25.1
- [PATCH 0/7] Undeprecate nanoMIPS and fix multiple bugs, Stefan Pejic, 2022/05/04
- [PATCH 4/7] target/mips: Fix emulation of nanoMips BNEC[32] instruction, Stefan Pejic, 2022/05/04
- [PATCH 5/7] target/mips: Fix handling of unaligned memory access for nanoMips ISA, Stefan Pejic, 2022/05/04
- [PATCH 1/7] target/mips: Fix emulation of nanoMips MTHLIP instruction,
Stefan Pejic <=
- [PATCH 3/7] target/mips: Fix emulation of nanoMips BPOSGE32C instruction, Stefan Pejic, 2022/05/04
- [PATCH 2/7] target/mips: Fix emulation of nanoMips EXTRV_S.H instruction, Stefan Pejic, 2022/05/04
- [PATCH 6/7] target/mips: Add missing default cases for some nanoMips pools, Stefan Pejic, 2022/05/04
- [PATCH 7/7] target/mips: Undeprecate nanoMips ISA support in QEMU, Stefan Pejic, 2022/05/04