[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v6 54/66] target/sparc: Remove DEBUG_UNALIGNED
From: |
Richard Henderson |
Subject: |
[PATCH v6 54/66] target/sparc: Remove DEBUG_UNALIGNED |
Date: |
Sat, 30 Oct 2021 10:16:23 -0700 |
The printf should have been qemu_log_mask, the parameters
themselves no longer compile, and because this is placed
before unwinding the PC is actively wrong.
We get better (and correct) logging on the other side of
raising the exception, in sparc_cpu_do_interrupt.
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
target/sparc/ldst_helper.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/target/sparc/ldst_helper.c b/target/sparc/ldst_helper.c
index bbf3601cb1..0549b6adf1 100644
--- a/target/sparc/ldst_helper.c
+++ b/target/sparc/ldst_helper.c
@@ -27,7 +27,6 @@
//#define DEBUG_MMU
//#define DEBUG_MXCC
-//#define DEBUG_UNALIGNED
//#define DEBUG_UNASSIGNED
//#define DEBUG_ASI
//#define DEBUG_CACHE_CONTROL
@@ -364,10 +363,6 @@ static void do_check_align(CPUSPARCState *env,
target_ulong addr,
uint32_t align, uintptr_t ra)
{
if (addr & align) {
-#ifdef DEBUG_UNALIGNED
- printf("Unaligned access to 0x" TARGET_FMT_lx " from 0x" TARGET_FMT_lx
- "\n", addr, env->pc);
-#endif
cpu_raise_exception_ra(env, TT_UNALIGNED, ra);
}
}
@@ -1968,10 +1963,6 @@ void QEMU_NORETURN
sparc_cpu_do_unaligned_access(CPUState *cs, vaddr addr,
SPARCCPU *cpu = SPARC_CPU(cs);
CPUSPARCState *env = &cpu->env;
-#ifdef DEBUG_UNALIGNED
- printf("Unaligned access to 0x" TARGET_FMT_lx " from 0x" TARGET_FMT_lx
- "\n", addr, env->pc);
-#endif
cpu_raise_exception_ra(env, TT_UNALIGNED, retaddr);
}
#endif
--
2.25.1
- [PATCH v6 51/66] linux-user/ppc: Remove POWERPC_EXCP_ALIGN handling, (continued)
- [PATCH v6 51/66] linux-user/ppc: Remove POWERPC_EXCP_ALIGN handling, Richard Henderson, 2021/10/30
- [PATCH v6 42/66] hw/core: Add TCGCPUOps.record_sigbus, Richard Henderson, 2021/10/30
- [PATCH v6 43/66] linux-user: Add cpu_loop_exit_sigbus, Richard Henderson, 2021/10/30
- [PATCH v6 50/66] target/ppc: Restrict ppc_cpu_do_unaligned_access to sysemu, Richard Henderson, 2021/10/30
- [PATCH v6 49/66] target/ppc: Set fault address in ppc_cpu_do_unaligned_access, Richard Henderson, 2021/10/30
- [PATCH v6 59/66] tcg: Add helper_unaligned_{ld, st} for user-only sigbus, Richard Henderson, 2021/10/30
- [PATCH v6 57/66] accel/tcg: Report unaligned atomics for user-only, Richard Henderson, 2021/10/30
- [PATCH v6 60/66] linux-user: Handle BUS_ADRALN in host_signal_handler, Richard Henderson, 2021/10/30
- [PATCH v6 56/66] target/sparc: Set fault address in sparc_cpu_do_unaligned_access, Richard Henderson, 2021/10/30
- [PATCH v6 62/66] linux-user: Disable more prctl subcodes, Richard Henderson, 2021/10/30
- [PATCH v6 54/66] target/sparc: Remove DEBUG_UNALIGNED,
Richard Henderson <=
- [PATCH v6 53/66] target/sh4: Set fault address in superh_cpu_do_unaligned_access, Richard Henderson, 2021/10/30
- [PATCH v6 65/66] target/hppa: Implement prctl_unalign_sigbus, Richard Henderson, 2021/10/30
- [PATCH v6 48/66] target/ppc: Move SPR_DSISR setting to powerpc_excp, Richard Henderson, 2021/10/30
- [PATCH v6 58/66] accel/tcg: Report unaligned load/store for user-only, Richard Henderson, 2021/10/30
- [PATCH v6 63/66] linux-user: Add code for PR_GET/SET_UNALIGN, Richard Henderson, 2021/10/30
- [PATCH v6 47/66] target/microblaze: Do not set MO_ALIGN for user-only, Richard Henderson, 2021/10/30
- [PATCH v6 55/66] target/sparc: Split out build_sfsr, Richard Henderson, 2021/10/30
- [PATCH v6 61/66] linux-user: Split out do_prctl and subroutines, Richard Henderson, 2021/10/30
- [PATCH v6 64/66] target/alpha: Implement prctl_unalign_sigbus, Richard Henderson, 2021/10/30
- [PATCH v6 66/66] target/sh4: Implement prctl_unalign_sigbus, Richard Henderson, 2021/10/30