Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-12-03 Thread Felix Fietkau
On 2012-11-20 11:05 AM, Felix Liao wrote: Hi Felix F, I'm Felix Liao from WatchGuard, I think we should spin_lock_irqsave and spin_unlock_irqrestore on sc_pcu_lock in the ath9k_tasklet() instead of spin_lock/spin_unlock, just to avoid the tasklet being interrupted by the hardware IRQ, do

Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-12-03 Thread Chaoxing Lin
On 2012-11-20 11:05 AM, Felix Liao wrote: Hi Felix F, I'm Felix Liao from WatchGuard, I think we should spin_lock_irqsave and spin_unlock_irqrestore on sc_pcu_lock in the ath9k_tasklet() instead of spin_lock/spin_unlock, just to avoid the tasklet being interrupted by the hardware IRQ, do

Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-11-20 Thread Felix Liao
-Original Message- From: adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] On Behalf Of Adrian Chadd Sent: Sunday, November 18, 2012 10:11 PM To: Felix Liao; Felix Fietkau Cc: ath9k-devel@lists.ath9k.org Subject: Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ On 12

Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-11-18 Thread Adrian Chadd
On 12 November 2012 01:54, Felix Liao felix.l...@watchguard.com wrote: Thanks for you information. It is running in AP mode, and uses compat-wireless-3.5-3, but the irq 16: nobody cared issue still happens when I update to compat-wireless-3.6.6-1. I found a mail list talk about this,

Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-11-12 Thread Felix Liao
of spin_lock/spin_unlock? Thanks, - Felix -Original Message- From: adrian.ch...@gmail.com [mailto:adrian.ch...@gmail.com] On Behalf Of Adrian Chadd Sent: Monday, November 12, 2012 5:44 PM To: Felix Liao Cc: ath9k-devel@lists.ath9k.org Subject: Re: [ath9k-devel] ath9k_tasklet() can be interrupted

Re: [ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-11-11 Thread Felix Liao
Hi ath9k team: Another interest question is since the ath9k interrupt was disabled, how can we get a new IRQ to interrupt the tasklet? So I investigated and found that the cause of the issue irq 16: nobody cared in my box is the ath9k's irq handler return too many IRQ_NONE since I read the ISR

[ath9k-devel] ath9k_tasklet() can be interrupted by a hardware IRQ

2012-11-08 Thread Felix Liao
Hi all, I met a kernel call trace below using the ath9k driver, we can see that the ath9k_tasklet() was interrupted by a hardware IRQ, and at this time the ath9k IRQ was disabled, so the kernel will complain that there is none irq handler for this interrupt. [ 60.977474] irq 16: nobody cared