From: Thomas Huth <thuth@redhat.com>
To: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, qemu-devel@nongnu.org
Cc: qemu-trivial@nongnu.org, Markus Armbruster <armbru@redhat.com>,
Artyom Tarasenko <atar4qemu@gmail.com>
Subject: [PATCH] hw/sparc64/sun4u_iommu: Fix crash when introspecting sun4u-iommu from the CLI
Date: Tue, 17 Mar 2026 09:58:39 +0100 [thread overview]
Message-ID: <20260317085839.445178-1-thuth@redhat.com> (raw)
From: Thomas Huth <thuth@redhat.com>
QEMU currently crashes when introspecting the sun4u-iommu device from the
command line interface:
$ ./qemu-system-sparc64 -display none -device sun4u-iommu,help
qemu-system-sparc64: ../../devel/qemu/system/physmem.c:1401:
register_multipage: Assertion `num_pages' failed.
Aborted (core dumped)
There does not seem to be a compelling reason for initializing the
memory regions from the instance_init function, so let's simply move
the code into a realize() function instead to fix this issue.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/sparc64/sun4u_iommu.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/hw/sparc64/sun4u_iommu.c b/hw/sparc64/sun4u_iommu.c
index 0188ce35d29..fe9d47b822f 100644
--- a/hw/sparc64/sun4u_iommu.c
+++ b/hw/sparc64/sun4u_iommu.c
@@ -290,13 +290,14 @@ static void iommu_reset(DeviceState *d)
memset(s->regs, 0, IOMMU_NREGS * sizeof(uint64_t));
}
-static void iommu_init(Object *obj)
+static void iommu_realize(DeviceState *ds, Error **errp)
{
- IOMMUState *s = SUN4U_IOMMU(obj);
- SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
+ IOMMUState *s = SUN4U_IOMMU(ds);
+ SysBusDevice *sbd = SYS_BUS_DEVICE(ds);
+ Object *obj = OBJECT(ds);
memory_region_init_iommu(&s->iommu, sizeof(s->iommu),
- TYPE_SUN4U_IOMMU_MEMORY_REGION, OBJECT(s),
+ TYPE_SUN4U_IOMMU_MEMORY_REGION, obj,
"iommu-sun4u", UINT64_MAX);
address_space_init(&s->iommu_as, MEMORY_REGION(&s->iommu), "iommu-as");
@@ -310,13 +311,13 @@ static void iommu_class_init(ObjectClass *klass, const void *data)
DeviceClass *dc = DEVICE_CLASS(klass);
device_class_set_legacy_reset(dc, iommu_reset);
+ dc->realize = iommu_realize;
}
static const TypeInfo iommu_info = {
.name = TYPE_SUN4U_IOMMU,
.parent = TYPE_SYS_BUS_DEVICE,
.instance_size = sizeof(IOMMUState),
- .instance_init = iommu_init,
.class_init = iommu_class_init,
};
--
2.53.0
next reply other threads:[~2026-03-17 8:59 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-17 8:58 Thomas Huth [this message]
2026-03-19 11:21 ` [PATCH] hw/sparc64/sun4u_iommu: Fix crash when introspecting sun4u-iommu from the CLI Markus Armbruster
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=20260317085839.445178-1-thuth@redhat.com \
--to=thuth@redhat.com \
--cc=armbru@redhat.com \
--cc=atar4qemu@gmail.com \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=qemu-devel@nongnu.org \
--cc=qemu-trivial@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is 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.