Re: ramdisk size is larger than 4MB

2010-08-03 Thread Shawn Jin
> I did more debugging and something is really weird though. When the > link address is changed to 0x80, when stepping through the kernel, > I actually got the kernel boot successfully. However I let the kernel > run through it would just crash. After crash the BDI2000 shows it > stopped at __d

[PATCH] memblock: Fix memblock_is_region_reserved() to return a boolean

2010-08-03 Thread Benjamin Herrenschmidt
All callers expect a boolean result which is true if the region overlaps a reserved region. However, the implementation actually returns -1 if there is no overlap, and a region index (0 based) if there is. Make it behave as callers (and common sense) expect. Signed-off-by: Benjamin Herrenschmidt

RE: [PATCH 2/3] P4080/mtd: Only make elbc nand driver detect nand flash partitions

2010-08-03 Thread Zang Roy-R61911
> -Original Message- > From: Kumar Gala [mailto:ga...@kernel.crashing.org] > Sent: Tuesday, August 03, 2010 20:58 PM > To: Zang Roy-R61911 > Cc: linuxppc-...@ozlabs.org list; Lan Chunhe-B25806; Gala Kumar-B11780 > Subject: Re: [PATCH 2/3] P4080/mtd: Only make elbc nand > driver detect

RE: [PATCH 1/3 v2] sdhci: Add auto CMD12 support for eSDHC driver

2010-08-03 Thread Zang Roy-R61911
> -Original Message- > From: Andrew Morton [mailto:a...@linux-foundation.org] > Sent: Wednesday, August 04, 2010 7:44 AM > To: Zang Roy-R61911 > Cc: linux-...@vger.kernel.org; linuxppc-...@ozlabs.org > Subject: Re: [PATCH 1/3 v2] sdhci: Add auto CMD12 support for > eSDHC driver > > On

Re: [PATCH 1/3 v2] sdhci: Add auto CMD12 support for eSDHC driver

2010-08-03 Thread Andrew Morton
On Tue, 3 Aug 2010 11:11:10 +0800 Roy Zang wrote: > --- a/drivers/mmc/host/sdhci.h > +++ b/drivers/mmc/host/sdhci.h > @@ -240,6 +240,8 @@ struct sdhci_host { > #define SDHCI_QUIRK_CAP_CLOCK_BASE_BROKEN(1<<25) > /* Controller cannot support End Attribute in NOP ADMA descriptor */ >

Re: [PATCH 1/3] powerpc: Optimise 64bit csum_partial

2010-08-03 Thread Segher Boessenkool
> > Hi Segher, > >> Not really. Do you know how many 16/32-bit words you can add before a >> 64-bit register can overflow? :-) > > Thats a very good point. I thought about using 32bit adds when writing > the copy and checksum routine, but came to the conclusion that it wouldn't > go > any faster t

Re: [PATCH 1/3] powerpc: Optimise 64bit csum_partial

2010-08-03 Thread Anton Blanchard
Hi Segher, > Not really. Do you know how many 16/32-bit words you can add before a > 64-bit register can overflow? :-) Thats a very good point. I thought about using 32bit adds when writing the copy and checksum routine, but came to the conclusion that it wouldn't go any faster than one using a

Re: [PATCH] powerpc/kdump: Stop all other CPUs before running crash handlers

2010-08-03 Thread Matt Evans
Anton Blanchard wrote: > During kdump we run the crash handlers first then stop all other CPUs. > We really want to stop all CPUs as close to the fail as possible and also > have a very controlled environment for running the crash handlers, so it > makes sense to reverse the order. > > Signed-off-

Re: [PATCH 8/9] arch/powerpc/kernel: Drop unnecessary null test

2010-08-03 Thread Grant Likely
On Tue, Aug 3, 2010 at 3:51 PM, Dan Carpenter wrote: > On Tue, Aug 03, 2010 at 11:35:17PM +0200, Julia Lawall wrote: >> diff --git a/arch/powerpc/kernel/pci_of_scan.c >> b/arch/powerpc/kernel/pci_of_scan.c >> index 6ddb795..62dd363 100644 >> --- a/arch/powerpc/kernel/pci_of_scan.c >> +++ b/arch/p

Re: [PATCH 8/9] arch/powerpc/kernel: Drop unnecessary null test

2010-08-03 Thread Dan Carpenter
On Tue, Aug 03, 2010 at 11:35:17PM +0200, Julia Lawall wrote: > diff --git a/arch/powerpc/kernel/pci_of_scan.c > b/arch/powerpc/kernel/pci_of_scan.c > index 6ddb795..62dd363 100644 > --- a/arch/powerpc/kernel/pci_of_scan.c > +++ b/arch/powerpc/kernel/pci_of_scan.c > @@ -336,8 +336,7 @@ static void

[PATCH 8/9] arch/powerpc/kernel: Drop unnecessary null test

2010-08-03 Thread Julia Lawall
From: Julia Lawall list_for_each_entry binds its first argument to a non-null value, and thus any null test on the value of that argument is superfluous. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ iterator I; expression x,E,E1,E2; statement S,S1

[PATCH 4/9] arch/powerpc/platforms/powermac: Drop unnecessary null test

2010-08-03 Thread Julia Lawall
From: Julia Lawall for_each_node_by_name binds its first argument to a non-null value, and thus any null test on the value of that argument is superfluous. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ iterator I; expression x,E; @@ I(x,...) { <..

[PATCH] arch/powerpc: Drop unnecessary of_node_put

2010-08-03 Thread Julia Lawall
From: Julia Lawall for_each_node_by_name only exits when its first argument is NULL, and a subsequent call to of_node_put on that argument is unnecessary. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ iterator name for_each_node_by_name; expression

Re: Commit 3da34aa brakes MSI support on MPC8308 (possibly all MPC83xx) [REPOST]

2010-08-03 Thread Wolfgang Denk
Hello Kumar, hello Kim, can you _please_ comment? Thanks. In message <20100729212043.4258c152...@gemini.denx.de> I wrote: > Dear Kumar & Kim, > > any comments on this issue? > > Thanks. > > In message <4c48b384.1020...@emcraft.com> Ilya Yanok wrote: > > Hi Kumar, Kim, Josh, everybody, > >

Re: [PATCH 00/27] KVM PPC PV framework v3

2010-08-03 Thread Scott Wood
On Sun, 1 Aug 2010 22:21:37 +0200 Alexander Graf wrote: > > On 01.08.2010, at 16:02, Avi Kivity wrote: > > > Looks reasonable. Since it's fair to say I understand nothing about > > powerpc, I'd like someone who does to review it and ack, please, with an > > emphasis on the interfaces. > > S

Re: [PATCH] powerpc: Dont require a dma_ops struct to set dma mask

2010-08-03 Thread roger blofeld
- Original Message > From: Kumar Gala > To: linuxppc-...@ozlabs.org > Sent: Mon, August 2, 2010 12:21:22 PM > Subject: [PATCH] powerpc: Dont require a dma_ops struct to set dma mask > > The only reason to require a dma_ops struct is to see if it has > implemented set_dma_mask. If not

Re: [PATCH 1/3] powerpc: Optimise 64bit csum_partial

2010-08-03 Thread Segher Boessenkool
On both POWER6 and POWER7 this should be as fast as we can go since we are limited by the latency of the adde instructions. Not really. Do you know how many 16/32-bit words you can add before a 64-bit register can overflow? :-) If you ever have to call this with more than 16GB of data to sum, t

[PATCH 9/9] v4 Update memory-hotplug documentation

2010-08-03 Thread Nathan Fontenot
Update the memory hotplug documentation to reflect the new behaviors of memory blocks reflected in sysfs. Signed-off-by: Nathan Fontenot --- Documentation/memory-hotplug.txt | 40 +++ 1 file changed, 24 insertions(+), 16 deletions(-) Index: linux-2.6/Docume

setting up of platform device archdata?

2010-08-03 Thread Kumar Gala
Greg, We are running into in issue in that we need the platform device archdata setup before platform_device_add gets called since we need some data in archdata setup to allow the bus notifiers to work properly. On PPC we need this to get the dma masks setup properly that we than use in the bu

[PATCH 8/9] v4 Define memory_block_size_bytes() for ppc/pseries

2010-08-03 Thread Nathan Fontenot
Define a version of memory_block_size_bytes() for powerpc/pseries such that a memory block spans an entire lmb. Signed-off-by: Nathan Fontenot --- arch/powerpc/platforms/pseries/hotplug-memory.c | 66 +++- 1 file changed, 53 insertions(+), 13 deletions(-) Index: linux-2.6/

[PATCH 7/9] v4 Update the node sysfs code

2010-08-03 Thread Nathan Fontenot
Update the node sysfs code to be aware of the new capability for a memory block to contain multiple memory sections. This requires an additional parameter to unregister_mem_sect_under_nodes so that we know which memory section of the memory block to unregister. Signed-off-by: Nathan Fontenot ---

[PATCH 6/9] v4 Update the find_memory_block declaration

2010-08-03 Thread Nathan Fontenot
Update the find_memory_block declaration to to take a struct mem_section * so that it matches the definition. Signed-off-by: Nathan Fontenot --- include/linux/memory.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-2.6/include/linux/memory.h ==

[PATCH 5/9] v4 Allow memory_block to span multiple memory sections

2010-08-03 Thread Nathan Fontenot
Update the memory sysfs code that each sysfs memory directory is now considered a memory block that can contain multiple memory sections per memory block. The default size of each memory block is SECTION_SIZE_BITS to maintain the current behavior of having a single memory section per memory block

[PATCH 4/9] v4 Add mutex for add/remove of memory blocks

2010-08-03 Thread Nathan Fontenot
Add a new mutex for use in adding and removing of memory blocks. This is needed to avoid any race conditions in which the same memory block could be added and removed at the same time. Signed-off-by: Nathan Fontenot --- drivers/base/memory.c |9 + 1 file changed, 9 insertions(+) In

[PATCH 3/9] v4 Add section count to memory_block

2010-08-03 Thread Nathan Fontenot
Add a section count property to the memory_block struct to track the number of memory sections that have been added/removed from a memory block. This allows us to know when the last memory section of a memory block has been removed so we can remove the memory block. Signed-off-by: Nathan Fontenot

[PATCH 2/9] v4 Add new phys_index properties

2010-08-03 Thread Nathan Fontenot
Update the 'phys_index' properties of a memory block to include a 'start_phys_index' which is the same as the current 'phys_index' property. The property still appears as 'phys_index' in sysfs but the memory_block struct name is updated to indicate the start and end values. This also adds an 'end_p

[PATCH 1/9] v4 Move the find_memory_block() routine up

2010-08-03 Thread Nathan Fontenot
Move the find_memory_block() routine up to avoid needing a forward declaration in subsequent patches. Signed-off-by: Nathan Fontenot --- drivers/base/memory.c | 62 +- 1 file changed, 31 insertions(+), 31 deletions(-) Index: linux-2.6/drivers/ba

[PATCH 0/9] v4 De-couple sysfs memory directories from memory sections

2010-08-03 Thread Nathan Fontenot
This set of patches de-couples the idea that there is a single directory in sysfs for each memory section. The intent of the patches is to reduce the number of sysfs directories created to resolve a boot-time performance issue. On very large systems boot time are getting very long (as seen on pow

RE: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-03 Thread Bounine, Alexandre
Yang Li pointed to these patches in his post from July 23, 2010. It would be nice to have these patches in mainline code. > -Original Message- > From: timur.t...@gmail.com [mailto:timur.t...@gmail.com] On Behalf Of Timur Tabi > Sent: Tuesday, August 03, 2010 9:02 AM > To: Bounine, Alexand

Re: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-03 Thread Timur Tabi
On Tue, Aug 3, 2010 at 7:17 AM, Bounine, Alexandre wrote: > This happened after change to book-e definitions. > There are patches that address this issue. And those patches should have been applied before 2.6.35 was released. Someone dropped the ball. 2.6.35 is broken for a number of PowerPC bo

Re: [PATCH 2/3] P4080/mtd: Only make elbc nand driver detect nand flash partitions

2010-08-03 Thread Kumar Gala
On Aug 2, 2010, at 11:45 PM, Roy Zang wrote: > From: Lan Chunhe-B25806 > > The former driver had the two functions: > > 1. detecting nand flash partitions; > 2. registering elbc interrupt. > > Now, second function is removed to fsl_lbc.c. > > Signed-off-by: Lan Chunhe-B25806 > Signed-off-by

Re: Issues to access Compact Flash Card on MPC8360E

2010-08-03 Thread Atul Deshmukh
Thanks a lot Anton, I was confused with "ata-generic" entry at p...@3.0 node. Now the things are pretty much clear.. Earlier we thought that IORD and IOWR pins would be from GPIO, so thought had to do bit-banging. But after going through the schematics, we come to know that pins could be from LGPL

RE: [PATCH v2 5/7] powerpc/85xx: Add MChk handler for SRIO port

2010-08-03 Thread Bounine, Alexandre
This happened after change to book-e definitions. There are patches that address this issue. > -Original Message- > From: Michael Neuling [mailto:mi...@neuling.org] > Sent: Tuesday, August 03, 2010 2:07 AM > To: Timur Tabi > Cc: Alexandre Bounine; linuxppc-dev@lists.ozlabs.org; linux-ker..

Re: ramdisk size is larger than 4MB

2010-08-03 Thread Shawn Jin
> I found the link_address in the wrapper shell script sets the _start > address. But after changing it to 0x80, the kernel failed to boot, > shown below. There must be something also needs proper adjustment. > What would that be? I did more debugging and something is really weird though. When