From owner-cvs-all@FreeBSD.ORG Sat Feb 2 21:31:18 2008 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4D3FE16A41B; Sat, 2 Feb 2008 21:31:18 +0000 (UTC) (envelope-from mav@FreeBSD.org) Received: from cmail.optima.ua (cmail.optima.ua [195.248.191.121]) by mx1.freebsd.org (Postfix) with ESMTP id 22DA213C43E; Sat, 2 Feb 2008 21:31:16 +0000 (UTC) (envelope-from mav@FreeBSD.org) X-Spam-Flag: SKIP X-Spam-Yversion: Spamooborona 1.7.0 Received: from [212.86.226.226] (account mav@alkar.net HELO [192.168.3.2]) by cmail.optima.ua (CommuniGate Pro SMTP 5.1.14) with ESMTPA id 72270392; Sat, 02 Feb 2008 23:31:16 +0200 Message-ID: <47A4E122.8080901@FreeBSD.org> Date: Sat, 02 Feb 2008 23:31:14 +0200 From: Alexander Motin User-Agent: Thunderbird 2.0.0.9 (Windows/20071031) MIME-Version: 1.0 To: Gleb Smirnoff References: <200801271501.m0RF1Hki089075@repoman.freebsd.org> <20080202201153.GL14339@FreeBSD.org> In-Reply-To: <20080202201153.GL14339@FreeBSD.org> Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 8bit Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Louis Mamakos , cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/netgraph/netflow ng_netflow.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 02 Feb 2008 21:31:18 -0000 Gleb Smirnoff пишет: > you should have asked me for review before committing! This is > not a bug, this is a feature. This was quite clear from the comments, > that you removed: > > - /* if export hook disconnected stop running expire(). */ > > This is intended behavior. We must not lose information unless > user explicitly wants to lose information. In the latter case > he will connect ng_hole(4) node to the "export" hook. But we must > not lose information if user runs some script that swaps receiving > node on the "export" hook. > > Please backout this change! Expire process was not depending completely on connected hook even before this commit. For example, every TCP session closing forces some data export. So even with export hook disconnected some data still will be lost and not just lost, but it was leading to memory leak which I have fixed with other commit. So if you insist that it was a feature then sorry. Then it should be documented and fixed to work correctly. But as soon as nobody notice that memory leak, probably nobody uses this feature actively. -- Alexander Motin