From owner-cvs-src@FreeBSD.ORG Thu Apr 15 13:31:13 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 327F016A4E0; Thu, 15 Apr 2004 13:31:13 -0700 (PDT) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 28C9D43D2F; Thu, 15 Apr 2004 13:31:13 -0700 (PDT) (envelope-from luigi@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.12.10/8.12.10) with ESMTP id i3FKVCGe095284; Thu, 15 Apr 2004 13:31:12 -0700 (PDT) (envelope-from luigi@repoman.freebsd.org) Received: (from luigi@localhost) by repoman.freebsd.org (8.12.10/8.12.10/Submit) id i3FKVBlD095276; Thu, 15 Apr 2004 13:31:11 -0700 (PDT) (envelope-from luigi) Message-Id: <200404152031.i3FKVBlD095276@repoman.freebsd.org> From: Luigi Rizzo Date: Thu, 15 Apr 2004 13:31:11 -0700 (PDT) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD 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 ... X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Apr 2004 20:31:13 -0000 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