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

[ Available lists | Index of cvs-all | Month of Apr 2004 | Week of 15 Apr 2004 | Raw email | View thread | Wrap long lines | Reply | Tag ]
From
Luigi Rizzo <luigi@FreeBSD.org>
Date
15 Apr 2004 13:31:13
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


[ Hide this part ]
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


Elapsed time: 0.086 seconds