[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH 2/2] pseries: Configure PCI bridge us
Re: [Qemu-ppc] [Qemu-devel] [PATCH 2/2] pseries: Configure PCI bridge using properties
Wed, 14 Mar 2012 14:30:15 +0100
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.2) Gecko/20120215 Thunderbird/10.0.2
Am 13.03.2012 04:50, schrieb David Gibson:
> On Sat, Mar 10, 2012 at 11:40:41PM +0100, Alexander Graf wrote:
>> On 08.03.2012, at 02:12, David Gibson wrote:
>>> Currently, the function spapr_create_phb() uses its parameters to
>>> initialize the correct memory windows for the new PCI Host Bridge
>>> (PHB). This is not the way things are supposed to be done with qdevs,
>>> and means you can't create extra PHBs easily using -device.
>>> Since pSeries machines can and do have many PHBs with various
>>> configurations, this is a real limitation, not just a theoretical.
>>> This patch, therefore, alters the PHB initialization code to use qdev
>>> properties to set these parameters of the new bridge, moving most of
>>> the code from spapr_create_phb() to spapr_phb_init().
>>> While we're at it, we change the naming of each PCI bus and its
>>> associated memory regions to be less arbitrary and make it easier to
>>> relate the guest and qemu views of memory to each other.
>>> Signed-off-by: Alexey Kardashevskiy <address@hidden>
>>> Signed-off-by: David Gibson <address@hidden>
>> Andreas, could you please (n)ack this version?
> Actually, can you review and merge this updated version instead.
> Turns out that libvirt and some other tools assume that there will be
> a PCI bus named simply "pci", which is no longer true with the patch
> above (we call each bus "address@hidden" to match the guest's device
> tree). The revised version calls the default bus simply "pci" to work
> around libvirt's PC centrism, and allows manually added extra busses
> to be named explicitly with a property, defaulting to "address@hidden".
I was happy with David's improvements but didn't feel comfortable acking
the overall patch.
Reviewed-by: Andreas Färber <address@hidden>
One minor improvement in case this gets gespun:
* spapr_phb_info could be made static const (was/is static)
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg