From owner-freebsd-stable@FreeBSD.ORG Thu Sep 28 19:35:32 2006 Return-Path: X-Original-To: freebsd-stable@freebsd.org Delivered-To: freebsd-stable@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id F1F4B16A407; Thu, 28 Sep 2006 19:35:31 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id 7185743D45; Thu, 28 Sep 2006 19:35:31 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [10.10.3.185] ([165.236.175.187]) (authenticated bits=0) by pooker.samsco.org (8.13.4/8.13.4) with ESMTP id k8SJZElK025978; Thu, 28 Sep 2006 13:35:20 -0600 (MDT) (envelope-from scottl@samsco.org) Message-ID: <451C23EB.3020809@samsco.org> Date: Thu, 28 Sep 2006 13:35:07 -0600 From: Scott Long User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.12) Gecko/20060206 X-Accept-Language: en-us, en MIME-Version: 1.0 To: "O. Hartmann" References: <451A1375.5080202@gneto.com> <20060927071538.GF22229@e-Gitt.NET><451A4189.5020906@samsco.org> <20060927152824.GJ22229@e-Gitt.NET><20060927155553.GB14563@icarus.home.lan><20060927155904.GM22229@e-Gitt.NET> <451AA7B1.5080202@samsco.org><20060927191402.GB932@turion.vk2pj.dyndns.org><20060927210349.GG14975@tnn.dglawrence.com> <451AEB02.2090806@samsco.org> <002201c6e290$45ece980$b3db87d4@multiplay.co.uk> <451BD89F.8080203@samsco.org> <451C1F6D.2020302@mail.uni-mainz.de> In-Reply-To: <451C1F6D.2020302@mail.uni-mainz.de> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=0.0 required=3.8 tests=none autolearn=failed version=3.1.1 X-Spam-Checker-Version: SpamAssassin 3.1.1 (2006-03-10) on pooker.samsco.org Cc: Peter Jeremy , Steven Hartland , John Baldwin , freebsd-stable@freebsd.org, Oliver Brandmueller Subject: Re: CALL FOR TESTERS! [Re: 6.2 SHOWSTOPPER - em completely unusable on 6.2] X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Sep 2006 19:35:32 -0000 O. Hartmann wrote: > Scott Long wrote: > >> All, >> >> Attached is my first cut at addressing the problems described in this >> thread. As I discussed earlier, the VM syncer thread is likely starving >> the USB interrupt thread. This causes the shared usb+network >> interrupt to remain masked, preventing network interrupts from being >> delivered, >> and thus triggering watchdog timeouts. >> >> This patch only addresses the USB driver. If your network card is >> sharing an interrupt with something other than (or additional to) USB, >> this might not work for you. Also, this patch is just a very rough >> proof-of-concept and is not meant for production use. But I'd like to >> get feedback now before I spend more time on this. If this works then >> I'll clean it up and make it suitable for the release, and I'll look at >> some of the other drivers like ichsmb. >> >> If this is to be fixed for 6.2, I need lots of feedback ASAP. So please >> do not be shy =-) The patch is at: >> >> http://people.freebsd.org/~scottl/usb_fastintr.diff >> >> Scott >> >> _______________________________________________ >> freebsd-stable@freebsd.org mailing list >> http://lists.freebsd.org/mailman/listinfo/freebsd-stable >> To unsubscribe, send any mail to "freebsd-stable-unsubscribe@freebsd.org" > > patch does not work on my box: > > cc -c -O2 -frename-registers -pipe -fno-strict-aliasing -march=athlon64 > -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes > -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual > -fformat-extensions -std=c99 -nostdinc -I- -I. -I/usr/src/sys > -I/usr/src/sys/contrib/altq -I/usr/src/sys/contrib/ipfilter > -I/usr/src/sys/contrib/pf -I/usr/src/sys/contrib/dev/ath > -I/usr/src/sys/contrib/dev/ath/freebsd -I/usr/src/sys/contrib/ngatm > -I/usr/src/sys/dev/twa -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include > opt_global.h -fno-common -finline-limit=8000 --param > inline-unit-growth=100 --param large-function-growth=1000 > -mcmodel=kernel -mno-red-zone -mfpmath=387 -mno-sse -mno-sse2 -mno-mmx > -mno-3dnow -msoft-float -fno-asynchronous-unwind-tables -ffreestanding > -Werror /usr/src/sys/dev/usb/usb.c > /usr/src/sys/dev/usb/usb.c: In function `usb_attach': > /usr/src/sys/dev/usb/usb.c:282: error: `usb_intr_task' undeclared (first > use in this function) > /usr/src/sys/dev/usb/usb.c:282: error: (Each undeclared identifier is > reported only once > /usr/src/sys/dev/usb/usb.c:282: error: for each function it appears in.) > /usr/src/sys/dev/usb/usb.c: At top level: > /usr/src/sys/dev/usb/usb.c:863: warning: 'usb_intr_task' defined but not > used > *** Error code 1 > > Stop in /usr/obj/usr/src/sys/THOR. > *** Error code 1 > > Stop in /usr/src. > *** Error code 1 > > Stop in /usr/src. > I accidentally posted a patch against HEAD, not RELENG_6. I'll correct that shortly. Scott