* [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
* 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 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
* [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 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
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.