Locking fixes for dc(4): please test!

John Baldwin jhb at FreeBSD.org
Fri Aug 12 12:58:48 GMT 2005


On Friday 12 August 2005 12:37 am, Marcel Moolenaar wrote:
> 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,

Does it respect any media settings from SRM?

-- 
John Baldwin <jhb at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org


More information about the freebsd-alpha mailing list