From owner-svn-src-all@FreeBSD.ORG Thu Sep 4 01:18:42 2014 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 13B54565; Thu, 4 Sep 2014 01:18:42 +0000 (UTC) Received: from smtp1.multiplay.co.uk (smtp1.multiplay.co.uk [85.236.96.35]) by mx1.freebsd.org (Postfix) with ESMTP id A01011335; Thu, 4 Sep 2014 01:18:41 +0000 (UTC) Received: by smtp1.multiplay.co.uk (Postfix, from userid 65534) id 2614C20E7088A; Thu, 4 Sep 2014 01:18:33 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on smtp1.multiplay.co.uk X-Spam-Level: ** X-Spam-Status: No, score=2.8 required=8.0 tests=AWL,BAYES_00,DOS_OE_TO_MX, FSL_HELO_NON_FQDN_1,RDNS_DYNAMIC,STOX_REPLY_TYPE,TVD_FINGER_02 autolearn=no version=3.3.1 Received: from r2d2 (82-69-141-170.dsl.in-addr.zen.co.uk [82.69.141.170]) by smtp1.multiplay.co.uk (Postfix) with ESMTPS id 734B020E70885; Thu, 4 Sep 2014 01:18:31 +0000 (UTC) Message-ID: <86292055B4114529874B693EEB441CB6@multiplay.co.uk> From: "Steven Hartland" To: "Andriy Gapon" , "Nikolai Lifanov" , "John Baldwin" References: <201408281950.s7SJo90I047213@svn.freebsd.org> <169C94ED141B435BACEADB04A4824717@multiplay.co.uk> <54072E20.10802@mail.lifanov.com> <2230377.GgKARkJyaG@ralph.baldwin.cx> <540778A2.3080809@mail.lifanov.com> <5407816B.9000401@FreeBSD.org> Subject: Re: svn commit: r270759 - in head/sys: cddl/compat/opensolaris/kern cddl/compat/opensolaris/sys cddl/contrib/opensolaris/uts/common/fs/zfs vm Date: Thu, 4 Sep 2014 02:18:33 +0100 MIME-Version: 1.0 Content-Type: text/plain; format=flowed; charset="Windows-1252"; reply-type=original Content-Transfer-Encoding: 7bit X-Priority: 3 X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook Express 6.00.2900.5931 X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.6157 X-Mailman-Approved-At: Thu, 04 Sep 2014 01:19:46 +0000 Cc: src-committers@FreeBSD.org, Peter Wemm , Alan Cox , svn-src-all@FreeBSD.org, Dmitry Morozovsky , "Matthew D. Fuller" , svn-src-head@FreeBSD.org 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: Thu, 04 Sep 2014 01:18:42 -0000 ----- Original Message ----- From: "Andriy Gapon" > on 03/09/2014 23:22 Nikolai Lifanov said the following: >> On 09/03/14 15:22, John Baldwin wrote: >>> On Wednesday, September 03, 2014 11:05:04 AM Nikolai Lifanov wrote: >>>> On 09/03/14 04:09, Steven Hartland wrote: >>>>> I'm looking to MFC this change so wanted to check if >>>>> anyone had an final feedback / objections? >>>>> >>>>> I know we currently have Alan's feedback on changing >>>>> the #ifdef __i386__ to #ifndef UMA_MD_SMALL_ALLOC >>>>> which sounds sensible but waiting Peter to comment on. >>>>> >>>>> Regards >>>>> Steve >>>> >>>> I have no technical input, but this change improves ARC usefulness for >>>> me quite a bit. I would like to see the improvement in 10-STABLE. >>> >>> Can you verify that the current 10-STABLE (as of today) with all the various >>> pagedaemon fixes still has ARC issues for your workload? >>> >> >> It doesn't have any issues, but I noticed the improvement on CURRENT. I >> observed that just after this change, my package builder is much more >> likely to retain MFU and not evict useful things from there (the port >> tree) after large builds. >> However, I run a lot more 10.0-RELEASE than CURRENT and I would like to >> see this improvement release-bound. >> >> I would be happy to test this on 10-STABLE if you think that this is >> relevant. > > > As noted before, unfortunately, this commit (plus its fixups) contains at least > two related but distinct changes. So, to separate the wheat from the chaff, > could you please try to comment out the following block in > sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c, function arc_reclaim_needed: > > if (kmem_free_count() < zfs_arc_free_target) { > DTRACE_PROBE2(arc__reclaim_freetarget, uint64_t, > kmem_free_count(), uint64_t, zfs_arc_free_target); > return (1); > } > > Alternatively, I think that the same effect can be achieved by setting sysctl > vfs.zfs.arc_free_target to the same value as vm.stats.vm.v_free_min. Thats correct that would achieve the same thing. > It's interesting to me whether you would still see the better performance or if > that improvement would be undone. Indeed that would be interesting, but we might find that its quite memory size dependent given the scaling so confirming HW details would be nice too. I'd also be interested to know who wins the free race between the VM and ARC when using that value. For those following this thread but not the review, I've added some additional information there which you might be interested in: https://reviews.freebsd.org/D702 Regards Steve