[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO spac
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space |
Date: |
Tue, 13 Oct 2015 14:55:30 +0100 |
On 13 October 2015 at 14:25, Peter Maydell <address@hidden> wrote:
> On 13 October 2015 at 14:19, Michael S. Tsirkin <address@hidden> wrote:
>> On Tue, Oct 13, 2015 at 02:12:34PM +0100, Peter Maydell wrote:
>>> Well, if we're going to go around and fix the machines which
>>> don't get things right, I guess. It's a shame the default for
>>> the global is "this machine is broken", because now every
>>> new machine will default unnecessarily to broken, and there's
>>> no way to grep the source tree for machines which need fixing.
>
>> It'd be easy enough to revert the logic if someone's willing to start on
>> this. I'm reluctant to make this patchset depend on changing all
>> machines, but if you think I'm wrong, pls let me know.
>
> Most machines don't have a PCI controller, luckily.
> I'll have a look at how many files it would touch...
So, first up I'm happy that the gpex and versatile
pci controllers don't have this problem (the way they
set up the mmio and io windows means there won't be
overlaps). That leaves the following pci controllers,
which I've listed with the source files which define machines
that use them:
hw/pci-host/pbm
hw/sparc64/sun4u.c
hw/pci-host/bonito
hw/mips/mips_fulong2e.c
hw/pci-host/grackle
hw/pc/mac_oldworld.c
hw/pci-host/piix
[the x86 systems]
hw/pci-host/ppce500
hw/ppc/e500.c
hw/pci-host/prep
hw/ppc/prep.c
hw/pci-host/q35
[x86 systems]
hw/pci-host/uninorth
hw/ppc/mac_newworld.c
hw/alpha/typhoon
used in hw/alpha/dp264.c
hw/mips/gt64xxx_pci
hw/mips/mips_malta.c
hw/ppc/ppc4xx_pci
hw/ppc/ppc440_bamboo.c
hw/ppc/spapr_pci
hw/ppc/spapr.c [already marked as '0 addrs ok']
hw/s390/s390-pci-bus
hw/s390/s390-virtio-ccw.c
hw/sh4/sh_pci
hw/sh4/rd2.c
So we'd need to touch perhaps fifteen files in total.
I don't insist we do that rather than applying this particular
patch, but I don't think it's a huge effort.
thanks
-- PMM
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, (continued)
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/15
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Alexander Gordeev, 2015/10/15
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Alexander Gordeev, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/15
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Laurent Vivier, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Michael S. Tsirkin, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Michael S. Tsirkin, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/13
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Laurent Vivier, 2015/10/16
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Michael S. Tsirkin, 2015/10/16
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/16
- Re: [Qemu-devel] [PATCH] hw/arm/virt: Allow zero address for PCI IO space, Laurent Vivier, 2015/10/16
- [Qemu-devel] [PATCH v2] hw/arm/virt: Allow zero address for PCI IO space, Alexander Gordeev, 2015/10/15
- Re: [Qemu-devel] [PATCH v2] hw/arm/virt: Allow zero address for PCI IO space, Peter Maydell, 2015/10/15