All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anup Patel <anup@brainfault.org>
To: Peter Maydell <peter.maydell@linaro.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Alistair Francis <Alistair.Francis@wdc.com>,
	Sagar Karandikar <sagark@eecs.berkeley.edu>
Cc: qemu-riscv@nongnu.org, Frank Chang <frank.chang@sifive.com>,
	Anup Patel <anup@brainfault.org>,
	qemu-devel@nongnu.org,
	Alistair Francis <alistair.francis@wdc.com>,
	Atish Patra <atishp@atishpatra.org>,
	Bin Meng <bmeng.cn@gmail.com>
Subject: [PATCH v9 12/23] target/riscv: Implement AIA interrupt filtering CSRs
Date: Fri,  4 Feb 2022 23:16:48 +0530	[thread overview]
Message-ID: <20220204174700.534953-13-anup@brainfault.org> (raw)
In-Reply-To: <20220204174700.534953-1-anup@brainfault.org>

From: Anup Patel <anup.patel@wdc.com>

The AIA specificaiton adds interrupt filtering support for M-mode
and HS-mode. Using AIA interrupt filtering M-mode and H-mode can
take local interrupt 13 or above and selectively inject same local
interrupt to lower privilege modes.

At the moment, we don't have any local interrupts above 12 so we
add dummy implementation (i.e. read zero and ignore write) of AIA
interrupt filtering CSRs.

Signed-off-by: Anup Patel <anup.patel@wdc.com>
Signed-off-by: Anup Patel <anup@brainfault.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Frank Chang <frank.chang@sifive.com>
---
 target/riscv/csr.c | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/target/riscv/csr.c b/target/riscv/csr.c
index 552dae1ef8..00f55f47ed 100644
--- a/target/riscv/csr.c
+++ b/target/riscv/csr.c
@@ -158,6 +158,15 @@ static RISCVException any32(CPURISCVState *env, int csrno)
 
 }
 
+static int aia_any(CPURISCVState *env, int csrno)
+{
+    if (!riscv_feature(env, RISCV_FEATURE_AIA)) {
+        return RISCV_EXCP_ILLEGAL_INST;
+    }
+
+    return any(env, csrno);
+}
+
 static int aia_any32(CPURISCVState *env, int csrno)
 {
     if (!riscv_feature(env, RISCV_FEATURE_AIA)) {
@@ -568,6 +577,12 @@ static RISCVException read_zero(CPURISCVState *env, int csrno,
     return RISCV_EXCP_NONE;
 }
 
+static RISCVException write_ignore(CPURISCVState *env, int csrno,
+                                   target_ulong val)
+{
+    return RISCV_EXCP_NONE;
+}
+
 static RISCVException read_mhartid(CPURISCVState *env, int csrno,
                                    target_ulong *val)
 {
@@ -2598,9 +2613,15 @@ riscv_csr_operations csr_ops[CSR_TABLE_SIZE] = {
     [CSR_MTVAL]    = { "mtval",    any,  read_mtval,    write_mtval    },
     [CSR_MIP]      = { "mip",      any,  NULL,    NULL, rmw_mip        },
 
+    /* Virtual Interrupts for Supervisor Level (AIA) */
+    [CSR_MVIEN]      = { "mvien", aia_any, read_zero, write_ignore },
+    [CSR_MVIP]       = { "mvip",  aia_any, read_zero, write_ignore },
+
     /* Machine-Level High-Half CSRs (AIA) */
     [CSR_MIDELEGH] = { "midelegh", aia_any32, NULL, NULL, rmw_midelegh },
     [CSR_MIEH]     = { "mieh",     aia_any32, NULL, NULL, rmw_mieh     },
+    [CSR_MVIENH]   = { "mvienh",   aia_any32, read_zero,  write_ignore },
+    [CSR_MVIPH]    = { "mviph",    aia_any32, read_zero,  write_ignore },
     [CSR_MIPH]     = { "miph",     aia_any32, NULL, NULL, rmw_miph     },
 
     /* Supervisor Trap Setup */
@@ -2654,12 +2675,14 @@ riscv_csr_operations csr_ops[CSR_TABLE_SIZE] = {
     [CSR_MTINST]      = { "mtinst",      hmode,   read_mtinst,      write_mtinst      },
 
     /* Virtual Interrupts and Interrupt Priorities (H-extension with AIA) */
+    [CSR_HVIEN]       = { "hvien",       aia_hmode, read_zero, write_ignore },
     [CSR_HVICTL]      = { "hvictl",      aia_hmode, read_hvictl, write_hvictl },
     [CSR_HVIPRIO1]    = { "hviprio1",    aia_hmode, read_hviprio1,   write_hviprio1 },
     [CSR_HVIPRIO2]    = { "hviprio2",    aia_hmode, read_hviprio2,   write_hviprio2 },
 
     /* Hypervisor and VS-Level High-Half CSRs (H-extension with AIA) */
     [CSR_HIDELEGH]    = { "hidelegh",    aia_hmode32, NULL, NULL, rmw_hidelegh },
+    [CSR_HVIENH]      = { "hvienh",      aia_hmode32, read_zero, write_ignore },
     [CSR_HVIPH]       = { "hviph",       aia_hmode32, NULL, NULL, rmw_hviph },
     [CSR_HVIPRIO1H]   = { "hviprio1h",   aia_hmode32, read_hviprio1h, write_hviprio1h },
     [CSR_HVIPRIO2H]   = { "hviprio2h",   aia_hmode32, read_hviprio2h, write_hviprio2h },
-- 
2.25.1



WARNING: multiple messages have this Message-ID (diff)
From: Anup Patel <anup@brainfault.org>
To: Peter Maydell <peter.maydell@linaro.org>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Alistair Francis <Alistair.Francis@wdc.com>,
	Sagar Karandikar <sagark@eecs.berkeley.edu>
Cc: Atish Patra <atishp@atishpatra.org>,
	Anup Patel <anup@brainfault.org>, Bin Meng <bmeng.cn@gmail.com>,
	qemu-riscv@nongnu.org, qemu-devel@nongnu.org,
	Alistair Francis <alistair.francis@wdc.com>,
	Frank Chang <frank.chang@sifive.com>
Subject: [PATCH v9 12/23] target/riscv: Implement AIA interrupt filtering CSRs
Date: Fri,  4 Feb 2022 23:16:48 +0530	[thread overview]
Message-ID: <20220204174700.534953-13-anup@brainfault.org> (raw)
In-Reply-To: <20220204174700.534953-1-anup@brainfault.org>

From: Anup Patel <anup.patel@wdc.com>

The AIA specificaiton adds interrupt filtering support for M-mode
and HS-mode. Using AIA interrupt filtering M-mode and H-mode can
take local interrupt 13 or above and selectively inject same local
interrupt to lower privilege modes.

At the moment, we don't have any local interrupts above 12 so we
add dummy implementation (i.e. read zero and ignore write) of AIA
interrupt filtering CSRs.

Signed-off-by: Anup Patel <anup.patel@wdc.com>
Signed-off-by: Anup Patel <anup@brainfault.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Frank Chang <frank.chang@sifive.com>
---
 target/riscv/csr.c | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/target/riscv/csr.c b/target/riscv/csr.c
index 552dae1ef8..00f55f47ed 100644
--- a/target/riscv/csr.c
+++ b/target/riscv/csr.c
@@ -158,6 +158,15 @@ static RISCVException any32(CPURISCVState *env, int csrno)
 
 }
 
+static int aia_any(CPURISCVState *env, int csrno)
+{
+    if (!riscv_feature(env, RISCV_FEATURE_AIA)) {
+        return RISCV_EXCP_ILLEGAL_INST;
+    }
+
+    return any(env, csrno);
+}
+
 static int aia_any32(CPURISCVState *env, int csrno)
 {
     if (!riscv_feature(env, RISCV_FEATURE_AIA)) {
@@ -568,6 +577,12 @@ static RISCVException read_zero(CPURISCVState *env, int csrno,
     return RISCV_EXCP_NONE;
 }
 
+static RISCVException write_ignore(CPURISCVState *env, int csrno,
+                                   target_ulong val)
+{
+    return RISCV_EXCP_NONE;
+}
+
 static RISCVException read_mhartid(CPURISCVState *env, int csrno,
                                    target_ulong *val)
 {
@@ -2598,9 +2613,15 @@ riscv_csr_operations csr_ops[CSR_TABLE_SIZE] = {
     [CSR_MTVAL]    = { "mtval",    any,  read_mtval,    write_mtval    },
     [CSR_MIP]      = { "mip",      any,  NULL,    NULL, rmw_mip        },
 
+    /* Virtual Interrupts for Supervisor Level (AIA) */
+    [CSR_MVIEN]      = { "mvien", aia_any, read_zero, write_ignore },
+    [CSR_MVIP]       = { "mvip",  aia_any, read_zero, write_ignore },
+
     /* Machine-Level High-Half CSRs (AIA) */
     [CSR_MIDELEGH] = { "midelegh", aia_any32, NULL, NULL, rmw_midelegh },
     [CSR_MIEH]     = { "mieh",     aia_any32, NULL, NULL, rmw_mieh     },
+    [CSR_MVIENH]   = { "mvienh",   aia_any32, read_zero,  write_ignore },
+    [CSR_MVIPH]    = { "mviph",    aia_any32, read_zero,  write_ignore },
     [CSR_MIPH]     = { "miph",     aia_any32, NULL, NULL, rmw_miph     },
 
     /* Supervisor Trap Setup */
@@ -2654,12 +2675,14 @@ riscv_csr_operations csr_ops[CSR_TABLE_SIZE] = {
     [CSR_MTINST]      = { "mtinst",      hmode,   read_mtinst,      write_mtinst      },
 
     /* Virtual Interrupts and Interrupt Priorities (H-extension with AIA) */
+    [CSR_HVIEN]       = { "hvien",       aia_hmode, read_zero, write_ignore },
     [CSR_HVICTL]      = { "hvictl",      aia_hmode, read_hvictl, write_hvictl },
     [CSR_HVIPRIO1]    = { "hviprio1",    aia_hmode, read_hviprio1,   write_hviprio1 },
     [CSR_HVIPRIO2]    = { "hviprio2",    aia_hmode, read_hviprio2,   write_hviprio2 },
 
     /* Hypervisor and VS-Level High-Half CSRs (H-extension with AIA) */
     [CSR_HIDELEGH]    = { "hidelegh",    aia_hmode32, NULL, NULL, rmw_hidelegh },
+    [CSR_HVIENH]      = { "hvienh",      aia_hmode32, read_zero, write_ignore },
     [CSR_HVIPH]       = { "hviph",       aia_hmode32, NULL, NULL, rmw_hviph },
     [CSR_HVIPRIO1H]   = { "hviprio1h",   aia_hmode32, read_hviprio1h, write_hviprio1h },
     [CSR_HVIPRIO2H]   = { "hviprio2h",   aia_hmode32, read_hviprio2h, write_hviprio2h },
-- 
2.25.1



  parent reply	other threads:[~2022-02-04 18:21 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-04 17:46 [PATCH v9 00/23] QEMU RISC-V AIA support Anup Patel
2022-02-04 17:46 ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 01/23] target/riscv: Fix trap cause for RV32 HS-mode CSR access from RV64 HS-mode Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 02/23] target/riscv: Implement SGEIP bit in hip and hie CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 03/23] target/riscv: Implement hgeie and hgeip CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 04/23] target/riscv: Improve delivery of guest external interrupts Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 05/23] target/riscv: Allow setting CPU feature from machine/device emulation Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 06/23] target/riscv: Add AIA cpu feature Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 07/23] target/riscv: Add defines for AIA CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 08/23] target/riscv: Allow AIA device emulation to set ireg rmw callback Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 09/23] target/riscv: Implement AIA local interrupt priorities Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 10/23] target/riscv: Implement AIA CSRs for 64 local interrupts on RV32 Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 11/23] target/riscv: Implement AIA hvictl and hviprioX CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` Anup Patel [this message]
2022-02-04 17:46   ` [PATCH v9 12/23] target/riscv: Implement AIA interrupt filtering CSRs Anup Patel
2022-02-04 17:46 ` [PATCH v9 13/23] target/riscv: Implement AIA mtopi, stopi, and vstopi CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 14/23] target/riscv: Implement AIA xiselect and xireg CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 15/23] target/riscv: Implement AIA IMSIC interface CSRs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 16/23] hw/riscv: virt: Use AIA INTC compatible string when available Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 17/23] target/riscv: Allow users to force enable AIA CSRs in HART Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 18/23] hw/intc: Add RISC-V AIA APLIC device emulation Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 19/23] hw/riscv: virt: Add optional AIA APLIC support to virt machine Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 20/23] hw/intc: Add RISC-V AIA IMSIC device emulation Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 21/23] hw/riscv: virt: Add optional AIA IMSIC support to virt machine Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 22/23] docs/system: riscv: Document AIA options for " Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-04 17:46 ` [PATCH v9 23/23] hw/riscv: virt: Increase maximum number of allowed CPUs Anup Patel
2022-02-04 17:46   ` Anup Patel
2022-02-08  4:16 ` [PATCH v9 00/23] QEMU RISC-V AIA support Alistair Francis
2022-02-08  4:16   ` Alistair Francis
2022-02-08  6:51   ` Alistair Francis
2022-02-08  6:51     ` Alistair Francis
2022-02-08  7:32     ` Anup Patel
2022-02-08  7:32       ` Anup Patel
2022-02-10  8:28     ` Atish Patra
2022-02-10  8:28       ` Atish Patra
2022-02-10 10:24       ` Anup Patel
2022-02-10 10:24         ` Anup Patel
2022-02-11  8:28         ` Alistair Francis
2022-02-11  8:28           ` Alistair Francis

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=20220204174700.534953-13-anup@brainfault.org \
    --to=anup@brainfault.org \
    --cc=Alistair.Francis@wdc.com \
    --cc=atishp@atishpatra.org \
    --cc=bmeng.cn@gmail.com \
    --cc=frank.chang@sifive.com \
    --cc=palmer@dabbelt.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=sagark@eecs.berkeley.edu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.