From owner-svn-src-all@FreeBSD.ORG Tue Nov 25 10:04:56 2014 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id C872120A; Tue, 25 Nov 2014 10:04:56 +0000 (UTC) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id 91B6CEC8; Tue, 25 Nov 2014 10:04:55 +0000 (UTC) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id MAA20607; Tue, 25 Nov 2014 12:06:46 +0200 (EET) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1XtCzN-000DbV-7t; Tue, 25 Nov 2014 12:04:53 +0200 Message-ID: <5474540D.2000607@FreeBSD.org> Date: Tue, 25 Nov 2014 12:03:57 +0200 From: Andriy Gapon User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: d@delphij.net, Xin LI , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org Subject: Re: svn commit: r273060 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs References: <201410132039.s9DKdpmh055573@svn.freebsd.org> <546F57B1.2020602@FreeBSD.org> <546F9FC3.10402@delphij.net> In-Reply-To: <546F9FC3.10402@delphij.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 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: Tue, 25 Nov 2014 10:04:56 -0000 On 21/11/2014 22:25, Xin Li wrote: > On 11/21/14 07:18, Andriy Gapon wrote: >> On 13/10/2014 23:39, Xin LI wrote: >>> Author: delphij Date: Mon Oct 13 20:39:51 2014 New Revision: >>> 273060 URL: https://svnweb.freebsd.org/changeset/base/273060 >>> >>> Log: Use write_psize instead of write_asize when doing >>> vdev_space_update. Without this change the accounting of L2ARC >>> usage would be wrong and give 16EB free space because the number >>> became negative and overflows. >>> >>> Obtained from: FreeNAS (issue #6239) > >> First, a link to the issue would be more convenient for reviewers. >> Here it is https://bugs.freenas.org/issues/6239 > >> Then, I would like to see a technical explanation for this change. >> I could not find any explanation here or in the FreeNAS issue or in >> the FreeNAS commit. > >> As far as I can see, all calls to vdev_space_update() in the ARC >> code are passed l2hdr->b_asize or a sum of b_asize fields of >> multiple buffers. Thus, I am really surprised with this change and >> would like to see reasoning behind it. > > Hmm I think you are right that my change doesn't make sense but I > can't remember the details either and suggests there is some other > issues that was covered up by this. Maybe you have some other changes in FreeNAS and the change in question is supposed to work along with them? Meanwhile, perhaps the discussed commit needs to be reverted? > It looks like that I was confused because the L2ARC clock hand > advances by buf_p_sz (unrelated but it should really be called > buf_a_size) and the way we calculate allocated L2ARC free. Agree about 'a' vs 'p'. And, BTW, we have some related changes in ClusterHQ that I will upstream. -- Andriy Gapon