From owner-cvs-src@FreeBSD.ORG  Tue Aug  2 18:36:39 2005
Return-Path: <owner-cvs-src@FreeBSD.ORG>
X-Original-To: cvs-src@FreeBSD.org
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 C939216A420;
	Tue,  2 Aug 2005 18:36:39 +0000 (GMT) (envelope-from jhb@FreeBSD.org)
Received: from mv.twc.weather.com (mv.twc.weather.com [65.212.71.225])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 08B9643D46;
	Tue,  2 Aug 2005 18:36:37 +0000 (GMT) (envelope-from jhb@FreeBSD.org)
Received: from [10.50.40.201] (Not Verified[65.202.103.25]) by
	mv.twc.weather.com with NetIQ MailMarshal (v6, 0, 3, 8)
	id <B42efc09e0002>; Tue, 02 Aug 2005 14:51:10 -0400
From: John Baldwin <jhb@FreeBSD.org>
To: Robert Watson <rwatson@FreeBSD.org>
Date: Tue, 2 Aug 2005 14:16:32 -0400
User-Agent: KMail/1.8
References: <200508021743.j72HhZRj009221@repoman.freebsd.org>
In-Reply-To: <200508021743.j72HhZRj009221@repoman.freebsd.org>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="iso-8859-6"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200508021416.33768.jhb@FreeBSD.org>
Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/net if_var.h
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 02 Aug 2005 18:36:40 -0000

On Tuesday 02 August 2005 01:43 pm, Robert Watson wrote:
> rwatson     2005-08-02 17:43:35 UTC
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/net              if_var.h
>   Log:
>   Add if_addr_mtx to struct ifnet, a mutex to protect ifnet-related address
>   lists.  Add accessor macros.
>
>   This changes the size of struct ifnet, but ideally, all ifnet consumers
>   are now using if_alloc() to allocate these structures rather than
>   embedding them into device driver softc's, so this won't modify the
>   network device driver ABI.
>
>   MFC after:      1 week

It still modifies the ABI because of all the code that does 'ifp->if_foo' 
unless you've added it to the end.

-- 
John Baldwin <jhb@FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org