From: Raymond Mao <raymondmaoca@gmail.com>
To: opensbi@lists.infradead.org
Cc: anup@brainfault.org, anup.patel@oss.qualcomm.com,
scott@riscstar.com, raymond.mao@riscstar.com,
robin.randhawa@sifive.com, samuel.holland@sifive.com,
peter.lin@sifive.com
Subject: [PATCH v2 01/12] lib: utils: add RPMI SYSTEM_IRQ service support
Date: Thu, 29 Oct 2026 11:59:18 -0400 [thread overview]
Message-ID: <20261029155929.586597-2-raymondmaoca@gmail.com> (raw)
In-Reply-To: <20261029155929.586597-1-raymondmaoca@gmail.com>
From: Raymond Mao <raymond.mao@riscstar.com>
Provide an SBI MPXY implementation of the RPMI SYSTEM_IRQ service
group, version 2.0, with GET_ATTRIBUTES, GET_TARGET_ATTRIBUTES,
GET_IRQ_ATTRIBUTES and SYSIRQ_CLAIM. Other services return NOTSUPP.
Expose backend callbacks for claiming and completing IRQs, allowing
delivery implementations to register domain-owned channels. Target
indices enumerate possible harts and CLAIM is restricted to the
calling domain and hart.
Keep claimed sources outstanding until the next valid CLAIM, after the
payload has handled the previous batch. Preserve failed completion
records for retry. This completion convention introduces no new RPMI
service and leaves backend device handling outside the protocol layer.
Signed-off-by: Raymond Mao <raymond.mao@riscstar.com>
---
include/sbi_utils/mailbox/rpmi_msgprot.h | 25 +++
include/sbi_utils/sysirq/rpmi_sysirq.h | 30 +++
lib/utils/mpxy/objects.mk | 2 +
lib/utils/sysirq/rpmi_sysirq.c | 266 +++++++++++++++++++++++
4 files changed, 323 insertions(+)
create mode 100644 include/sbi_utils/sysirq/rpmi_sysirq.h
create mode 100644 lib/utils/sysirq/rpmi_sysirq.c
diff --git a/include/sbi_utils/mailbox/rpmi_msgprot.h b/include/sbi_utils/mailbox/rpmi_msgprot.h
index 7ae4abe0..f9d1d177 100644
--- a/include/sbi_utils/mailbox/rpmi_msgprot.h
+++ b/include/sbi_utils/mailbox/rpmi_msgprot.h
@@ -226,6 +226,7 @@ enum rpmi_servicegroup_id {
RPMI_SRVGRP_PERFORMANCE = 0x0000A,
RPMI_SRVGRP_MANAGEMENT_MODE = 0x000B,
RPMI_SRVGRP_LOGGING = 0x000E,
+ RPMI_SRVGRP_SYSTEM_IRQ = 0x000F,
RPMI_SRVGRP_ID_MAX_COUNT,
/* Reserved range for service groups */
@@ -290,6 +291,30 @@ struct rpmi_base_get_platform_info_resp {
char plat_info[];
};
+/** RPMI System IRQ ServiceGroup Service IDs */
+enum rpmi_sysirq_service_id {
+ RPMI_SYSIRQ_SRV_ENABLE_NOTIFICATION = 0x01,
+ RPMI_SYSIRQ_SRV_GET_ATTRIBUTES = 0x02,
+ RPMI_SYSIRQ_SRV_GET_TARGET_ATTRIBUTES = 0x03,
+ RPMI_SYSIRQ_SRV_GET_IRQ_ATTRIBUTES = 0x04,
+ RPMI_SYSIRQ_SRV_SET_IRQ_STATE = 0x05,
+ RPMI_SYSIRQ_SRV_GET_IRQ_STATE = 0x06,
+ RPMI_SYSIRQ_SRV_SET_IRQ_TARGET = 0x07,
+ RPMI_SYSIRQ_SRV_GET_IRQ_TARGET = 0x08,
+ RPMI_SYSIRQ_SRV_CLAIM = 0x09,
+ RPMI_SYSIRQ_SRV_ID_MAX_COUNT,
+};
+
+struct rpmi_sysirq_claim_req {
+ u32 sys_target_index;
+};
+
+struct rpmi_sysirq_claim_resp {
+ s32 status;
+ u32 returned;
+ u32 sys_irq_index[];
+};
+
/** RPMI System MSI ServiceGroup Service IDs */
enum rpmi_sysmsi_service_id {
RPMI_SYSMSI_SRV_ENABLE_NOTIFICATION = 0x01,
diff --git a/include/sbi_utils/sysirq/rpmi_sysirq.h b/include/sbi_utils/sysirq/rpmi_sysirq.h
new file mode 100644
index 00000000..0af2f33b
--- /dev/null
+++ b/include/sbi_utils/sysirq/rpmi_sysirq.h
@@ -0,0 +1,30 @@
+/* SPDX-License-Identifier: BSD-2-Clause */
+/*
+ * Copyright (c) 2026 RISCstar Solutions Corporation.
+ *
+ * Author: Raymond Mao <raymond.mao@riscstar.com>
+ */
+
+#ifndef __RPMI_SYSIRQ_H__
+#define __RPMI_SYSIRQ_H__
+
+#include <sbi/sbi_types.h>
+
+struct sbi_domain;
+
+/* Callbacks operate on the calling domain and hart; ops must remain valid. */
+struct rpmi_sysirq_ops {
+ int (*claim)(u32 channel_id, u32 *irqs, u32 max_count, u32 *count);
+ int (*complete)(u32 channel_id, u32 irq);
+ void (*drained)(void);
+};
+
+int rpmi_sysirq_register_channel(struct sbi_domain *dom, u32 channel_id,
+ u32 num_irqs, const struct rpmi_sysirq_ops *ops);
+
+bool rpmi_sysirq_available(void);
+
+/* Complete this domain/hart's batches, then ask backends to consider return. */
+int rpmi_sysirq_complete_thishart(u32 *completed);
+
+#endif
diff --git a/lib/utils/mpxy/objects.mk b/lib/utils/mpxy/objects.mk
index 1f1bf0a0..a59a7175 100644
--- a/lib/utils/mpxy/objects.mk
+++ b/lib/utils/mpxy/objects.mk
@@ -32,3 +32,5 @@ libsbiutils-objs-$(CONFIG_FDT_MPXY_RPMI_MM) += mpxy/fdt_mpxy_rpmi_mm.o
carray-fdt_mpxy_drivers-$(CONFIG_FDT_MPXY_RPMI_LOGGING) += fdt_mpxy_rpmi_logging
libsbiutils-objs-$(CONFIG_FDT_MPXY_RPMI_LOGGING) += mpxy/fdt_mpxy_rpmi_logging.o
+
+libsbiutils-objs-y += sysirq/rpmi_sysirq.o
diff --git a/lib/utils/sysirq/rpmi_sysirq.c b/lib/utils/sysirq/rpmi_sysirq.c
new file mode 100644
index 00000000..8fa37887
--- /dev/null
+++ b/lib/utils/sysirq/rpmi_sysirq.c
@@ -0,0 +1,266 @@
+// SPDX-License-Identifier: BSD-2-Clause
+/*
+ * Copyright (c) 2026 RISCstar Solutions Corporation.
+ *
+ * Author: Raymond Mao <raymond.mao@riscstar.com>
+ */
+
+#include <sbi/sbi_byteorder.h>
+#include <sbi/sbi_console.h>
+#include <sbi/sbi_domain.h>
+#include <sbi/sbi_error.h>
+#include <sbi/sbi_heap.h>
+#include <sbi/sbi_mpxy.h>
+#include <sbi/sbi_scratch.h>
+#include <sbi/sbi_string.h>
+#include <sbi_utils/mailbox/rpmi_msgprot.h>
+#include <sbi_utils/sysirq/rpmi_sysirq.h>
+
+#define SYSIRQ_BATCH_MAX 63U
+#define SYSIRQ_MSG_MAX_LEN ((SYSIRQ_BATCH_MAX + 2) * sizeof(u32))
+#define SYSIRQ_ATTR_RESPONSE_LEN (8 * sizeof(u32))
+
+struct rpmi_sysirq_target {
+ u32 count;
+ u32 claimed[SYSIRQ_BATCH_MAX];
+};
+
+struct rpmi_sysirq_channel {
+ struct sbi_dlist node;
+ struct sbi_mpxy_channel channel;
+ const struct rpmi_sysirq_ops *ops;
+ u32 num_irqs;
+ struct rpmi_sysirq_target targets[];
+};
+
+static SBI_LIST_HEAD(sysirq_channels);
+
+static int sysirq_complete_batch(struct rpmi_sysirq_channel *sc,
+ struct rpmi_sysirq_target *target,
+ u32 *completed)
+{
+ u32 irq;
+ int rc;
+
+ while (target->count) {
+ irq = target->claimed[target->count - 1];
+ rc = sc->ops->complete(sc->channel.channel_id, irq);
+ if (rc)
+ return rc;
+ target->count--;
+ (*completed)++;
+ }
+ return SBI_OK;
+}
+
+bool rpmi_sysirq_available(void)
+{
+ return !sbi_list_empty(&sysirq_channels);
+}
+
+int rpmi_sysirq_complete_thishart(u32 *completed)
+{
+ struct sbi_domain *dom = sbi_domain_thishart_ptr();
+ struct rpmi_sysirq_channel *sc;
+ u32 index = 0, hidx;
+ int rc;
+
+ if (!dom || !completed)
+ return SBI_EINVAL;
+ *completed = 0;
+ sbi_hartmask_for_each_hartindex(hidx, dom->possible_harts) {
+ if (hidx == current_hartindex())
+ break;
+ index++;
+ }
+ if (index == sbi_hartmask_weight(dom->possible_harts))
+ return SBI_EINVAL;
+
+ sbi_list_for_each_entry(sc, &sysirq_channels, node) {
+ if (sc->channel.owner_domain != dom)
+ continue;
+ rc = sysirq_complete_batch(sc, &sc->targets[index], completed);
+ if (rc)
+ return rc;
+ }
+
+ /* Only consider return once every owned channel completed successfully. */
+ sbi_list_for_each_entry(sc, &sysirq_channels, node) {
+ if (sc->channel.owner_domain == dom && sc->ops->drained)
+ sc->ops->drained();
+ }
+ return SBI_OK;
+}
+
+static int sysirq_read_attributes(struct sbi_mpxy_channel *channel,
+ u32 *out, u32 base, u32 count)
+{
+ u32 attrs[] = { RPMI_SRVGRP_SYSTEM_IRQ, RPMI_VERSION(2, 0), 0, 0 };
+ u32 index, i;
+
+ if (base < SBI_MPXY_ATTR_MSGPROTO_ATTR_START)
+ return SBI_EBAD_RANGE;
+ index = base - SBI_MPXY_ATTR_MSGPROTO_ATTR_START;
+ if (index >= array_size(attrs) || count > array_size(attrs) - index)
+ return SBI_EBAD_RANGE;
+
+ for (i = 0; i < count; i++)
+ out[i] = cpu_to_le32(attrs[index + i]);
+
+ return SBI_OK;
+}
+
+static u32 sysirq_target_hartindex(struct sbi_domain *dom, u32 target)
+{
+ u32 index = 0, hidx;
+
+ sbi_hartmask_for_each_hartindex(hidx, dom->possible_harts) {
+ if (index++ == target)
+ return hidx;
+ }
+
+ return -1U;
+}
+
+static int sysirq_send_message(struct sbi_mpxy_channel *channel,
+ u32 msg_id, void *msg, u32 msg_len,
+ void *response, u32 response_max_len,
+ unsigned long *response_len)
+{
+ struct rpmi_sysirq_channel *sc =
+ container_of(channel, struct rpmi_sysirq_channel, channel);
+ struct sbi_domain *dom = channel->owner_domain;
+ struct rpmi_sysirq_target *target;
+ u32 *out = response;
+ u32 index = 0, hartindex, count, max_count, i, completed = 0;
+ s32 status = RPMI_SUCCESS;
+ int rc;
+
+ if (!response || !response_len || response_max_len < sizeof(u32))
+ return SBI_EINVAL;
+ *response_len = sizeof(u32);
+ if (sbi_domain_thishart_ptr() != dom) {
+ status = RPMI_ERR_DENIED;
+ goto done;
+ }
+
+ switch (msg_id) {
+ case RPMI_SYSIRQ_SRV_GET_ATTRIBUTES:
+ if (msg_len) {
+ status = RPMI_ERR_INVALID_PARAM;
+ break;
+ }
+ if (response_max_len < SYSIRQ_ATTR_RESPONSE_LEN)
+ return SBI_EINVAL;
+ sbi_memset(out, 0, SYSIRQ_ATTR_RESPONSE_LEN);
+ out[1] = cpu_to_le32(sbi_hartmask_weight(dom->possible_harts));
+ out[2] = cpu_to_le32(sc->num_irqs);
+ *response_len = SYSIRQ_ATTR_RESPONSE_LEN;
+ break;
+ case RPMI_SYSIRQ_SRV_GET_TARGET_ATTRIBUTES:
+ case RPMI_SYSIRQ_SRV_GET_IRQ_ATTRIBUTES:
+ case RPMI_SYSIRQ_SRV_CLAIM:
+ if (!msg || msg_len != sizeof(u32)) {
+ status = RPMI_ERR_INVALID_PARAM;
+ break;
+ }
+ /* MPXY uses the same shared buffer for request and response. */
+ index = le32_to_cpu(*(u32 *)msg);
+ if (msg_id == RPMI_SYSIRQ_SRV_GET_IRQ_ATTRIBUTES) {
+ if (index >= sc->num_irqs) {
+ status = RPMI_ERR_INVALID_PARAM;
+ break;
+ }
+ if (response_max_len < SYSIRQ_ATTR_RESPONSE_LEN)
+ return SBI_EINVAL;
+ sbi_memset(out, 0, SYSIRQ_ATTR_RESPONSE_LEN);
+ sbi_snprintf((char *)&out[3], 20, "virq%u", index);
+ *response_len = SYSIRQ_ATTR_RESPONSE_LEN;
+ break;
+ }
+ hartindex = sysirq_target_hartindex(dom, index);
+ if (!sbi_hartindex_valid(hartindex)) {
+ status = RPMI_ERR_INVALID_PARAM;
+ break;
+ }
+ if (msg_id == RPMI_SYSIRQ_SRV_GET_TARGET_ATTRIBUTES) {
+ if (response_max_len < SYSIRQ_ATTR_RESPONSE_LEN)
+ return SBI_EINVAL;
+ sbi_memset(out, 0, SYSIRQ_ATTR_RESPONSE_LEN);
+ out[2] = cpu_to_le32(sbi_hartindex_to_hartid(hartindex));
+ sbi_snprintf((char *)&out[3], 20, "hart%u",
+ sbi_hartindex_to_hartid(hartindex));
+ *response_len = SYSIRQ_ATTR_RESPONSE_LEN;
+ break;
+ }
+ if (hartindex != current_hartindex()) {
+ status = RPMI_ERR_DENIED;
+ break;
+ }
+ if (response_max_len > channel->attrs.msg_data_maxlen)
+ response_max_len = channel->attrs.msg_data_maxlen;
+ if (response_max_len < 3 * sizeof(u32))
+ return SBI_EINVAL;
+ max_count = response_max_len / sizeof(u32) - 2;
+ target = &sc->targets[index];
+ if (max_count > array_size(target->claimed))
+ max_count = array_size(target->claimed);
+ /* A new CLAIM follows device handling of the previous batch. */
+ rc = sysirq_complete_batch(sc, target, &completed);
+ if (rc) {
+ status = RPMI_ERR_FAILED;
+ goto done;
+ }
+ rc = sc->ops->claim(channel->channel_id, target->claimed,
+ max_count, &count);
+ if (rc) {
+ status = RPMI_ERR_FAILED;
+ break;
+ }
+ target->count = count;
+ out[1] = cpu_to_le32(count);
+ for (i = 0; i < count; i++)
+ out[i + 2] = cpu_to_le32(target->claimed[i]);
+ *response_len = (count + 2) * sizeof(u32);
+ break;
+ default:
+ status = RPMI_ERR_NOTSUPP;
+ break;
+ }
+
+done:
+ out[0] = cpu_to_le32((u32)status);
+ return SBI_OK;
+}
+
+int rpmi_sysirq_register_channel(struct sbi_domain *dom, u32 channel_id,
+ u32 num_irqs, const struct rpmi_sysirq_ops *ops)
+{
+ struct rpmi_sysirq_channel *sc;
+ int rc;
+
+ if (!dom || !dom->possible_harts || !num_irqs || !ops ||
+ !ops->claim || !ops->complete)
+ return SBI_EINVAL;
+
+ sc = sbi_zalloc(sizeof(*sc) +
+ sbi_hartmask_weight(dom->possible_harts) * sizeof(sc->targets[0]));
+ if (!sc)
+ return SBI_ENOMEM;
+ sc->ops = ops;
+ sc->num_irqs = num_irqs;
+ sc->channel.channel_id = channel_id;
+ sc->channel.owner_domain = dom;
+ sc->channel.attrs.msg_proto_id = SBI_MPXY_MSGPROTO_RPMI_ID;
+ sc->channel.attrs.msg_proto_version = RPMI_VERSION(2, 0);
+ sc->channel.attrs.msg_data_maxlen = SYSIRQ_MSG_MAX_LEN;
+ sc->channel.read_attributes = sysirq_read_attributes;
+ sc->channel.send_message_with_response = sysirq_send_message;
+ rc = sbi_mpxy_register_channel(&sc->channel);
+ if (rc)
+ sbi_free(sc);
+ else
+ sbi_list_add_tail(&sc->node, &sysirq_channels);
+
+ return rc;
+}
--
2.25.1
--
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi
next prev parent reply other threads:[~2026-10-29 15:59 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-29 15:59 [PATCH v2 00/12] Introduce Virtual IRQ (VIRQ) framework Raymond Mao
2026-10-29 15:59 ` Raymond Mao [this message]
2026-10-29 15:59 ` [PATCH v2 02/12] lib: sbi: add experimental Yield ECALL extension Raymond Mao
2026-10-29 15:59 ` [PATCH v2 03/12] lib: irqchip: add S-mode notification helpers Raymond Mao
2026-10-29 15:59 ` [PATCH v2 04/12] lib: sbi: domain: support VIRQ couriering domain context switch Raymond Mao
2026-10-29 15:59 ` [PATCH v2 05/12] lib: sbi: domain: ensure boot_hartid is assigned Raymond Mao
2026-10-29 15:59 ` [PATCH v2 06/12] lib: sbi: Add Virtual IRQ (VIRQ) subsystem Raymond Mao
2026-10-29 15:59 ` [PATCH v2 07/12] lib: sbi: domain: add domain lookup by name Raymond Mao
2026-10-29 15:59 ` [PATCH v2 08/12] lib: utils: irqchip: derive APLIC targets from sysirq nodes Raymond Mao
2026-10-29 15:59 ` [PATCH v2 09/12] lib: utils: irqchip: Add helper to register VIRQ courier handler Raymond Mao
2026-10-29 15:59 ` [PATCH v2 10/12] lib: utils: fdt: parse sysirq routing from DT Raymond Mao
2026-10-29 15:59 ` [PATCH v2 11/12] docs: domain: document sysirq VIRQ mapping and routing rules Raymond Mao
2026-10-29 15:59 ` [PATCH v2 12/12] [NOT-FOR-UPSTREAM] platform: generic: Add sample dts overlays for testing hwirq/domain route rules Raymond Mao
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=20261029155929.586597-2-raymondmaoca@gmail.com \
--to=raymondmaoca@gmail.com \
--cc=anup.patel@oss.qualcomm.com \
--cc=anup@brainfault.org \
--cc=opensbi@lists.infradead.org \
--cc=peter.lin@sifive.com \
--cc=raymond.mao@riscstar.com \
--cc=robin.randhawa@sifive.com \
--cc=samuel.holland@sifive.com \
--cc=scott@riscstar.com \
/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