[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] pci: pass bridge update to secondary bus
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH] pci: pass bridge update to secondary bus |
Date: |
Wed, 7 Jul 2010 17:31:39 +0000 |
On Tue, Jul 6, 2010 at 11:23 AM, Michael S. Tsirkin <address@hidden> wrote:
> bridge config write should trigger updates
> on the secondary bus. never on the primary bus.
If this is true, shouldn't updates happen on all buses from secondary
to subordinate? Do we know which of these are immediately below
primary bus?
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
>
> Compile-tested only.
> Isaku Yamahata, could you review this please?
> You wrote the code, and you seem to have some bridged setups.
>
> hw/pci.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/hw/pci.c b/hw/pci.c
> index 926cf63..011d83e 100644
> --- a/hw/pci.c
> +++ b/hw/pci.c
> @@ -1513,7 +1513,9 @@ static void pci_bridge_write_config(PCIDevice *d,
> /* memory base/limit, prefetchable base/limit and
> io base/limit upper 16 */
> ranges_overlap(address, len, PCI_MEMORY_BASE, 20)) {
> - pci_bridge_update_mappings(d->bus);
> + PCIBridge *s = container_of(d, PCIBridge, dev);
> + PCIBus *secondary_bus = &s->bus;
> + pci_bridge_update_mappings(secondary_bus);
> }
> }
>
> --
> 1.7.2.rc0.14.g41c1c
>
>