Re: [PATCH] powerpc/cell: Only iterate over online nodes in cbe_init_pm_irq()

2013-05-21 Thread Michael Ellerman
On Fri, May 17, 2013 at 05:45:05PM +0200, Dennis Schridde wrote: Hello! Just wanted to remind you: The patchto fix cbe_init_pm_irq() that Michael and Grant sent me is still not included in Linux 3.8.12. I didn't push that one to stable because it just fixes a warning. If you want it you'll

Re: [PATCH] powerpc/cell: Only iterate over online nodes in cbe_init_pm_irq()

2013-05-17 Thread Dennis Schridde
Hello! Just wanted to remind you: The patchto fix cbe_init_pm_irq() that Michael and Grant sent me is still not included in Linux 3.8.12. --Dennis Am Dienstag, 23. April 2013, 22:14:51 schrieb Michael Ellerman: None of the cell platforms support CPU hotplug, so we should iterate only over

[PATCH] powerpc/cell: Only iterate over online nodes in cbe_init_pm_irq()

2013-04-23 Thread Michael Ellerman
None of the cell platforms support CPU hotplug, so we should iterate only over online nodes when setting PMU interrupts. This also fixes a warning during boot when NODES_SHIFT is large enough: WARNING: at /scratch/michael/src/kmk/linus/kernel/irq/irqdomain.c:766 ... NIP [c00db278]

Re: [PATCH] powerpc/cell: Only iterate over online nodes in cbe_init_pm_irq()

2013-04-23 Thread Michael Ellerman
On Tue, Apr 23, 2013 at 02:45:50PM +0200, Dennis Schridde wrote: Hello everyone! I have been testing this patch (given to me by Grant Likely grant.lik...@secretlab.ca) with various kernel versions (3.6.2, 3.6.11, 3.8.6, 3.8.8) since November and can confirm that it solves part of the IRQ

Re: [PATCH] powerpc/cell: Only iterate over online nodes in cbe_init_pm_irq()

2013-04-23 Thread Dennis Schridde
Hello! Please find an up-to-date dmesg log attached. It was created from a Linux 3.8.8 kernel with these two patches applied: - for_each_node(node) { + for_each_online_node(node) { - return distance; + return ((a == b) ? LOCAL_DISTANCE : REMOTE_DISTANCE);