Return-path: Received: from www.linutronix.de ([62.245.132.108]:57123 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758459Ab2CFJwX (ORCPT ); Tue, 6 Mar 2012 04:52:23 -0500 Date: Tue, 6 Mar 2012 10:52:19 +0100 (CET) From: Thomas Gleixner To: Linus Torvalds cc: Stefan Lippers-Hollmann , Sven Joachim , Greg KH , LKML , stable@vger.kernel.org, Andrew Morton , Alan Cox , Jonathan Nieder , linux-wireless@vger.kernel.org, Stefano Brivio Subject: Re: [ 57/72] genirq: Unmask oneshot irqs when thread was not woken In-Reply-To: Message-ID: (sfid-20120306_105252_111615_8915B1FF) References: <20120228010511.GA8453@kroah.com> <20120228010434.412979550@linuxfoundation.org> <87hay4dqjr.fsf@turtle.gmx.de> <201203050143.24541.s.L-H@gmx.de> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, 6 Mar 2012, Thomas Gleixner wrote: > On Mon, 5 Mar 2012, Linus Torvalds wrote: > > > Thomas, should we just plan on reverting that commit from mainline? It > > clearly causes regressions. > > Give me a day or two to figure out why it breaks stuff. I have no idea > why it should wreckage anything. Hmm. This is interesting. The b43 driver has a primary handler which can return IRQ_NONE. So up to that change the interrupt line was kept disabled when that happened. Possibly the driver relies on that behaviour. Digging for a machine with a b43.