[Top][All Lists]

[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

From: Andreas Färber
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH 2/2] pseries: Configure PCI bridge using properties
Date: Wed, 14 Mar 2012 14:30:15 +0100
User-agent: 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

reply via email to

[Prev in Thread] Current Thread [Next in Thread]