Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 11 Aug 2005 21:37:53 -0700
From:      Marcel Moolenaar <marcel@xcllnt.net>
To:        John Baldwin <jhb@FreeBSD.org>
Cc:        alpha@FreeBSD.org, current@FreeBSD.org
Subject:   Re: Locking fixes for dc(4): please test!
Message-ID:  <A747614E-F2A7-4991-A424-934142B00BB9@xcllnt.net>
In-Reply-To: <200508111722.09584.jhb@FreeBSD.org>
References:  <200508041255.05839.jhb@FreeBSD.org> <200508101656.23255.jhb@FreeBSD.org> <B76A5A98-6A32-4181-AB7D-644A0723A1E9@xcllnt.net> <200508111722.09584.jhb@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Aug 11, 2005, at 2:22 PM, John Baldwin wrote:

>> 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.

Yup, that fixes half the problems. I added the following at line 3403
to fix the other half:

     struct mii_data *mii;
     mii = device_get_softc(sc->dc_miibus);

With that, if_dc seems to work fine. haven't pushed it though...
FYI,

-- 
  Marcel Moolenaar         USPA: A-39004          marcel@xcllnt.net





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?A747614E-F2A7-4991-A424-934142B00BB9>