linux-kernel - Re: [34-longterm 029/247] iwlagn: enable only rfkill interrupt when device is down

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4E04825A.2000805@windriver.com>
Date:	Fri, 24 Jun 2011 08:26:02 -0400
From:	Paul Gortmaker <paul.gortmaker@...driver.com>
To:	Stanislaw Gruszka <sgruszka@...hat.com>
CC:	<stable@...nel.org>, <linux-kernel@...r.kernel.org>,
	<stable-review@...nel.org>,
	"John W. Linville" <linville@...driver.com>
Subject: Re: [34-longterm 029/247] iwlagn: enable only rfkill interrupt when
 device is down

On 11-06-24 04:16 AM, Stanislaw Gruszka wrote:
> On Thu, Jun 23, 2011 at 01:17:52PM -0400, Paul Gortmaker wrote:
>> From: Stanislaw Gruszka <sgruszka@...hat.com>
>>
>>                    -------------------
>>     This is a commit scheduled for the next v2.6.34 longterm release.
>>     If you see a problem with using this for longterm, please comment.
>>                    -------------------
>>
>> commit 554d1d027b19265c4aa3f718b3126d2b86e09a08 upstream.
>>
>> Since commit 6cd0b1cb872b3bf9fc5de4536404206ab74bafdd "iwlagn: fix
>> hw-rfkill while the interface is down", we enable interrupts when
>> device is not ready to receive them. However hardware, when it is in
>> some inconsistent state, can generate other than rfkill interrupts
>> and crash the system. I can reproduce crash with "kernel BUG at
>> drivers/net/wireless/iwlwifi/iwl-agn.c:1010!" message, when forcing
>> firmware restarts.
>>
>> To fix only enable rfkill interrupt when down device and after probe.
>> I checked patch on laptop with 5100 device, rfkill change is still
>> passed to user space when device is down.
> 
> This needs follow up:
> 
> commit 3dd823e6b86407aed1a025041d8f1df77e43a9c8
> Author: Don Fry <donald.h.fry@...el.com>
> Date:   Sun Feb 6 09:29:45 2011 -0800
> 
>     iwlagn: Re-enable RF_KILL interrupt when down
> 
> I'm not sure if that commit is included in the patch series.
> 
> Stanislaw

Thanks for the review; the 3dd823e commit was already in the
series (sent as 049/247) fortunately.

Paul.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/