Re: [PATCHv4 2/2] powerpc: implement arch_scale_smt_power for Power7

2010-02-23 Thread Michael Neuling
In message <1266942281.11845.521.ca...@laptop> you wrote: > On Tue, 2010-02-23 at 17:08 +1100, Michael Neuling wrote: > > > I have some comments on the code inline but... > > > > So when I run this, I don't get processes pulled down to the lower > > threads. A simple test case of running 1 CPU

Re: [PATCH 2/4] Fix G5 thermal shutdown

2010-02-23 Thread Benjamin Herrenschmidt
On Tue, 2010-02-23 at 12:48 +, David Woodhouse wrote: > On Tue, 2010-02-09 at 11:55 +1100, Benjamin Herrenschmidt wrote: > > On Fri, 2010-02-05 at 08:52 -0500, Josh Boyer wrote: > > > This changes the thresholds for the liquid cooled G5 thermal > > > shutdown mechanism to prevent an errant shut

[Fwd: [PATCH] windfarm: init sysfs attributes]

2010-02-23 Thread Benjamin Herrenschmidt
--- Begin Message --- This is required for lockdep. Signed-off-by: Johannes Berg --- drivers/macintosh/windfarm_core.c |1 + drivers/macintosh/windfarm_smu_controls.c |1 + --- wireless-testing.orig/drivers/macintosh/windfarm_smu_controls.c 2010-02-20 16:05:55.738721565 +01

Re: simpleImage.XX and large kernels

2010-02-23 Thread David Gibson
On Mon, Feb 08, 2010 at 08:47:44PM +1000, John Williams wrote: > Hi, > > I'm looking at the simpleImage.XXX make target (PPC 405/440), and it > seems that by default the arch/powerpc/boot/wrapper script places the > bootwrapper at 0x40, effectively setting a limit on the maximum > bootable ker

Re: [PATCH 3/4] Provide VIO modalias

2010-02-23 Thread Brian King
On 02/23/2010 06:50 AM, David Woodhouse wrote: >> +static ssize_t modalias_show(struct device *dev, struct device_attribute >> *attr, >> + char *buf) >> +{ >> +const struct vio_dev *vio_dev = to_vio_dev(dev); >> +struct device_node *dn; >> +const char *cp; >> +

[PATCH 2/2] powerpc: Add power management support to VIO bus

2010-02-23 Thread Brian King
Adds support for suspend/resume for VIO devices. This is needed for support for HMC initiated hibernation. Signed-off-by: Brian King --- arch/powerpc/kernel/vio.c | 24 1 file changed, 24 insertions(+) diff -puN arch/powerpc/kernel/vio.c~powerpc_vio_bus_pm arch/powe

[PATCH 1/2] powerpc: Partition hibernation support

2010-02-23 Thread Brian King
Enables support for HMC initiated partition hibernation. This is a firmware assisted hibernation, since the firmware handles writing the memory out to disk, along with other partition information, so we just mimic suspend to ram. Signed-off-by: Brian King --- arch/powerpc/Kconfig

[PATCH] powerpc/booke: Fix a couple typos in the advanced ptrace code

2010-02-23 Thread Dave Kleikamp
powerpc/booke: Fix a couple typos in the advanced ptrace code Found and fixed a couple typos in the advanced ptrace patches. (These patches are currently in benh's next tree.) Signed-off-by: Dave Kleikamp Cc: Benjamin Herrenschmidt Cc: linuxppc-dev list --- diff --git a/arch/powerpc/include/

Re: [PATCH v5 0/5] PCI: try "pci=use_crs" again

2010-02-23 Thread Jesse Barnes
On Tue, 23 Feb 2010 10:24:15 -0700 Bjorn Helgaas wrote: > Historically, Linux has assumed a single PCI host bridge, with that > bridge claiming all the address space left after RAM and legacy > devices are taken out. > > If the system contains multiple host bridges, we can no longer > operate un

[PATCH v5 5/5] x86/PCI: use host bridge _CRS info by default on 2008 and newer machines

2010-02-23 Thread Bjorn Helgaas
The main benefit of using ACPI host bridge window information is that we can do better resource allocation in systems with multiple host bridges, e.g., http://bugzilla.kernel.org/show_bug.cgi?id=14183 Sometimes we need _CRS information even if we only have one host bridge, e.g., https://bugs.laun

[PATCH v5 4/5] PCI: augment bus resource table with a list

2010-02-23 Thread Bjorn Helgaas
Previously we used a table of size PCI_BUS_NUM_RESOURCES (16) for resources forwarded to a bus by its upstream bridge. We've increased this size several times when the table overflowed. But there's no good limit on the number of resources because host bridges and subtractive decode bridges can f

[PATCH v5 3/5] PCI: add pci_bus_for_each_resource(), remove direct bus->resource[] refs

2010-02-23 Thread Bjorn Helgaas
No functional change; this converts loops that iterate from 0 to PCI_BUS_NUM_RESOURCES through pci_bus resource[] table to use the pci_bus_for_each_resource() iterator instead. This doesn't change the way resources are stored; it merely removes dependencies on the fact that they're in a table. S

[PATCH v5 2/5] PCI: read bridge windows before filling in subtractive decode resources

2010-02-23 Thread Bjorn Helgaas
No functional change; this fills in the bus subtractive decode resources after reading the bridge window information rather than before. Also, print out the subtractive decode resources as we already do for the positive decode windows. Signed-off-by: Bjorn Helgaas --- drivers/pci/probe.c |

[PATCH v5 1/5] PCI: split up pci_read_bridge_bases()

2010-02-23 Thread Bjorn Helgaas
No functional change; this breaks up pci_read_bridge_bases() into separate pieces for the I/O, memory, and prefetchable memory windows, similar to how Yinghai recently split up pci_setup_bridge() in 68e84ff3bdc. Signed-off-by: Bjorn Helgaas --- drivers/pci/probe.c | 54 ++

[PATCH v5 0/5] PCI: try "pci=use_crs" again

2010-02-23 Thread Bjorn Helgaas
Historically, Linux has assumed a single PCI host bridge, with that bridge claiming all the address space left after RAM and legacy devices are taken out. If the system contains multiple host bridges, we can no longer operate under that assumption. We have to know what parts of the address space

Re: [PATCHv4 2/2] powerpc: implement arch_scale_smt_power for Power7

2010-02-23 Thread Peter Zijlstra
On Tue, 2010-02-23 at 17:24 +0100, Peter Zijlstra wrote: > > busiest_cpu = group_first_cpu(sds->busiest); > if (cpu_rq(this_cpu)->nr_running || this_cpu > busiest_cpu) > return 0; Hmm, we could change the bit in find_busiest_group() to: if (idle == CPU_IDLE &&

Re: [PATCHv4 2/2] powerpc: implement arch_scale_smt_power for Power7

2010-02-23 Thread Peter Zijlstra
On Tue, 2010-02-23 at 17:08 +1100, Michael Neuling wrote: > I have some comments on the code inline but... > > So when I run this, I don't get processes pulled down to the lower > threads. A simple test case of running 1 CPU intensive process at > SCHED_OTHER on a machine with 2 way SMT system

Re: [PATCH] powerpc: Set a smaller value for RECLAIM_DISTANCE to enable zone reclaim

2010-02-23 Thread Mel Gorman
On Tue, Feb 23, 2010 at 12:55:51PM +1100, Anton Blanchard wrote: > > Hi Mel, > I'm afraid I'm on vacation at the moment. This mail is costing me shots with penaltys every minute it's open. It'll be early next week before I can look at this closely. Sorry. > > You're pretty much on the button

Re: [PATCH v4 4/5] PCI: augment bus resource table with a list

2010-02-23 Thread Bjorn Helgaas
Here's the update (only added EXPORT_SYMBOL_GPL for pci_resource_n). I'll repost the whole series if that's easier for you. Bjorn commit 72b4d74167af9cec8e68cd6d54eda5d3cb894c57 Author: Bjorn Helgaas Date: Tue Feb 9 15:33:27 2010 -0700 PCI: augment bus resource table with a list

Re: [PATCH v4 04/11] mtd: Add MPC5121 NAND Flash Controller driver

2010-02-23 Thread Kári Davíðsson
Any special reason this in MPC5121 Revision 2 only? I did not see anything that contradicted the rev 3 manual in the code. But there is one harmless bug (since the NFC_CMD_FAIL symbol is never used), i.e. git diff diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c inde

Re: [PATCH 4/4] Fix iMac iSight PCI bridge setup

2010-02-23 Thread David Woodhouse
On Fri, 2010-02-05 at 20:03 -0500, Josh Boyer wrote: > On Sat, Feb 06, 2010 at 07:57:46AM +1100, Benjamin Herrenschmidt wrote: > >On Fri, 2010-02-05 at 08:55 -0500, Josh Boyer wrote: > >> This works around the PCIe bridge setup on the iMac iSight boxen. > > > >Is this still needed ? I though I had

Re: [PATCH 3/4] Provide VIO modalias

2010-02-23 Thread David Woodhouse
On Fri, 2010-02-05 at 08:53 -0500, Josh Boyer wrote: > Provide a modalias entry for VIO devices in sysfs. I believe > this was another initrd generation bugfix for anaconda. > > --- Signed-off-by: David Woodhouse Not entirely sure why this didn't get upstream when I first did it; maybe I just

Re: [PATCH 2/4] Fix G5 thermal shutdown

2010-02-23 Thread David Woodhouse
On Tue, 2010-02-09 at 11:55 +1100, Benjamin Herrenschmidt wrote: > On Fri, 2010-02-05 at 08:52 -0500, Josh Boyer wrote: > > This changes the thresholds for the liquid cooled G5 thermal > > shutdown mechanism to prevent an errant shutdown. > > > > This has been carried since about Fedora Core 5. I

Re: [PATCH 1/4] Add ps3_storage module alias

2010-02-23 Thread David Woodhouse
On Fri, 2010-02-05 at 14:49 +0100, Geert Uytterhoeven wrote: > On Fri, 5 Feb 2010, Josh Boyer wrote: > > This allows the upgrade path from older kernel releases to work > > with the Fedora mkinitrd tool. > > > > --- > > > > --- linux-2.6.22.ppc64/drivers/block/ps3disk.c~ 2007-07-25 > > 16:

Re: [Patch 1/1] PPC64-HWBKPT: Implement hw-breakpoints for PPC64

2010-02-23 Thread K.Prasad
On Mon, Feb 22, 2010 at 06:47:46PM +0530, K.Prasad wrote: > On Sun, Feb 21, 2010 at 02:01:37AM +0100, Frederic Weisbecker wrote: > > On Mon, Feb 15, 2010 at 11:29:14AM +0530, K.Prasad wrote: [snipped] > > Also, do you think addr/len/type is enough to abstract out > > any ppc breakpoints? > > > > T