Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread Benjamin Herrenschmidt
On Mon, 2011-02-07 at 10:22 +0530, K.Prasad wrote: > Okay! Another wrapper of "#ifndef CONFIG_HAVE_HW_BREAKPOINT" around > the > definition of 'set_debug_reg_defaults'. Can you send a patch ? > There's indeed too much sprinkling of #ifdefs in the code, but most of > it would go away when the Book

Re: [PATCH v2 3/3] powerpc: Add IO event interrupt support

2011-02-06 Thread Benjamin Herrenschmidt
On Wed, 2011-01-05 at 16:44 -0600, Tseng-Hui (Frank) Lin wrote: > This patch adds support for handling IO Event interrupts which come > through at the /event-sources/ibm,io-events device tree node. .../... The previous patches regarding the error log sometimes call it error log and sometimes eve

Re: [PATCH 2/6] nvram: Capture oops/panic reports in ibm, oops-log partition

2011-02-06 Thread Benjamin Herrenschmidt
On Sat, 2010-11-13 at 20:15 -0800, Jim Keniston wrote: > Create the ibm,oops-log NVRAM partition, and capture the end of the printk > buffer in it when there's an oops or panic. If we can't create the > ibm,oops-log partition, capture the oops/panic report in ibm,rtas-log. Won't the parsing tools

Re: [PATCH 1/6] nvram: Generalize code for OS partitions in NVRAM

2011-02-06 Thread Benjamin Herrenschmidt
On Sat, 2010-11-13 at 20:15 -0800, Jim Keniston wrote: > Adapt the functions used to create and write to the RTAS-log partition > to work with any OS-type partition. > > Signed-off-by: Jim Keniston > --- Overall pretty good (sorry for taking that long to review, I've been swamped down). Just a h

Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread K.Prasad
On Mon, Feb 07, 2011 at 02:10:39PM +1100, Benjamin Herrenschmidt wrote: > On Mon, 2011-02-07 at 08:26 +0530, K.Prasad wrote: > > On Mon, Feb 07, 2011 at 09:54:13AM +1100, Benjamin Herrenschmidt wrote: > > > A typo in the #ifdef statement makes us never call it > > > in flush_thread() > > > > > > >

Re: [PATCH 5/6] nvram: Slim down zlib_deflate workspace when possible

2011-02-06 Thread Benjamin Herrenschmidt
On Sat, 2010-11-13 at 20:15 -0800, Jim Keniston wrote: > Instead of always creating a huge (268K) deflate_workspace with the > maximum compression parameters (windowBits=15, memLevel=8), allow the > caller to obtain a smaller workspace (24K in our case) by specifying > smaller parameter values -- v

Re: [PATCH] sched: provide scheduler_ipi() callback in response to smp_send_reschedule()

2011-02-06 Thread Benjamin Herrenschmidt
On Mon, 2011-01-17 at 12:07 +0100, Peter Zijlstra wrote: > diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c > index 68034bb..7ee0fc3 100644 > --- a/arch/powerpc/kernel/smp.c > +++ b/arch/powerpc/kernel/smp.c > @@ -128,6 +128,7 @@ static irqreturn_t call_function_action(int irq, vo

[PATCH] powerpc/pseries: Reduce HVCS driver insanity

2011-02-06 Thread Benjamin Herrenschmidt
The HVCS driver, for those who don't know, is a driver for the "server" side of the IBM virtual terminal mechanism allowing Linux partitions to act as terminal servers under IBM PowerVM hypervisor. It's almost never used on the field at the moment. However, it's part of our configs, and in its cur

[git pull] Please pull powerpc.git merge branch

2011-02-06 Thread Benjamin Herrenschmidt
Hi Linus ! Here are a few powerpc bug fixes for you. Mostly Anton and Jesse fixing up the new VPNH feature (which was added this cycle) and a handful of other fixes worth having. Cheers, Ben. The following changes since commit 8dbdea8444d303a772bceb1ba963f0e3273bfc5e: Merge branch 'ixp4xx' of

Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread Benjamin Herrenschmidt
On Mon, 2011-02-07 at 08:26 +0530, K.Prasad wrote: > On Mon, Feb 07, 2011 at 09:54:13AM +1100, Benjamin Herrenschmidt wrote: > > A typo in the #ifdef statement makes us never call it > > in flush_thread() > > > > I wish it never compiled for such typos :-) > > > > Signed-off-by: Benjamin Herren

Re: [PATCH -mm 2/6] powerpc: convert little-endian bitops macros to static inline functions

2011-02-06 Thread Akinobu Mita
2011/2/7 Benjamin Herrenschmidt : > On Thu, 2011-01-27 at 22:56 +0900, Akinobu Mita wrote: >> (This patch is intended to be folded into the patch in -mm: >> powerpc-introduce-little-endian-bitops.patch) >> >> The little-endian bitops on powerpc are written as preprocessor >> macros with the cast to

Re: [PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread K.Prasad
On Mon, Feb 07, 2011 at 09:54:13AM +1100, Benjamin Herrenschmidt wrote: > A typo in the #ifdef statement makes us never call it > in flush_thread() > I wish it never compiled for such typos :-) > Signed-off-by: Benjamin Herrenschmidt Acked-by: K.Prasad > --- > > diff --git a/arch/powerpc/ke

Re: [PATCH 3/4]Define memory_block_size_bytes for powerpc/pseries

2011-02-06 Thread Greg KH
On Mon, Feb 07, 2011 at 10:39:23AM +1100, Benjamin Herrenschmidt wrote: > On Thu, 2011-01-20 at 10:45 -0600, Nathan Fontenot wrote: > > 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 > > Re

Re: [PATCH v3 1/4] powerpc: Removing support for 'protected-sources'

2011-02-06 Thread Benjamin Herrenschmidt
On Sun, 2011-02-06 at 19:32 -0600, Meador Inge wrote: > So barring the removal of protected sources, does the inclusion of the > "pic-no-reset" property seem reasonable? Sure. Ben. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://l

Re: [PATCH v3 1/4] powerpc: Removing support for 'protected-sources'

2011-02-06 Thread Meador Inge
On 02/06/2011 05:35 PM, Benjamin Herrenschmidt wrote: On Fri, 2011-02-04 at 17:25 -0600, Meador Inge wrote: In a recent thread [1,2,3] concerning device trees for AMP systems, the question of whether we really need 'protected-sources' arose. The general consensus was that a new boolean property

Re: [PATCH] watchdog: booke_wdt: clean up status messages

2011-02-06 Thread Benjamin Herrenschmidt
On Mon, 2011-01-17 at 15:29 -0600, Timur Tabi wrote: > Improve the status messages that are displayed during some operations of the > PowerPC watchdog timer driver. When the watchdog is enabled, the timeout is > displayed as a number of seconds, instead of an obscure "period". The > "period" > i

Re: [PATCH 1/2] video, sm501: add OF binding to support SM501

2011-02-06 Thread Benjamin Herrenschmidt
On Sat, 2010-12-04 at 09:23 +0100, Heiko Schocher wrote: > - add binding to OF, compatible name "smi,sm501" > > - add read/write functions for using this driver > also on powerpc plattforms .../... Who plans to merge that patch series ? I'm happy for whoever is doing that to take the powerpc

RE: [PATCH] e500: Erratum cpu a005 workaround

2011-02-06 Thread Benjamin Herrenschmidt
> > > > This isn't the way to do this. We normally add entries in > > cputable.c an add a new cpu_feature_bit for the errata. > > > > Than above we'd do: > > > > if (cur_cpu_spec->cpu_features & CPU_FTR_E500_A005_ERRATUM) > > > > > > IMHO, a cpu erratum is not a cpu feature. > See there're

[PATCH 21/23] hvc_console: Fix race between hvc_close and hvc_remove

2011-02-06 Thread Willy Tarreau
2.6.27.58-stable review patch. If anyone has any objections, please let us know. -- From: Amit Shah commit e74d098c66543d0731de62eb747ccd5b636a6f4c upstream. Alan pointed out a race in the code where hvc_remove is invoked. The recent virtio_console work is the first user of h

Re: [PATCH 3/4]Define memory_block_size_bytes for powerpc/pseries

2011-02-06 Thread Benjamin Herrenschmidt
On Thu, 2011-01-20 at 10:45 -0600, Nathan Fontenot wrote: > 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 > Reviewed-by: Robin Holt Hi Nathan ! Is somebody from -mm picking the rest of the se

Re: [PATCH v3 1/4] powerpc: Removing support for 'protected-sources'

2011-02-06 Thread Benjamin Herrenschmidt
On Fri, 2011-02-04 at 17:25 -0600, Meador Inge wrote: > In a recent thread [1,2,3] concerning device trees for AMP systems, the > question of whether we really need 'protected-sources' arose. The general > consensus was that a new boolean property 'pic-no-reset' (described in more > detail in a fo

Re: [PATCH -mm 2/6] powerpc: convert little-endian bitops macros to static inline functions

2011-02-06 Thread Benjamin Herrenschmidt
On Thu, 2011-01-27 at 22:56 +0900, Akinobu Mita wrote: > (This patch is intended to be folded into the patch in -mm: > powerpc-introduce-little-endian-bitops.patch) > > The little-endian bitops on powerpc are written as preprocessor > macros with the cast to "unsigned long *". > This means that ev

[PATCH] powerpc: Fix call to flush_ptrace_hw_breakpoint()

2011-02-06 Thread Benjamin Herrenschmidt
A typo in the #ifdef statement makes us never call it in flush_thread() Signed-off-by: Benjamin Herrenschmidt --- diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index 7a1d5cb..c4e4eab 100644 --- a/arch/powerpc/kernel/process.c +++ b/arch/powerpc/kernel/process.c @@ -6