qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Subject: [PATCH v7 56/64] target/nios2: Implement EIC interrupt processing
Date: Thu, 21 Apr 2022 08:17:27 -0700	[thread overview]
Message-ID: <20220421151735.31996-57-richard.henderson@linaro.org> (raw)
In-Reply-To: <20220421151735.31996-1-richard.henderson@linaro.org>

This is the cpu side of the operation.  Register one irq line,
called EIC.  Split out the rather different processing to a
separate function.

Delay initialization of gpio irqs until realize.  We need to
provide a window after init in which the board can set eic_present.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/nios2/cpu.h    |  8 ++++
 target/nios2/cpu.c    | 92 +++++++++++++++++++++++++++++++++----------
 target/nios2/helper.c | 51 +++++++++++++++++++++++-
 3 files changed, 129 insertions(+), 22 deletions(-)

diff --git a/target/nios2/cpu.h b/target/nios2/cpu.h
index c2fedead54..e25d3cdb11 100644
--- a/target/nios2/cpu.h
+++ b/target/nios2/cpu.h
@@ -114,6 +114,7 @@ FIELD(CR_STATUS, CRS, 10, 6)
 FIELD(CR_STATUS, PRS, 16, 6)
 FIELD(CR_STATUS, NMI, 22, 1)
 FIELD(CR_STATUS, RSIE, 23, 1)
+FIELD(CR_STATUS, SRS, 31, 1)  /* only in sstatus */
 
 #define CR_STATUS_PIE    R_CR_STATUS_PIE_MASK
 #define CR_STATUS_U      R_CR_STATUS_U_MASK
@@ -121,6 +122,7 @@ FIELD(CR_STATUS, RSIE, 23, 1)
 #define CR_STATUS_IH     R_CR_STATUS_IH_MASK
 #define CR_STATUS_NMI    R_CR_STATUS_NMI_MASK
 #define CR_STATUS_RSIE   R_CR_STATUS_RSIE_MASK
+#define CR_STATUS_SRS    R_CR_STATUS_SRS_MASK
 
 FIELD(CR_EXCEPTION, CAUSE, 2, 5)
 FIELD(CR_EXCEPTION, ECCFTL, 31, 1)
@@ -234,6 +236,12 @@ struct ArchCPU {
 
     /* Bits within each control register which are reserved or readonly. */
     ControlRegState cr_state[NUM_CR_REGS];
+
+    /* External Interrupt Controller Interface */
+    uint32_t rha; /* Requested handler address */
+    uint32_t ril; /* Requested interrupt level */
+    uint32_t rrs; /* Requested register set */
+    bool rnmi;    /* Requested nonmaskable interrupt */
 };
 
 
diff --git a/target/nios2/cpu.c b/target/nios2/cpu.c
index d043c02fcd..19b2409974 100644
--- a/target/nios2/cpu.c
+++ b/target/nios2/cpu.c
@@ -63,7 +63,19 @@ static void nios2_cpu_reset(DeviceState *dev)
 }
 
 #ifndef CONFIG_USER_ONLY
-static void nios2_cpu_set_irq(void *opaque, int irq, int level)
+static void eic_set_irq(void *opaque, int irq, int level)
+{
+    Nios2CPU *cpu = opaque;
+    CPUState *cs = CPU(cpu);
+
+    if (level) {
+        cpu_interrupt(cs, CPU_INTERRUPT_HARD);
+    } else {
+        cpu_reset_interrupt(cs, CPU_INTERRUPT_HARD);
+    }
+}
+
+static void iic_set_irq(void *opaque, int irq, int level)
 {
     Nios2CPU *cpu = opaque;
     CPUNios2State *env = &cpu->env;
@@ -87,15 +99,6 @@ static void nios2_cpu_initfn(Object *obj)
 
 #if !defined(CONFIG_USER_ONLY)
     mmu_init(&cpu->env);
-
-    /*
-     * These interrupt lines model the IIC (internal interrupt
-     * controller). QEMU does not currently support the EIC
-     * (external interrupt controller) -- if we did it would be
-     * a separate device in hw/intc with a custom interface to
-     * the CPU, and boards using it would not wire up these IRQ lines.
-     */
-    qdev_init_gpio_in_named(DEVICE(cpu), nios2_cpu_set_irq, "IRQ", 32);
 #endif
 }
 
@@ -128,10 +131,18 @@ static void realize_cr_status(CPUState *cs)
     RO_REG(CR_EXCEPTION);
     WR_REG(CR_BADADDR);
 
-    /* TODO: These control registers are not present with the EIC. */
-    RO_FIELD(CR_STATUS, RSIE);
-    WR_REG(CR_IENABLE);
-    RO_REG(CR_IPENDING);
+    if (cpu->eic_present) {
+        WR_FIELD(CR_STATUS, RSIE);
+        RO_FIELD(CR_STATUS, NMI);
+        WR_FIELD(CR_STATUS, PRS);
+        RO_FIELD(CR_STATUS, CRS);
+        WR_FIELD(CR_STATUS, IL);
+        WR_FIELD(CR_STATUS, IH);
+    } else {
+        RO_FIELD(CR_STATUS, RSIE);
+        WR_REG(CR_IENABLE);
+        RO_REG(CR_IPENDING);
+    }
 
     if (cpu->mmu_present) {
         WR_FIELD(CR_STATUS, U);
@@ -170,6 +181,14 @@ static void nios2_cpu_realizefn(DeviceState *dev, Error **errp)
     Nios2CPUClass *ncc = NIOS2_CPU_GET_CLASS(dev);
     Error *local_err = NULL;
 
+#ifndef CONFIG_USER_ONLY
+    if (cpu->eic_present) {
+        qdev_init_gpio_in_named(DEVICE(cpu), eic_set_irq, "EIC", 1);
+    } else {
+        qdev_init_gpio_in_named(DEVICE(cpu), iic_set_irq, "IRQ", 32);
+    }
+#endif
+
     cpu_exec_realizefn(cs, &local_err);
     if (local_err != NULL) {
         error_propagate(errp, local_err);
@@ -187,17 +206,48 @@ static void nios2_cpu_realizefn(DeviceState *dev, Error **errp)
 }
 
 #ifndef CONFIG_USER_ONLY
+static bool eic_take_interrupt(Nios2CPU *cpu)
+{
+    CPUNios2State *env = &cpu->env;
+    const uint32_t status = env->ctrl[CR_STATUS];
+
+    if (cpu->rnmi) {
+        return !(status & CR_STATUS_NMI);
+    }
+    if (!(status & CR_STATUS_PIE)) {
+        return false;
+    }
+    if (cpu->ril <= FIELD_EX32(status, CR_STATUS, IL)) {
+        return false;
+    }
+    if (cpu->rrs != FIELD_EX32(status, CR_STATUS, CRS)) {
+        return true;
+    }
+    return status & CR_STATUS_RSIE;
+}
+
+static bool iic_take_interrupt(Nios2CPU *cpu)
+{
+    CPUNios2State *env = &cpu->env;
+
+    if (!(env->ctrl[CR_STATUS] & CR_STATUS_PIE)) {
+        return false;
+    }
+    return env->ctrl[CR_IPENDING] & env->ctrl[CR_IENABLE];
+}
+
 static bool nios2_cpu_exec_interrupt(CPUState *cs, int interrupt_request)
 {
     Nios2CPU *cpu = NIOS2_CPU(cs);
-    CPUNios2State *env = &cpu->env;
 
-    if ((interrupt_request & CPU_INTERRUPT_HARD) &&
-        (env->ctrl[CR_STATUS] & CR_STATUS_PIE) &&
-        (env->ctrl[CR_IPENDING] & env->ctrl[CR_IENABLE])) {
-        cs->exception_index = EXCP_IRQ;
-        nios2_cpu_do_interrupt(cs);
-        return true;
+    if (interrupt_request & CPU_INTERRUPT_HARD) {
+        if (cpu->eic_present
+            ? eic_take_interrupt(cpu)
+            : iic_take_interrupt(cpu)) {
+            cs->exception_index = EXCP_IRQ;
+            nios2_cpu_do_interrupt(cs);
+            return true;
+        }
     }
     return false;
 }
diff --git a/target/nios2/helper.c b/target/nios2/helper.c
index 2e9fea4a01..e256d1528e 100644
--- a/target/nios2/helper.c
+++ b/target/nios2/helper.c
@@ -37,6 +37,10 @@ static void do_exception(Nios2CPU *cpu, uint32_t exception_addr,
     uint32_t old_status = env->ctrl[CR_STATUS];
     uint32_t new_status = old_status;
 
+    /* With shadow regs, exceptions are always taken into CRS 0. */
+    new_status &= ~R_CR_STATUS_CRS_MASK;
+    env->regs = env->shadow_regs[0];
+
     if ((old_status & CR_STATUS_EH) == 0) {
         int r_ea = R_EA, cr_es = CR_ESTATUS;
 
@@ -60,6 +64,14 @@ static void do_exception(Nios2CPU *cpu, uint32_t exception_addr,
                                        CR_TLBMISC_DBL);
             env->ctrl[CR_TLBMISC] |= tlbmisc_set;
         }
+
+        /*
+         * With shadow regs, and EH == 0, PRS is set from CRS.
+         * At least, so says Table 3-9, and some other text,
+         * though Table 3-38 says otherwise.
+         */
+        new_status = FIELD_DP32(new_status, CR_STATUS, PRS,
+                                FIELD_EX32(old_status, CR_STATUS, CRS));
     }
 
     new_status &= ~(CR_STATUS_PIE | CR_STATUS_U);
@@ -77,6 +89,39 @@ static void do_iic_irq(Nios2CPU *cpu)
     do_exception(cpu, cpu->exception_addr, 0, false);
 }
 
+static void do_eic_irq(Nios2CPU *cpu)
+{
+    CPUNios2State *env = &cpu->env;
+    uint32_t old_status = env->ctrl[CR_STATUS];
+    uint32_t new_status = old_status;
+    uint32_t old_rs = FIELD_EX32(old_status, CR_STATUS, CRS);
+    uint32_t new_rs = cpu->rrs;
+
+    new_status = FIELD_DP32(new_status, CR_STATUS, CRS, new_rs);
+    new_status = FIELD_DP32(new_status, CR_STATUS, IL, cpu->ril);
+    new_status = FIELD_DP32(new_status, CR_STATUS, NMI, cpu->rnmi);
+    new_status &= ~(CR_STATUS_RSIE | CR_STATUS_U);
+    new_status |= CR_STATUS_IH;
+
+    if (!(new_status & CR_STATUS_EH)) {
+        new_status = FIELD_DP32(new_status, CR_STATUS, PRS, old_rs);
+        if (new_rs == 0) {
+            env->ctrl[CR_ESTATUS] = old_status;
+        } else {
+            if (new_rs != old_rs) {
+                old_status |= CR_STATUS_SRS;
+            }
+            env->shadow_regs[new_rs][R_SSTATUS] = old_status;
+        }
+        env->shadow_regs[new_rs][R_EA] = env->pc + 4;
+    }
+
+    env->ctrl[CR_STATUS] = new_status;
+    nios2_update_crs(env);
+
+    env->pc = cpu->rha;
+}
+
 void nios2_cpu_do_interrupt(CPUState *cs)
 {
     Nios2CPU *cpu = NIOS2_CPU(cs);
@@ -142,7 +187,11 @@ void nios2_cpu_do_interrupt(CPUState *cs)
 
     switch (cs->exception_index) {
     case EXCP_IRQ:
-        do_iic_irq(cpu);
+        if (cpu->eic_present) {
+            do_eic_irq(cpu);
+        } else {
+            do_iic_irq(cpu);
+        }
         break;
 
     case EXCP_TLB_D:
-- 
2.34.1



  parent reply	other threads:[~2022-04-21 16:34 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-21 15:16 [PATCH v7 00/64] nios2 fixes, cleanups, shadow reg sets Richard Henderson
2022-04-21 15:16 ` [PATCH v7 01/64] linux-user/nios2: Hoist pc advance to the top of EXCP_TRAP Richard Henderson
2022-04-21 15:16 ` [PATCH v7 02/64] linux-user/nios2: Fix clone child return Richard Henderson
2022-04-21 15:16 ` [PATCH v7 03/64] linux-user/nios2: Drop syscall 0 "workaround" Richard Henderson
2022-04-21 15:16 ` [PATCH v7 04/64] linux-user/nios2: Adjust error return Richard Henderson
2022-04-21 15:16 ` [PATCH v7 05/64] linux-user/nios2: Handle special qemu syscall return values Richard Henderson
2022-04-21 15:16 ` [PATCH v7 06/64] linux-user/nios2: Remove do_sigreturn Richard Henderson
2022-04-21 15:16 ` [PATCH v7 07/64] linux-user/nios2: Use QEMU_ESIGRETURN from do_rt_sigreturn Richard Henderson
2022-04-21 15:16 ` [PATCH v7 08/64] tests/tcg/nios2: Re-enable linux-user tests Richard Henderson
2022-04-21 15:16 ` [PATCH v7 09/64] target/nios2: Remove user-only nios2_cpu_do_interrupt Richard Henderson
2022-04-22 12:51   ` Peter Maydell
2022-04-21 15:16 ` [PATCH v7 10/64] target/nios2: Remove nios2_cpu_record_sigsegv Richard Henderson
2022-04-22 12:53   ` Peter Maydell
2022-04-21 15:16 ` [PATCH v7 11/64] target/nios2: Build helper.c for system only Richard Henderson
2022-04-22 12:54   ` Peter Maydell
2022-04-21 15:16 ` [PATCH v7 12/64] linux-user/nios2: Use force_sig_fault for EXCP_DEBUG Richard Henderson
2022-04-22 12:54   ` Peter Maydell
2022-04-21 15:16 ` [PATCH v7 13/64] target/nios2: Check supervisor on eret Richard Henderson
2022-04-21 15:16 ` [PATCH v7 14/64] target/nios2: Stop generating code if gen_check_supervisor fails Richard Henderson
2022-04-21 15:16 ` [PATCH v7 15/64] target/nios2: Add NUM_GP_REGS and NUM_CP_REGS Richard Henderson
2022-04-21 15:16 ` [PATCH v7 16/64] target/nios2: Split PC out of env->regs[] Richard Henderson
2022-04-21 15:16 ` [PATCH v7 17/64] target/nios2: Split out helper for eret instruction Richard Henderson
2022-04-21 15:16 ` [PATCH v7 18/64] target/nios2: Fix BRET instruction Richard Henderson
2022-04-21 15:16 ` [PATCH v7 19/64] target/nios2: Do not create TCGv for control registers Richard Henderson
2022-04-21 15:16 ` [PATCH v7 20/64] linux-user/nios2: Only initialize SP and PC in target_cpu_copy_regs Richard Henderson
2022-04-21 15:16 ` [PATCH v7 21/64] target/nios2: Remove cpu_interrupts_enabled Richard Henderson
2022-04-21 15:16 ` [PATCH v7 22/64] target/nios2: Split control registers away from general registers Richard Henderson
2022-04-21 15:16 ` [PATCH v7 23/64] target/nios2: Clean up nios2_cpu_dump_state Richard Henderson
2022-04-21 15:16 ` [PATCH v7 24/64] target/nios2: Use hw/registerfields.h for CR_STATUS fields Richard Henderson
2022-04-21 15:16 ` [PATCH v7 25/64] target/nios2: Use hw/registerfields.h for CR_EXCEPTION fields Richard Henderson
2022-04-21 15:16 ` [PATCH v7 26/64] target/nios2: Use hw/registerfields.h for CR_TLBADDR fields Richard Henderson
2022-04-21 15:16 ` [PATCH v7 27/64] target/nios2: Use hw/registerfields.h for CR_TLBACC fields Richard Henderson
2022-04-21 15:16 ` [PATCH v7 28/64] target/nios2: Rename CR_TLBMISC_WR to CR_TLBMISC_WE Richard Henderson
2022-04-21 15:17 ` [PATCH v7 29/64] target/nios2: Use hw/registerfields.h for CR_TLBMISC fields Richard Henderson
2022-04-21 15:17 ` [PATCH v7 30/64] target/nios2: Move R_FOO and CR_BAR into enumerations Richard Henderson
2022-04-21 15:17 ` [PATCH v7 31/64] target/nios2: Create EXCP_SEMIHOST for semi-hosting Richard Henderson
2022-04-21 15:17 ` [PATCH v7 32/64] target/nios2: Clean up nios2_cpu_do_interrupt Richard Henderson
2022-04-21 15:17 ` [PATCH v7 33/64] target/nios2: Hoist CPU_LOG_INT logging Richard Henderson
2022-04-21 15:17 ` [PATCH v7 34/64] target/nios2: Handle EXCP_UNALIGN and EXCP_UALIGND Richard Henderson
2022-04-22 12:56   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 35/64] target/nios2: Cleanup set of CR_EXCEPTION for do_interrupt Richard Henderson
2022-04-21 15:17 ` [PATCH v7 36/64] target/nios2: Clean up handling of tlbmisc in do_exception Richard Henderson
2022-04-21 15:17 ` [PATCH v7 37/64] target/nios2: Prevent writes to read-only or reserved control fields Richard Henderson
2022-04-22 12:56   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 38/64] target/nios2: Implement cpuid Richard Henderson
2022-04-21 15:17 ` [PATCH v7 39/64] target/nios2: Implement CR_STATUS.RSIE Richard Henderson
2022-04-21 15:17 ` [PATCH v7 40/64] target/nios2: Remove CPU_INTERRUPT_NMI Richard Henderson
2022-04-21 15:17 ` [PATCH v7 41/64] target/nios2: Support division error exception Richard Henderson
2022-04-21 15:17 ` [PATCH v7 42/64] target/nios2: Use tcg_constant_tl Richard Henderson
2022-04-21 15:17 ` [PATCH v7 43/64] target/nios2: Split out named structs for [IRJ]_TYPE Richard Henderson
2022-04-22 12:57   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 44/64] target/nios2: Split out helpers for gen_* translate macros Richard Henderson
2022-04-22 13:16   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 45/64] target/nios2: Introduce dest_gpr Richard Henderson
2022-04-21 15:17 ` [PATCH v7 46/64] target/nios2: Drop CR_STATUS_EH from tb->flags Richard Henderson
2022-04-21 15:17 ` [PATCH v7 47/64] target/nios2: Enable unaligned traps for system mode Richard Henderson
2022-04-21 15:17 ` [PATCH v7 48/64] target/nios2: Create gen_jumpr Richard Henderson
2022-04-21 15:17 ` [PATCH v7 49/64] target/nios2: Hoist set of is_jmp into gen_goto_tb Richard Henderson
2022-04-21 15:17 ` [PATCH v7 50/64] target/nios2: Use gen_goto_tb for DISAS_TOO_MANY Richard Henderson
2022-04-21 15:17 ` [PATCH v7 51/64] target/nios2: Use tcg_gen_lookup_and_goto_ptr Richard Henderson
2022-04-21 15:17 ` [PATCH v7 52/64] target/nios2: Implement Misaligned destination exception Richard Henderson
2022-04-21 15:17 ` [PATCH v7 53/64] target/nios2: Introduce shadow register sets Richard Henderson
2022-04-22 13:21   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 54/64] target/nios2: Implement rdprs, wrprs Richard Henderson
2022-04-21 15:17 ` [PATCH v7 55/64] target/nios2: Update helper_eret for shadow registers Richard Henderson
2022-04-21 15:17 ` Richard Henderson [this message]
2022-04-22 13:24   ` [PATCH v7 56/64] target/nios2: Implement EIC interrupt processing Peter Maydell
2022-04-21 15:17 ` [PATCH v7 57/64] target/nios2: Advance pc when raising exceptions Richard Henderson
2022-04-22 13:00   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 58/64] linux-user/nios2: Handle various SIGILL exceptions Richard Henderson
2022-04-21 15:17 ` [PATCH v7 59/64] hw/intc: Vectored Interrupt Controller (VIC) Richard Henderson
2022-04-21 15:17 ` [PATCH v7 60/64] hw/nios2: Introduce Nios2MachineState Richard Henderson
2022-04-21 15:17 ` [PATCH v7 61/64] hw/nios2: Move memory regions into Nios2Machine Richard Henderson
2022-04-21 15:17 ` [PATCH v7 62/64] hw/nios2: Machine with a Vectored Interrupt Controller Richard Henderson
2022-04-21 15:17 ` [PATCH v7 63/64] tests/tcg/nios2: Add semihosting multiarch tests Richard Henderson
2022-04-22 13:26   ` Peter Maydell
2022-04-21 15:17 ` [PATCH v7 64/64] tests/tcg/nios2: Add test-shadow-1 Richard Henderson
2022-04-22 13:26   ` Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220421151735.31996-57-richard.henderson@linaro.org \
    --to=richard.henderson@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).