Re: [PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-16 Thread Benjamin Herrenschmidt
On Sun, 2013-06-16 at 15:27 +0800, Gavin Shan wrote: > Thanks for the review, Ben. > > >Getting better but: > > > > - I still don't like having a kthread for that. Why not use schedule_work() > > ? > > > > Ok. Will update it with schedule_work() in next revision :-) > > > - We already have

Re: [PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-16 Thread Gavin Shan
On Sun, Jun 16, 2013 at 03:12:11PM +1000, Benjamin Herrenschmidt wrote: >On Sat, 2013-06-15 at 17:03 +0800, Gavin Shan wrote: >> On PowerNV platform, the EEH event is produced either by detect >> on accessing config or I/O registers, or by interrupts dedicated >> for EEH report. The patch adds supp

Re: [PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-15 Thread Benjamin Herrenschmidt
On Sat, 2013-06-15 at 17:03 +0800, Gavin Shan wrote: > On PowerNV platform, the EEH event is produced either by detect > on accessing config or I/O registers, or by interrupts dedicated > for EEH report. The patch adds support to process the interrupts > dedicated for EEH report. > > Firstly, the

[PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-15 Thread Gavin Shan
On PowerNV platform, the EEH event is produced either by detect on accessing config or I/O registers, or by interrupts dedicated for EEH report. The patch adds support to process the interrupts dedicated for EEH report. Firstly, the kernel thread will be waken up to process incoming interrupt. The

Re: [PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-12 Thread Gavin Shan
On Tue, Jun 11, 2013 at 06:13:55PM +1000, Benjamin Herrenschmidt wrote: >On Wed, 2013-06-05 at 15:34 +0800, Gavin Shan wrote: > >> diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h >> index d1fd5d4..68ac408 100644 >> --- a/arch/powerpc/include/asm/eeh.h >> +++ b/arch/powe

Re: [PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-11 Thread Benjamin Herrenschmidt
On Wed, 2013-06-05 at 15:34 +0800, Gavin Shan wrote: > diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h > index d1fd5d4..68ac408 100644 > --- a/arch/powerpc/include/asm/eeh.h > +++ b/arch/powerpc/include/asm/eeh.h > @@ -209,6 +209,12 @@ void eeh_add_device_tree_late(str

[PATCH 21/27] powerpc/eeh: Process interrupts caused by EEH

2013-06-05 Thread Gavin Shan
On PowerNV platform, the EEH event is produced either by detect on accessing config or I/O registers, or by interrupts dedicated for EEH report. The patch adds support to process the interrupts dedicated for EEH report. Firstly, the kernel thread will be waken up to process incoming interrupt. The