@samitouri / QOSamiQemu / commits / f84a406f4d

hw/virtio/virtio-pci: Remove VIRTIO_PCI_FLAG_INIT_DEVERR definition

VIRTIO_PCI_FLAG_INIT_DEVERR was only used by the hw_compat_2_8[] array, via the 'x-pcie-deverr-init=off' property. We removed all machines using that array, lets remove all the code around VIRTIO_PCI_FLAG_INIT_DEVERR (see commit 9a4c0e220d8 for similar VIRTIO_PCI_FLAG_* enum removal). Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-ID: <20250501210456.89071-9-philmd@linaro.org> Reviewed-by: Igor Mammedov <imammedo@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com> Message-ID: <20260225092024.794595-8-thuth@redhat.com>

Philippe Mathieu-Daudé committed Feb 25, 2026 at 10:20 UTC f84a406f4d2de1f88b32590cec382d26b060a655
2 files changed +2 -10
hw/virtio/virtio-pci.c
+2 -6
@@ -2307,10 +2307,8 @@ static void virtio_pci_realize(PCIDevice *pci_dev, Error **errp)
2307 proxy->last_pcie_cap_offset += PCI_ERR_SIZEOF;
2308 }
2309
2310 - if (proxy->flags & VIRTIO_PCI_FLAG_INIT_DEVERR) {
2311 - /* Init error enabling flags */
2312 - pcie_cap_deverr_init(pci_dev);
2313 - }
2310 + /* Init error enabling flags */
2311 + pcie_cap_deverr_init(pci_dev);
2312
2313 if (proxy->flags & VIRTIO_PCI_FLAG_INIT_LNKCTL) {
2314 /* Init Link Control Register */
@@ -2446,8 +2444,6 @@ static const Property virtio_pci_properties[] = {
2444 VIRTIO_PCI_FLAG_ATS_BIT, false),
2445 DEFINE_PROP_BIT("x-ats-page-aligned", VirtIOPCIProxy, flags,
2446 VIRTIO_PCI_FLAG_ATS_PAGE_ALIGNED_BIT, true),
2449 - DEFINE_PROP_BIT("x-pcie-deverr-init", VirtIOPCIProxy, flags,
2450 - VIRTIO_PCI_FLAG_INIT_DEVERR_BIT, true),
2447 DEFINE_PROP_BIT("x-pcie-lnkctl-init", VirtIOPCIProxy, flags,
2448 VIRTIO_PCI_FLAG_INIT_LNKCTL_BIT, true),
2449 DEFINE_PROP_BIT("x-pcie-pm-init", VirtIOPCIProxy, flags,
include/hw/virtio/virtio-pci.h
-4
@@ -35,7 +35,6 @@ enum {
35 VIRTIO_PCI_FLAG_MODERN_PIO_NOTIFY_BIT,
36 VIRTIO_PCI_FLAG_PAGE_PER_VQ_BIT,
37 VIRTIO_PCI_FLAG_ATS_BIT,
38 - VIRTIO_PCI_FLAG_INIT_DEVERR_BIT,
38 VIRTIO_PCI_FLAG_INIT_LNKCTL_BIT,
39 VIRTIO_PCI_FLAG_INIT_PM_BIT,
40 VIRTIO_PCI_FLAG_INIT_FLR_BIT,
@@ -63,9 +62,6 @@ enum {
62 /* address space translation service */
63 #define VIRTIO_PCI_FLAG_ATS (1 << VIRTIO_PCI_FLAG_ATS_BIT)
64
66 -/* Init error enabling flags */
67 -#define VIRTIO_PCI_FLAG_INIT_DEVERR (1 << VIRTIO_PCI_FLAG_INIT_DEVERR_BIT)
68 -
65 /* Init Link Control register */
66 #define VIRTIO_PCI_FLAG_INIT_LNKCTL (1 << VIRTIO_PCI_FLAG_INIT_LNKCTL_BIT)
67