diff options
| author | luigi <luigi@FreeBSD.org> | 2004-04-15 20:31:10 +0000 |
|---|---|---|
| committer | luigi <luigi@FreeBSD.org> | 2004-04-15 20:31:10 +0000 |
| commit | 7d7307d2e1689754b4def7b381a46fd20919d442 (patch) | |
| tree | 5724ee7f72224819d2453cd24c370e176d84429f /sys/dev/arl/if_arl.c | |
| parent | fa9222585e2ae94c9da702e4b6a4fd959416bd33 (diff) | |
| download | FreeBSD-src-7d7307d2e1689754b4def7b381a46fd20919d442.zip FreeBSD-src-7d7307d2e1689754b4def7b381a46fd20919d442.tar.gz | |
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.
Diffstat (limited to 'sys/dev/arl/if_arl.c')
| -rw-r--r-- | sys/dev/arl/if_arl.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sys/dev/arl/if_arl.c b/sys/dev/arl/if_arl.c index f3495fb..9a69870 100644 --- a/sys/dev/arl/if_arl.c +++ b/sys/dev/arl/if_arl.c @@ -701,9 +701,6 @@ arl_init(xsc) D(("init\n")); - if (TAILQ_EMPTY(&ifp->if_addrhead)) - return; - s = splimp(); if (ARL_CHECKREG(sc)) |
