From owner-freebsd-alpha@FreeBSD.ORG Thu Aug 11 21:22:57 2005 Return-Path: X-Original-To: alpha@freebsd.org Delivered-To: freebsd-alpha@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id F106E16A41F; Thu, 11 Aug 2005 21:22:57 +0000 (GMT) (envelope-from jhb@FreeBSD.org) Received: from mv.twc.weather.com (mv.twc.weather.com [65.212.71.225]) by mx1.FreeBSD.org (Postfix) with ESMTP id 7742443D45; Thu, 11 Aug 2005 21:22:57 +0000 (GMT) (envelope-from jhb@FreeBSD.org) Received: from [10.50.40.201] (Not Verified[65.202.103.25]) by mv.twc.weather.com with NetIQ MailMarshal (v6, 0, 3, 8) id ; Thu, 11 Aug 2005 17:37:40 -0400 From: John Baldwin To: Marcel Moolenaar Date: Thu, 11 Aug 2005 17:22:07 -0400 User-Agent: KMail/1.8 References: <200508041255.05839.jhb@FreeBSD.org> <200508101656.23255.jhb@FreeBSD.org> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200508111722.09584.jhb@FreeBSD.org> Cc: alpha@freebsd.org, current@freebsd.org Subject: Re: Locking fixes for dc(4): please test! X-BeenThere: freebsd-alpha@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to the Alpha List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 11 Aug 2005 21:22:58 -0000 On Thursday 11 August 2005 04:27 pm, Marcel Moolenaar wrote: > On Aug 10, 2005, at 1:56 PM, John Baldwin wrote: > > On 2005.08.04 12:55:05 -0400, John Baldwin wrote: > >> I have some fixes for the locking in the dc(4) driver, please test > >> them > >> if you have some dc(4) hardware, thanks! > >> > >> http://www.FreeBSD.org/~jhb/patches/dc_locking.patch > > > > I've updated the patch to use callout_init_mtx() and would > > appreciate it if > > the folks who tested it earlier could retest it. Also, can someone > > with an > > Alpha please test this as I messed around with the SRM_MEDIA code > > some in > > order to make the locking in dc_init_locked() sane and it needs > > testing on an > > Alpha. > > It doesn't build on alpha: > > ds10% cd /sys/alpha/compile/DS10 > ds10% make > cc -c -O -pipe -mcpu=ev6 -mieee -Wall -Wredundant-decls -Wnested- > externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith - > Winline -Wcast-qual -fformat-extensions -std=c99 -g -nostdinc -I- - > I. -I../../.. -I../../../contrib/dev/acpica -I../../../contrib/altq - > I../../../contrib/ipfilter -I../../../contrib/pf -I../../../contrib/ > dev/ath -I../../../contrib/dev/ath/freebsd -I../../../contrib/ngatm - > I../../../dev/twa -D_KERNEL -include opt_global.h -fno-common - > finline-limit=15000 --param inline-unit-growth=100 --param large- > function-growth=1000 -mno-fp-regs -ffixed-8 -Wa,-mev6 -ffreestanding > -Werror ../../../pci/if_dc.c > ../../../pci/if_dc.c: In function `dc_init': > ../../../pci/if_dc.c:3407: error: `ifp' undeclared (first use in this > function) > ../../../pci/if_dc.c:3407: error: (Each undeclared identifier is > reported only once > ../../../pci/if_dc.c:3407: error: for each function it appears in.) > ../../../pci/if_dc.c:3407: error: `mii' undeclared (first use in this > function) > *** Error code 1 > > ...The SRM_MEDIA macro is probably only defined on alpha... Yeah it is, which is why I wanted someone with an Alpha to test it. You should be able to replace 'ifp' with 'sc->dc_ifp' on line 3407. I'll fix the patch and re-upload it as well. Thanks. -- John Baldwin <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" = http://www.FreeBSD.org