Return-Path: <linux-kernel-owner+willy=40w.ods.org-S932203AbWBJVbW@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S932203AbWBJVbW (ORCPT <rfc822;willy@w.ods.org>);
	Fri, 10 Feb 2006 16:31:22 -0500
Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932206AbWBJVbW
	(ORCPT <rfc822;linux-kernel-outgoing>);
	Fri, 10 Feb 2006 16:31:22 -0500
Received: from hummeroutlaws.com ([12.161.0.3]:54027 "EHLO atpro.com")
	by vger.kernel.org with ESMTP id S932203AbWBJVbV (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Fri, 10 Feb 2006 16:31:21 -0500
From: "Jim Crilly" <jim@why.dont.jablowme.net>
Date: Fri, 10 Feb 2006 16:30:58 -0500
To: Andrew Morton <akpm@osdl.org>
Cc: linux-kernel@vger.kernel.org, jgarzik@pobox.com
Subject: Re: 2.6.16-rc2-mm1
Message-ID: <20060210213058.GE11297@voodoo>
Mail-Followup-To: Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org, jgarzik@pobox.com
References: <20060207220627.345107c3.akpm@osdl.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20060207220627.345107c3.akpm@osdl.org>
User-Agent: Mutt/1.5.11+cvs20060126
Sender: linux-kernel-owner@vger.kernel.org
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1764
Lines: 43

On 02/07/06 10:06:27PM -0800, Andrew Morton wrote:
> 
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.16-rc2/2.6.16-rc2-mm1/
>

I got the following BUG with the tulip driver because there's a mdelay(1)
at the end of the tulip_media_select() function. Removing the mdelay(1)
is trivial so I've attached a patch, but I'm not sure if it's the
correct fix so I've CC'd Jeff Garzik.

BUG: warning at drivers/net/tulip/media.c:402/tulip_select_media()
<f0c8e288> tulip_select_media+0x7b8/0x7db [tulip]   <b02103ac> dma_pool_free+0xc4/0x10e
<f0c9130b> t21142_lnk_change+0x1af/0x4f4 [tulip]   <f0c7c490> finish_urb+0x98/0xc0 [ohci_hcd]
<f0c8d374> tulip_interrupt+0x65f/0x803 [tulip]   <f0c7e153> ohci_irq+0x148/0x16d [ohci_hcd]
<b013cb3f> handle_IRQ_event+0x20/0x4c   <b013cbf7> __do_IRQ+0x8c/0xdd
<b0105250> do_IRQ+0x3c/0x54
  =======================
<b0103662> common_interrupt+0x1a/0x20   <b0101aa6> default_idle+0x0/0x55
<b0101ad2> default_idle+0x2c/0x55   <b0101b8a> cpu_idle+0x8f/0xae
<b02f8707> start_kernel+0x37f/0x386

Signed-off-by: Jim Crilly <jim@why.dont.jablowme.net>

---
drivers/net/tulip/media.c |    2 --
 1 file changed, 2 deletions(-)

--- linux-2.6.16-rc2-mm1/drivers/net/tulip/media.c  2006-02-09 22:13:36.403653502 -0500
+++ linux-2.6.16-rc2-mm1-new/drivers/net/tulip/media.c  2006-02-10 15:51:41.821983228 -0500
@@ -399,8 +399,6 @@ void tulip_select_media(struct net_devic

  tp->csr6 = new_csr6 | (tp->csr6 & 0xfdff) | (tp->full_duplex ? 0x0200 : 0);

- mdelay(1);
-
  return;
 }
  
-
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/