From owner-svn-src-head@freebsd.org Fri Aug 14 09:45:06 2015 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 405289B862B; Fri, 14 Aug 2015 09:45:06 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id 465D71454; Fri, 14 Aug 2015 09:45:04 +0000 (UTC) (envelope-from avg@FreeBSD.org) 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 MAA13156; Fri, 14 Aug 2015 12:45:02 +0300 (EEST) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1ZQBXn-0002cc-Fp; Fri, 14 Aug 2015 12:45:02 +0300 Subject: Re: svn commit: r286763 - in head/sys: cddl/contrib/opensolaris/uts/common cddl/contrib/opensolaris/uts/common/fs/zfs cddl/contrib/opensolaris/uts/common/fs/zfs/sys conf To: Alexander Motin , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org References: <201508140931.t7E9V8nF054188@repo.freebsd.org> <55CDB74F.6070708@FreeBSD.org> <55CDB802.9040507@FreeBSD.org> From: Andriy Gapon Message-ID: <55CDB864.7050100@FreeBSD.org> Date: Fri, 14 Aug 2015 12:44:04 +0300 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <55CDB802.9040507@FreeBSD.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 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: Fri, 14 Aug 2015 09:45:06 -0000 On 14/08/2015 12:42, Alexander Motin wrote: > On 14.08.2015 12:39, Andriy Gapon wrote: >> On 14/08/2015 12:31, Alexander Motin wrote: >>> Author: mav >>> Date: Fri Aug 14 09:31:07 2015 >>> New Revision: 286763 >>> URL: https://svnweb.freebsd.org/changeset/base/286763 >>> >>> Log: >>> MFV r277431: 5497 lock contention on arcs_mtx >>> >>> Reviewed by: George Wilson >>> Reviewed by: Matthew Ahrens >>> Reviewed by: Richard Elling >>> Approved by: Dan McDonald >>> Author: Prakash Surya >>> >>> illumos/illumos-gate@244781f10dcd82684fd8163c016540667842f203 >>> >>> This patch attempts to reduce lock contention on the current arc_state_t >>> mutexes. These mutexes are used liberally to protect the number of LRU >>> lists within the ARC (e.g. ARC_mru, ARC_mfu, etc). The granularity at >>> which these locks are acquired has been shown to greatly affect the >>> performance of highly concurrent, cached workloads. >>> >>> Added: >>> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/multilist.c (contents, props changed) >>> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/multilist.h (contents, props changed) >>> Modified: >>> head/sys/cddl/contrib/opensolaris/uts/common/Makefile.files >>> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c >>> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_pool.c >>> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/arc.h >>> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zio_inject.c >>> head/sys/conf/files >>> Directory Properties: >>> head/sys/cddl/contrib/opensolaris/ (props changed) >>> >>> Modified: head/sys/cddl/contrib/opensolaris/uts/common/Makefile.files >>> ============================================================================== >>> --- head/sys/cddl/contrib/opensolaris/uts/common/Makefile.files Fri Aug 14 09:25:54 2015 (r286762) >>> +++ head/sys/cddl/contrib/opensolaris/uts/common/Makefile.files Fri Aug 14 09:31:07 2015 (r286763) >>> @@ -68,6 +68,7 @@ ZFS_COMMON_OBJS += \ >>> lz4.o \ >>> lzjb.o \ >>> metaslab.o \ >>> + multilist.o \ >>> range_tree.o \ >>> refcount.o \ >>> rrwlock.o \ >> >> Thank you! >> The new file also has to be added to sys/conf/files for those who >> compile ZFS into a kernel. > > And it was added just below. Have I missed something? > Oops, sorry for the noise. -- Andriy Gapon