* [PATCH] ppc/pnv: Add initial P9/10 SBE model
@ 2022-08-11 7:54 Nicholas Piggin
2022-08-11 8:04 ` Cédric Le Goater
0 siblings, 1 reply; 4+ messages in thread
From: Nicholas Piggin @ 2022-08-11 7:54 UTC (permalink / raw)
To: qemu-ppc; +Cc: Nicholas Piggin, qemu-devel, Cédric Le Goater
The SBE (Self Boot Engine) are on-chip microcontrollers that perform
early boot steps, as well as provide some runtime facilities (e.g.,
timer, secure register access, MPIPL). The latter facilities are
accessed mostly via a message system called SBEFIFO.
This driver provides initial emulation for the SBE runtime registers
and a very basic SBEFIFO implementation that provides the timer
command. This covers the basic SBE behaviour expected by skiboot when
booting.
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
---
hw/ppc/meson.build | 1 +
hw/ppc/pnv.c | 25 +++
hw/ppc/pnv_sbe.c | 429 +++++++++++++++++++++++++++++++++++++
hw/ppc/pnv_xscom.c | 3 +
include/hw/ppc/pnv.h | 3 +
include/hw/ppc/pnv_sbe.h | 55 +++++
include/hw/ppc/pnv_xscom.h | 12 ++
7 files changed, 528 insertions(+)
create mode 100644 hw/ppc/pnv_sbe.c
create mode 100644 include/hw/ppc/pnv_sbe.h
diff --git a/hw/ppc/meson.build b/hw/ppc/meson.build
index aa4c8e6a2e..62801923f3 100644
--- a/hw/ppc/meson.build
+++ b/hw/ppc/meson.build
@@ -46,6 +46,7 @@ ppc_ss.add(when: 'CONFIG_POWERNV', if_true: files(
'pnv_lpc.c',
'pnv_psi.c',
'pnv_occ.c',
+ 'pnv_sbe.c',
'pnv_bmc.c',
'pnv_homer.c',
'pnv_pnor.c',
diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
index d3f77c8367..7ff1f464d3 100644
--- a/hw/ppc/pnv.c
+++ b/hw/ppc/pnv.c
@@ -1397,6 +1397,8 @@ static void pnv_chip_power9_instance_init(Object *obj)
object_initialize_child(obj, "occ", &chip9->occ, TYPE_PNV9_OCC);
+ object_initialize_child(obj, "sbe", &chip9->sbe, TYPE_PNV9_SBE);
+
object_initialize_child(obj, "homer", &chip9->homer, TYPE_PNV9_HOMER);
/* Number of PECs is the chip default */
@@ -1549,6 +1551,17 @@ static void pnv_chip_power9_realize(DeviceState *dev, Error **errp)
memory_region_add_subregion(get_system_memory(), PNV9_OCC_SENSOR_BASE(chip),
&chip9->occ.sram_regs);
+ /* SBE */
+ if (!qdev_realize(DEVICE(&chip9->sbe), NULL, errp)) {
+ return;
+ }
+ pnv_xscom_add_subregion(chip, PNV9_XSCOM_SBE_CTRL_BASE,
+ &chip9->sbe.xscom_ctrl_regs);
+ pnv_xscom_add_subregion(chip, PNV9_XSCOM_SBE_MBOX_BASE,
+ &chip9->sbe.xscom_mbox_regs);
+ qdev_connect_gpio_out(DEVICE(&chip9->sbe), 0, qdev_get_gpio_in(
+ DEVICE(&chip9->psi), PSIHB9_IRQ_PSU));
+
/* HOMER */
object_property_set_link(OBJECT(&chip9->homer), "chip", OBJECT(chip),
&error_abort);
@@ -1613,6 +1626,7 @@ static void pnv_chip_power10_instance_init(Object *obj)
object_initialize_child(obj, "psi", &chip10->psi, TYPE_PNV10_PSI);
object_initialize_child(obj, "lpc", &chip10->lpc, TYPE_PNV10_LPC);
object_initialize_child(obj, "occ", &chip10->occ, TYPE_PNV10_OCC);
+ object_initialize_child(obj, "sbe", &chip10->sbe, TYPE_PNV10_SBE);
object_initialize_child(obj, "homer", &chip10->homer, TYPE_PNV10_HOMER);
chip->num_pecs = pcc->num_pecs;
@@ -1754,6 +1768,17 @@ static void pnv_chip_power10_realize(DeviceState *dev, Error **errp)
PNV10_OCC_SENSOR_BASE(chip),
&chip10->occ.sram_regs);
+ /* SBE */
+ if (!qdev_realize(DEVICE(&chip10->sbe), NULL, errp)) {
+ return;
+ }
+ pnv_xscom_add_subregion(chip, PNV10_XSCOM_SBE_CTRL_BASE,
+ &chip10->sbe.xscom_ctrl_regs);
+ pnv_xscom_add_subregion(chip, PNV10_XSCOM_SBE_MBOX_BASE,
+ &chip10->sbe.xscom_mbox_regs);
+ qdev_connect_gpio_out(DEVICE(&chip10->sbe), 0, qdev_get_gpio_in(
+ DEVICE(&chip10->psi), PSIHB9_IRQ_PSU));
+
/* HOMER */
object_property_set_link(OBJECT(&chip10->homer), "chip", OBJECT(chip),
&error_abort);
diff --git a/hw/ppc/pnv_sbe.c b/hw/ppc/pnv_sbe.c
new file mode 100644
index 0000000000..559502b7a6
--- /dev/null
+++ b/hw/ppc/pnv_sbe.c
@@ -0,0 +1,429 @@
+/*
+ * QEMU PowerPC PowerNV Emulation of some SBE behaviour
+ *
+ * Copyright (c) 2022, IBM Corporation.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License, version 2, as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include "qemu/osdep.h"
+#include "target/ppc/cpu.h"
+#include "qapi/error.h"
+#include "qemu/log.h"
+#include "qemu/module.h"
+#include "hw/irq.h"
+#include "hw/qdev-properties.h"
+#include "hw/ppc/pnv.h"
+#include "hw/ppc/pnv_xscom.h"
+#include "hw/ppc/pnv_sbe.h"
+
+#define DEBUG_SBE_REG 0
+#define DEBUG_SBE_MSG 0
+#define DEBUG_SBE_CMD 0
+
+#define sbe_reg_dbg(fmt, ...) do { \
+ if (DEBUG_SBE_REG) { \
+ fprintf(stderr, "SBE reg: %s: " fmt, __func__, ## __VA_ARGS__); \
+ } \
+} while (0)
+
+#define sbe_msg_dbg(fmt, ...) do { \
+ if (DEBUG_SBE_MSG) { \
+ fprintf(stderr, "SBE msg: %s: " fmt, __func__, ## __VA_ARGS__); \
+ } \
+} while (0)
+
+#define sbe_cmd_dbg(fmt, ...) do { \
+ if (DEBUG_SBE_CMD) { \
+ fprintf(stderr, "SBE cmd: %s: " fmt, __func__, ## __VA_ARGS__); \
+ } \
+} while (0)
+
+
+/* Most register and command definitions come from skiboot */
+
+/*
+ * SBE MBOX register address
+ * Reg 0 - 3 : Host to send command packets to SBE
+ * Reg 4 - 7 : SBE to send response packets to Host
+ */
+#define PSU_HOST_SBE_MBOX_REG0 0x00000000
+#define PSU_HOST_SBE_MBOX_REG1 0x00000001
+#define PSU_HOST_SBE_MBOX_REG2 0x00000002
+#define PSU_HOST_SBE_MBOX_REG3 0x00000003
+#define PSU_HOST_SBE_MBOX_REG4 0x00000004
+#define PSU_HOST_SBE_MBOX_REG5 0x00000005
+#define PSU_HOST_SBE_MBOX_REG6 0x00000006
+#define PSU_HOST_SBE_MBOX_REG7 0x00000007
+#define PSU_SBE_DOORBELL_REG_RW 0x00000010
+#define PSU_SBE_DOORBELL_REG_AND 0x00000011
+#define PSU_SBE_DOORBELL_REG_OR 0x00000012
+#define PSU_HOST_DOORBELL_REG_RW 0x00000013
+#define PSU_HOST_DOORBELL_REG_AND 0x00000014
+#define PSU_HOST_DOORBELL_REG_OR 0x00000015
+
+/*
+ * Doorbell register to trigger SBE interrupt. Set by OPAL to inform
+ * the SBE about a waiting message in the Host/SBE mailbox registers
+ */
+#define HOST_SBE_MSG_WAITING PPC_BIT(0)
+
+/*
+ * Doorbell register for host bridge interrupt. Set by the SBE to inform
+ * host about a response message in the Host/SBE mailbox registers
+ */
+#define SBE_HOST_RESPONSE_WAITING PPC_BIT(0)
+#define SBE_HOST_MSG_READ PPC_BIT(1)
+#define SBE_HOST_STOP15_EXIT PPC_BIT(2)
+#define SBE_HOST_RESET PPC_BIT(3)
+#define SBE_HOST_PASSTHROUGH PPC_BIT(4)
+#define SBE_HOST_TIMER_EXPIRY PPC_BIT(14)
+#define SBE_HOST_RESPONSE_MASK (PPC_BITMASK(0, 4) | SBE_HOST_TIMER_EXPIRY)
+
+/* SBE Control Register */
+#define SBE_CONTROL_REG_RW 0x00000000
+
+/* SBE interrupt s0/s1 bits */
+#define SBE_CONTROL_REG_S0 PPC_BIT(14)
+#define SBE_CONTROL_REG_S1 PPC_BIT(15)
+
+struct sbe_msg {
+ uint64_t reg[4];
+};
+
+static uint64_t pnv_sbe_power9_xscom_ctrl_read(void *opaque, hwaddr addr,
+ unsigned size)
+{
+ uint32_t offset = addr >> 3;
+ uint64_t val = 0;
+
+ switch (offset) {
+ default:
+ qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
+ HWADDR_PRIx "\n", addr >> 3);
+ }
+
+ sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
+
+ return val;
+}
+
+static void pnv_sbe_power9_xscom_ctrl_write(void *opaque, hwaddr addr,
+ uint64_t val, unsigned size)
+{
+ uint32_t offset = addr >> 3;
+
+ sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
+
+ switch (offset) {
+ default:
+ qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
+ HWADDR_PRIx "\n", addr >> 3);
+ }
+}
+
+static const MemoryRegionOps pnv_sbe_power9_xscom_ctrl_ops = {
+ .read = pnv_sbe_power9_xscom_ctrl_read,
+ .write = pnv_sbe_power9_xscom_ctrl_write,
+ .valid.min_access_size = 8,
+ .valid.max_access_size = 8,
+ .impl.min_access_size = 8,
+ .impl.max_access_size = 8,
+ .endianness = DEVICE_BIG_ENDIAN,
+};
+
+static void pnv_sbe_set_host_doorbell(PnvSBE *sbe, uint64_t val)
+{
+ val &= SBE_HOST_RESPONSE_MASK; /* Is this right? What does HW do? */
+ sbe->host_doorbell = val;
+
+ sbe_reg_dbg("val:%lx\n", val);
+ qemu_set_irq(sbe->psi_irq, !!val);
+}
+
+/* SBE Target Type */
+#define SBE_TARGET_TYPE_PROC 0x00
+#define SBE_TARGET_TYPE_EX 0x01
+#define SBE_TARGET_TYPE_PERV 0x02
+#define SBE_TARGET_TYPE_MCS 0x03
+#define SBE_TARGET_TYPE_EQ 0x04
+#define SBE_TARGET_TYPE_CORE 0x05
+
+/* SBE MBOX command class */
+#define SBE_MCLASS_FIRST 0xD1
+#define SBE_MCLASS_CORE_STATE 0xD1
+#define SBE_MCLASS_SCOM 0xD2
+#define SBE_MCLASS_RING 0xD3
+#define SBE_MCLASS_TIMER 0xD4
+#define SBE_MCLASS_MPIPL 0xD5
+#define SBE_MCLASS_SECURITY 0xD6
+#define SBE_MCLASS_GENERIC 0xD7
+#define SBE_MCLASS_LAST 0xD7
+
+/*
+ * Commands are provided in xxyy form where:
+ * - xx : command class
+ * - yy : command
+ *
+ * Both request and response message uses same seq ID,
+ * command class and command.
+ */
+#define SBE_CMD_CTRL_DEADMAN_LOOP 0xD101
+#define SBE_CMD_MULTI_SCOM 0xD201
+#define SBE_CMD_PUT_RING_FORM_IMAGE 0xD301
+#define SBE_CMD_CONTROL_TIMER 0xD401
+#define SBE_CMD_GET_ARCHITECTED_REG 0xD501
+#define SBE_CMD_CLR_ARCHITECTED_REG 0xD502
+#define SBE_CMD_SET_UNSEC_MEM_WINDOW 0xD601
+#define SBE_CMD_GET_SBE_FFDC 0xD701
+#define SBE_CMD_GET_CAPABILITY 0xD702
+#define SBE_CMD_READ_SBE_SEEPROM 0xD703
+#define SBE_CMD_SET_FFDC_ADDR 0xD704
+#define SBE_CMD_QUIESCE_SBE 0xD705
+#define SBE_CMD_SET_FABRIC_ID_MAP 0xD706
+#define SBE_CMD_STASH_MPIPL_CONFIG 0xD707
+
+/* SBE MBOX control flags */
+
+/* Generic flags */
+#define SBE_CMD_CTRL_RESP_REQ 0x0100
+#define SBE_CMD_CTRL_ACK_REQ 0x0200
+
+/* Deadman loop */
+#define CTRL_DEADMAN_LOOP_START 0x0001
+#define CTRL_DEADMAN_LOOP_STOP 0x0002
+
+/* Control timer */
+#define CONTROL_TIMER_START 0x0001
+#define CONTROL_TIMER_STOP 0x0002
+
+/* Stash MPIPL config */
+#define SBE_STASH_KEY_SKIBOOT_BASE 0x03
+
+static void sbe_timer(void *opaque)
+{
+ PnvSBE *sbe = opaque;
+
+ sbe_cmd_dbg("timer expired\n");
+ pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | SBE_HOST_TIMER_EXPIRY);
+}
+
+static void do_sbe_msg(PnvSBE *sbe)
+{
+ struct sbe_msg msg;
+ uint16_t cmd, ctrl_flags, seq_id;
+ int i;
+
+ memset(&msg, 0, sizeof(msg));
+
+ for (i = 0; i < 4; i++) {
+ msg.reg[i] = sbe->mbox[i];
+ }
+
+ cmd = msg.reg[0];
+ seq_id = msg.reg[0] >> 16;
+ ctrl_flags = msg.reg[0] >> 32;
+ sbe_msg_dbg("cmd:%x seq:%d ctrlflags:%x\n", cmd, seq_id, ctrl_flags);
+
+ if (ctrl_flags & SBE_CMD_CTRL_ACK_REQ) {
+ pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | SBE_HOST_MSG_READ);
+ }
+
+ switch (cmd) {
+ case SBE_CMD_CONTROL_TIMER:
+ if (ctrl_flags & CONTROL_TIMER_START) {
+ uint64_t us = msg.reg[1];
+ sbe_cmd_dbg("timer start us:%lu\n", us);
+ timer_mod(sbe->timer, qemu_clock_get_us(QEMU_CLOCK_VIRTUAL) + us);
+ }
+ if (ctrl_flags & CONTROL_TIMER_STOP) {
+ sbe_cmd_dbg("timer stop\n");
+ timer_del(sbe->timer);
+ }
+ break;
+ default:
+ qemu_log_mask(LOG_UNIMP, "SBE Unimplemented command: 0x%x\n", cmd);
+ }
+}
+
+static void pnv_sbe_set_sbe_doorbell(PnvSBE *sbe, uint64_t val)
+{
+ val &= HOST_SBE_MSG_WAITING;
+ sbe->sbe_doorbell = val;
+
+ if (val & HOST_SBE_MSG_WAITING) {
+ sbe->sbe_doorbell &= ~HOST_SBE_MSG_WAITING;
+ do_sbe_msg(sbe);
+ }
+}
+
+static uint64_t pnv_sbe_power9_xscom_mbox_read(void *opaque, hwaddr addr,
+ unsigned size)
+{
+ PnvSBE *sbe = PNV_SBE(opaque);
+ uint32_t offset = addr >> 3;
+ uint64_t val = 0;
+
+ if (offset <= PSU_HOST_SBE_MBOX_REG7) {
+ uint32_t idx = offset - PSU_HOST_SBE_MBOX_REG0;
+ val = sbe->mbox[idx];
+ } else {
+ switch (offset) {
+ case PSU_SBE_DOORBELL_REG_RW:
+ val = sbe->sbe_doorbell;
+ break;
+ case PSU_HOST_DOORBELL_REG_RW:
+ val = sbe->host_doorbell;
+ break;
+ default:
+ qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
+ HWADDR_PRIx "\n", addr >> 3);
+ }
+ }
+
+ sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
+
+ return val;
+}
+
+static void pnv_sbe_power9_xscom_mbox_write(void *opaque, hwaddr addr,
+ uint64_t val, unsigned size)
+{
+ PnvSBE *sbe = PNV_SBE(opaque);
+ uint32_t offset = addr >> 3;
+
+ sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
+
+ if (offset <= PSU_HOST_SBE_MBOX_REG7) {
+ uint32_t idx = offset - PSU_HOST_SBE_MBOX_REG0;
+ sbe->mbox[idx] = val;
+ } else {
+ switch (offset) {
+ case PSU_SBE_DOORBELL_REG_RW:
+ pnv_sbe_set_sbe_doorbell(sbe, val);
+ break;
+ case PSU_SBE_DOORBELL_REG_AND:
+ pnv_sbe_set_sbe_doorbell(sbe, sbe->sbe_doorbell & val);
+ break;
+ case PSU_SBE_DOORBELL_REG_OR:
+ pnv_sbe_set_sbe_doorbell(sbe, sbe->sbe_doorbell | val);
+ break;
+
+ case PSU_HOST_DOORBELL_REG_RW:
+ pnv_sbe_set_host_doorbell(sbe, val);
+ break;
+ case PSU_HOST_DOORBELL_REG_AND:
+ pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell & val);
+ break;
+ case PSU_HOST_DOORBELL_REG_OR:
+ pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | val);
+ break;
+
+ default:
+ qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
+ HWADDR_PRIx "\n", addr >> 3);
+ }
+ }
+}
+
+static const MemoryRegionOps pnv_sbe_power9_xscom_mbox_ops = {
+ .read = pnv_sbe_power9_xscom_mbox_read,
+ .write = pnv_sbe_power9_xscom_mbox_write,
+ .valid.min_access_size = 8,
+ .valid.max_access_size = 8,
+ .impl.min_access_size = 8,
+ .impl.max_access_size = 8,
+ .endianness = DEVICE_BIG_ENDIAN,
+};
+
+static void pnv_sbe_power9_class_init(ObjectClass *klass, void *data)
+{
+ PnvSBEClass *psc = PNV_SBE_CLASS(klass);
+ DeviceClass *dc = DEVICE_CLASS(klass);
+
+ dc->desc = "PowerNV SBE Controller (POWER9)";
+ psc->xscom_ctrl_size = PNV9_XSCOM_SBE_CTRL_SIZE;
+ psc->xscom_ctrl_ops = &pnv_sbe_power9_xscom_ctrl_ops;
+ psc->xscom_mbox_size = PNV9_XSCOM_SBE_MBOX_SIZE;
+ psc->xscom_mbox_ops = &pnv_sbe_power9_xscom_mbox_ops;
+}
+
+static const TypeInfo pnv_sbe_power9_type_info = {
+ .name = TYPE_PNV9_SBE,
+ .parent = TYPE_PNV_SBE,
+ .instance_size = sizeof(PnvSBE),
+ .class_init = pnv_sbe_power9_class_init,
+};
+
+static void pnv_sbe_power10_class_init(ObjectClass *klass, void *data)
+{
+ PnvSBEClass *psc = PNV_SBE_CLASS(klass);
+ DeviceClass *dc = DEVICE_CLASS(klass);
+
+ dc->desc = "PowerNV SBE Controller (POWER10)";
+ psc->xscom_ctrl_size = PNV10_XSCOM_SBE_CTRL_SIZE;
+ psc->xscom_ctrl_ops = &pnv_sbe_power9_xscom_ctrl_ops;
+ psc->xscom_mbox_size = PNV10_XSCOM_SBE_MBOX_SIZE;
+ psc->xscom_mbox_ops = &pnv_sbe_power9_xscom_mbox_ops;
+}
+
+static const TypeInfo pnv_sbe_power10_type_info = {
+ .name = TYPE_PNV10_SBE,
+ .parent = TYPE_PNV9_SBE,
+ .class_init = pnv_sbe_power10_class_init,
+};
+
+static void pnv_sbe_realize(DeviceState *dev, Error **errp)
+{
+ PnvSBE *sbe = PNV_SBE(dev);
+ PnvSBEClass *psc = PNV_SBE_GET_CLASS(sbe);
+
+ /* XScom regions for SBE registers */
+ pnv_xscom_region_init(&sbe->xscom_ctrl_regs, OBJECT(dev),
+ psc->xscom_ctrl_ops, sbe, "xscom-sbe-ctrl",
+ psc->xscom_ctrl_size);
+ pnv_xscom_region_init(&sbe->xscom_mbox_regs, OBJECT(dev),
+ psc->xscom_mbox_ops, sbe, "xscom-sbe-mbox",
+ psc->xscom_mbox_size);
+
+ qdev_init_gpio_out(DEVICE(dev), &sbe->psi_irq, 1);
+
+ sbe->timer = timer_new_us(QEMU_CLOCK_VIRTUAL, sbe_timer, sbe);
+}
+
+static void pnv_sbe_class_init(ObjectClass *klass, void *data)
+{
+ DeviceClass *dc = DEVICE_CLASS(klass);
+
+ dc->realize = pnv_sbe_realize;
+ dc->desc = "PowerNV SBE Controller";
+ dc->user_creatable = false;
+}
+
+static const TypeInfo pnv_sbe_type_info = {
+ .name = TYPE_PNV_SBE,
+ .parent = TYPE_DEVICE,
+ .instance_size = sizeof(PnvSBE),
+ .class_init = pnv_sbe_class_init,
+ .class_size = sizeof(PnvSBEClass),
+ .abstract = true,
+};
+
+static void pnv_sbe_register_types(void)
+{
+ type_register_static(&pnv_sbe_type_info);
+ type_register_static(&pnv_sbe_power9_type_info);
+ type_register_static(&pnv_sbe_power10_type_info);
+}
+
+type_init(pnv_sbe_register_types);
diff --git a/hw/ppc/pnv_xscom.c b/hw/ppc/pnv_xscom.c
index 9ce018dbc2..79f10de57f 100644
--- a/hw/ppc/pnv_xscom.c
+++ b/hw/ppc/pnv_xscom.c
@@ -295,6 +295,9 @@ int pnv_dt_xscom(PnvChip *chip, void *fdt, int root_offset,
_FDT((fdt_setprop(fdt, xscom_offset, "reg", reg, sizeof(reg))));
_FDT((fdt_setprop(fdt, xscom_offset, "compatible", compat, compat_size)));
_FDT((fdt_setprop(fdt, xscom_offset, "scom-controller", NULL, 0)));
+ if (chip->chip_id == 0) {
+ _FDT((fdt_setprop(fdt, xscom_offset, "primary", NULL, 0)));
+ }
args.fdt = fdt;
args.xscom_offset = xscom_offset;
diff --git a/include/hw/ppc/pnv.h b/include/hw/ppc/pnv.h
index b991194223..37c303bf36 100644
--- a/include/hw/ppc/pnv.h
+++ b/include/hw/ppc/pnv.h
@@ -27,6 +27,7 @@
#include "hw/ppc/pnv_pnor.h"
#include "hw/ppc/pnv_psi.h"
#include "hw/ppc/pnv_occ.h"
+#include "hw/ppc/pnv_sbe.h"
#include "hw/ppc/pnv_homer.h"
#include "hw/ppc/pnv_xive.h"
#include "hw/ppc/pnv_core.h"
@@ -100,6 +101,7 @@ struct Pnv9Chip {
Pnv9Psi psi;
PnvLpcController lpc;
PnvOCC occ;
+ PnvSBE sbe;
PnvHomer homer;
uint32_t nr_quads;
@@ -129,6 +131,7 @@ struct Pnv10Chip {
Pnv9Psi psi;
PnvLpcController lpc;
PnvOCC occ;
+ PnvSBE sbe;
PnvHomer homer;
uint32_t nr_quads;
diff --git a/include/hw/ppc/pnv_sbe.h b/include/hw/ppc/pnv_sbe.h
new file mode 100644
index 0000000000..f54a3ae9ba
--- /dev/null
+++ b/include/hw/ppc/pnv_sbe.h
@@ -0,0 +1,55 @@
+/*
+ * QEMU PowerPC PowerNV Emulation of some SBE behaviour
+ *
+ * Copyright (c) 2022, IBM Corporation.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, see <http://www.gnu.org/licenses/>.
+ */
+
+#ifndef PPC_PNV_SBE_H
+#define PPC_PNV_SBE_H
+
+#include "qom/object.h"
+
+#define TYPE_PNV_SBE "pnv-sbe"
+OBJECT_DECLARE_TYPE(PnvSBE, PnvSBEClass, PNV_SBE)
+#define TYPE_PNV9_SBE TYPE_PNV_SBE "-POWER9"
+DECLARE_INSTANCE_CHECKER(PnvSBE, PNV9_SBE, TYPE_PNV9_SBE)
+#define TYPE_PNV10_SBE TYPE_PNV_SBE "-POWER10"
+DECLARE_INSTANCE_CHECKER(PnvSBE, PNV10_SBE, TYPE_PNV10_SBE)
+
+struct PnvSBE {
+ DeviceState xd;
+
+ uint64_t mbox[8];
+ uint64_t sbe_doorbell;
+ uint64_t host_doorbell;
+
+ qemu_irq psi_irq;
+ QEMUTimer *timer;
+
+ MemoryRegion xscom_mbox_regs;
+ MemoryRegion xscom_ctrl_regs;
+};
+
+struct PnvSBEClass {
+ DeviceClass parent_class;
+
+ int xscom_ctrl_size;
+ int xscom_mbox_size;
+ const MemoryRegionOps *xscom_ctrl_ops;
+ const MemoryRegionOps *xscom_mbox_ops;
+};
+
+#endif /* PPC_PNV_SBE_H */
diff --git a/include/hw/ppc/pnv_xscom.h b/include/hw/ppc/pnv_xscom.h
index 7c7440de0c..c6e9ef8dd2 100644
--- a/include/hw/ppc/pnv_xscom.h
+++ b/include/hw/ppc/pnv_xscom.h
@@ -92,6 +92,12 @@ struct PnvXScomInterfaceClass {
#define PNV9_XSCOM_OCC_BASE PNV_XSCOM_OCC_BASE
#define PNV9_XSCOM_OCC_SIZE 0x8000
+#define PNV9_XSCOM_SBE_CTRL_BASE 0x00050008
+#define PNV9_XSCOM_SBE_CTRL_SIZE 0x1
+
+#define PNV9_XSCOM_SBE_MBOX_BASE 0x000D0050
+#define PNV9_XSCOM_SBE_MBOX_SIZE 0x16
+
#define PNV9_XSCOM_PBA_BASE 0x5012b00
#define PNV9_XSCOM_PBA_SIZE 0x40
@@ -134,6 +140,12 @@ struct PnvXScomInterfaceClass {
#define PNV10_XSCOM_OCC_BASE PNV9_XSCOM_OCC_BASE
#define PNV10_XSCOM_OCC_SIZE PNV9_XSCOM_OCC_SIZE
+#define PNV10_XSCOM_SBE_CTRL_BASE PNV9_XSCOM_SBE_CTRL_BASE
+#define PNV10_XSCOM_SBE_CTRL_SIZE PNV9_XSCOM_SBE_CTRL_SIZE
+
+#define PNV10_XSCOM_SBE_MBOX_BASE PNV9_XSCOM_SBE_MBOX_BASE
+#define PNV10_XSCOM_SBE_MBOX_SIZE PNV9_XSCOM_SBE_MBOX_SIZE
+
#define PNV10_XSCOM_PBA_BASE 0x01010CDA
#define PNV10_XSCOM_PBA_SIZE 0x40
--
2.35.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ppc/pnv: Add initial P9/10 SBE model
2022-08-11 7:54 [PATCH] ppc/pnv: Add initial P9/10 SBE model Nicholas Piggin
@ 2022-08-11 8:04 ` Cédric Le Goater
2023-05-26 12:20 ` Cédric Le Goater
0 siblings, 1 reply; 4+ messages in thread
From: Cédric Le Goater @ 2022-08-11 8:04 UTC (permalink / raw)
To: Nicholas Piggin, qemu-ppc; +Cc: qemu-devel
On 8/11/22 09:54, Nicholas Piggin wrote:
> The SBE (Self Boot Engine) are on-chip microcontrollers that perform
> early boot steps, as well as provide some runtime facilities (e.g.,
> timer, secure register access, MPIPL). The latter facilities are
> accessed mostly via a message system called SBEFIFO.
>
> This driver provides initial emulation for the SBE runtime registers
> and a very basic SBEFIFO implementation that provides the timer
> command. This covers the basic SBE behaviour expected by skiboot when
> booting.
>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
> ---
> hw/ppc/meson.build | 1 +
> hw/ppc/pnv.c | 25 +++
> hw/ppc/pnv_sbe.c | 429 +++++++++++++++++++++++++++++++++++++
> hw/ppc/pnv_xscom.c | 3 +
> include/hw/ppc/pnv.h | 3 +
> include/hw/ppc/pnv_sbe.h | 55 +++++
> include/hw/ppc/pnv_xscom.h | 12 ++
> 7 files changed, 528 insertions(+)
> create mode 100644 hw/ppc/pnv_sbe.c
> create mode 100644 include/hw/ppc/pnv_sbe.h
>
> diff --git a/hw/ppc/meson.build b/hw/ppc/meson.build
> index aa4c8e6a2e..62801923f3 100644
> --- a/hw/ppc/meson.build
> +++ b/hw/ppc/meson.build
> @@ -46,6 +46,7 @@ ppc_ss.add(when: 'CONFIG_POWERNV', if_true: files(
> 'pnv_lpc.c',
> 'pnv_psi.c',
> 'pnv_occ.c',
> + 'pnv_sbe.c',
> 'pnv_bmc.c',
> 'pnv_homer.c',
> 'pnv_pnor.c',
> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> index d3f77c8367..7ff1f464d3 100644
> --- a/hw/ppc/pnv.c
> +++ b/hw/ppc/pnv.c
> @@ -1397,6 +1397,8 @@ static void pnv_chip_power9_instance_init(Object *obj)
>
> object_initialize_child(obj, "occ", &chip9->occ, TYPE_PNV9_OCC);
>
> + object_initialize_child(obj, "sbe", &chip9->sbe, TYPE_PNV9_SBE);
> +
> object_initialize_child(obj, "homer", &chip9->homer, TYPE_PNV9_HOMER);
>
> /* Number of PECs is the chip default */
> @@ -1549,6 +1551,17 @@ static void pnv_chip_power9_realize(DeviceState *dev, Error **errp)
> memory_region_add_subregion(get_system_memory(), PNV9_OCC_SENSOR_BASE(chip),
> &chip9->occ.sram_regs);
>
> + /* SBE */
> + if (!qdev_realize(DEVICE(&chip9->sbe), NULL, errp)) {
> + return;
> + }
> + pnv_xscom_add_subregion(chip, PNV9_XSCOM_SBE_CTRL_BASE,
> + &chip9->sbe.xscom_ctrl_regs);
> + pnv_xscom_add_subregion(chip, PNV9_XSCOM_SBE_MBOX_BASE,
> + &chip9->sbe.xscom_mbox_regs);
> + qdev_connect_gpio_out(DEVICE(&chip9->sbe), 0, qdev_get_gpio_in(
> + DEVICE(&chip9->psi), PSIHB9_IRQ_PSU));
> +
> /* HOMER */
> object_property_set_link(OBJECT(&chip9->homer), "chip", OBJECT(chip),
> &error_abort);
> @@ -1613,6 +1626,7 @@ static void pnv_chip_power10_instance_init(Object *obj)
> object_initialize_child(obj, "psi", &chip10->psi, TYPE_PNV10_PSI);
> object_initialize_child(obj, "lpc", &chip10->lpc, TYPE_PNV10_LPC);
> object_initialize_child(obj, "occ", &chip10->occ, TYPE_PNV10_OCC);
> + object_initialize_child(obj, "sbe", &chip10->sbe, TYPE_PNV10_SBE);
> object_initialize_child(obj, "homer", &chip10->homer, TYPE_PNV10_HOMER);
>
> chip->num_pecs = pcc->num_pecs;
> @@ -1754,6 +1768,17 @@ static void pnv_chip_power10_realize(DeviceState *dev, Error **errp)
> PNV10_OCC_SENSOR_BASE(chip),
> &chip10->occ.sram_regs);
>
> + /* SBE */
> + if (!qdev_realize(DEVICE(&chip10->sbe), NULL, errp)) {
> + return;
> + }
> + pnv_xscom_add_subregion(chip, PNV10_XSCOM_SBE_CTRL_BASE,
> + &chip10->sbe.xscom_ctrl_regs);
> + pnv_xscom_add_subregion(chip, PNV10_XSCOM_SBE_MBOX_BASE,
> + &chip10->sbe.xscom_mbox_regs);
> + qdev_connect_gpio_out(DEVICE(&chip10->sbe), 0, qdev_get_gpio_in(
> + DEVICE(&chip10->psi), PSIHB9_IRQ_PSU));
> +
> /* HOMER */
> object_property_set_link(OBJECT(&chip10->homer), "chip", OBJECT(chip),
> &error_abort);
> diff --git a/hw/ppc/pnv_sbe.c b/hw/ppc/pnv_sbe.c
> new file mode 100644
> index 0000000000..559502b7a6
> --- /dev/null
> +++ b/hw/ppc/pnv_sbe.c
> @@ -0,0 +1,429 @@
> +/*
> + * QEMU PowerPC PowerNV Emulation of some SBE behaviour
> + *
> + * Copyright (c) 2022, IBM Corporation.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License, version 2, as
> + * published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "target/ppc/cpu.h"
> +#include "qapi/error.h"
> +#include "qemu/log.h"
> +#include "qemu/module.h"
> +#include "hw/irq.h"
> +#include "hw/qdev-properties.h"
> +#include "hw/ppc/pnv.h"
> +#include "hw/ppc/pnv_xscom.h"
> +#include "hw/ppc/pnv_sbe.h"
> +
> +#define DEBUG_SBE_REG 0
> +#define DEBUG_SBE_MSG 0
> +#define DEBUG_SBE_CMD 0
> +
> +#define sbe_reg_dbg(fmt, ...) do { \
> + if (DEBUG_SBE_REG) { \
> + fprintf(stderr, "SBE reg: %s: " fmt, __func__, ## __VA_ARGS__); \
> + } \
> +} while (0)
> +
> +#define sbe_msg_dbg(fmt, ...) do { \
> + if (DEBUG_SBE_MSG) { \
> + fprintf(stderr, "SBE msg: %s: " fmt, __func__, ## __VA_ARGS__); \
> + } \
> +} while (0)
> +
> +#define sbe_cmd_dbg(fmt, ...) do { \
> + if (DEBUG_SBE_CMD) { \
> + fprintf(stderr, "SBE cmd: %s: " fmt, __func__, ## __VA_ARGS__); \
> + } \
> +} while (0)
Please use trace events instead, with a pnv_sbe_ prefix.
Apart from that, LGTM !
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Thanks,
C.
> +
> +/* Most register and command definitions come from skiboot */
> +
> +/*
> + * SBE MBOX register address
> + * Reg 0 - 3 : Host to send command packets to SBE
> + * Reg 4 - 7 : SBE to send response packets to Host
> + */
> +#define PSU_HOST_SBE_MBOX_REG0 0x00000000
> +#define PSU_HOST_SBE_MBOX_REG1 0x00000001
> +#define PSU_HOST_SBE_MBOX_REG2 0x00000002
> +#define PSU_HOST_SBE_MBOX_REG3 0x00000003
> +#define PSU_HOST_SBE_MBOX_REG4 0x00000004
> +#define PSU_HOST_SBE_MBOX_REG5 0x00000005
> +#define PSU_HOST_SBE_MBOX_REG6 0x00000006
> +#define PSU_HOST_SBE_MBOX_REG7 0x00000007
> +#define PSU_SBE_DOORBELL_REG_RW 0x00000010
> +#define PSU_SBE_DOORBELL_REG_AND 0x00000011
> +#define PSU_SBE_DOORBELL_REG_OR 0x00000012
> +#define PSU_HOST_DOORBELL_REG_RW 0x00000013
> +#define PSU_HOST_DOORBELL_REG_AND 0x00000014
> +#define PSU_HOST_DOORBELL_REG_OR 0x00000015
> +
> +/*
> + * Doorbell register to trigger SBE interrupt. Set by OPAL to inform
> + * the SBE about a waiting message in the Host/SBE mailbox registers
> + */
> +#define HOST_SBE_MSG_WAITING PPC_BIT(0)
> +
> +/*
> + * Doorbell register for host bridge interrupt. Set by the SBE to inform
> + * host about a response message in the Host/SBE mailbox registers
> + */
> +#define SBE_HOST_RESPONSE_WAITING PPC_BIT(0)
> +#define SBE_HOST_MSG_READ PPC_BIT(1)
> +#define SBE_HOST_STOP15_EXIT PPC_BIT(2)
> +#define SBE_HOST_RESET PPC_BIT(3)
> +#define SBE_HOST_PASSTHROUGH PPC_BIT(4)
> +#define SBE_HOST_TIMER_EXPIRY PPC_BIT(14)
> +#define SBE_HOST_RESPONSE_MASK (PPC_BITMASK(0, 4) | SBE_HOST_TIMER_EXPIRY)
> +
> +/* SBE Control Register */
> +#define SBE_CONTROL_REG_RW 0x00000000
> +
> +/* SBE interrupt s0/s1 bits */
> +#define SBE_CONTROL_REG_S0 PPC_BIT(14)
> +#define SBE_CONTROL_REG_S1 PPC_BIT(15)
> +
> +struct sbe_msg {
> + uint64_t reg[4];
> +};
> +
> +static uint64_t pnv_sbe_power9_xscom_ctrl_read(void *opaque, hwaddr addr,
> + unsigned size)
> +{
> + uint32_t offset = addr >> 3;
> + uint64_t val = 0;
> +
> + switch (offset) {
> + default:
> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
> + HWADDR_PRIx "\n", addr >> 3);
> + }
> +
> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
> +
> + return val;
> +}
> +
> +static void pnv_sbe_power9_xscom_ctrl_write(void *opaque, hwaddr addr,
> + uint64_t val, unsigned size)
> +{
> + uint32_t offset = addr >> 3;
> +
> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
> +
> + switch (offset) {
> + default:
> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
> + HWADDR_PRIx "\n", addr >> 3);
> + }
> +}
> +
> +static const MemoryRegionOps pnv_sbe_power9_xscom_ctrl_ops = {
> + .read = pnv_sbe_power9_xscom_ctrl_read,
> + .write = pnv_sbe_power9_xscom_ctrl_write,
> + .valid.min_access_size = 8,
> + .valid.max_access_size = 8,
> + .impl.min_access_size = 8,
> + .impl.max_access_size = 8,
> + .endianness = DEVICE_BIG_ENDIAN,
> +};
> +
> +static void pnv_sbe_set_host_doorbell(PnvSBE *sbe, uint64_t val)
> +{
> + val &= SBE_HOST_RESPONSE_MASK; /* Is this right? What does HW do? */
> + sbe->host_doorbell = val;
> +
> + sbe_reg_dbg("val:%lx\n", val);
> + qemu_set_irq(sbe->psi_irq, !!val);
> +}
> +
> +/* SBE Target Type */
> +#define SBE_TARGET_TYPE_PROC 0x00
> +#define SBE_TARGET_TYPE_EX 0x01
> +#define SBE_TARGET_TYPE_PERV 0x02
> +#define SBE_TARGET_TYPE_MCS 0x03
> +#define SBE_TARGET_TYPE_EQ 0x04
> +#define SBE_TARGET_TYPE_CORE 0x05
> +
> +/* SBE MBOX command class */
> +#define SBE_MCLASS_FIRST 0xD1
> +#define SBE_MCLASS_CORE_STATE 0xD1
> +#define SBE_MCLASS_SCOM 0xD2
> +#define SBE_MCLASS_RING 0xD3
> +#define SBE_MCLASS_TIMER 0xD4
> +#define SBE_MCLASS_MPIPL 0xD5
> +#define SBE_MCLASS_SECURITY 0xD6
> +#define SBE_MCLASS_GENERIC 0xD7
> +#define SBE_MCLASS_LAST 0xD7
> +
> +/*
> + * Commands are provided in xxyy form where:
> + * - xx : command class
> + * - yy : command
> + *
> + * Both request and response message uses same seq ID,
> + * command class and command.
> + */
> +#define SBE_CMD_CTRL_DEADMAN_LOOP 0xD101
> +#define SBE_CMD_MULTI_SCOM 0xD201
> +#define SBE_CMD_PUT_RING_FORM_IMAGE 0xD301
> +#define SBE_CMD_CONTROL_TIMER 0xD401
> +#define SBE_CMD_GET_ARCHITECTED_REG 0xD501
> +#define SBE_CMD_CLR_ARCHITECTED_REG 0xD502
> +#define SBE_CMD_SET_UNSEC_MEM_WINDOW 0xD601
> +#define SBE_CMD_GET_SBE_FFDC 0xD701
> +#define SBE_CMD_GET_CAPABILITY 0xD702
> +#define SBE_CMD_READ_SBE_SEEPROM 0xD703
> +#define SBE_CMD_SET_FFDC_ADDR 0xD704
> +#define SBE_CMD_QUIESCE_SBE 0xD705
> +#define SBE_CMD_SET_FABRIC_ID_MAP 0xD706
> +#define SBE_CMD_STASH_MPIPL_CONFIG 0xD707
> +
> +/* SBE MBOX control flags */
> +
> +/* Generic flags */
> +#define SBE_CMD_CTRL_RESP_REQ 0x0100
> +#define SBE_CMD_CTRL_ACK_REQ 0x0200
> +
> +/* Deadman loop */
> +#define CTRL_DEADMAN_LOOP_START 0x0001
> +#define CTRL_DEADMAN_LOOP_STOP 0x0002
> +
> +/* Control timer */
> +#define CONTROL_TIMER_START 0x0001
> +#define CONTROL_TIMER_STOP 0x0002
> +
> +/* Stash MPIPL config */
> +#define SBE_STASH_KEY_SKIBOOT_BASE 0x03
> +
> +static void sbe_timer(void *opaque)
> +{
> + PnvSBE *sbe = opaque;
> +
> + sbe_cmd_dbg("timer expired\n");
> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | SBE_HOST_TIMER_EXPIRY);
> +}
> +
> +static void do_sbe_msg(PnvSBE *sbe)
> +{
> + struct sbe_msg msg;
> + uint16_t cmd, ctrl_flags, seq_id;
> + int i;
> +
> + memset(&msg, 0, sizeof(msg));
> +
> + for (i = 0; i < 4; i++) {
> + msg.reg[i] = sbe->mbox[i];
> + }
> +
> + cmd = msg.reg[0];
> + seq_id = msg.reg[0] >> 16;
> + ctrl_flags = msg.reg[0] >> 32;
> + sbe_msg_dbg("cmd:%x seq:%d ctrlflags:%x\n", cmd, seq_id, ctrl_flags);
> +
> + if (ctrl_flags & SBE_CMD_CTRL_ACK_REQ) {
> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | SBE_HOST_MSG_READ);
> + }
> +
> + switch (cmd) {
> + case SBE_CMD_CONTROL_TIMER:
> + if (ctrl_flags & CONTROL_TIMER_START) {
> + uint64_t us = msg.reg[1];
> + sbe_cmd_dbg("timer start us:%lu\n", us);
> + timer_mod(sbe->timer, qemu_clock_get_us(QEMU_CLOCK_VIRTUAL) + us);
> + }
> + if (ctrl_flags & CONTROL_TIMER_STOP) {
> + sbe_cmd_dbg("timer stop\n");
> + timer_del(sbe->timer);
> + }
> + break;
> + default:
> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented command: 0x%x\n", cmd);
> + }
> +}
> +
> +static void pnv_sbe_set_sbe_doorbell(PnvSBE *sbe, uint64_t val)
> +{
> + val &= HOST_SBE_MSG_WAITING;
> + sbe->sbe_doorbell = val;
> +
> + if (val & HOST_SBE_MSG_WAITING) {
> + sbe->sbe_doorbell &= ~HOST_SBE_MSG_WAITING;
> + do_sbe_msg(sbe);
> + }
> +}
> +
> +static uint64_t pnv_sbe_power9_xscom_mbox_read(void *opaque, hwaddr addr,
> + unsigned size)
> +{
> + PnvSBE *sbe = PNV_SBE(opaque);
> + uint32_t offset = addr >> 3;
> + uint64_t val = 0;
> +
> + if (offset <= PSU_HOST_SBE_MBOX_REG7) {
> + uint32_t idx = offset - PSU_HOST_SBE_MBOX_REG0;
> + val = sbe->mbox[idx];
> + } else {
> + switch (offset) {
> + case PSU_SBE_DOORBELL_REG_RW:
> + val = sbe->sbe_doorbell;
> + break;
> + case PSU_HOST_DOORBELL_REG_RW:
> + val = sbe->host_doorbell;
> + break;
> + default:
> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
> + HWADDR_PRIx "\n", addr >> 3);
> + }
> + }
> +
> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
> +
> + return val;
> +}
> +
> +static void pnv_sbe_power9_xscom_mbox_write(void *opaque, hwaddr addr,
> + uint64_t val, unsigned size)
> +{
> + PnvSBE *sbe = PNV_SBE(opaque);
> + uint32_t offset = addr >> 3;
> +
> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
> +
> + if (offset <= PSU_HOST_SBE_MBOX_REG7) {
> + uint32_t idx = offset - PSU_HOST_SBE_MBOX_REG0;
> + sbe->mbox[idx] = val;
> + } else {
> + switch (offset) {
> + case PSU_SBE_DOORBELL_REG_RW:
> + pnv_sbe_set_sbe_doorbell(sbe, val);
> + break;
> + case PSU_SBE_DOORBELL_REG_AND:
> + pnv_sbe_set_sbe_doorbell(sbe, sbe->sbe_doorbell & val);
> + break;
> + case PSU_SBE_DOORBELL_REG_OR:
> + pnv_sbe_set_sbe_doorbell(sbe, sbe->sbe_doorbell | val);
> + break;
> +
> + case PSU_HOST_DOORBELL_REG_RW:
> + pnv_sbe_set_host_doorbell(sbe, val);
> + break;
> + case PSU_HOST_DOORBELL_REG_AND:
> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell & val);
> + break;
> + case PSU_HOST_DOORBELL_REG_OR:
> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | val);
> + break;
> +
> + default:
> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
> + HWADDR_PRIx "\n", addr >> 3);
> + }
> + }
> +}
> +
> +static const MemoryRegionOps pnv_sbe_power9_xscom_mbox_ops = {
> + .read = pnv_sbe_power9_xscom_mbox_read,
> + .write = pnv_sbe_power9_xscom_mbox_write,
> + .valid.min_access_size = 8,
> + .valid.max_access_size = 8,
> + .impl.min_access_size = 8,
> + .impl.max_access_size = 8,
> + .endianness = DEVICE_BIG_ENDIAN,
> +};
> +
> +static void pnv_sbe_power9_class_init(ObjectClass *klass, void *data)
> +{
> + PnvSBEClass *psc = PNV_SBE_CLASS(klass);
> + DeviceClass *dc = DEVICE_CLASS(klass);
> +
> + dc->desc = "PowerNV SBE Controller (POWER9)";
> + psc->xscom_ctrl_size = PNV9_XSCOM_SBE_CTRL_SIZE;
> + psc->xscom_ctrl_ops = &pnv_sbe_power9_xscom_ctrl_ops;
> + psc->xscom_mbox_size = PNV9_XSCOM_SBE_MBOX_SIZE;
> + psc->xscom_mbox_ops = &pnv_sbe_power9_xscom_mbox_ops;
> +}
> +
> +static const TypeInfo pnv_sbe_power9_type_info = {
> + .name = TYPE_PNV9_SBE,
> + .parent = TYPE_PNV_SBE,
> + .instance_size = sizeof(PnvSBE),
> + .class_init = pnv_sbe_power9_class_init,
> +};
> +
> +static void pnv_sbe_power10_class_init(ObjectClass *klass, void *data)
> +{
> + PnvSBEClass *psc = PNV_SBE_CLASS(klass);
> + DeviceClass *dc = DEVICE_CLASS(klass);
> +
> + dc->desc = "PowerNV SBE Controller (POWER10)";
> + psc->xscom_ctrl_size = PNV10_XSCOM_SBE_CTRL_SIZE;
> + psc->xscom_ctrl_ops = &pnv_sbe_power9_xscom_ctrl_ops;
> + psc->xscom_mbox_size = PNV10_XSCOM_SBE_MBOX_SIZE;
> + psc->xscom_mbox_ops = &pnv_sbe_power9_xscom_mbox_ops;
> +}
> +
> +static const TypeInfo pnv_sbe_power10_type_info = {
> + .name = TYPE_PNV10_SBE,
> + .parent = TYPE_PNV9_SBE,
> + .class_init = pnv_sbe_power10_class_init,
> +};
> +
> +static void pnv_sbe_realize(DeviceState *dev, Error **errp)
> +{
> + PnvSBE *sbe = PNV_SBE(dev);
> + PnvSBEClass *psc = PNV_SBE_GET_CLASS(sbe);
> +
> + /* XScom regions for SBE registers */
> + pnv_xscom_region_init(&sbe->xscom_ctrl_regs, OBJECT(dev),
> + psc->xscom_ctrl_ops, sbe, "xscom-sbe-ctrl",
> + psc->xscom_ctrl_size);
> + pnv_xscom_region_init(&sbe->xscom_mbox_regs, OBJECT(dev),
> + psc->xscom_mbox_ops, sbe, "xscom-sbe-mbox",
> + psc->xscom_mbox_size);
> +
> + qdev_init_gpio_out(DEVICE(dev), &sbe->psi_irq, 1);
> +
> + sbe->timer = timer_new_us(QEMU_CLOCK_VIRTUAL, sbe_timer, sbe);
> +}
> +
> +static void pnv_sbe_class_init(ObjectClass *klass, void *data)
> +{
> + DeviceClass *dc = DEVICE_CLASS(klass);
> +
> + dc->realize = pnv_sbe_realize;
> + dc->desc = "PowerNV SBE Controller";
> + dc->user_creatable = false;
> +}
> +
> +static const TypeInfo pnv_sbe_type_info = {
> + .name = TYPE_PNV_SBE,
> + .parent = TYPE_DEVICE,
> + .instance_size = sizeof(PnvSBE),
> + .class_init = pnv_sbe_class_init,
> + .class_size = sizeof(PnvSBEClass),
> + .abstract = true,
> +};
> +
> +static void pnv_sbe_register_types(void)
> +{
> + type_register_static(&pnv_sbe_type_info);
> + type_register_static(&pnv_sbe_power9_type_info);
> + type_register_static(&pnv_sbe_power10_type_info);
> +}
> +
> +type_init(pnv_sbe_register_types);
> diff --git a/hw/ppc/pnv_xscom.c b/hw/ppc/pnv_xscom.c
> index 9ce018dbc2..79f10de57f 100644
> --- a/hw/ppc/pnv_xscom.c
> +++ b/hw/ppc/pnv_xscom.c
> @@ -295,6 +295,9 @@ int pnv_dt_xscom(PnvChip *chip, void *fdt, int root_offset,
> _FDT((fdt_setprop(fdt, xscom_offset, "reg", reg, sizeof(reg))));
> _FDT((fdt_setprop(fdt, xscom_offset, "compatible", compat, compat_size)));
> _FDT((fdt_setprop(fdt, xscom_offset, "scom-controller", NULL, 0)));
> + if (chip->chip_id == 0) {
> + _FDT((fdt_setprop(fdt, xscom_offset, "primary", NULL, 0)));
> + }
>
> args.fdt = fdt;
> args.xscom_offset = xscom_offset;
> diff --git a/include/hw/ppc/pnv.h b/include/hw/ppc/pnv.h
> index b991194223..37c303bf36 100644
> --- a/include/hw/ppc/pnv.h
> +++ b/include/hw/ppc/pnv.h
> @@ -27,6 +27,7 @@
> #include "hw/ppc/pnv_pnor.h"
> #include "hw/ppc/pnv_psi.h"
> #include "hw/ppc/pnv_occ.h"
> +#include "hw/ppc/pnv_sbe.h"
> #include "hw/ppc/pnv_homer.h"
> #include "hw/ppc/pnv_xive.h"
> #include "hw/ppc/pnv_core.h"
> @@ -100,6 +101,7 @@ struct Pnv9Chip {
> Pnv9Psi psi;
> PnvLpcController lpc;
> PnvOCC occ;
> + PnvSBE sbe;
> PnvHomer homer;
>
> uint32_t nr_quads;
> @@ -129,6 +131,7 @@ struct Pnv10Chip {
> Pnv9Psi psi;
> PnvLpcController lpc;
> PnvOCC occ;
> + PnvSBE sbe;
> PnvHomer homer;
>
> uint32_t nr_quads;
> diff --git a/include/hw/ppc/pnv_sbe.h b/include/hw/ppc/pnv_sbe.h
> new file mode 100644
> index 0000000000..f54a3ae9ba
> --- /dev/null
> +++ b/include/hw/ppc/pnv_sbe.h
> @@ -0,0 +1,55 @@
> +/*
> + * QEMU PowerPC PowerNV Emulation of some SBE behaviour
> + *
> + * Copyright (c) 2022, IBM Corporation.
> + *
> + * This library is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2.1 of the License, or (at your option) any later version.
> + *
> + * This library is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with this library; if not, see <http://www.gnu.org/licenses/>.
> + */
> +
> +#ifndef PPC_PNV_SBE_H
> +#define PPC_PNV_SBE_H
> +
> +#include "qom/object.h"
> +
> +#define TYPE_PNV_SBE "pnv-sbe"
> +OBJECT_DECLARE_TYPE(PnvSBE, PnvSBEClass, PNV_SBE)
> +#define TYPE_PNV9_SBE TYPE_PNV_SBE "-POWER9"
> +DECLARE_INSTANCE_CHECKER(PnvSBE, PNV9_SBE, TYPE_PNV9_SBE)
> +#define TYPE_PNV10_SBE TYPE_PNV_SBE "-POWER10"
> +DECLARE_INSTANCE_CHECKER(PnvSBE, PNV10_SBE, TYPE_PNV10_SBE)
> +
> +struct PnvSBE {
> + DeviceState xd;
> +
> + uint64_t mbox[8];
> + uint64_t sbe_doorbell;
> + uint64_t host_doorbell;
> +
> + qemu_irq psi_irq;
> + QEMUTimer *timer;
> +
> + MemoryRegion xscom_mbox_regs;
> + MemoryRegion xscom_ctrl_regs;
> +};
> +
> +struct PnvSBEClass {
> + DeviceClass parent_class;
> +
> + int xscom_ctrl_size;
> + int xscom_mbox_size;
> + const MemoryRegionOps *xscom_ctrl_ops;
> + const MemoryRegionOps *xscom_mbox_ops;
> +};
> +
> +#endif /* PPC_PNV_SBE_H */
> diff --git a/include/hw/ppc/pnv_xscom.h b/include/hw/ppc/pnv_xscom.h
> index 7c7440de0c..c6e9ef8dd2 100644
> --- a/include/hw/ppc/pnv_xscom.h
> +++ b/include/hw/ppc/pnv_xscom.h
> @@ -92,6 +92,12 @@ struct PnvXScomInterfaceClass {
> #define PNV9_XSCOM_OCC_BASE PNV_XSCOM_OCC_BASE
> #define PNV9_XSCOM_OCC_SIZE 0x8000
>
> +#define PNV9_XSCOM_SBE_CTRL_BASE 0x00050008
> +#define PNV9_XSCOM_SBE_CTRL_SIZE 0x1
> +
> +#define PNV9_XSCOM_SBE_MBOX_BASE 0x000D0050
> +#define PNV9_XSCOM_SBE_MBOX_SIZE 0x16
> +
> #define PNV9_XSCOM_PBA_BASE 0x5012b00
> #define PNV9_XSCOM_PBA_SIZE 0x40
>
> @@ -134,6 +140,12 @@ struct PnvXScomInterfaceClass {
> #define PNV10_XSCOM_OCC_BASE PNV9_XSCOM_OCC_BASE
> #define PNV10_XSCOM_OCC_SIZE PNV9_XSCOM_OCC_SIZE
>
> +#define PNV10_XSCOM_SBE_CTRL_BASE PNV9_XSCOM_SBE_CTRL_BASE
> +#define PNV10_XSCOM_SBE_CTRL_SIZE PNV9_XSCOM_SBE_CTRL_SIZE
> +
> +#define PNV10_XSCOM_SBE_MBOX_BASE PNV9_XSCOM_SBE_MBOX_BASE
> +#define PNV10_XSCOM_SBE_MBOX_SIZE PNV9_XSCOM_SBE_MBOX_SIZE
> +
> #define PNV10_XSCOM_PBA_BASE 0x01010CDA
> #define PNV10_XSCOM_PBA_SIZE 0x40
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ppc/pnv: Add initial P9/10 SBE model
2022-08-11 8:04 ` Cédric Le Goater
@ 2023-05-26 12:20 ` Cédric Le Goater
2023-05-29 1:51 ` Nicholas Piggin
0 siblings, 1 reply; 4+ messages in thread
From: Cédric Le Goater @ 2023-05-26 12:20 UTC (permalink / raw)
To: Nicholas Piggin, qemu-ppc; +Cc: qemu-devel, Frederic Barrat
On 8/11/22 10:04, Cédric Le Goater wrote:
> On 8/11/22 09:54, Nicholas Piggin wrote:
>> The SBE (Self Boot Engine) are on-chip microcontrollers that perform
>> early boot steps, as well as provide some runtime facilities (e.g.,
>> timer, secure register access, MPIPL). The latter facilities are
>> accessed mostly via a message system called SBEFIFO.
>>
>> This driver provides initial emulation for the SBE runtime registers
>> and a very basic SBEFIFO implementation that provides the timer
>> command. This covers the basic SBE behaviour expected by skiboot when
>> booting.
>>
>> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
>> ---
>> hw/ppc/meson.build | 1 +
>> hw/ppc/pnv.c | 25 +++
>> hw/ppc/pnv_sbe.c | 429 +++++++++++++++++++++++++++++++++++++
>> hw/ppc/pnv_xscom.c | 3 +
>> include/hw/ppc/pnv.h | 3 +
>> include/hw/ppc/pnv_sbe.h | 55 +++++
>> include/hw/ppc/pnv_xscom.h | 12 ++
>> 7 files changed, 528 insertions(+)
>> create mode 100644 hw/ppc/pnv_sbe.c
>> create mode 100644 include/hw/ppc/pnv_sbe.h
>>
>> diff --git a/hw/ppc/meson.build b/hw/ppc/meson.build
>> index aa4c8e6a2e..62801923f3 100644
>> --- a/hw/ppc/meson.build
>> +++ b/hw/ppc/meson.build
>> @@ -46,6 +46,7 @@ ppc_ss.add(when: 'CONFIG_POWERNV', if_true: files(
>> 'pnv_lpc.c',
>> 'pnv_psi.c',
>> 'pnv_occ.c',
>> + 'pnv_sbe.c',
>> 'pnv_bmc.c',
>> 'pnv_homer.c',
>> 'pnv_pnor.c',
>> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
>> index d3f77c8367..7ff1f464d3 100644
>> --- a/hw/ppc/pnv.c
>> +++ b/hw/ppc/pnv.c
>> @@ -1397,6 +1397,8 @@ static void pnv_chip_power9_instance_init(Object *obj)
>> object_initialize_child(obj, "occ", &chip9->occ, TYPE_PNV9_OCC);
>> + object_initialize_child(obj, "sbe", &chip9->sbe, TYPE_PNV9_SBE);
>> +
>> object_initialize_child(obj, "homer", &chip9->homer, TYPE_PNV9_HOMER);
>> /* Number of PECs is the chip default */
>> @@ -1549,6 +1551,17 @@ static void pnv_chip_power9_realize(DeviceState *dev, Error **errp)
>> memory_region_add_subregion(get_system_memory(), PNV9_OCC_SENSOR_BASE(chip),
>> &chip9->occ.sram_regs);
>> + /* SBE */
>> + if (!qdev_realize(DEVICE(&chip9->sbe), NULL, errp)) {
>> + return;
>> + }
>> + pnv_xscom_add_subregion(chip, PNV9_XSCOM_SBE_CTRL_BASE,
>> + &chip9->sbe.xscom_ctrl_regs);
>> + pnv_xscom_add_subregion(chip, PNV9_XSCOM_SBE_MBOX_BASE,
>> + &chip9->sbe.xscom_mbox_regs);
>> + qdev_connect_gpio_out(DEVICE(&chip9->sbe), 0, qdev_get_gpio_in(
>> + DEVICE(&chip9->psi), PSIHB9_IRQ_PSU));
>> +
>> /* HOMER */
>> object_property_set_link(OBJECT(&chip9->homer), "chip", OBJECT(chip),
>> &error_abort);
>> @@ -1613,6 +1626,7 @@ static void pnv_chip_power10_instance_init(Object *obj)
>> object_initialize_child(obj, "psi", &chip10->psi, TYPE_PNV10_PSI);
>> object_initialize_child(obj, "lpc", &chip10->lpc, TYPE_PNV10_LPC);
>> object_initialize_child(obj, "occ", &chip10->occ, TYPE_PNV10_OCC);
>> + object_initialize_child(obj, "sbe", &chip10->sbe, TYPE_PNV10_SBE);
>> object_initialize_child(obj, "homer", &chip10->homer, TYPE_PNV10_HOMER);
>> chip->num_pecs = pcc->num_pecs;
>> @@ -1754,6 +1768,17 @@ static void pnv_chip_power10_realize(DeviceState *dev, Error **errp)
>> PNV10_OCC_SENSOR_BASE(chip),
>> &chip10->occ.sram_regs);
>> + /* SBE */
>> + if (!qdev_realize(DEVICE(&chip10->sbe), NULL, errp)) {
>> + return;
>> + }
>> + pnv_xscom_add_subregion(chip, PNV10_XSCOM_SBE_CTRL_BASE,
>> + &chip10->sbe.xscom_ctrl_regs);
>> + pnv_xscom_add_subregion(chip, PNV10_XSCOM_SBE_MBOX_BASE,
>> + &chip10->sbe.xscom_mbox_regs);
>> + qdev_connect_gpio_out(DEVICE(&chip10->sbe), 0, qdev_get_gpio_in(
>> + DEVICE(&chip10->psi), PSIHB9_IRQ_PSU));
>> +
>> /* HOMER */
>> object_property_set_link(OBJECT(&chip10->homer), "chip", OBJECT(chip),
>> &error_abort);
>> diff --git a/hw/ppc/pnv_sbe.c b/hw/ppc/pnv_sbe.c
>> new file mode 100644
>> index 0000000000..559502b7a6
>> --- /dev/null
>> +++ b/hw/ppc/pnv_sbe.c
>> @@ -0,0 +1,429 @@
>> +/*
>> + * QEMU PowerPC PowerNV Emulation of some SBE behaviour
>> + *
>> + * Copyright (c) 2022, IBM Corporation.
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License, version 2, as
>> + * published by the Free Software Foundation.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU General Public License
>> + * along with this program; if not, see <http://www.gnu.org/licenses/>.
>> + */
>> +
>> +#include "qemu/osdep.h"
>> +#include "target/ppc/cpu.h"
>> +#include "qapi/error.h"
>> +#include "qemu/log.h"
>> +#include "qemu/module.h"
>> +#include "hw/irq.h"
>> +#include "hw/qdev-properties.h"
>> +#include "hw/ppc/pnv.h"
>> +#include "hw/ppc/pnv_xscom.h"
>> +#include "hw/ppc/pnv_sbe.h"
>> +
>> +#define DEBUG_SBE_REG 0
>> +#define DEBUG_SBE_MSG 0
>> +#define DEBUG_SBE_CMD 0
>> +
>> +#define sbe_reg_dbg(fmt, ...) do { \
>> + if (DEBUG_SBE_REG) { \
>> + fprintf(stderr, "SBE reg: %s: " fmt, __func__, ## __VA_ARGS__); \
>> + } \
>> +} while (0)
>> +
>> +#define sbe_msg_dbg(fmt, ...) do { \
>> + if (DEBUG_SBE_MSG) { \
>> + fprintf(stderr, "SBE msg: %s: " fmt, __func__, ## __VA_ARGS__); \
>> + } \
>> +} while (0)
>> +
>> +#define sbe_cmd_dbg(fmt, ...) do { \
>> + if (DEBUG_SBE_CMD) { \
>> + fprintf(stderr, "SBE cmd: %s: " fmt, __func__, ## __VA_ARGS__); \
>> + } \
>> +} while (0)
>
> Please use trace events instead, with a pnv_sbe_ prefix.
>
> Apart from that, LGTM !
>
> Reviewed-by: Cédric Le Goater <clg@kaod.org>
This looked like an interesting extension to the baremetal models.
Did you abandon the idea ? if not, it would be good material for
QEMU 8.1. Could you resend please ? (with the TOD patches)
Thanks,
C.
> Thanks,
>
> C.
>
>
>> +
>> +/* Most register and command definitions come from skiboot */
>> +
>> +/*
>> + * SBE MBOX register address
>> + * Reg 0 - 3 : Host to send command packets to SBE
>> + * Reg 4 - 7 : SBE to send response packets to Host
>> + */
>> +#define PSU_HOST_SBE_MBOX_REG0 0x00000000
>> +#define PSU_HOST_SBE_MBOX_REG1 0x00000001
>> +#define PSU_HOST_SBE_MBOX_REG2 0x00000002
>> +#define PSU_HOST_SBE_MBOX_REG3 0x00000003
>> +#define PSU_HOST_SBE_MBOX_REG4 0x00000004
>> +#define PSU_HOST_SBE_MBOX_REG5 0x00000005
>> +#define PSU_HOST_SBE_MBOX_REG6 0x00000006
>> +#define PSU_HOST_SBE_MBOX_REG7 0x00000007
>> +#define PSU_SBE_DOORBELL_REG_RW 0x00000010
>> +#define PSU_SBE_DOORBELL_REG_AND 0x00000011
>> +#define PSU_SBE_DOORBELL_REG_OR 0x00000012
>> +#define PSU_HOST_DOORBELL_REG_RW 0x00000013
>> +#define PSU_HOST_DOORBELL_REG_AND 0x00000014
>> +#define PSU_HOST_DOORBELL_REG_OR 0x00000015
>> +
>> +/*
>> + * Doorbell register to trigger SBE interrupt. Set by OPAL to inform
>> + * the SBE about a waiting message in the Host/SBE mailbox registers
>> + */
>> +#define HOST_SBE_MSG_WAITING PPC_BIT(0)
>> +
>> +/*
>> + * Doorbell register for host bridge interrupt. Set by the SBE to inform
>> + * host about a response message in the Host/SBE mailbox registers
>> + */
>> +#define SBE_HOST_RESPONSE_WAITING PPC_BIT(0)
>> +#define SBE_HOST_MSG_READ PPC_BIT(1)
>> +#define SBE_HOST_STOP15_EXIT PPC_BIT(2)
>> +#define SBE_HOST_RESET PPC_BIT(3)
>> +#define SBE_HOST_PASSTHROUGH PPC_BIT(4)
>> +#define SBE_HOST_TIMER_EXPIRY PPC_BIT(14)
>> +#define SBE_HOST_RESPONSE_MASK (PPC_BITMASK(0, 4) | SBE_HOST_TIMER_EXPIRY)
>> +
>> +/* SBE Control Register */
>> +#define SBE_CONTROL_REG_RW 0x00000000
>> +
>> +/* SBE interrupt s0/s1 bits */
>> +#define SBE_CONTROL_REG_S0 PPC_BIT(14)
>> +#define SBE_CONTROL_REG_S1 PPC_BIT(15)
>> +
>> +struct sbe_msg {
>> + uint64_t reg[4];
>> +};
>> +
>> +static uint64_t pnv_sbe_power9_xscom_ctrl_read(void *opaque, hwaddr addr,
>> + unsigned size)
>> +{
>> + uint32_t offset = addr >> 3;
>> + uint64_t val = 0;
>> +
>> + switch (offset) {
>> + default:
>> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
>> + HWADDR_PRIx "\n", addr >> 3);
>> + }
>> +
>> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
>> +
>> + return val;
>> +}
>> +
>> +static void pnv_sbe_power9_xscom_ctrl_write(void *opaque, hwaddr addr,
>> + uint64_t val, unsigned size)
>> +{
>> + uint32_t offset = addr >> 3;
>> +
>> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
>> +
>> + switch (offset) {
>> + default:
>> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
>> + HWADDR_PRIx "\n", addr >> 3);
>> + }
>> +}
>> +
>> +static const MemoryRegionOps pnv_sbe_power9_xscom_ctrl_ops = {
>> + .read = pnv_sbe_power9_xscom_ctrl_read,
>> + .write = pnv_sbe_power9_xscom_ctrl_write,
>> + .valid.min_access_size = 8,
>> + .valid.max_access_size = 8,
>> + .impl.min_access_size = 8,
>> + .impl.max_access_size = 8,
>> + .endianness = DEVICE_BIG_ENDIAN,
>> +};
>> +
>> +static void pnv_sbe_set_host_doorbell(PnvSBE *sbe, uint64_t val)
>> +{
>> + val &= SBE_HOST_RESPONSE_MASK; /* Is this right? What does HW do? */
>> + sbe->host_doorbell = val;
>> +
>> + sbe_reg_dbg("val:%lx\n", val);
>> + qemu_set_irq(sbe->psi_irq, !!val);
>> +}
>> +
>> +/* SBE Target Type */
>> +#define SBE_TARGET_TYPE_PROC 0x00
>> +#define SBE_TARGET_TYPE_EX 0x01
>> +#define SBE_TARGET_TYPE_PERV 0x02
>> +#define SBE_TARGET_TYPE_MCS 0x03
>> +#define SBE_TARGET_TYPE_EQ 0x04
>> +#define SBE_TARGET_TYPE_CORE 0x05
>> +
>> +/* SBE MBOX command class */
>> +#define SBE_MCLASS_FIRST 0xD1
>> +#define SBE_MCLASS_CORE_STATE 0xD1
>> +#define SBE_MCLASS_SCOM 0xD2
>> +#define SBE_MCLASS_RING 0xD3
>> +#define SBE_MCLASS_TIMER 0xD4
>> +#define SBE_MCLASS_MPIPL 0xD5
>> +#define SBE_MCLASS_SECURITY 0xD6
>> +#define SBE_MCLASS_GENERIC 0xD7
>> +#define SBE_MCLASS_LAST 0xD7
>> +
>> +/*
>> + * Commands are provided in xxyy form where:
>> + * - xx : command class
>> + * - yy : command
>> + *
>> + * Both request and response message uses same seq ID,
>> + * command class and command.
>> + */
>> +#define SBE_CMD_CTRL_DEADMAN_LOOP 0xD101
>> +#define SBE_CMD_MULTI_SCOM 0xD201
>> +#define SBE_CMD_PUT_RING_FORM_IMAGE 0xD301
>> +#define SBE_CMD_CONTROL_TIMER 0xD401
>> +#define SBE_CMD_GET_ARCHITECTED_REG 0xD501
>> +#define SBE_CMD_CLR_ARCHITECTED_REG 0xD502
>> +#define SBE_CMD_SET_UNSEC_MEM_WINDOW 0xD601
>> +#define SBE_CMD_GET_SBE_FFDC 0xD701
>> +#define SBE_CMD_GET_CAPABILITY 0xD702
>> +#define SBE_CMD_READ_SBE_SEEPROM 0xD703
>> +#define SBE_CMD_SET_FFDC_ADDR 0xD704
>> +#define SBE_CMD_QUIESCE_SBE 0xD705
>> +#define SBE_CMD_SET_FABRIC_ID_MAP 0xD706
>> +#define SBE_CMD_STASH_MPIPL_CONFIG 0xD707
>> +
>> +/* SBE MBOX control flags */
>> +
>> +/* Generic flags */
>> +#define SBE_CMD_CTRL_RESP_REQ 0x0100
>> +#define SBE_CMD_CTRL_ACK_REQ 0x0200
>> +
>> +/* Deadman loop */
>> +#define CTRL_DEADMAN_LOOP_START 0x0001
>> +#define CTRL_DEADMAN_LOOP_STOP 0x0002
>> +
>> +/* Control timer */
>> +#define CONTROL_TIMER_START 0x0001
>> +#define CONTROL_TIMER_STOP 0x0002
>> +
>> +/* Stash MPIPL config */
>> +#define SBE_STASH_KEY_SKIBOOT_BASE 0x03
>> +
>> +static void sbe_timer(void *opaque)
>> +{
>> + PnvSBE *sbe = opaque;
>> +
>> + sbe_cmd_dbg("timer expired\n");
>> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | SBE_HOST_TIMER_EXPIRY);
>> +}
>> +
>> +static void do_sbe_msg(PnvSBE *sbe)
>> +{
>> + struct sbe_msg msg;
>> + uint16_t cmd, ctrl_flags, seq_id;
>> + int i;
>> +
>> + memset(&msg, 0, sizeof(msg));
>> +
>> + for (i = 0; i < 4; i++) {
>> + msg.reg[i] = sbe->mbox[i];
>> + }
>> +
>> + cmd = msg.reg[0];
>> + seq_id = msg.reg[0] >> 16;
>> + ctrl_flags = msg.reg[0] >> 32;
>> + sbe_msg_dbg("cmd:%x seq:%d ctrlflags:%x\n", cmd, seq_id, ctrl_flags);
>> +
>> + if (ctrl_flags & SBE_CMD_CTRL_ACK_REQ) {
>> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | SBE_HOST_MSG_READ);
>> + }
>> +
>> + switch (cmd) {
>> + case SBE_CMD_CONTROL_TIMER:
>> + if (ctrl_flags & CONTROL_TIMER_START) {
>> + uint64_t us = msg.reg[1];
>> + sbe_cmd_dbg("timer start us:%lu\n", us);
>> + timer_mod(sbe->timer, qemu_clock_get_us(QEMU_CLOCK_VIRTUAL) + us);
>> + }
>> + if (ctrl_flags & CONTROL_TIMER_STOP) {
>> + sbe_cmd_dbg("timer stop\n");
>> + timer_del(sbe->timer);
>> + }
>> + break;
>> + default:
>> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented command: 0x%x\n", cmd);
>> + }
>> +}
>> +
>> +static void pnv_sbe_set_sbe_doorbell(PnvSBE *sbe, uint64_t val)
>> +{
>> + val &= HOST_SBE_MSG_WAITING;
>> + sbe->sbe_doorbell = val;
>> +
>> + if (val & HOST_SBE_MSG_WAITING) {
>> + sbe->sbe_doorbell &= ~HOST_SBE_MSG_WAITING;
>> + do_sbe_msg(sbe);
>> + }
>> +}
>> +
>> +static uint64_t pnv_sbe_power9_xscom_mbox_read(void *opaque, hwaddr addr,
>> + unsigned size)
>> +{
>> + PnvSBE *sbe = PNV_SBE(opaque);
>> + uint32_t offset = addr >> 3;
>> + uint64_t val = 0;
>> +
>> + if (offset <= PSU_HOST_SBE_MBOX_REG7) {
>> + uint32_t idx = offset - PSU_HOST_SBE_MBOX_REG0;
>> + val = sbe->mbox[idx];
>> + } else {
>> + switch (offset) {
>> + case PSU_SBE_DOORBELL_REG_RW:
>> + val = sbe->sbe_doorbell;
>> + break;
>> + case PSU_HOST_DOORBELL_REG_RW:
>> + val = sbe->host_doorbell;
>> + break;
>> + default:
>> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
>> + HWADDR_PRIx "\n", addr >> 3);
>> + }
>> + }
>> +
>> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
>> +
>> + return val;
>> +}
>> +
>> +static void pnv_sbe_power9_xscom_mbox_write(void *opaque, hwaddr addr,
>> + uint64_t val, unsigned size)
>> +{
>> + PnvSBE *sbe = PNV_SBE(opaque);
>> + uint32_t offset = addr >> 3;
>> +
>> + sbe_reg_dbg("addr:0x%"HWADDR_PRIx" offset:%x val:%lx\n", addr, offset, val);
>> +
>> + if (offset <= PSU_HOST_SBE_MBOX_REG7) {
>> + uint32_t idx = offset - PSU_HOST_SBE_MBOX_REG0;
>> + sbe->mbox[idx] = val;
>> + } else {
>> + switch (offset) {
>> + case PSU_SBE_DOORBELL_REG_RW:
>> + pnv_sbe_set_sbe_doorbell(sbe, val);
>> + break;
>> + case PSU_SBE_DOORBELL_REG_AND:
>> + pnv_sbe_set_sbe_doorbell(sbe, sbe->sbe_doorbell & val);
>> + break;
>> + case PSU_SBE_DOORBELL_REG_OR:
>> + pnv_sbe_set_sbe_doorbell(sbe, sbe->sbe_doorbell | val);
>> + break;
>> +
>> + case PSU_HOST_DOORBELL_REG_RW:
>> + pnv_sbe_set_host_doorbell(sbe, val);
>> + break;
>> + case PSU_HOST_DOORBELL_REG_AND:
>> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell & val);
>> + break;
>> + case PSU_HOST_DOORBELL_REG_OR:
>> + pnv_sbe_set_host_doorbell(sbe, sbe->host_doorbell | val);
>> + break;
>> +
>> + default:
>> + qemu_log_mask(LOG_UNIMP, "SBE Unimplemented register: Ox%"
>> + HWADDR_PRIx "\n", addr >> 3);
>> + }
>> + }
>> +}
>> +
>> +static const MemoryRegionOps pnv_sbe_power9_xscom_mbox_ops = {
>> + .read = pnv_sbe_power9_xscom_mbox_read,
>> + .write = pnv_sbe_power9_xscom_mbox_write,
>> + .valid.min_access_size = 8,
>> + .valid.max_access_size = 8,
>> + .impl.min_access_size = 8,
>> + .impl.max_access_size = 8,
>> + .endianness = DEVICE_BIG_ENDIAN,
>> +};
>> +
>> +static void pnv_sbe_power9_class_init(ObjectClass *klass, void *data)
>> +{
>> + PnvSBEClass *psc = PNV_SBE_CLASS(klass);
>> + DeviceClass *dc = DEVICE_CLASS(klass);
>> +
>> + dc->desc = "PowerNV SBE Controller (POWER9)";
>> + psc->xscom_ctrl_size = PNV9_XSCOM_SBE_CTRL_SIZE;
>> + psc->xscom_ctrl_ops = &pnv_sbe_power9_xscom_ctrl_ops;
>> + psc->xscom_mbox_size = PNV9_XSCOM_SBE_MBOX_SIZE;
>> + psc->xscom_mbox_ops = &pnv_sbe_power9_xscom_mbox_ops;
>> +}
>> +
>> +static const TypeInfo pnv_sbe_power9_type_info = {
>> + .name = TYPE_PNV9_SBE,
>> + .parent = TYPE_PNV_SBE,
>> + .instance_size = sizeof(PnvSBE),
>> + .class_init = pnv_sbe_power9_class_init,
>> +};
>> +
>> +static void pnv_sbe_power10_class_init(ObjectClass *klass, void *data)
>> +{
>> + PnvSBEClass *psc = PNV_SBE_CLASS(klass);
>> + DeviceClass *dc = DEVICE_CLASS(klass);
>> +
>> + dc->desc = "PowerNV SBE Controller (POWER10)";
>> + psc->xscom_ctrl_size = PNV10_XSCOM_SBE_CTRL_SIZE;
>> + psc->xscom_ctrl_ops = &pnv_sbe_power9_xscom_ctrl_ops;
>> + psc->xscom_mbox_size = PNV10_XSCOM_SBE_MBOX_SIZE;
>> + psc->xscom_mbox_ops = &pnv_sbe_power9_xscom_mbox_ops;
>> +}
>> +
>> +static const TypeInfo pnv_sbe_power10_type_info = {
>> + .name = TYPE_PNV10_SBE,
>> + .parent = TYPE_PNV9_SBE,
>> + .class_init = pnv_sbe_power10_class_init,
>> +};
>> +
>> +static void pnv_sbe_realize(DeviceState *dev, Error **errp)
>> +{
>> + PnvSBE *sbe = PNV_SBE(dev);
>> + PnvSBEClass *psc = PNV_SBE_GET_CLASS(sbe);
>> +
>> + /* XScom regions for SBE registers */
>> + pnv_xscom_region_init(&sbe->xscom_ctrl_regs, OBJECT(dev),
>> + psc->xscom_ctrl_ops, sbe, "xscom-sbe-ctrl",
>> + psc->xscom_ctrl_size);
>> + pnv_xscom_region_init(&sbe->xscom_mbox_regs, OBJECT(dev),
>> + psc->xscom_mbox_ops, sbe, "xscom-sbe-mbox",
>> + psc->xscom_mbox_size);
>> +
>> + qdev_init_gpio_out(DEVICE(dev), &sbe->psi_irq, 1);
>> +
>> + sbe->timer = timer_new_us(QEMU_CLOCK_VIRTUAL, sbe_timer, sbe);
>> +}
>> +
>> +static void pnv_sbe_class_init(ObjectClass *klass, void *data)
>> +{
>> + DeviceClass *dc = DEVICE_CLASS(klass);
>> +
>> + dc->realize = pnv_sbe_realize;
>> + dc->desc = "PowerNV SBE Controller";
>> + dc->user_creatable = false;
>> +}
>> +
>> +static const TypeInfo pnv_sbe_type_info = {
>> + .name = TYPE_PNV_SBE,
>> + .parent = TYPE_DEVICE,
>> + .instance_size = sizeof(PnvSBE),
>> + .class_init = pnv_sbe_class_init,
>> + .class_size = sizeof(PnvSBEClass),
>> + .abstract = true,
>> +};
>> +
>> +static void pnv_sbe_register_types(void)
>> +{
>> + type_register_static(&pnv_sbe_type_info);
>> + type_register_static(&pnv_sbe_power9_type_info);
>> + type_register_static(&pnv_sbe_power10_type_info);
>> +}
>> +
>> +type_init(pnv_sbe_register_types);
>> diff --git a/hw/ppc/pnv_xscom.c b/hw/ppc/pnv_xscom.c
>> index 9ce018dbc2..79f10de57f 100644
>> --- a/hw/ppc/pnv_xscom.c
>> +++ b/hw/ppc/pnv_xscom.c
>> @@ -295,6 +295,9 @@ int pnv_dt_xscom(PnvChip *chip, void *fdt, int root_offset,
>> _FDT((fdt_setprop(fdt, xscom_offset, "reg", reg, sizeof(reg))));
>> _FDT((fdt_setprop(fdt, xscom_offset, "compatible", compat, compat_size)));
>> _FDT((fdt_setprop(fdt, xscom_offset, "scom-controller", NULL, 0)));
>> + if (chip->chip_id == 0) {
>> + _FDT((fdt_setprop(fdt, xscom_offset, "primary", NULL, 0)));
>> + }
>> args.fdt = fdt;
>> args.xscom_offset = xscom_offset;
>> diff --git a/include/hw/ppc/pnv.h b/include/hw/ppc/pnv.h
>> index b991194223..37c303bf36 100644
>> --- a/include/hw/ppc/pnv.h
>> +++ b/include/hw/ppc/pnv.h
>> @@ -27,6 +27,7 @@
>> #include "hw/ppc/pnv_pnor.h"
>> #include "hw/ppc/pnv_psi.h"
>> #include "hw/ppc/pnv_occ.h"
>> +#include "hw/ppc/pnv_sbe.h"
>> #include "hw/ppc/pnv_homer.h"
>> #include "hw/ppc/pnv_xive.h"
>> #include "hw/ppc/pnv_core.h"
>> @@ -100,6 +101,7 @@ struct Pnv9Chip {
>> Pnv9Psi psi;
>> PnvLpcController lpc;
>> PnvOCC occ;
>> + PnvSBE sbe;
>> PnvHomer homer;
>> uint32_t nr_quads;
>> @@ -129,6 +131,7 @@ struct Pnv10Chip {
>> Pnv9Psi psi;
>> PnvLpcController lpc;
>> PnvOCC occ;
>> + PnvSBE sbe;
>> PnvHomer homer;
>> uint32_t nr_quads;
>> diff --git a/include/hw/ppc/pnv_sbe.h b/include/hw/ppc/pnv_sbe.h
>> new file mode 100644
>> index 0000000000..f54a3ae9ba
>> --- /dev/null
>> +++ b/include/hw/ppc/pnv_sbe.h
>> @@ -0,0 +1,55 @@
>> +/*
>> + * QEMU PowerPC PowerNV Emulation of some SBE behaviour
>> + *
>> + * Copyright (c) 2022, IBM Corporation.
>> + *
>> + * This library is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU Lesser General Public
>> + * License as published by the Free Software Foundation; either
>> + * version 2.1 of the License, or (at your option) any later version.
>> + *
>> + * This library is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>> + * Lesser General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU Lesser General Public
>> + * License along with this library; if not, see <http://www.gnu.org/licenses/>.
>> + */
>> +
>> +#ifndef PPC_PNV_SBE_H
>> +#define PPC_PNV_SBE_H
>> +
>> +#include "qom/object.h"
>> +
>> +#define TYPE_PNV_SBE "pnv-sbe"
>> +OBJECT_DECLARE_TYPE(PnvSBE, PnvSBEClass, PNV_SBE)
>> +#define TYPE_PNV9_SBE TYPE_PNV_SBE "-POWER9"
>> +DECLARE_INSTANCE_CHECKER(PnvSBE, PNV9_SBE, TYPE_PNV9_SBE)
>> +#define TYPE_PNV10_SBE TYPE_PNV_SBE "-POWER10"
>> +DECLARE_INSTANCE_CHECKER(PnvSBE, PNV10_SBE, TYPE_PNV10_SBE)
>> +
>> +struct PnvSBE {
>> + DeviceState xd;
>> +
>> + uint64_t mbox[8];
>> + uint64_t sbe_doorbell;
>> + uint64_t host_doorbell;
>> +
>> + qemu_irq psi_irq;
>> + QEMUTimer *timer;
>> +
>> + MemoryRegion xscom_mbox_regs;
>> + MemoryRegion xscom_ctrl_regs;
>> +};
>> +
>> +struct PnvSBEClass {
>> + DeviceClass parent_class;
>> +
>> + int xscom_ctrl_size;
>> + int xscom_mbox_size;
>> + const MemoryRegionOps *xscom_ctrl_ops;
>> + const MemoryRegionOps *xscom_mbox_ops;
>> +};
>> +
>> +#endif /* PPC_PNV_SBE_H */
>> diff --git a/include/hw/ppc/pnv_xscom.h b/include/hw/ppc/pnv_xscom.h
>> index 7c7440de0c..c6e9ef8dd2 100644
>> --- a/include/hw/ppc/pnv_xscom.h
>> +++ b/include/hw/ppc/pnv_xscom.h
>> @@ -92,6 +92,12 @@ struct PnvXScomInterfaceClass {
>> #define PNV9_XSCOM_OCC_BASE PNV_XSCOM_OCC_BASE
>> #define PNV9_XSCOM_OCC_SIZE 0x8000
>> +#define PNV9_XSCOM_SBE_CTRL_BASE 0x00050008
>> +#define PNV9_XSCOM_SBE_CTRL_SIZE 0x1
>> +
>> +#define PNV9_XSCOM_SBE_MBOX_BASE 0x000D0050
>> +#define PNV9_XSCOM_SBE_MBOX_SIZE 0x16
>> +
>> #define PNV9_XSCOM_PBA_BASE 0x5012b00
>> #define PNV9_XSCOM_PBA_SIZE 0x40
>> @@ -134,6 +140,12 @@ struct PnvXScomInterfaceClass {
>> #define PNV10_XSCOM_OCC_BASE PNV9_XSCOM_OCC_BASE
>> #define PNV10_XSCOM_OCC_SIZE PNV9_XSCOM_OCC_SIZE
>> +#define PNV10_XSCOM_SBE_CTRL_BASE PNV9_XSCOM_SBE_CTRL_BASE
>> +#define PNV10_XSCOM_SBE_CTRL_SIZE PNV9_XSCOM_SBE_CTRL_SIZE
>> +
>> +#define PNV10_XSCOM_SBE_MBOX_BASE PNV9_XSCOM_SBE_MBOX_BASE
>> +#define PNV10_XSCOM_SBE_MBOX_SIZE PNV9_XSCOM_SBE_MBOX_SIZE
>> +
>> #define PNV10_XSCOM_PBA_BASE 0x01010CDA
>> #define PNV10_XSCOM_PBA_SIZE 0x40
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ppc/pnv: Add initial P9/10 SBE model
2023-05-26 12:20 ` Cédric Le Goater
@ 2023-05-29 1:51 ` Nicholas Piggin
0 siblings, 0 replies; 4+ messages in thread
From: Nicholas Piggin @ 2023-05-29 1:51 UTC (permalink / raw)
To: Cédric Le Goater, qemu-ppc; +Cc: qemu-devel, Frederic Barrat
On Fri May 26, 2023 at 10:20 PM AEST, Cédric Le Goater wrote:
> On 8/11/22 10:04, Cédric Le Goater wrote:
> > On 8/11/22 09:54, Nicholas Piggin wrote:
[snip]
> >> +
> >> +#define sbe_cmd_dbg(fmt, ...) do { \
> >> + if (DEBUG_SBE_CMD) { \
> >> + fprintf(stderr, "SBE cmd: %s: " fmt, __func__, ## __VA_ARGS__); \
> >> + } \
> >> +} while (0)
> >
> > Please use trace events instead, with a pnv_sbe_ prefix.
> >
> > Apart from that, LGTM !
> >
> > Reviewed-by: Cédric Le Goater <clg@kaod.org>
>
>
>
>
> This looked like an interesting extension to the baremetal models.
> Did you abandon the idea ? if not, it would be good material for
> QEMU 8.1. Could you resend please ? (with the TOD patches)
Hey Cedric,
Yeah I kind of forgot about these for a while, sorry :( Too much
stuff going on... But by an amazing coincidence I have just started to
look at these again in the past few days, so I should try to get it
merged.
Thanks,
Nick
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-05-29 1:51 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-11 7:54 [PATCH] ppc/pnv: Add initial P9/10 SBE model Nicholas Piggin
2022-08-11 8:04 ` Cédric Le Goater
2023-05-26 12:20 ` Cédric Le Goater
2023-05-29 1:51 ` Nicholas Piggin
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.