From owner-cvs-all@FreeBSD.ORG Tue Oct 19 18:02:55 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8815916A4CE; Tue, 19 Oct 2004 18:02:55 +0000 (GMT) Received: from mailhub.sweetdreamsracing.biz (mailhub.sweetdreamsracing.biz [66.92.171.106]) by mx1.FreeBSD.org (Postfix) with ESMTP id 2B06143D1F; Tue, 19 Oct 2004 18:02:55 +0000 (GMT) (envelope-from culverk@sweetdreamsracing.biz) Received: by mailhub.sweetdreamsracing.biz (Postfix, from userid 80) id 49A6561AB; Tue, 19 Oct 2004 14:02:37 -0400 (EDT) Received: from 141.156.69.109 ([141.156.69.109]) by www.sweetdreamsracing.biz (Horde) with HTTP for ; Tue, 19 Oct 2004 14:02:37 -0400 Message-ID: <20041019140237.60488wg8oss88ocg@www.sweetdreamsracing.biz> Date: Tue, 19 Oct 2004 14:02:37 -0400 From: Kenneth Culver To: John-Mark Gurney References: <200410182306.i9IN6DDx016512@repoman.freebsd.org> <20041019101752.e8ws08k8kcw8408c@www.sweetdreamsracing.biz> <20041019174306.GK22681@funkthat.com> In-Reply-To: <20041019174306.GK22681@funkthat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format="flowed" Content-Disposition: inline Content-Transfer-Encoding: 7bit User-Agent: Internet Messaging Program (IMP) 4.0-cvs cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/pci if_de.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Oct 2004 18:02:55 -0000 Quoting John-Mark Gurney : > Kenneth Culver wrote this message on Tue, Oct 19, 2004 at 10:17 -0400: >> Quoting John-Mark Gurney : >> >> >jmg 2004-10-18 23:06:13 UTC >> > >> > FreeBSD src repository >> > >> > Modified files: >> > sys/pci if_de.c >> > Log: >> > fix (for me) the problems where if_de gets really slow after time >> > (usually taking 20 seconds to transmit a packet).. no longer fall back >> > to only transmitting one packet (instead of the entire queue) after we >> > have processed the entire send queue... I have no idea why we didn't >> > start seeing this problem ~6 years ago when this code was introduced... >> >> Do you get an error on boot with this device about it running in >> Giant. I was >> seeing similar problems to you and I was seeing a message at boot about this >> device running in Giant, so I set debug.mpsafenet to 0, and the problem went >> away. > > yes, I do: > de0: port 0xd800-0xd87f mem > 0xe8000000-0xe800007f > irq 10 at device 9.0 on pci0 > de0: [GIANT-LOCKED] > de0: 21140A [10-100Mb/s] pass 2.2 > de0: Ethernet address: 00:c0:f0:1f:21:f9 > de0: if_start running deferred for Giant > > but even with those messages about Giant, we did find that the interface > ioctl was being called w/o Giant... also, the warning about Giant > mean that the network subsystem is suppose to call all of the driver's > routines w/ Giant held to appear as if debug.mpsafenet has been set to 0.. > > I have a patch that addes some Giant requireds, but they can't be commited > till the ioctl problem is fixed... > > If you are interested in tracking down the bug, I'll be more than happy > to review and commit any changes... The driver needs to be both locked > and busdma'ified... > Well the machine I'm using this interface on is in a production machine. If I can find another card and put it in my personal machine I'll see if I can track down the problem. However I haven't done any kernel work in BSD since 4.x, so it might take me a while to come up to speed on the locking and busdma'ing. Ken