[PATCH 2/2] powerpc: Fix build without CONFIG_PCI

2011-08-04 Thread Benjamin Herrenschmidt
Commit fea80311a939a746533a6d7e7c3183729d6a3faf "iomap: make IOPORT/PCI mapping functions conditional" Broke powerpc build without CONFIG_PCI as we would still define pci_iomap(), which overlaps with the new empty inline in the headers. Make our implementation conditional on CONFIG_PCI Signed-of

[PATCH] powerpc/4xx: Fix build of PCI code on 405

2011-08-04 Thread Benjamin Herrenschmidt
Commit 112d1fe9f7715db423ffeec5ac1beccff6093dc4 "powerpc/4xx: Add check_link to struct ppc4xx_pciex_hwops" inadvertently broke 405 builds due to some functions being over protected by an ifdef CONFIG_44x. Move them back out. Signed-off-by: Benjamin Herrenschmidt --- arch/powerpc/sysdev/ppc4xx_p

[PATCH] powerpc: Make KVM_GUEST default to n

2011-08-04 Thread Anton Blanchard
KVM_GUEST adds a 1 MB array to the kernel (kvm_tmp) which grew my kernel enough to cause it to fail to boot. Dynamically allocating or reducing the size of this array is a good idea, but in the meantime I think it makes sense to make KVM_GUEST default to n in order to minimise surprises. Signed-

MESH SCSI driver not working

2011-08-04 Thread Pol Vangheluwe
On Fri, 2010-03-19 at 00:21 +0100, Stef Simoens wrote: > Hello, > > Some time ago (July 24th 2009 my mailbox says) I emailed you and the > linuxppc-dev list about my problems booting from the mesh SCSI > controller. > > I just compiled 2.6.31 (actually, gentoo-sources-2.6.31-r10); but the > probl

Re: [PATCH 03/15] [PS3] Add region 1 memory early

2011-08-04 Thread Geoff Levand
Hi Hector, On 08/03/2011 05:08 PM, Hector Martin wrote: > On 08/04/2011 12:32 AM, Geoff Levand wrote: >> We need an explanation of this change. Sorry for such a terse request. What I meant was that this is a significant change to how high mem is managed, so the patch needs a comment explaining t

Re: [Cbe-oss-dev] [PATCH 07/15] ps3flash: Refuse to work in lpars other than OtherOS

2011-08-04 Thread Geert Uytterhoeven
On Thu, Aug 4, 2011 at 18:40, Andre Heider wrote: > On Thu, Aug 4, 2011 at 12:34 AM, Geoff Levand wrote: >> On 08/01/2011 01:02 PM, Andre Heider wrote: >>> --- a/drivers/char/ps3flash.c >>> +++ b/drivers/char/ps3flash.c >>> @@ -25,6 +25,7 @@ >>> >>>  #include >>>  #include >>> +#include >>> >>

Re: [PATCH 02/15] [PS3] Get lv1 high memory region from devtree

2011-08-04 Thread Geoff Levand
On 08/03/2011 06:19 PM, Hector Martin wrote: > On 08/04/2011 12:30 AM, Geoff Levand wrote: >> How would a kexec based bootloader work? If it's kernel were to allocate >> high mem and the bootloader program uses the high mem, how could it tell >> that kernel not to destroy the region on shutdown? >

Re: [PATCH 07/15] ps3flash: Refuse to work in lpars other than OtherOS

2011-08-04 Thread Andre Heider
On Thu, Aug 4, 2011 at 12:34 AM, Geoff Levand wrote: > On 08/01/2011 01:02 PM, Andre Heider wrote: >> The driver implements a character and misc device, meant for the >> axed OtherOS to exchange various settings with GameOS. >> Since Firmware 3.21 there is no GameOS support anymore to write these

Re: [PATCH 01/15] [PS3] Add udbg driver using the PS3 gelic Ethernet device

2011-08-04 Thread Andre Heider
On Thu, Aug 4, 2011 at 12:32 AM, Geoff Levand wrote: > On 08/01/2011 01:02 PM, Andre Heider wrote: >> --- /dev/null >> +++ b/arch/powerpc/platforms/ps3/gelic_udbg.c >> @@ -0,0 +1,272 @@ >> +/* >> + * arch/powerpc/platforms/ps3/gelic_udbg.c > > Don't put file names in files.  When the file gets mov

Re: [PATCH 05/15] ps3: Detect the current lpar environment

2011-08-04 Thread Andre Heider
On Thu, Aug 4, 2011 at 12:31 AM, Geoff Levand wrote: > On 08/01/2011 01:02 PM, Andre Heider wrote: >> --- >>  arch/powerpc/include/asm/ps3.h          |    7 +++ >>  arch/powerpc/platforms/ps3/platform.h   |    4 >>  arch/powerpc/platforms/ps3/repository.c |   19 +++ >>  ar

Re: [PATCH 00/15] ps3: Support more than the OtherOS lpar

2011-08-04 Thread Andre Heider
Hi Geoff, On Thu, Aug 4, 2011 at 12:23 AM, Geoff Levand wrote: > Hi Andre, > > On 08/01/2011 01:02 PM, Andre Heider wrote: >> This series addresses various issues and extends support when running >> in lpars like GameOS. Included are some patches from Hector Martin, which >> I found useful. > > M

Re: kvm PCI assignment & VFIO ramblings

2011-08-04 Thread Joerg Roedel
On Mon, Aug 01, 2011 at 02:27:36PM -0600, Alex Williamson wrote: > It's not clear to me how we could skip it. With VT-d, we'd have to > implement an emulated interrupt remapper and hope that the guest picks > unused indexes in the host interrupt remapping table before it could do > anything useful

Re: kvm PCI assignment & VFIO ramblings

2011-08-04 Thread Joerg Roedel
Hi Ben, thanks for your detailed introduction to the requirements for POWER. Its good to know that the granularity problem is not x86-only. On Sat, Jul 30, 2011 at 09:58:53AM +1000, Benjamin Herrenschmidt wrote: > In IBM POWER land, we call this a "partitionable endpoint" (the term > "endpoint" h

Re: kvm PCI assignment & VFIO ramblings

2011-08-04 Thread Joerg Roedel
On Sat, Jul 30, 2011 at 12:20:08PM -0600, Alex Williamson wrote: > On Sat, 2011-07-30 at 09:58 +1000, Benjamin Herrenschmidt wrote: > > - The -minimum- granularity of pass-through is not always a single > > device and not always under SW control > > But IMHO, we need to preserve the granularity of

Re: [PATCH 03/15] [PS3] Add region 1 memory early

2011-08-04 Thread Hector Martin
On 08/04/2011 09:05 AM, Geert Uytterhoeven wrote: > The reason for that is to make sure the allocations will succeed. > Chances are very > slim you can allocate a contiguous 9 MiB buffer at any arbitrary time. Fair enough, but then they don't need to happen as early as they do now; any time during

Re: [PATCH 03/15] [PS3] Add region 1 memory early

2011-08-04 Thread Geert Uytterhoeven
On Thu, Aug 4, 2011 at 02:08, Hector Martin wrote: > tight. Can we get rid of the ps3flash and ps3fb preallocations to save > bootmem and just allocate them during device init like the other drivers > do? What is the reason for preallocating these? The reason for that is to make sure the allocati