From owner-svn-src-head@FreeBSD.ORG  Mon Oct 27 02:51:08 2014
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTPS id 9198AA17;
 Mon, 27 Oct 2014 02:51:08 +0000 (UTC)
Received: from lauren.room52.net (lauren.room52.net [210.50.193.198])
 by mx1.freebsd.org (Postfix) with ESMTP id 5627DA41;
 Mon, 27 Oct 2014 02:51:08 +0000 (UTC)
Received: from lgwl-lstewart2.corp.netflix.com
 (c110-22-7-57.eburwd3.vic.optusnet.com.au [110.22.7.57])
 by lauren.room52.net (Postfix) with ESMTPSA id A713B7E824;
 Mon, 27 Oct 2014 13:43:32 +1100 (EST)
Message-ID: <544DB140.7070807@freebsd.org>
Date: Mon, 27 Oct 2014 13:43:12 +1100
From: Lawrence Stewart <lstewart@freebsd.org>
User-Agent: Mozilla/5.0 (X11; FreeBSD amd64;
 rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: Hiroki Sato <hrs@freebsd.org>, src-committers@freebsd.org, 
 svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject: Re: svn commit: r272386 - in head: sbin/ifconfig share/man/man4
 sys/net
References: <201410012137.s91LbXL4025967@svn.freebsd.org>
In-Reply-To: <201410012137.s91LbXL4025967@svn.freebsd.org>
X-Enigmail-Version: 1.6
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
X-Spam-Status: No, score=0.0 required=5.0 tests=UNPARSEABLE_RELAY
 autolearn=unavailable version=3.3.2
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on lauren.room52.net
X-BeenThere: svn-src-head@freebsd.org
X-Mailman-Version: 2.1.18-1
Precedence: list
List-Id: SVN commit messages for the src tree for head/-current
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head/>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 27 Oct 2014 02:51:08 -0000

Hi Hiroki-san,

On 10/02/14 07:37, Hiroki Sato wrote:
> Author: hrs
> Date: Wed Oct  1 21:37:32 2014
> New Revision: 272386
> URL: https://svnweb.freebsd.org/changeset/base/272386
> 
> Log:
>   Virtualize lagg(4) cloner.  This change fixes a panic when tearing down
>   if_lagg(4) interfaces which were cloned in a vnet jail.
>   
>   Sysctl nodes which are dynamically generated for each cloned interface
>   (net.link.lagg.N.*) have been removed, and use_flowid and flowid_shift
>   ifconfig(8) parameters have been added instead.  Flags and per-interface
>   statistics counters are displayed in "ifconfig -v".
>   
>   CR:	D842
> 
> Modified:
>   head/sbin/ifconfig/ifconfig.8
>   head/sbin/ifconfig/iflagg.c
>   head/share/man/man4/lagg.4
>   head/sys/net/ieee8023ad_lacp.c
>   head/sys/net/if_lagg.c
>   head/sys/net/if_lagg.h

What was the reasoning for removing the net.link.lagg.* sysctls? We use
them at Netflix and the new way of obtaining the information via
ifconfig -v is less structured and significantly higher overhead by
comparison. Would be good to have the sysctls back or an equivalent low
overhead way of obtaining the previously available information.

Cheers,
Lawrence