Date: Thu, 04 Sep 2014 00:00:27 +0300 From: Andriy Gapon <avg@FreeBSD.org> To: Nikolai Lifanov <lifanov@mail.lifanov.com>, John Baldwin <jhb@FreeBSD.org> Cc: src-committers@FreeBSD.org, Peter Wemm <peter@wemm.org>, Alan Cox <alc@rice.edu>, svn-src-all@FreeBSD.org, Dmitry Morozovsky <marck@rinet.ru>, "Matthew D. Fuller" <fullermd@over-yonder.net>, Steven Hartland <killing@multiplay.co.uk>, svn-src-head@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 Message-ID: <5407816B.9000401@FreeBSD.org> In-Reply-To: <540778A2.3080809@mail.lifanov.com> 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>
next in thread | previous in thread | raw e-mail | index | archive | help
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.
It's interesting to me whether you would still see the better performance or if
that improvement would be undone.
Thanks!
--
Andriy Gapon
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5407816B.9000401>