Series comparison

-[PATCH v3 0/2] AHCI cleanup
+[PATCH v4 0/2] AHCI cleanup
 This series fixes some runtime overhead when handling interrupts in AHCIState.
 It then extracts the SysBus implementation into a dedicated file for separation
 of concerns.
+v4:
+* Port away from qemu_allocate_irq() while at it
 v3:
 * Remove extra PCI include in ahci-internal.h
 * Extract SysBus implementation into dedicated file
 Supersedes: 20241205114453.1848-1-shentey@gmail.com
 Bernhard Beschow (2):
   hw/ide/ahci: Decouple from PCI
   hw/ide/ahci: Extract TYPE_SYSBUS_AHCI into dedicated file
- hw/ide/ahci-internal.h |   1 -
+ hw/ide/ahci-internal.h    |   1 -
- include/hw/ide/ahci.h  |   2 -
+ include/hw/ide/ahci-pci.h |   2 +
- hw/ide/ahci-sysbus.c   |  91 +++++++++++++++++++++++++++++++++++
+ include/hw/ide/ahci.h     |   2 -
- hw/ide/ahci.c          | 106 ++---------------------------------------
+ hw/ide/ahci-sysbus.c      |  91 ++++++++++++++++++++++++++++++++
- hw/ide/ich.c           |  17 ++++++-
+ hw/ide/ahci.c             | 106 ++------------------------------------
- hw/arm/Kconfig         |  10 ++--
+ hw/ide/ich.c              |  19 +++++--
- hw/ide/Kconfig         |   4 ++
+ hw/arm/Kconfig            |  10 ++--
- hw/ide/meson.build     |   1 +
+ hw/ide/Kconfig            |   4 ++
-files changed, 121 insertions(+), 111 deletions(-)
+ hw/ide/meson.build        |   1 +
 files changed, 122 insertions(+), 114 deletions(-)
  create mode 100644 hw/ide/ahci-sysbus.c
 --
 .47.1

-[PATCH v3 1/2] hw/ide/ahci: Decouple from PCI
+[PATCH v4 1/2] hw/ide/ahci: Decouple from PCI
 ...
 reflected in Kconfig.
 Reported-by: Peter Xu <peterx@redhat.com>
 Inspired-by: Philippe Mathieu-Daudé <philmd@linaro.org>
 Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
-Supersedes: 20241121100152.65476-1-philmd@linaro.org
 ---
- hw/ide/ahci-internal.h |  1 -
+ hw/ide/ahci-internal.h    |  1 -
- include/hw/ide/ahci.h  |  2 --
+ include/hw/ide/ahci-pci.h |  2 ++
- hw/ide/ahci.c          | 39 ++++-----------------------------------
+ include/hw/ide/ahci.h     |  2 --
- hw/ide/ich.c           | 17 ++++++++++++++++-
+ hw/ide/ahci.c             | 39 ++++-----------------------------------
-files changed, 20 insertions(+), 39 deletions(-)
+ hw/ide/ich.c              | 19 +++++++++++++++----
 files changed, 21 insertions(+), 42 deletions(-)
 diff --git a/hw/ide/ahci-internal.h b/hw/ide/ahci-internal.h
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/ide/ahci-internal.h
 +++ b/hw/ide/ahci-internal.h
 ...
  #include "hw/ide/ahci.h"
 -#include "hw/pci/pci_device.h"
  #include "ide-internal.h"
  #define AHCI_MEM_BAR_SIZE         0x1000
+diff --git a/include/hw/ide/ahci-pci.h b/include/hw/ide/ahci-pci.h
+index XXXXXXX..XXXXXXX 100644
+--- a/include/hw/ide/ahci-pci.h
++++ b/include/hw/ide/ahci-pci.h
+@@ -XXX,XX +XXX,XX @@
+ #include "qom/object.h"
+ #include "hw/ide/ahci.h"
+ #include "hw/pci/pci_device.h"
++#include "hw/irq.h"
+ #define TYPE_ICH9_AHCI "ich9-ahci"
+ OBJECT_DECLARE_SIMPLE_TYPE(AHCIPCIState, ICH9_AHCI)
+@@ -XXX,XX +XXX,XX @@ struct AHCIPCIState {
+     PCIDevice parent_obj;
+     AHCIState ahci;
++    IRQState irq;
+ };
+ #endif
 diff --git a/include/hw/ide/ahci.h b/include/hw/ide/ahci.h
 index XXXXXXX..XXXXXXX 100644
 --- a/include/hw/ide/ahci.h
 +++ b/include/hw/ide/ahci.h
 @@ -XXX,XX +XXX,XX @@ typedef struct AHCIControlRegs {
 ...
                            "ahci", AHCI_MEM_BAR_SIZE);
 diff --git a/hw/ide/ich.c b/hw/ide/ich.c
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/ide/ich.c
 +++ b/hw/ide/ich.c
+@@ -XXX,XX +XXX,XX @@
+  */
+ #include "qemu/osdep.h"
+-#include "hw/irq.h"
+ #include "hw/pci/msi.h"
+ #include "hw/pci/pci.h"
+ #include "migration/vmstate.h"
 @@ -XXX,XX +XXX,XX @@ static const VMStateDescription vmstate_ich9_ahci = {
      },
  };
 +static void pci_ich9_ahci_update_irq(void *opaque, int irq_num, int level)
 +{
 +    PCIDevice *pci_dev = opaque;
-+
-+    assert(irq_num == 0);
 +
 +    if (msi_enabled(pci_dev)) {
 +        if (level) {
 +            msi_notify(pci_dev, 0);
 +        }
 ...
 +}
 +
  static void pci_ich9_reset(DeviceState *dev)
  {
      AHCIPCIState *d = ICH9_AHCI(dev);
+@@ -XXX,XX +XXX,XX @@ static void pci_ich9_ahci_init(Object *obj)
+ {
+     AHCIPCIState *d = ICH9_AHCI(obj);
++    qemu_init_irq(&d->irq, pci_ich9_ahci_update_irq, d, 0);
+     ahci_init(&d->ahci, DEVICE(obj));
++    d->ahci.irq = &d->irq;
+ }
+ static void pci_ich9_ahci_realize(PCIDevice *dev, Error **errp)
 @@ -XXX,XX +XXX,XX @@ static void pci_ich9_ahci_realize(PCIDevice *dev, Error **errp)
      /* XXX Software should program this register */
      dev->config[0x90]   = 1 << 6; /* Address Map Register - AHCI mode */
 -    d->ahci.irq = pci_allocate_irq(dev);
-+    d->ahci.irq = qemu_allocate_irq(pci_ich9_ahci_update_irq, d, 0);
+-
      pci_register_bar(dev, ICH9_IDP_BAR, PCI_BASE_ADDRESS_SPACE_IO,
                       &d->ahci.idp);
+     pci_register_bar(dev, ICH9_MEM_BAR, PCI_BASE_ADDRESS_SPACE_MEMORY,
+@@ -XXX,XX +XXX,XX @@ static void pci_ich9_uninit(PCIDevice *dev)
+     msi_uninit(dev);
+     ahci_uninit(&d->ahci);
+-    qemu_free_irq(d->ahci.irq);
+ }
+ static void ich_ahci_class_init(ObjectClass *klass, void *data)
 --
 .47.1

-[PATCH v3 2/2] hw/ide/ahci: Extract TYPE_SYSBUS_AHCI into dedicated file
+[PATCH v4 2/2] hw/ide/ahci: Extract TYPE_SYSBUS_AHCI into dedicated file
 Implement in dedicated file, just like TYPE_ICH9_AHCI.
 Signed-off-by: Bernhard Beschow <shentey@gmail.com>
 ---
  hw/ide/ahci-sysbus.c | 91 ++++++++++++++++++++++++++++++++++++++++++++
  hw/ide/ahci.c        | 67 --------------------------------
  hw/arm/Kconfig       | 10 ++---
  hw/ide/Kconfig       |  4 ++
  hw/ide/meson.build   |  1 +
 files changed, 101 insertions(+), 72 deletions(-)
  create mode 100644 hw/ide/ahci-sysbus.c
 diff --git a/hw/ide/ahci-sysbus.c b/hw/ide/ahci-sysbus.c
 new file mode 100644
 index XXXXXXX..XXXXXXX
 --- /dev/null
 +++ b/hw/ide/ahci-sysbus.c
 @@ -XXX,XX +XXX,XX @@
 +/*
 + * QEMU AHCI Emulation (MMIO-mapped devices)
 + *
 + * Copyright (c) 2010 qiaochong@loongson.cn
 + * Copyright (c) 2010 Roland Elek <elek.roland@gmail.com>
 + * Copyright (c) 2010 Sebastian Herbszt <herbszt@gmx.de>
 + * Copyright (c) 2010 Alexander Graf <agraf@suse.de>
 + *
 + * 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/>.
 + *
 + */
 +
 +#include "qemu/osdep.h"
 +#include "exec/address-spaces.h"
 +#include "hw/qdev-properties.h"
 +#include "migration/vmstate.h"
 +
 +#include "hw/ide/ahci-sysbus.h"
 +#include "ahci-internal.h"
 +
 +static const VMStateDescription vmstate_sysbus_ahci = {
 +    .name = "sysbus-ahci",
 +    .fields = (const VMStateField[]) {
 +        VMSTATE_AHCI(ahci, SysbusAHCIState),
 +        VMSTATE_END_OF_LIST()
 +    },
 +};
 +
 +static void sysbus_ahci_reset(DeviceState *dev)
 +{
 +    SysbusAHCIState *s = SYSBUS_AHCI(dev);
 +
 +    ahci_reset(&s->ahci);
 +}
 +
 +static void sysbus_ahci_init(Object *obj)
 +{
 +    SysbusAHCIState *s = SYSBUS_AHCI(obj);
 +    SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
 +
 +    ahci_init(&s->ahci, DEVICE(obj));
 +
 +    sysbus_init_mmio(sbd, &s->ahci.mem);
 +    sysbus_init_irq(sbd, &s->ahci.irq);
 +}
 +
 +static void sysbus_ahci_realize(DeviceState *dev, Error **errp)
 +{
 +    SysbusAHCIState *s = SYSBUS_AHCI(dev);
 +
 +    ahci_realize(&s->ahci, dev, &address_space_memory);
 +}
 +
 +static Property sysbus_ahci_properties[] = {
 +    DEFINE_PROP_UINT32("num-ports", SysbusAHCIState, ahci.ports, 1),
 +    DEFINE_PROP_END_OF_LIST(),
 +};
 +
 +static void sysbus_ahci_class_init(ObjectClass *klass, void *data)
 +{
 +    DeviceClass *dc = DEVICE_CLASS(klass);
 +
 +    dc->realize = sysbus_ahci_realize;
 +    dc->vmsd = &vmstate_sysbus_ahci;
 +    device_class_set_props(dc, sysbus_ahci_properties);
 +    device_class_set_legacy_reset(dc, sysbus_ahci_reset);
 +    set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
 +}
 +
 +static const TypeInfo sysbus_ahci_types[] = {
 +    {
 +        .name          = TYPE_SYSBUS_AHCI,
 +        .parent        = TYPE_SYS_BUS_DEVICE,
 +        .instance_size = sizeof(SysbusAHCIState),
 +        .instance_init = sysbus_ahci_init,
 +        .class_init    = sysbus_ahci_class_init,
 +    },
 +};
 +
 +DEFINE_TYPES(sysbus_ahci_types)
 diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/ide/ahci.c
 +++ b/hw/ide/ahci.c
 @@ -XXX,XX +XXX,XX @@
  #include "qemu/osdep.h"
  #include "hw/irq.h"
 -#include "hw/qdev-properties.h"
  #include "migration/vmstate.h"
  #include "qemu/error-report.h"
  #include "qemu/log.h"
  #include "qemu/main-loop.h"
 -#include "qemu/module.h"
  #include "sysemu/block-backend.h"
  #include "sysemu/dma.h"
 -#include "hw/ide/ahci-sysbus.h"
  #include "ahci-internal.h"
  #include "ide-internal.h"
 @@ -XXX,XX +XXX,XX @@ const VMStateDescription vmstate_ahci = {
      },
  };
 -static const VMStateDescription vmstate_sysbus_ahci = {
 -    .name = "sysbus-ahci",
 -    .fields = (const VMStateField[]) {
 -        VMSTATE_AHCI(ahci, SysbusAHCIState),
 -        VMSTATE_END_OF_LIST()
 -    },
 -};
 -
 -static void sysbus_ahci_reset(DeviceState *dev)
 -{
 -    SysbusAHCIState *s = SYSBUS_AHCI(dev);
 -
 -    ahci_reset(&s->ahci);
 -}
 -
 -static void sysbus_ahci_init(Object *obj)
 -{
 -    SysbusAHCIState *s = SYSBUS_AHCI(obj);
 -    SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
 -
 -    ahci_init(&s->ahci, DEVICE(obj));
 -
 -    sysbus_init_mmio(sbd, &s->ahci.mem);
 -    sysbus_init_irq(sbd, &s->ahci.irq);
 -}
 -
 -static void sysbus_ahci_realize(DeviceState *dev, Error **errp)
 -{
 -    SysbusAHCIState *s = SYSBUS_AHCI(dev);
 -
 -    ahci_realize(&s->ahci, dev, &address_space_memory);
 -}
 -
 -static Property sysbus_ahci_properties[] = {
 -    DEFINE_PROP_UINT32("num-ports", SysbusAHCIState, ahci.ports, 1),
 -    DEFINE_PROP_END_OF_LIST(),
 -};
 -
 -static void sysbus_ahci_class_init(ObjectClass *klass, void *data)
 -{
 -    DeviceClass *dc = DEVICE_CLASS(klass);
 -
 -    dc->realize = sysbus_ahci_realize;
 -    dc->vmsd = &vmstate_sysbus_ahci;
 -    device_class_set_props(dc, sysbus_ahci_properties);
 -    device_class_set_legacy_reset(dc, sysbus_ahci_reset);
 -    set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
 -}
 -
 -static const TypeInfo sysbus_ahci_info = {
 -    .name          = TYPE_SYSBUS_AHCI,
 -    .parent        = TYPE_SYS_BUS_DEVICE,
 -    .instance_size = sizeof(SysbusAHCIState),
 -    .instance_init = sysbus_ahci_init,
 -    .class_init    = sysbus_ahci_class_init,
 -};
 -
 -static void sysbus_ahci_register_types(void)
 -{
 -    type_register_static(&sysbus_ahci_info);
 -}
 -
 -type_init(sysbus_ahci_register_types)
 -
  void ahci_ide_create_devs(AHCIState *ahci, DriveInfo **hd)
  {
      int i;
 diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/arm/Kconfig
 +++ b/hw/arm/Kconfig
 @@ -XXX,XX +XXX,XX @@ config HIGHBANK
      depends on TCG && ARM
      select A9MPCORE
      select A15MPCORE
 -    select AHCI
 +    select AHCI_SYSBUS
      select ARM_TIMER # sp804
      select ARM_V7M
      select PL011 if !HAVE_RUST # UART
 @@ -XXX,XX +XXX,XX @@ config SBSA_REF
      depends on TCG && AARCH64
      imply PCI_DEVICES
      select DEVICE_TREE
 -    select AHCI
 +    select AHCI_SYSBUS
      select ARM_SMMUV3
      select GPIO_KEY
      select PCI_EXPRESS
 @@ -XXX,XX +XXX,XX @@ config ARM_V7M
  config ALLWINNER_A10
      bool
 -    select AHCI
 +    select AHCI_SYSBUS
      select ALLWINNER_A10_PIT
      select ALLWINNER_A10_PIC
      select ALLWINNER_A10_CCM
 @@ -XXX,XX +XXX,XX @@ config ALLWINNER_H3
  config ALLWINNER_R40
      bool
      default y if TCG && ARM
 -    select AHCI
 +    select AHCI_SYSBUS
      select ALLWINNER_SRAMC
      select ALLWINNER_A10_PIT
      select ALLWINNER_WDT
 @@ -XXX,XX +XXX,XX @@ config XLNX_ZYNQMP_ARM
      bool
      default y if PIXMAN
      depends on TCG && AARCH64
 -    select AHCI
 +    select AHCI_SYSBUS
      select ARM_GIC
      select CADENCE
      select CPU_CLUSTER
 diff --git a/hw/ide/Kconfig b/hw/ide/Kconfig
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/ide/Kconfig
 +++ b/hw/ide/Kconfig
 @@ -XXX,XX +XXX,XX @@ config AHCI_ICH9
      depends on PCI
      select AHCI
 +config AHCI_SYSBUS
 +    bool
 +    select AHCI
 +
  config IDE_SII3112
      bool
      select IDE_PCI
 diff --git a/hw/ide/meson.build b/hw/ide/meson.build
 index XXXXXXX..XXXXXXX 100644
 --- a/hw/ide/meson.build
 +++ b/hw/ide/meson.build
 @@ -XXX,XX +XXX,XX @@
  system_ss.add(when: 'CONFIG_AHCI', if_true: files('ahci.c'))
  system_ss.add(when: 'CONFIG_AHCI_ICH9', if_true: files('ich.c'))
 +system_ss.add(when: 'CONFIG_AHCI_SYSBUS', if_true: files('ahci-sysbus.c'))
  system_ss.add(when: 'CONFIG_ALLWINNER_A10', if_true: files('ahci-allwinner.c'))
  system_ss.add(when: 'CONFIG_IDE_BUS', if_true: files('ide-bus.c'))
  system_ss.add(when: 'CONFIG_IDE_CF', if_true: files('cf.c'))
 --
 .47.1

In some adhoc profiling booting Linux VMs, it's observed that ahci_irq_lower()
can be a hot path (10000+ triggers until login prompt appears). Even though the
parent device never changes, this method re-determines whether the parent device
is a PCI device or not using the rather expensive object_dynamic_cast()
function. Avoid this overhead by pushing the interrupt handling to the parent
device, essentially turning AHCIState into an "IP block".

Note that this change also frees AHCIState from the PCI dependency which wasn't
reflected in Kconfig.

Reported-by: Peter Xu <peterx@redhat.com>
Inspired-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Bernhard Beschow <shentey@gmail.com>

--
Supersedes: 20241121100152.65476-1-philmd@linaro.org
---
 hw/ide/ahci-internal.h |  1 -
 include/hw/ide/ahci.h  |  2 --
 hw/ide/ahci.c          | 39 ++++-----------------------------------
 hw/ide/ich.c           | 17 ++++++++++++++++-
 4 files changed, 20 insertions(+), 39 deletions(-)

diff --git a/hw/ide/ahci-internal.h b/hw/ide/ahci-internal.h
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ahci-internal.h
+++ b/hw/ide/ahci-internal.h
@@ -XXX,XX +XXX,XX @@
 #define HW_IDE_AHCI_INTERNAL_H
 
 #include "hw/ide/ahci.h"
-#include "hw/pci/pci_device.h"
 #include "ide-internal.h"
 
 #define AHCI_MEM_BAR_SIZE         0x1000
diff --git a/include/hw/ide/ahci.h b/include/hw/ide/ahci.h
index XXXXXXX..XXXXXXX 100644
--- a/include/hw/ide/ahci.h
+++ b/include/hw/ide/ahci.h
@@ -XXX,XX +XXX,XX @@ typedef struct AHCIControlRegs {
 } AHCIControlRegs;
 
 typedef struct AHCIState {
-    DeviceState *container;
-
     AHCIDevice *dev;
     AHCIControlRegs control_regs;
     MemoryRegion mem;
diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ahci.c
+++ b/hw/ide/ahci.c
@@ -XXX,XX +XXX,XX @@
 
 #include "qemu/osdep.h"
 #include "hw/irq.h"
-#include "hw/pci/msi.h"
-#include "hw/pci/pci.h"
 #include "hw/qdev-properties.h"
 #include "migration/vmstate.h"
 
@@ -XXX,XX +XXX,XX @@
 #include "qemu/module.h"
 #include "sysemu/block-backend.h"
 #include "sysemu/dma.h"
-#include "hw/ide/pci.h"
-#include "hw/ide/ahci-pci.h"
 #include "hw/ide/ahci-sysbus.h"
 #include "ahci-internal.h"
 #include "ide-internal.h"
@@ -XXX,XX +XXX,XX @@ static uint32_t ahci_port_read(AHCIState *s, int port, int offset)
     return val;
 }
 
-static void ahci_irq_raise(AHCIState *s)
-{
-    DeviceState *dev_state = s->container;
-    PCIDevice *pci_dev = (PCIDevice *) object_dynamic_cast(OBJECT(dev_state),
-                                                           TYPE_PCI_DEVICE);
-
-    trace_ahci_irq_raise(s);
-
-    if (pci_dev && msi_enabled(pci_dev)) {
-        msi_notify(pci_dev, 0);
-    } else {
-        qemu_irq_raise(s->irq);
-    }
-}
-
-static void ahci_irq_lower(AHCIState *s)
-{
-    DeviceState *dev_state = s->container;
-    PCIDevice *pci_dev = (PCIDevice *) object_dynamic_cast(OBJECT(dev_state),
-                                                           TYPE_PCI_DEVICE);
-
-    trace_ahci_irq_lower(s);
-
-    if (!pci_dev || !msi_enabled(pci_dev)) {
-        qemu_irq_lower(s->irq);
-    }
-}
-
 static void ahci_check_irq(AHCIState *s)
 {
     int i;
@@ -XXX,XX +XXX,XX @@ static void ahci_check_irq(AHCIState *s)
     trace_ahci_check_irq(s, old_irq, s->control_regs.irqstatus);
     if (s->control_regs.irqstatus &&
         (s->control_regs.ghc & HOST_CTL_IRQ_EN)) {
-            ahci_irq_raise(s);
+        trace_ahci_irq_raise(s);
+        qemu_irq_raise(s->irq);
     } else {
-        ahci_irq_lower(s);
+        trace_ahci_irq_lower(s);
+        qemu_irq_lower(s->irq);
     }
 }
 
@@ -XXX,XX +XXX,XX @@ static const IDEDMAOps ahci_dma_ops = {
 
 void ahci_init(AHCIState *s, DeviceState *qdev)
 {
-    s->container = qdev;
     /* XXX BAR size should be 1k, but that breaks, so bump it to 4k for now */
     memory_region_init_io(&s->mem, OBJECT(qdev), &ahci_mem_ops, s,
                           "ahci", AHCI_MEM_BAR_SIZE);
diff --git a/hw/ide/ich.c b/hw/ide/ich.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ich.c
+++ b/hw/ide/ich.c
@@ -XXX,XX +XXX,XX @@ static const VMStateDescription vmstate_ich9_ahci = {
     },
 };
 
+static void pci_ich9_ahci_update_irq(void *opaque, int irq_num, int level)
+{
+    PCIDevice *pci_dev = opaque;
+
+    assert(irq_num == 0);
+
+    if (msi_enabled(pci_dev)) {
+        if (level) {
+            msi_notify(pci_dev, 0);
+        }
+    } else {
+        pci_set_irq(pci_dev, level);
+    }
+}
+
 static void pci_ich9_reset(DeviceState *dev)
 {
     AHCIPCIState *d = ICH9_AHCI(dev);
@@ -XXX,XX +XXX,XX @@ static void pci_ich9_ahci_realize(PCIDevice *dev, Error **errp)
     /* XXX Software should program this register */
     dev->config[0x90]   = 1 << 6; /* Address Map Register - AHCI mode */
 
-    d->ahci.irq = pci_allocate_irq(dev);
+    d->ahci.irq = qemu_allocate_irq(pci_ich9_ahci_update_irq, d, 0);
 
     pci_register_bar(dev, ICH9_IDP_BAR, PCI_BASE_ADDRESS_SPACE_IO,
                      &d->ahci.idp);
-- 
2.47.1

Implement in dedicated file, just like TYPE_ICH9_AHCI.

Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
 hw/ide/ahci-sysbus.c | 91 ++++++++++++++++++++++++++++++++++++++++++++
 hw/ide/ahci.c        | 67 --------------------------------
 hw/arm/Kconfig       | 10 ++---
 hw/ide/Kconfig       |  4 ++
 hw/ide/meson.build   |  1 +
 5 files changed, 101 insertions(+), 72 deletions(-)
 create mode 100644 hw/ide/ahci-sysbus.c

diff --git a/hw/ide/ahci-sysbus.c b/hw/ide/ahci-sysbus.c
new file mode 100644
index XXXXXXX..XXXXXXX
--- /dev/null
+++ b/hw/ide/ahci-sysbus.c
@@ -XXX,XX +XXX,XX @@
+/*
+ * QEMU AHCI Emulation (MMIO-mapped devices)
+ *
+ * Copyright (c) 2010 qiaochong@loongson.cn
+ * Copyright (c) 2010 Roland Elek <elek.roland@gmail.com>
+ * Copyright (c) 2010 Sebastian Herbszt <herbszt@gmx.de>
+ * Copyright (c) 2010 Alexander Graf <agraf@suse.de>
+ *
+ * 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/>.
+ *
+ */
+
+#include "qemu/osdep.h"
+#include "exec/address-spaces.h"
+#include "hw/qdev-properties.h"
+#include "migration/vmstate.h"
+
+#include "hw/ide/ahci-sysbus.h"
+#include "ahci-internal.h"
+
+static const VMStateDescription vmstate_sysbus_ahci = {
+    .name = "sysbus-ahci",
+    .fields = (const VMStateField[]) {
+        VMSTATE_AHCI(ahci, SysbusAHCIState),
+        VMSTATE_END_OF_LIST()
+    },
+};
+
+static void sysbus_ahci_reset(DeviceState *dev)
+{
+    SysbusAHCIState *s = SYSBUS_AHCI(dev);
+
+    ahci_reset(&s->ahci);
+}
+
+static void sysbus_ahci_init(Object *obj)
+{
+    SysbusAHCIState *s = SYSBUS_AHCI(obj);
+    SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
+
+    ahci_init(&s->ahci, DEVICE(obj));
+
+    sysbus_init_mmio(sbd, &s->ahci.mem);
+    sysbus_init_irq(sbd, &s->ahci.irq);
+}
+
+static void sysbus_ahci_realize(DeviceState *dev, Error **errp)
+{
+    SysbusAHCIState *s = SYSBUS_AHCI(dev);
+
+    ahci_realize(&s->ahci, dev, &address_space_memory);
+}
+
+static Property sysbus_ahci_properties[] = {
+    DEFINE_PROP_UINT32("num-ports", SysbusAHCIState, ahci.ports, 1),
+    DEFINE_PROP_END_OF_LIST(),
+};
+
+static void sysbus_ahci_class_init(ObjectClass *klass, void *data)
+{
+    DeviceClass *dc = DEVICE_CLASS(klass);
+
+    dc->realize = sysbus_ahci_realize;
+    dc->vmsd = &vmstate_sysbus_ahci;
+    device_class_set_props(dc, sysbus_ahci_properties);
+    device_class_set_legacy_reset(dc, sysbus_ahci_reset);
+    set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
+}
+
+static const TypeInfo sysbus_ahci_types[] = {
+    {
+        .name          = TYPE_SYSBUS_AHCI,
+        .parent        = TYPE_SYS_BUS_DEVICE,
+        .instance_size = sizeof(SysbusAHCIState),
+        .instance_init = sysbus_ahci_init,
+        .class_init    = sysbus_ahci_class_init,
+    },
+};
+
+DEFINE_TYPES(sysbus_ahci_types)
diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ahci.c
+++ b/hw/ide/ahci.c
@@ -XXX,XX +XXX,XX @@
 
 #include "qemu/osdep.h"
 #include "hw/irq.h"
-#include "hw/qdev-properties.h"
 #include "migration/vmstate.h"
 
 #include "qemu/error-report.h"
 #include "qemu/log.h"
 #include "qemu/main-loop.h"
-#include "qemu/module.h"
 #include "sysemu/block-backend.h"
 #include "sysemu/dma.h"
-#include "hw/ide/ahci-sysbus.h"
 #include "ahci-internal.h"
 #include "ide-internal.h"
 
@@ -XXX,XX +XXX,XX @@ const VMStateDescription vmstate_ahci = {
     },
 };
 
-static const VMStateDescription vmstate_sysbus_ahci = {
-    .name = "sysbus-ahci",
-    .fields = (const VMStateField[]) {
-        VMSTATE_AHCI(ahci, SysbusAHCIState),
-        VMSTATE_END_OF_LIST()
-    },
-};
-
-static void sysbus_ahci_reset(DeviceState *dev)
-{
-    SysbusAHCIState *s = SYSBUS_AHCI(dev);
-
-    ahci_reset(&s->ahci);
-}
-
-static void sysbus_ahci_init(Object *obj)
-{
-    SysbusAHCIState *s = SYSBUS_AHCI(obj);
-    SysBusDevice *sbd = SYS_BUS_DEVICE(obj);
-
-    ahci_init(&s->ahci, DEVICE(obj));
-
-    sysbus_init_mmio(sbd, &s->ahci.mem);
-    sysbus_init_irq(sbd, &s->ahci.irq);
-}
-
-static void sysbus_ahci_realize(DeviceState *dev, Error **errp)
-{
-    SysbusAHCIState *s = SYSBUS_AHCI(dev);
-
-    ahci_realize(&s->ahci, dev, &address_space_memory);
-}
-
-static Property sysbus_ahci_properties[] = {
-    DEFINE_PROP_UINT32("num-ports", SysbusAHCIState, ahci.ports, 1),
-    DEFINE_PROP_END_OF_LIST(),
-};
-
-static void sysbus_ahci_class_init(ObjectClass *klass, void *data)
-{
-    DeviceClass *dc = DEVICE_CLASS(klass);
-
-    dc->realize = sysbus_ahci_realize;
-    dc->vmsd = &vmstate_sysbus_ahci;
-    device_class_set_props(dc, sysbus_ahci_properties);
-    device_class_set_legacy_reset(dc, sysbus_ahci_reset);
-    set_bit(DEVICE_CATEGORY_STORAGE, dc->categories);
-}
-
-static const TypeInfo sysbus_ahci_info = {
-    .name          = TYPE_SYSBUS_AHCI,
-    .parent        = TYPE_SYS_BUS_DEVICE,
-    .instance_size = sizeof(SysbusAHCIState),
-    .instance_init = sysbus_ahci_init,
-    .class_init    = sysbus_ahci_class_init,
-};
-
-static void sysbus_ahci_register_types(void)
-{
-    type_register_static(&sysbus_ahci_info);
-}
-
-type_init(sysbus_ahci_register_types)
-
 void ahci_ide_create_devs(AHCIState *ahci, DriveInfo **hd)
 {
     int i;
diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index XXXXXXX..XXXXXXX 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -XXX,XX +XXX,XX @@ config HIGHBANK
     depends on TCG && ARM
     select A9MPCORE
     select A15MPCORE
-    select AHCI
+    select AHCI_SYSBUS
     select ARM_TIMER # sp804
     select ARM_V7M
     select PL011 if !HAVE_RUST # UART
@@ -XXX,XX +XXX,XX @@ config SBSA_REF
     depends on TCG && AARCH64
     imply PCI_DEVICES
     select DEVICE_TREE
-    select AHCI
+    select AHCI_SYSBUS
     select ARM_SMMUV3
     select GPIO_KEY
     select PCI_EXPRESS
@@ -XXX,XX +XXX,XX @@ config ARM_V7M
 
 config ALLWINNER_A10
     bool
-    select AHCI
+    select AHCI_SYSBUS
     select ALLWINNER_A10_PIT
     select ALLWINNER_A10_PIC
     select ALLWINNER_A10_CCM
@@ -XXX,XX +XXX,XX @@ config ALLWINNER_H3
 config ALLWINNER_R40
     bool
     default y if TCG && ARM
-    select AHCI
+    select AHCI_SYSBUS
     select ALLWINNER_SRAMC
     select ALLWINNER_A10_PIT
     select ALLWINNER_WDT
@@ -XXX,XX +XXX,XX @@ config XLNX_ZYNQMP_ARM
     bool
     default y if PIXMAN
     depends on TCG && AARCH64
-    select AHCI
+    select AHCI_SYSBUS
     select ARM_GIC
     select CADENCE
     select CPU_CLUSTER
diff --git a/hw/ide/Kconfig b/hw/ide/Kconfig
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/Kconfig
+++ b/hw/ide/Kconfig
@@ -XXX,XX +XXX,XX @@ config AHCI_ICH9
     depends on PCI
     select AHCI
 
+config AHCI_SYSBUS
+    bool
+    select AHCI
+
 config IDE_SII3112
     bool
     select IDE_PCI
diff --git a/hw/ide/meson.build b/hw/ide/meson.build
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/meson.build
+++ b/hw/ide/meson.build
@@ -XXX,XX +XXX,XX @@
 system_ss.add(when: 'CONFIG_AHCI', if_true: files('ahci.c'))
 system_ss.add(when: 'CONFIG_AHCI_ICH9', if_true: files('ich.c'))
+system_ss.add(when: 'CONFIG_AHCI_SYSBUS', if_true: files('ahci-sysbus.c'))
 system_ss.add(when: 'CONFIG_ALLWINNER_A10', if_true: files('ahci-allwinner.c'))
 system_ss.add(when: 'CONFIG_IDE_BUS', if_true: files('ide-bus.c'))
 system_ss.add(when: 'CONFIG_IDE_CF', if_true: files('cf.c'))
-- 
2.47.1

Note that this change also frees AHCIState from the PCI dependency which wasn't
reflected in Kconfig.

Reported-by: Peter Xu <peterx@redhat.com>
Inspired-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Bernhard Beschow <shentey@gmail.com>
---
 hw/ide/ahci-internal.h    |  1 -
 include/hw/ide/ahci-pci.h |  2 ++
 include/hw/ide/ahci.h     |  2 --
 hw/ide/ahci.c             | 39 ++++-----------------------------------
 hw/ide/ich.c              | 19 +++++++++++++++----
 5 files changed, 21 insertions(+), 42 deletions(-)

diff --git a/hw/ide/ahci-internal.h b/hw/ide/ahci-internal.h
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ahci-internal.h
+++ b/hw/ide/ahci-internal.h
@@ -XXX,XX +XXX,XX @@
 #define HW_IDE_AHCI_INTERNAL_H
 
 #include "hw/ide/ahci.h"
-#include "hw/pci/pci_device.h"
 #include "ide-internal.h"
 
 #define AHCI_MEM_BAR_SIZE         0x1000
diff --git a/include/hw/ide/ahci-pci.h b/include/hw/ide/ahci-pci.h
index XXXXXXX..XXXXXXX 100644
--- a/include/hw/ide/ahci-pci.h
+++ b/include/hw/ide/ahci-pci.h
@@ -XXX,XX +XXX,XX @@
 #include "qom/object.h"
 #include "hw/ide/ahci.h"
 #include "hw/pci/pci_device.h"
+#include "hw/irq.h"
 
 #define TYPE_ICH9_AHCI "ich9-ahci"
 OBJECT_DECLARE_SIMPLE_TYPE(AHCIPCIState, ICH9_AHCI)
@@ -XXX,XX +XXX,XX @@ struct AHCIPCIState {
     PCIDevice parent_obj;
 
     AHCIState ahci;
+    IRQState irq;
 };
 
 #endif
diff --git a/include/hw/ide/ahci.h b/include/hw/ide/ahci.h
index XXXXXXX..XXXXXXX 100644
--- a/include/hw/ide/ahci.h
+++ b/include/hw/ide/ahci.h
@@ -XXX,XX +XXX,XX @@ typedef struct AHCIControlRegs {
 } AHCIControlRegs;
 
 typedef struct AHCIState {
-    DeviceState *container;
-
     AHCIDevice *dev;
     AHCIControlRegs control_regs;
     MemoryRegion mem;
diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ahci.c
+++ b/hw/ide/ahci.c
@@ -XXX,XX +XXX,XX @@
 
 #include "qemu/osdep.h"
 #include "hw/irq.h"
-#include "hw/pci/msi.h"
-#include "hw/pci/pci.h"
 #include "hw/qdev-properties.h"
 #include "migration/vmstate.h"
 
@@ -XXX,XX +XXX,XX @@
 #include "qemu/module.h"
 #include "sysemu/block-backend.h"
 #include "sysemu/dma.h"
-#include "hw/ide/pci.h"
-#include "hw/ide/ahci-pci.h"
 #include "hw/ide/ahci-sysbus.h"
 #include "ahci-internal.h"
 #include "ide-internal.h"
@@ -XXX,XX +XXX,XX @@ static uint32_t ahci_port_read(AHCIState *s, int port, int offset)
     return val;
 }
 
-static void ahci_irq_raise(AHCIState *s)
-{
-    DeviceState *dev_state = s->container;
-    PCIDevice *pci_dev = (PCIDevice *) object_dynamic_cast(OBJECT(dev_state),
-                                                           TYPE_PCI_DEVICE);
-
-    trace_ahci_irq_raise(s);
-
-    if (pci_dev && msi_enabled(pci_dev)) {
-        msi_notify(pci_dev, 0);
-    } else {
-        qemu_irq_raise(s->irq);
-    }
-}
-
-static void ahci_irq_lower(AHCIState *s)
-{
-    DeviceState *dev_state = s->container;
-    PCIDevice *pci_dev = (PCIDevice *) object_dynamic_cast(OBJECT(dev_state),
-                                                           TYPE_PCI_DEVICE);
-
-    trace_ahci_irq_lower(s);
-
-    if (!pci_dev || !msi_enabled(pci_dev)) {
-        qemu_irq_lower(s->irq);
-    }
-}
-
 static void ahci_check_irq(AHCIState *s)
 {
     int i;
@@ -XXX,XX +XXX,XX @@ static void ahci_check_irq(AHCIState *s)
     trace_ahci_check_irq(s, old_irq, s->control_regs.irqstatus);
     if (s->control_regs.irqstatus &&
         (s->control_regs.ghc & HOST_CTL_IRQ_EN)) {
-            ahci_irq_raise(s);
+        trace_ahci_irq_raise(s);
+        qemu_irq_raise(s->irq);
     } else {
-        ahci_irq_lower(s);
+        trace_ahci_irq_lower(s);
+        qemu_irq_lower(s->irq);
     }
 }
 
@@ -XXX,XX +XXX,XX @@ static const IDEDMAOps ahci_dma_ops = {
 
 void ahci_init(AHCIState *s, DeviceState *qdev)
 {
-    s->container = qdev;
     /* XXX BAR size should be 1k, but that breaks, so bump it to 4k for now */
     memory_region_init_io(&s->mem, OBJECT(qdev), &ahci_mem_ops, s,
                           "ahci", AHCI_MEM_BAR_SIZE);
diff --git a/hw/ide/ich.c b/hw/ide/ich.c
index XXXXXXX..XXXXXXX 100644
--- a/hw/ide/ich.c
+++ b/hw/ide/ich.c
@@ -XXX,XX +XXX,XX @@
  */
 
 #include "qemu/osdep.h"
-#include "hw/irq.h"
 #include "hw/pci/msi.h"
 #include "hw/pci/pci.h"
 #include "migration/vmstate.h"
@@ -XXX,XX +XXX,XX @@ static const VMStateDescription vmstate_ich9_ahci = {
     },
 };
 
+static void pci_ich9_ahci_update_irq(void *opaque, int irq_num, int level)
+{
+    PCIDevice *pci_dev = opaque;
+
+    if (msi_enabled(pci_dev)) {
+        if (level) {
+            msi_notify(pci_dev, 0);
+        }
+    } else {
+        pci_set_irq(pci_dev, level);
+    }
+}
+
 static void pci_ich9_reset(DeviceState *dev)
 {
     AHCIPCIState *d = ICH9_AHCI(dev);
@@ -XXX,XX +XXX,XX @@ static void pci_ich9_ahci_init(Object *obj)
 {
     AHCIPCIState *d = ICH9_AHCI(obj);
 
+    qemu_init_irq(&d->irq, pci_ich9_ahci_update_irq, d, 0);
     ahci_init(&d->ahci, DEVICE(obj));
+    d->ahci.irq = &d->irq;
 }
 
 static void pci_ich9_ahci_realize(PCIDevice *dev, Error **errp)
@@ -XXX,XX +XXX,XX @@ static void pci_ich9_ahci_realize(PCIDevice *dev, Error **errp)
     /* XXX Software should program this register */
     dev->config[0x90]   = 1 << 6; /* Address Map Register - AHCI mode */
 
-    d->ahci.irq = pci_allocate_irq(dev);
-
     pci_register_bar(dev, ICH9_IDP_BAR, PCI_BASE_ADDRESS_SPACE_IO,
                      &d->ahci.idp);
     pci_register_bar(dev, ICH9_MEM_BAR, PCI_BASE_ADDRESS_SPACE_MEMORY,
@@ -XXX,XX +XXX,XX @@ static void pci_ich9_uninit(PCIDevice *dev)
 
     msi_uninit(dev);
     ahci_uninit(&d->ahci);
-    qemu_free_irq(d->ahci.irq);
 }
 
 static void ich_ahci_class_init(ObjectClass *klass, void *data)
-- 
2.47.1

Implement in dedicated file, just like TYPE_ICH9_AHCI.