From: Thomas Huth Date: Thu, 14 Nov 2024 10:46:53 +0000 (+0100) Subject: hw/misc/mos6522: Fix bad class definition of the MOS6522 device X-Git-Tag: v9.2.0-rc1~8^2~5 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c3d7c18b0d616cf7fb3c1f325503e1462307209d;p=thirdparty%2Fqemu.git hw/misc/mos6522: Fix bad class definition of the MOS6522 device When compiling QEMU with --enable-cfi, the "q800" m68k machine currently crashes very early, when the q800_machine_init() function tries to wire the interrupts of the "via1" device. This happens because TYPE_MOS6522_Q800_VIA1 is supposed to be a proper SysBus device, but its parent (TYPE_MOS6522) has a mistake in its class definition where it is only derived from DeviceClass, and not from SysBusDeviceClass, so we end up in funny memory access issues here. Using the right class hierarchy for the MOS6522 device fixes the problem. Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2675 Signed-off-by: Thomas Huth Fixes: 51f233ec92 ("misc: introduce new mos6522 VIA device") Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland Message-ID: <20241114104653.963812-1-thuth@redhat.com> Signed-off-by: Philippe Mathieu-Daudé --- diff --git a/include/hw/misc/mos6522.h b/include/hw/misc/mos6522.h index fba45668aba..920871a598b 100644 --- a/include/hw/misc/mos6522.h +++ b/include/hw/misc/mos6522.h @@ -154,7 +154,7 @@ struct MOS6522State { OBJECT_DECLARE_TYPE(MOS6522State, MOS6522DeviceClass, MOS6522) struct MOS6522DeviceClass { - DeviceClass parent_class; + SysBusDeviceClass parent_class; ResettablePhases parent_phases; void (*portB_write)(MOS6522State *dev);