[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/5] hw/alpha/dp264: Fix wiring of PIC -> CPU interrupt
From: |
Bernhard Beschow |
Subject: |
[PATCH 2/5] hw/alpha/dp264: Fix wiring of PIC -> CPU interrupt |
Date: |
Sat, 4 Mar 2023 12:40:40 +0100 |
Commit cef2e7148e32 ("hw/isa/i82378: Remove intermediate IRQ forwarder")
passes s->cpu_intr to i8259_init() in i82378_realize() directly. However, s-
>cpu_intr isn't initialized yet since that happens after the south bridge's
pci_realize_and_unref() in board code. Fix this by initializing s->cpu_intr
before realizing the south bridge.
Fixes: cef2e7148e32 ("hw/isa/i82378: Remove intermediate IRQ forwarder")
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
hw/alpha/dp264.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/hw/alpha/dp264.c b/hw/alpha/dp264.c
index 4161f559a7..e92295ac86 100644
--- a/hw/alpha/dp264.c
+++ b/hw/alpha/dp264.c
@@ -16,6 +16,7 @@
#include "hw/ide/pci.h"
#include "hw/isa/superio.h"
#include "net/net.h"
+#include "qapi/error.h"
#include "qemu/cutils.h"
#include "qemu/datadir.h"
@@ -110,11 +111,12 @@ static void clipper_init(MachineState *machine)
* Importantly, we need to provide a PCI device node for it, otherwise
* some operating systems won't notice there's an ISA bus to configure.
*/
- i82378_dev = DEVICE(pci_create_simple(pci_bus, PCI_DEVFN(7, 0), "i82378"));
- isa_bus = ISA_BUS(qdev_get_child_bus(i82378_dev, "isa.0"));
-
+ i82378_dev = DEVICE(pci_new(PCI_DEVFN(7, 0), "i82378"));
/* Connect the ISA PIC to the Typhoon IRQ used for ISA interrupts. */
qdev_connect_gpio_out(i82378_dev, 0, isa_irq);
+ qdev_realize_and_unref(i82378_dev, BUS(pci_bus), &error_fatal);
+
+ isa_bus = ISA_BUS(qdev_get_child_bus(i82378_dev, "isa.0"));
/* Since we have an SRM-compatible PALcode, use the SRM epoch. */
mc146818_rtc_init(isa_bus, 1900, rtc_irq);
--
2.39.2
- [PATCH 0/5] Fix recent PIC -> CPU interrupt wiring regressions, Bernhard Beschow, 2023/03/04
- [PATCH 1/5] hw/isa/vt82c686: Fix wiring of PIC -> CPU interrupt, Bernhard Beschow, 2023/03/04
- [PATCH 3/5] hw/ppc/prep: Fix wiring of PIC -> CPU interrupt, Bernhard Beschow, 2023/03/04
- [PATCH 2/5] hw/alpha/dp264: Fix wiring of PIC -> CPU interrupt,
Bernhard Beschow <=
- [PATCH 4/5] hw/pci/pci: Remove multifunction parameter from pci_create_simple_multifunction(), Bernhard Beschow, 2023/03/04
- [PATCH 5/5] hw/pci/pci: Remove multifunction parameter from pci_new_multifunction(), Bernhard Beschow, 2023/03/04
- Re: [PATCH 0/5] Fix recent PIC -> CPU interrupt wiring regressions, Bernhard Beschow, 2023/03/04
- Re: [PATCH 0/5] Fix recent PIC -> CPU interrupt wiring regressions, BALATON Zoltan, 2023/03/04
- Re: [PATCH 0/5] Fix recent PIC -> CPU interrupt wiring regressions, Michael S. Tsirkin, 2023/03/05
- Re: [PATCH 0/5] Fix recent PIC -> CPU interrupt wiring regressions, Mark Cave-Ayland, 2023/03/06
- Re: [PATCH 0/5] Fix recent PIC -> CPU interrupt wiring regressions, Michael S. Tsirkin, 2023/03/07