* [Qemu-devel] [PATCH for-1.6 v2 0/2] PCI bridge fixes and cleanup
@ 2013-08-05 14:36 Andreas Färber
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space Andreas Färber
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type Andreas Färber
0 siblings, 2 replies; 8+ messages in thread
From: Andreas Färber @ 2013-08-05 14:36 UTC (permalink / raw)
To: qemu-devel; +Cc: Gerd Hoffmann, Andreas Färber, Michael S. Tsirkin
Hello Michael,
This mini-series cleans up Gerd's commit message and adds the generalization on top.
Both look safe for 1.6 to me; doing it in two steps not only allows to record
Gerd's authorship but allows to cherry-pick his patch for 1.5 while mine is 1.6+.
Regards,
Andreas
Gerd's v1 -> v2:
* Reworked commit message: Added qemu-stable, clarified scope and change.
* Appended patch
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Andreas Färber (1):
pci-bridge: Move common class initializations to base type
Gerd Hoffmann (1):
i82801b11: Fix i82801b11 PCI host bridge config space
hw/pci-bridge/dec.c | 4 ----
hw/pci-bridge/i82801b11.c | 1 -
hw/pci-bridge/ioh3420.c | 1 -
hw/pci-bridge/pci_bridge_dev.c | 1 -
hw/pci-bridge/xio3130_downstream.c | 1 -
hw/pci-bridge/xio3130_upstream.c | 1 -
hw/pci-host/apb.c | 4 ----
hw/pci/pci_bridge.c | 12 ++++++++++++
8 files changed, 12 insertions(+), 13 deletions(-)
--
1.8.1.4
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space
2013-08-05 14:36 [Qemu-devel] [PATCH for-1.6 v2 0/2] PCI bridge fixes and cleanup Andreas Färber
@ 2013-08-05 14:36 ` Andreas Färber
2013-08-05 15:37 ` Michael S. Tsirkin
2013-08-12 8:51 ` Gerd Hoffmann
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type Andreas Färber
1 sibling, 2 replies; 8+ messages in thread
From: Andreas Färber @ 2013-08-05 14:36 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-stable, Gerd Hoffmann, Andreas Färber, Michael S. Tsirkin
From: Gerd Hoffmann <kraxel@redhat.com>
pci_bridge_write_config() was not being used.
Cc: qemu-stable@nongnu.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
---
hw/pci-bridge/i82801b11.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
index 8a5e426..14cd7fd 100644
--- a/hw/pci-bridge/i82801b11.c
+++ b/hw/pci-bridge/i82801b11.c
@@ -90,6 +90,7 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
k->revision = ICH9_D2P_A2_REVISION;
k->init = i82801b11_bridge_initfn;
+ k->config_write = pci_bridge_write_config;
set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
}
--
1.8.1.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type
2013-08-05 14:36 [Qemu-devel] [PATCH for-1.6 v2 0/2] PCI bridge fixes and cleanup Andreas Färber
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space Andreas Färber
@ 2013-08-05 14:36 ` Andreas Färber
2013-08-05 14:41 ` Paolo Bonzini
1 sibling, 1 reply; 8+ messages in thread
From: Andreas Färber @ 2013-08-05 14:36 UTC (permalink / raw)
To: qemu-devel
Cc: Alexander Graf, open list:New World, Gerd Hoffmann,
Andreas Färber, Michael S. Tsirkin
Set PCIDeviceClass::is_bridge as well as PCIDeviceClass::config_write(),
PCIDeviceClass::exit() and DeviceClass::reset() defaults directly for
TYPE_PCI_BRIDGE (complements f055e96bd4311d287c0e03faec02a1bdbc351925).
This avoids another pci_bridge_config_write() missing somewhere, unless
explicitly overridden.
Reported-by: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
---
hw/pci-bridge/dec.c | 4 ----
hw/pci-bridge/i82801b11.c | 2 --
hw/pci-bridge/ioh3420.c | 1 -
hw/pci-bridge/pci_bridge_dev.c | 1 -
hw/pci-bridge/xio3130_downstream.c | 1 -
hw/pci-bridge/xio3130_upstream.c | 1 -
hw/pci-host/apb.c | 4 ----
hw/pci/pci_bridge.c | 12 ++++++++++++
8 files changed, 12 insertions(+), 14 deletions(-)
diff --git a/hw/pci-bridge/dec.c b/hw/pci-bridge/dec.c
index e5e3be8..379f9f0 100644
--- a/hw/pci-bridge/dec.c
+++ b/hw/pci-bridge/dec.c
@@ -62,13 +62,9 @@ static void dec_21154_pci_bridge_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
k->init = dec_pci_bridge_initfn;
- k->exit = pci_bridge_exitfn;
k->vendor_id = PCI_VENDOR_ID_DEC;
k->device_id = PCI_DEVICE_ID_DEC_21154;
- k->config_write = pci_bridge_write_config;
- k->is_bridge = 1;
dc->desc = "DEC 21154 PCI-PCI bridge";
- dc->reset = pci_bridge_reset;
dc->vmsd = &vmstate_pci_device;
}
diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
index 14cd7fd..ab0c8e1 100644
--- a/hw/pci-bridge/i82801b11.c
+++ b/hw/pci-bridge/i82801b11.c
@@ -85,12 +85,10 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
DeviceClass *dc = DEVICE_CLASS(klass);
- k->is_bridge = 1;
k->vendor_id = PCI_VENDOR_ID_INTEL;
k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
k->revision = ICH9_D2P_A2_REVISION;
k->init = i82801b11_bridge_initfn;
- k->config_write = pci_bridge_write_config;
set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
}
diff --git a/hw/pci-bridge/ioh3420.c b/hw/pci-bridge/ioh3420.c
index 0f7f209..2748b06 100644
--- a/hw/pci-bridge/ioh3420.c
+++ b/hw/pci-bridge/ioh3420.c
@@ -200,7 +200,6 @@ static void ioh3420_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
k->is_express = 1;
- k->is_bridge = 1;
k->config_write = ioh3420_write_config;
k->init = ioh3420_initfn;
k->exit = ioh3420_exitfn;
diff --git a/hw/pci-bridge/pci_bridge_dev.c b/hw/pci-bridge/pci_bridge_dev.c
index a9392c7..716d0af 100644
--- a/hw/pci-bridge/pci_bridge_dev.c
+++ b/hw/pci-bridge/pci_bridge_dev.c
@@ -142,7 +142,6 @@ static void pci_bridge_dev_class_init(ObjectClass *klass, void *data)
k->vendor_id = PCI_VENDOR_ID_REDHAT;
k->device_id = PCI_DEVICE_ID_REDHAT_BRIDGE;
k->class_id = PCI_CLASS_BRIDGE_PCI;
- k->is_bridge = 1,
dc->desc = "Standard PCI Bridge";
dc->reset = qdev_pci_bridge_dev_reset;
dc->props = pci_bridge_dev_properties;
diff --git a/hw/pci-bridge/xio3130_downstream.c b/hw/pci-bridge/xio3130_downstream.c
index 94f9781..07da9bf 100644
--- a/hw/pci-bridge/xio3130_downstream.c
+++ b/hw/pci-bridge/xio3130_downstream.c
@@ -167,7 +167,6 @@ static void xio3130_downstream_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
k->is_express = 1;
- k->is_bridge = 1;
k->config_write = xio3130_downstream_write_config;
k->init = xio3130_downstream_initfn;
k->exit = xio3130_downstream_exitfn;
diff --git a/hw/pci-bridge/xio3130_upstream.c b/hw/pci-bridge/xio3130_upstream.c
index 59f97f6..def08c8 100644
--- a/hw/pci-bridge/xio3130_upstream.c
+++ b/hw/pci-bridge/xio3130_upstream.c
@@ -146,7 +146,6 @@ static void xio3130_upstream_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
k->is_express = 1;
- k->is_bridge = 1;
k->config_write = xio3130_upstream_write_config;
k->init = xio3130_upstream_initfn;
k->exit = xio3130_upstream_exitfn;
diff --git a/hw/pci-host/apb.c b/hw/pci-host/apb.c
index 92f289f..1cb2ef4 100644
--- a/hw/pci-host/apb.c
+++ b/hw/pci-host/apb.c
@@ -553,14 +553,10 @@ static void pbm_pci_bridge_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
k->init = apb_pci_bridge_initfn;
- k->exit = pci_bridge_exitfn;
k->vendor_id = PCI_VENDOR_ID_SUN;
k->device_id = PCI_DEVICE_ID_SUN_SIMBA;
k->revision = 0x11;
- k->config_write = pci_bridge_write_config;
- k->is_bridge = 1;
set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
- dc->reset = pci_bridge_reset;
dc->vmsd = &vmstate_pci_device;
}
diff --git a/hw/pci/pci_bridge.c b/hw/pci/pci_bridge.c
index a90671d..23a5c7d 100644
--- a/hw/pci/pci_bridge.c
+++ b/hw/pci/pci_bridge.c
@@ -405,11 +405,23 @@ void pci_bridge_map_irq(PCIBridge *br, const char* bus_name,
br->bus_name = bus_name;
}
+static void pci_bridge_class_init(ObjectClass *oc, void *data)
+{
+ DeviceClass *dc = DEVICE_CLASS(oc);
+ PCIDeviceClass *pdc = PCI_DEVICE_CLASS(oc);
+
+ pdc->is_bridge = 1;
+ pdc->exit = pci_bridge_exitfn;
+ pdc->config_write = pci_bridge_write_config;
+ dc->reset = pci_bridge_reset;
+}
+
static const TypeInfo pci_bridge_type_info = {
.name = TYPE_PCI_BRIDGE,
.parent = TYPE_PCI_DEVICE,
.instance_size = sizeof(PCIBridge),
.abstract = true,
+ .class_init = pci_bridge_class_init,
};
static void pci_bridge_register_types(void)
--
1.8.1.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type Andreas Färber
@ 2013-08-05 14:41 ` Paolo Bonzini
2013-08-05 16:22 ` Andreas Färber
0 siblings, 1 reply; 8+ messages in thread
From: Paolo Bonzini @ 2013-08-05 14:41 UTC (permalink / raw)
To: Andreas Färber, qemu-ppc, agraf, qemu-devel
On 08/05/2013 04:36 PM, Andreas Färber wrote:
> Set PCIDeviceClass::is_bridge as well as PCIDeviceClass::config_write(),
> PCIDeviceClass::exit() and DeviceClass::reset() defaults directly for
> TYPE_PCI_BRIDGE (complements f055e96bd4311d287c0e03faec02a1bdbc351925).
>
> This avoids another pci_bridge_config_write() missing somewhere, unless
> explicitly overridden.
>
> Reported-by: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Andreas Färber <afaerber@suse.de>
Why for 1.6?
Paolo
> ---
> hw/pci-bridge/dec.c | 4 ----
> hw/pci-bridge/i82801b11.c | 2 --
> hw/pci-bridge/ioh3420.c | 1 -
> hw/pci-bridge/pci_bridge_dev.c | 1 -
> hw/pci-bridge/xio3130_downstream.c | 1 -
> hw/pci-bridge/xio3130_upstream.c | 1 -
> hw/pci-host/apb.c | 4 ----
> hw/pci/pci_bridge.c | 12 ++++++++++++
> 8 files changed, 12 insertions(+), 14 deletions(-)
>
> diff --git a/hw/pci-bridge/dec.c b/hw/pci-bridge/dec.c
> index e5e3be8..379f9f0 100644
> --- a/hw/pci-bridge/dec.c
> +++ b/hw/pci-bridge/dec.c
> @@ -62,13 +62,9 @@ static void dec_21154_pci_bridge_class_init(ObjectClass *klass, void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
>
> k->init = dec_pci_bridge_initfn;
> - k->exit = pci_bridge_exitfn;
> k->vendor_id = PCI_VENDOR_ID_DEC;
> k->device_id = PCI_DEVICE_ID_DEC_21154;
> - k->config_write = pci_bridge_write_config;
> - k->is_bridge = 1;
> dc->desc = "DEC 21154 PCI-PCI bridge";
> - dc->reset = pci_bridge_reset;
> dc->vmsd = &vmstate_pci_device;
> }
>
> diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
> index 14cd7fd..ab0c8e1 100644
> --- a/hw/pci-bridge/i82801b11.c
> +++ b/hw/pci-bridge/i82801b11.c
> @@ -85,12 +85,10 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
> DeviceClass *dc = DEVICE_CLASS(klass);
>
> - k->is_bridge = 1;
> k->vendor_id = PCI_VENDOR_ID_INTEL;
> k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
> k->revision = ICH9_D2P_A2_REVISION;
> k->init = i82801b11_bridge_initfn;
> - k->config_write = pci_bridge_write_config;
> set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> diff --git a/hw/pci-bridge/ioh3420.c b/hw/pci-bridge/ioh3420.c
> index 0f7f209..2748b06 100644
> --- a/hw/pci-bridge/ioh3420.c
> +++ b/hw/pci-bridge/ioh3420.c
> @@ -200,7 +200,6 @@ static void ioh3420_class_init(ObjectClass *klass, void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
>
> k->is_express = 1;
> - k->is_bridge = 1;
> k->config_write = ioh3420_write_config;
> k->init = ioh3420_initfn;
> k->exit = ioh3420_exitfn;
> diff --git a/hw/pci-bridge/pci_bridge_dev.c b/hw/pci-bridge/pci_bridge_dev.c
> index a9392c7..716d0af 100644
> --- a/hw/pci-bridge/pci_bridge_dev.c
> +++ b/hw/pci-bridge/pci_bridge_dev.c
> @@ -142,7 +142,6 @@ static void pci_bridge_dev_class_init(ObjectClass *klass, void *data)
> k->vendor_id = PCI_VENDOR_ID_REDHAT;
> k->device_id = PCI_DEVICE_ID_REDHAT_BRIDGE;
> k->class_id = PCI_CLASS_BRIDGE_PCI;
> - k->is_bridge = 1,
> dc->desc = "Standard PCI Bridge";
> dc->reset = qdev_pci_bridge_dev_reset;
> dc->props = pci_bridge_dev_properties;
> diff --git a/hw/pci-bridge/xio3130_downstream.c b/hw/pci-bridge/xio3130_downstream.c
> index 94f9781..07da9bf 100644
> --- a/hw/pci-bridge/xio3130_downstream.c
> +++ b/hw/pci-bridge/xio3130_downstream.c
> @@ -167,7 +167,6 @@ static void xio3130_downstream_class_init(ObjectClass *klass, void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
>
> k->is_express = 1;
> - k->is_bridge = 1;
> k->config_write = xio3130_downstream_write_config;
> k->init = xio3130_downstream_initfn;
> k->exit = xio3130_downstream_exitfn;
> diff --git a/hw/pci-bridge/xio3130_upstream.c b/hw/pci-bridge/xio3130_upstream.c
> index 59f97f6..def08c8 100644
> --- a/hw/pci-bridge/xio3130_upstream.c
> +++ b/hw/pci-bridge/xio3130_upstream.c
> @@ -146,7 +146,6 @@ static void xio3130_upstream_class_init(ObjectClass *klass, void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
>
> k->is_express = 1;
> - k->is_bridge = 1;
> k->config_write = xio3130_upstream_write_config;
> k->init = xio3130_upstream_initfn;
> k->exit = xio3130_upstream_exitfn;
> diff --git a/hw/pci-host/apb.c b/hw/pci-host/apb.c
> index 92f289f..1cb2ef4 100644
> --- a/hw/pci-host/apb.c
> +++ b/hw/pci-host/apb.c
> @@ -553,14 +553,10 @@ static void pbm_pci_bridge_class_init(ObjectClass *klass, void *data)
> PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
>
> k->init = apb_pci_bridge_initfn;
> - k->exit = pci_bridge_exitfn;
> k->vendor_id = PCI_VENDOR_ID_SUN;
> k->device_id = PCI_DEVICE_ID_SUN_SIMBA;
> k->revision = 0x11;
> - k->config_write = pci_bridge_write_config;
> - k->is_bridge = 1;
> set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> - dc->reset = pci_bridge_reset;
> dc->vmsd = &vmstate_pci_device;
> }
>
> diff --git a/hw/pci/pci_bridge.c b/hw/pci/pci_bridge.c
> index a90671d..23a5c7d 100644
> --- a/hw/pci/pci_bridge.c
> +++ b/hw/pci/pci_bridge.c
> @@ -405,11 +405,23 @@ void pci_bridge_map_irq(PCIBridge *br, const char* bus_name,
> br->bus_name = bus_name;
> }
>
> +static void pci_bridge_class_init(ObjectClass *oc, void *data)
> +{
> + DeviceClass *dc = DEVICE_CLASS(oc);
> + PCIDeviceClass *pdc = PCI_DEVICE_CLASS(oc);
> +
> + pdc->is_bridge = 1;
> + pdc->exit = pci_bridge_exitfn;
> + pdc->config_write = pci_bridge_write_config;
> + dc->reset = pci_bridge_reset;
> +}
> +
> static const TypeInfo pci_bridge_type_info = {
> .name = TYPE_PCI_BRIDGE,
> .parent = TYPE_PCI_DEVICE,
> .instance_size = sizeof(PCIBridge),
> .abstract = true,
> + .class_init = pci_bridge_class_init,
> };
>
> static void pci_bridge_register_types(void)
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space Andreas Färber
@ 2013-08-05 15:37 ` Michael S. Tsirkin
2013-08-12 8:51 ` Gerd Hoffmann
1 sibling, 0 replies; 8+ messages in thread
From: Michael S. Tsirkin @ 2013-08-05 15:37 UTC (permalink / raw)
To: Andreas Färber; +Cc: qemu-stable, qemu-devel, Gerd Hoffmann
On Mon, Aug 05, 2013 at 04:36:40PM +0200, Andreas Färber wrote:
> From: Gerd Hoffmann <kraxel@redhat.com>
>
> pci_bridge_write_config() was not being used.
>
> Cc: qemu-stable@nongnu.org
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Andreas Färber <afaerber@suse.de>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
> ---
> hw/pci-bridge/i82801b11.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
> index 8a5e426..14cd7fd 100644
> --- a/hw/pci-bridge/i82801b11.c
> +++ b/hw/pci-bridge/i82801b11.c
> @@ -90,6 +90,7 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
> k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
> k->revision = ICH9_D2P_A2_REVISION;
> k->init = i82801b11_bridge_initfn;
> + k->config_write = pci_bridge_write_config;
> set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> }
>
> --
> 1.8.1.4
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type
2013-08-05 14:41 ` Paolo Bonzini
@ 2013-08-05 16:22 ` Andreas Färber
0 siblings, 0 replies; 8+ messages in thread
From: Andreas Färber @ 2013-08-05 16:22 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: qemu-ppc, agraf, qemu-devel
Am 05.08.2013 16:41, schrieb Paolo Bonzini:
> On 08/05/2013 04:36 PM, Andreas Färber wrote:
>> Set PCIDeviceClass::is_bridge as well as PCIDeviceClass::config_write(),
>> PCIDeviceClass::exit() and DeviceClass::reset() defaults directly for
>> TYPE_PCI_BRIDGE (complements f055e96bd4311d287c0e03faec02a1bdbc351925).
>>
>> This avoids another pci_bridge_config_write() missing somewhere, unless
>> explicitly overridden.
>>
>> Reported-by: Gerd Hoffmann <kraxel@redhat.com>
>> Signed-off-by: Andreas Färber <afaerber@suse.de>
>
> Why for 1.6?
This is The Real Fix. We could squash it as one patch, but then we can't
backport it.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space Andreas Färber
2013-08-05 15:37 ` Michael S. Tsirkin
@ 2013-08-12 8:51 ` Gerd Hoffmann
2013-08-12 8:53 ` Andreas Färber
1 sibling, 1 reply; 8+ messages in thread
From: Gerd Hoffmann @ 2013-08-12 8:51 UTC (permalink / raw)
To: Andreas Färber
Cc: Michael S. Tsirkin, qemu-devel, Anthony Liguori, qemu-stable
On 08/05/13 16:36, Andreas Färber wrote:
> From: Gerd Hoffmann <kraxel@redhat.com>
>
> pci_bridge_write_config() was not being used.
>
> Cc: qemu-stable@nongnu.org
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Andreas Färber <afaerber@suse.de>
Ping? I see it's not yet in master, can we *please* get this in?
It is save to take just this one-liner and defer 2/2 to 1.7.
cheers,
Gerd
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space
2013-08-12 8:51 ` Gerd Hoffmann
@ 2013-08-12 8:53 ` Andreas Färber
0 siblings, 0 replies; 8+ messages in thread
From: Andreas Färber @ 2013-08-12 8:53 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: qemu-stable, Gerd Hoffmann, Anthony Liguori, qemu-devel
Am 12.08.2013 10:51, schrieb Gerd Hoffmann:
> On 08/05/13 16:36, Andreas Färber wrote:
>> From: Gerd Hoffmann <kraxel@redhat.com>
>>
>> pci_bridge_write_config() was not being used.
>>
>> Cc: qemu-stable@nongnu.org
>> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
>> Signed-off-by: Andreas Färber <afaerber@suse.de>
>
> Ping? I see it's not yet in master, can we *please* get this in?
> It is save to take just this one-liner and defer 2/2 to 1.7.
Michael, please decide as PCI maintainer and send a pull. Thanks!
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-08-12 8:54 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-05 14:36 [Qemu-devel] [PATCH for-1.6 v2 0/2] PCI bridge fixes and cleanup Andreas Färber
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 1/2] i82801b11: Fix i82801b11 PCI host bridge config space Andreas Färber
2013-08-05 15:37 ` Michael S. Tsirkin
2013-08-12 8:51 ` Gerd Hoffmann
2013-08-12 8:53 ` Andreas Färber
2013-08-05 14:36 ` [Qemu-devel] [PATCH for-1.6 v2 2/2] pci-bridge: Move common class initializations to base type Andreas Färber
2013-08-05 14:41 ` Paolo Bonzini
2013-08-05 16:22 ` Andreas Färber
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.