Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757596AbXE1Kvm (ORCPT ); Mon, 28 May 2007 06:51:42 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750700AbXE1Kvc (ORCPT ); Mon, 28 May 2007 06:51:32 -0400 Received: from static-ip-62-75-166-246.inaddr.intergenia.de ([62.75.166.246]:55569 "EHLO vs166246.vserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750699AbXE1Kvb (ORCPT ); Mon, 28 May 2007 06:51:31 -0400 From: Michael Buesch To: Maximilian Engelhardt Subject: Re: b44: regression in 2.6.22 (resend) Date: Mon, 28 May 2007 12:49:55 +0200 User-Agent: KMail/1.9.6 Cc: "linux-kernel" , "linux-wireless" , Stephen Hemminger , Arnaldo Carvalho de Melo , Jeff Garzik , Gary Zambrano , netdev@vger.kernel.org, Andrew Morton References: <20070525172431.60affaca@freepuppy> <200705280224.40506.mb@bu3sch.de> <200705280240.17910.maxi@daemonizer.de> In-Reply-To: <200705280240.17910.maxi@daemonizer.de> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200705281249.56106.mb@bu3sch.de> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1148 Lines: 35 Can you also test the following patch? I think there's a bug in b44 that is doesn't properly discard shared IRQs, so it might possibly generate a NAPI storm, dunno. Worth a try. Index: linux-2.6.22-rc3/drivers/net/b44.c =================================================================== --- linux-2.6.22-rc3.orig/drivers/net/b44.c 2007-05-27 23:01:44.000000000 +0200 +++ linux-2.6.22-rc3/drivers/net/b44.c 2007-05-28 12:48:27.000000000 +0200 @@ -911,6 +911,8 @@ static irqreturn_t b44_interrupt(int irq spin_lock(&bp->lock); istat = br32(bp, B44_ISTAT); + if (istat == 0xFFFFFFFF) + goto out; /* Shared IRQ not for us */ imask = br32(bp, B44_IMASK); /* The interrupt mask register controls which interrupt bits @@ -942,6 +944,7 @@ irq_ack: bw32(bp, B44_ISTAT, istat); br32(bp, B44_ISTAT); } +out: spin_unlock(&bp->lock); return IRQ_RETVAL(handled); } -- Greetings Michael. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/