From owner-svn-src-head@FreeBSD.ORG Mon Jul 9 08:32:41 2012 Return-Path: Delivered-To: svn-src-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 00B64106566B; Mon, 9 Jul 2012 08:32:41 +0000 (UTC) (envelope-from melifaro@FreeBSD.org) Received: from mail.ipfw.ru (unknown [IPv6:2a01:4f8:120:6141::2]) by mx1.freebsd.org (Postfix) with ESMTP id B240B8FC12; Mon, 9 Jul 2012 08:32:40 +0000 (UTC) Received: from v6.mpls.in ([2a02:978:2::5] helo=ws.su29.net) by mail.ipfw.ru with esmtpsa (TLSv1:CAMELLIA256-SHA:256) (Exim 4.76 (FreeBSD)) (envelope-from ) id 1So9RG-00080m-FK; Mon, 09 Jul 2012 12:35:26 +0400 Message-ID: <4FFA9723.5000301@FreeBSD.org> Date: Mon, 09 Jul 2012 12:32:35 +0400 From: "Alexander V. Chernikov" User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120121 Thunderbird/9.0 MIME-Version: 1.0 To: Hiroki Sato References: <201207090716.q697GJ7A001973@svn.freebsd.org> <4FFA894D.9050104@FreeBSD.org> <20120709.170813.339720376082380726.hrs@allbsd.org> In-Reply-To: <20120709.170813.339720376082380726.hrs@allbsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r238277 - in head: etc/defaults etc/rc.d sbin/ipfw share/man/man5 sys/netinet/ipfw X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Jul 2012 08:32:41 -0000 On 09.07.2012 12:08, Hiroki Sato wrote: > "Alexander V. Chernikov" wrote > in<4FFA894D.9050104@FreeBSD.org>: > > I meant there was no strong objection. I am sorry for not commenting > your implementation, but at least for ipfw0 it is difficult to > decouple ifnet and bpf because the primary consumer is tcpdump(8), > which depends on NET_RT_IFLIST to find the target. Probably your tcpdump -i still works with interface name supplied. > solution can be used for usbdump(8). The reason why I committed the > patch now is there are reports that these pseudo interfaces made some > applications confused and/or caused some performance degradation on > 9.0R, and wanted to fix it in some way. Do you plan to take this to 9.1 ? > > I am still open for more sophisticated implementation and have no > objection to replace mine with it. Do you have an idea about > converting it with a loadable module? Personally I think that the right way is to add user<>kernel interface for requesting interface list since this is the most major stopper for doing BPF-only providers. However this should be discussed with rpaulo@ and delphij@ (so most probably this skips 9.1). And, as fallback solution we can probably add separate ipfwlog module which is quite easy but much less clean. > > -- Hiroki