From owner-svn-src-all@FreeBSD.ORG Fri Nov 11 14:23:20 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C6D98106564A; Fri, 11 Nov 2011 14:23:20 +0000 (UTC) (envelope-from minimarmot@gmail.com) Received: from mail-gx0-f182.google.com (mail-gx0-f182.google.com [209.85.161.182]) by mx1.freebsd.org (Postfix) with ESMTP id 5BF908FC12; Fri, 11 Nov 2011 14:23:19 +0000 (UTC) Received: by ggnk3 with SMTP id k3so6013486ggn.13 for ; Fri, 11 Nov 2011 06:23:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=RaoFeCL53I7C6hq7MYe7DQSfl+3Dy8EYsDfSV8c+xAk=; b=Kk5MwJdG1/LlXJu88LIjNOhUMfFw9RzZcthnPCGxk25b+gDnRHvTHU1Q0vrL3hNzld JoEt25gl1xj2XlSy4FwV4Y1A1mcW21PlbaeKkGXoh3ob1woKpg8aELW1rEEMSwjRPKEL 9cN86F4U4GQOJ7A564JTmgPD8JadbAvBdC1hA= MIME-Version: 1.0 Received: by 10.101.42.18 with SMTP id u18mr5622707anj.50.1321021399488; Fri, 11 Nov 2011 06:23:19 -0800 (PST) Received: by 10.236.110.174 with HTTP; Fri, 11 Nov 2011 06:23:19 -0800 (PST) In-Reply-To: <9A674915-EF68-4CEA-93CB-CDE349082019@FreeBSD.org> References: <201110201558.p9KFw550081440@svn.freebsd.org> <9A674915-EF68-4CEA-93CB-CDE349082019@FreeBSD.org> Date: Fri, 11 Nov 2011 09:23:19 -0500 Message-ID: From: Ben Kaduk To: Gleb Smirnoff Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, "Bjoern A. Zeeb" , src-committers@freebsd.org, svn-src-stable-9@freebsd.org Subject: Re: svn commit: r226572 - stable/9/sys/netinet X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Nov 2011 14:23:21 -0000 On Thu, Oct 20, 2011 at 12:40 PM, Bjoern A. Zeeb wrote: > On 20. Oct 2011, at 15:58 , Gleb Smirnoff wrote: > >> Author: glebius >> Date: Thu Oct 20 15:58:05 2011 >> New Revision: 226572 >> URL: http://svn.freebsd.org/changeset/base/226572 >> >> Log: >> =A0MFhead 226401,226402: >> >> =A0 =A0Remove last remnants of classful addressing: >> >> =A0 =A0- Remove ia_net, ia_netmask, ia_netbroadcast from struct in_ifadd= r. >> =A0 =A0- Remove net.inet.ip.subnetsarelocal, I bet no one need it in 201= 1. >> =A0 =A0- fix bug when we were not forwarding to a host which matches cla= ssful >> =A0 =A0 =A0net address. For example router having 192.168.x.y/16 network= attached, >> =A0 =A0 =A0would not forward traffic to 192.168.*.0, which are legal IPs= in >> =A0 =A0 =A0CIDR world. >> =A0 =A0- For compatibility, leave autoguessing of mask based on class. >> >> =A0 =A0Reviewed by: =A0andre, bz, rwatson >> >> =A0 =A0Add support for IPv4 /31 prefixes, as described in RFC3021. >> >> =A0 =A0To run a /31 network, participating hosts MUST drop support >> =A0 =A0for directed broadcasts, and treat the first and last addresses >> =A0 =A0on subnet as unicast. The broadcast address for the prefix >> =A0 =A0should be the link local broadcast address, INADDR_BROADCAST. > ... >> Modified: stable/9/sys/netinet/in_var.h >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D >> --- stable/9/sys/netinet/in_var.h =A0 =A0 Thu Oct 20 15:55:01 2011 =A0 = =A0 =A0 =A0(r226571) >> +++ stable/9/sys/netinet/in_var.h =A0 =A0 Thu Oct 20 15:58:05 2011 =A0 = =A0 =A0 =A0(r226572) >> @@ -60,12 +60,9 @@ struct in_ifaddr { >> =A0 =A0 =A0 struct =A0ifaddr ia_ifa; =A0 =A0 =A0 =A0 =A0/* protocol-inde= pendent info */ >> #define =A0 =A0 =A0 ia_ifp =A0 =A0 =A0 =A0 =A0ia_ifa.ifa_ifp >> #define ia_flags =A0 =A0 =A0ia_ifa.ifa_flags >> - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = =A0 /* ia_{,sub}net{,mask} in host order */ >> - =A0 =A0 u_long =A0ia_net; =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 /* network n= umber of interface */ >> - =A0 =A0 u_long =A0ia_netmask; =A0 =A0 =A0 =A0 =A0 =A0 /* mask of net p= art */ >> - =A0 =A0 u_long =A0ia_subnet; =A0 =A0 =A0 =A0 =A0 =A0 =A0/* subnet numb= er, including net */ >> - =A0 =A0 u_long =A0ia_subnetmask; =A0 =A0 =A0 =A0 =A0/* mask of subnet = part */ >> - =A0 =A0 struct =A0in_addr ia_netbroadcast; /* to recognize net broadca= sts */ >> + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = =A0 /* ia_subnet{,mask} in host order */ >> + =A0 =A0 u_long =A0ia_subnet; =A0 =A0 =A0 =A0 =A0 =A0 =A0/* subnet addr= ess */ >> + =A0 =A0 u_long =A0ia_subnetmask; =A0 =A0 =A0 =A0 =A0/* mask of subnet = */ >> =A0 =A0 =A0 LIST_ENTRY(in_ifaddr) ia_hash; =A0/* entry in bucket of inet= addresses */ >> =A0 =A0 =A0 TAILQ_ENTRY(in_ifaddr) ia_link; /* list of internet addresse= s */ >> =A0 =A0 =A0 struct =A0sockaddr_in ia_addr; =A0 =A0/* reserve space for i= nterface name */ > > This should at least break parts of netstat and ifmcstat as I had told yo= u in > private email before. Just so we're clear, it changes the API (at the RC stage!), causing at least the net/openafs port to fail to compile. I presume, given the commit message, that I am just not supposed to use ia_net and ia_netmask any more, and that ia_submet and ia_subnetmask are presumed to be sufficient? -Ben Kaduk