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