Date: Thu, 15 Apr 2004 13:31:11 -0700 (PDT) From: Luigi Rizzo <luigi@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/dev/arl if_arl.c src/sys/dev/cs if_cs.c src/sys/dev/ed if_ed.c src/sys/dev/ex if_ex.c src/sys/dev/fe if_fe.c src/sys/dev/lnc if_lnc.c src/sys/dev/sbni if_sbni.c src/sys/dev/sn if_sn.c src/sys/dev/wl if_wl.c src/sys/dev/xe ... Message-ID: <200404152031.i3FKVBlD095276@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
luigi 2004/04/15 13:31:11 PDT
FreeBSD src repository
Modified files:
sys/dev/arl if_arl.c
sys/dev/cs if_cs.c
sys/dev/ed if_ed.c
sys/dev/ex if_ex.c
sys/dev/fe if_fe.c
sys/dev/lnc if_lnc.c
sys/dev/sbni if_sbni.c
sys/dev/sn if_sn.c
sys/dev/wl if_wl.c
sys/dev/xe if_xe.c
Log:
Remove improper use of if_addrhead in device drivers to check
if the link-level address has been initialized already.
The majority of modern drivers never does this and works fine, which
makes me think that the check is totally unnecessary and a residue
of cut&paste from other drivers.
This change is done to simplify locking because now almost none of the
drivers uses this field. The exceptions are "ct" "ctau" and "cx"
where i am not sure if i can remove that part.
Revision Changes Path
1.4 +0 -3 src/sys/dev/arl/if_arl.c
1.31 +0 -4 src/sys/dev/cs/if_cs.c
1.229 +0 -4 src/sys/dev/ed/if_ed.c
1.50 +0 -3 src/sys/dev/ex/if_ex.c
1.83 +0 -8 src/sys/dev/fe/if_fe.c
1.106 +0 -7 src/sys/dev/lnc/if_lnc.c
1.16 +0 -4 src/sys/dev/sbni/if_sbni.c
1.38 +0 -18 src/sys/dev/sn/if_sn.c
1.60 +0 -2 src/sys/dev/wl/if_wl.c
1.49 +0 -2 src/sys/dev/xe/if_xe.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200404152031.i3FKVBlD095276>