RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-21 Thread anish kumar
> > > Sent: Friday, October 12, 2012 11:25 PM > > > > To: Liu, Chuansheng > > > > Cc: Thomas Gleixner; linux-kernel@vger.kernel.org > > > > Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support > > > > with irq > > >

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-21 Thread anish kumar
To: Liu, Chuansheng Cc: Thomas Gleixner; linux-kernel@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: On SMP an interrupt which is raised after the ack

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread anish kumar
u, Chuansheng > > > Cc: Thomas Gleixner; linux-kernel@vger.kernel.org > > > Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with > > > irq > > > thread > > > > > > On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: >

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: > > > But IRQS_ONESHOT does not work well for edge interrupt. > > > And pasting the IRQS_ONESHOT description: > > > * IRQS_ONESHOT- irq is not unmasked in primary handler > > > > Right, and edge type interrupts doe not support it. >

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: > > On SMP an interrupt which is raised after the ack() again before the > > handler finishes, can invoke another delivery on a different CPU, > > which then sees the IRQ_INPROGESS flag, masks it and flags it > > PENDING. When the primary handler on the

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
ubject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with > > irq > > thread > > > > On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: > > > > On SMP an interrupt which is raised after the ack() again before the > > > > handl

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread anish kumar
ernel.org > > Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with > > irq > > thread > > > > On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: > > > > On SMP an interrupt which is raised after the ack() again before t

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
> -Original Message- > From: anish kumar [mailto:anish198519851...@gmail.com] > Sent: Friday, October 12, 2012 11:25 PM > To: Liu, Chuansheng > Cc: Thomas Gleixner; linux-kernel@vger.kernel.org > Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread anish kumar
On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: > > On SMP an interrupt which is raised after the ack() again before the > > handler finishes, can invoke another delivery on a different CPU, > > which then sees the IRQ_INPROGESS flag, masks it and flags it > > PENDING. When the primary

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
> On SMP an interrupt which is raised after the ack() again before the > handler finishes, can invoke another delivery on a different CPU, > which then sees the IRQ_INPROGESS flag, masks it and flags it > PENDING. When the primary handler on the first CPU returns, it sees > the PENDING flag,

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
> To: Liu, Chuansheng > Cc: linux-kernel@vger.kernel.org > Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq > thread > > On Fri, 12 Oct 2012, Liu, Chuansheng wrote: > > > -Original Message- > > > From: Thomas Gleixner [mailto:t.

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
y, October 12, 2012 8:32 PM > > > > To: Liu, Chuansheng > > > > Cc: linux-kernel@vger.kernel.org > > > > Subject: Re: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support > > with irq > > > > thread > > > > > > > > On

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
FLAG IRQS_ONESHOT does not work well for edge interrupt. * IRQS_ONESHOT - irq is not unmasked in primary handler Is it possible give some notification or remind? Thanks. I took some time to know it:) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Friday, October 12, 2012 8:46 PM > To: Liu, Chuansheng > Cc: linux-kernel@vger.kernel.org > Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq > thread >

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: > > -Original Message- > > From: Thomas Gleixner [mailto:t...@linutronix.de] > > Sent: Friday, October 12, 2012 8:32 PM > > To: Liu, Chuansheng > > Cc: linux-kernel@vger.kernel.org > > Subject: Re: [PATCH] gen

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
> -Original Message- > From: Thomas Gleixner [mailto:t...@linutronix.de] > Sent: Friday, October 12, 2012 8:32 PM > To: Liu, Chuansheng > Cc: linux-kernel@vger.kernel.org > Subject: Re: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq > thread >

Re: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Chuansheng Liu wrote: > > In our system, there is one edge interrupt, and we want it to be > irq thread with IRQS_ONESHOT, and found in handle_edge_irq(), > even with IRQS_ONESHOT, the irq is still unmasked without care of > flag IRQS_ONESHOT. > > It causes IRQS_ONESHOT can

Re: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Chuansheng Liu wrote: In our system, there is one edge interrupt, and we want it to be irq thread with IRQS_ONESHOT, and found in handle_edge_irq(), even with IRQS_ONESHOT, the irq is still unmasked without care of flag IRQS_ONESHOT. It causes IRQS_ONESHOT can not

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
-Original Message- From: Thomas Gleixner [mailto:t...@linutronix.de] Sent: Friday, October 12, 2012 8:32 PM To: Liu, Chuansheng Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread On Fri, 12 Oct 2012, Chuansheng

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: -Original Message- From: Thomas Gleixner [mailto:t...@linutronix.de] Sent: Friday, October 12, 2012 8:32 PM To: Liu, Chuansheng Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
-Original Message- From: Thomas Gleixner [mailto:t...@linutronix.de] Sent: Friday, October 12, 2012 8:46 PM To: Liu, Chuansheng Cc: linux-kernel@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread On Fri, 12 Oct 2012, Liu

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
FLAG IRQS_ONESHOT does not work well for edge interrupt. * IRQS_ONESHOT - irq is not unmasked in primary handler Is it possible give some notification or remind? Thanks. I took some time to know it:) -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: -Original Message- From: Thomas Gleixner [mailto:t...@linutronix.de] Sent: Friday, October 12, 2012 8:46 PM To: Liu, Chuansheng Cc: linux-kernel@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
, Chuansheng Cc: linux-kernel@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread On Fri, 12 Oct 2012, Liu, Chuansheng wrote: -Original Message- From: Thomas Gleixner [mailto:t...@linutronix.de] Sent: Friday, October 12, 2012 8:46 PM

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
On SMP an interrupt which is raised after the ack() again before the handler finishes, can invoke another delivery on a different CPU, which then sees the IRQ_INPROGESS flag, masks it and flags it PENDING. When the primary handler on the first CPU returns, it sees the PENDING flag, unmasks

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread anish kumar
On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: On SMP an interrupt which is raised after the ack() again before the handler finishes, can invoke another delivery on a different CPU, which then sees the IRQ_INPROGESS flag, masks it and flags it PENDING. When the primary handler

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Liu, Chuansheng
-Original Message- From: anish kumar [mailto:anish198519851...@gmail.com] Sent: Friday, October 12, 2012 11:25 PM To: Liu, Chuansheng Cc: Thomas Gleixner; linux-kernel@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread On Fri

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread anish kumar
: for edge interrupt IRQS_ONESHOT support with irq thread On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: On SMP an interrupt which is raised after the ack() again before the handler finishes, can invoke another delivery on a different CPU, which then sees the IRQ_INPROGESS

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: -Original Message- From: anish kumar [mailto:anish198519851...@gmail.com] Sent: Friday, October 12, 2012 11:25 PM To: Liu, Chuansheng Cc: Thomas Gleixner; linux-kernel@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: On SMP an interrupt which is raised after the ack() again before the handler finishes, can invoke another delivery on a different CPU, which then sees the IRQ_INPROGESS flag, masks it and flags it PENDING. When the primary handler on the first

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread Thomas Gleixner
On Fri, 12 Oct 2012, Liu, Chuansheng wrote: But IRQS_ONESHOT does not work well for edge interrupt. And pasting the IRQS_ONESHOT description: * IRQS_ONESHOT- irq is not unmasked in primary handler Right, and edge type interrupts doe not support it. Can we do

RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread

2012-10-12 Thread anish kumar
@vger.kernel.org Subject: RE: [PATCH] genirq: for edge interrupt IRQS_ONESHOT support with irq thread On Fri, 2012-10-12 at 14:57 +, Liu, Chuansheng wrote: On SMP an interrupt which is raised after the ack() again before the handler finishes, can invoke another delivery