[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 10/14] target/ppc: 405: Debug exception cleanup
From: |
Fabiano Rosas |
Subject: |
[PATCH v2 10/14] target/ppc: 405: Debug exception cleanup |
Date: |
Tue, 18 Jan 2022 15:44:44 -0300 |
The current Debug exception dispatch is the BookE one, so it is
different from the 405. We effectively don't support the 405 Debug
exception.
This patch removes the BookE code and moves the DEBUG into the "not
implemented" block.
Note that there is in theory a functional change here since we now
abort when a Debug exception happens. However, given how it was never
implemented, I don't believe this to have ever been dispatched for the
405.
Signed-off-by: Fabiano Rosas <farosas@linux.ibm.com>
---
target/ppc/excp_helper.c | 18 ++++--------------
1 file changed, 4 insertions(+), 14 deletions(-)
diff --git a/target/ppc/excp_helper.c b/target/ppc/excp_helper.c
index d263f20002..84ec7e094a 100644
--- a/target/ppc/excp_helper.c
+++ b/target/ppc/excp_helper.c
@@ -539,23 +539,13 @@ static void powerpc_excp_40x(PowerPCCPU *cpu, int excp)
case POWERPC_EXCP_DTLB: /* Data TLB error */
case POWERPC_EXCP_ITLB: /* Instruction TLB error */
break;
- case POWERPC_EXCP_DEBUG: /* Debug interrupt */
- if (env->flags & POWERPC_FLAG_DE) {
- /* FIXME: choose one or the other based on CPU type */
- srr0 = SPR_BOOKE_DSRR0;
- srr1 = SPR_BOOKE_DSRR1;
-
- env->spr[SPR_BOOKE_CSRR0] = env->nip;
- env->spr[SPR_BOOKE_CSRR1] = msr;
-
- /* DBSR already modified by caller */
- } else {
- cpu_abort(cs, "Debug exception triggered on unsupported model\n");
- }
- break;
case POWERPC_EXCP_PIT: /* Programmable interval timer interrupt */
trace_ppc_excp_print("PIT");
break;
+ case POWERPC_EXCP_DEBUG: /* Debug interrupt */
+ cpu_abort(cs, "%s exception not implemented\n",
+ powerpc_excp_name(excp));
+ break;
default:
cpu_abort(cs, "Invalid PowerPC exception %d. Aborting\n", excp);
break;
--
2.33.1
- Re: [PATCH v2 11/14] target/ppc: 405: Data Storage exception cleanup, (continued)
[PATCH v2 02/14] target/ppc: 405: Add missing MSR_ME bit, Fabiano Rosas, 2022/01/18
[PATCH v2 13/14] target/ppc: 405: Program exception cleanup, Fabiano Rosas, 2022/01/18
Re: [PATCH v2 13/14] target/ppc: 405: Program exception cleanup, Cédric Le Goater, 2022/01/25
[PATCH v2 10/14] target/ppc: 405: Debug exception cleanup,
Fabiano Rosas <=
[PATCH v2 05/14] target/ppc: 405: Critical exceptions cleanup, Fabiano Rosas, 2022/01/18
[PATCH v2 09/14] target/ppc: 405: Alignment exception cleanup, Fabiano Rosas, 2022/01/18
[PATCH v2 12/14] target/ppc: 405: Instruction storage interrupt cleanup, Fabiano Rosas, 2022/01/18
[PATCH v2 14/14] target/ppc: 405: Watchdog timer exception cleanup, Fabiano Rosas, 2022/01/18