From owner-svn-src-head@FreeBSD.ORG Sat Aug 24 16:44:25 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id C409E332; Sat, 24 Aug 2013 16:44:25 +0000 (UTC) (envelope-from marck@rinet.ru) Received: from woozle.rinet.ru (woozle.rinet.ru [195.54.192.68]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 4A37427F1; Sat, 24 Aug 2013 16:44:25 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by woozle.rinet.ru (8.14.5/8.14.5) with ESMTP id r7OGiNi6097555; Sat, 24 Aug 2013 20:44:23 +0400 (MSK) (envelope-from marck@rinet.ru) Date: Sat, 24 Aug 2013 20:44:23 +0400 (MSK) From: Dmitry Morozovsky To: Alexander Motin Subject: Re: svn commit: r254787 - head/sys/kern In-Reply-To: <5218DE6C.4060700@FreeBSD.org> Message-ID: References: <201308241441.r7OEfnFx060671@svn.freebsd.org> <5218D22F.8000009@FreeBSD.org> <5218DE6C.4060700@FreeBSD.org> User-Agent: Alpine 2.00 (BSF 1167 2008-08-23) X-NCC-RegID: ru.rinet X-OpenPGP-Key-ID: 6B691B03 MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (woozle.rinet.ru [0.0.0.0]); Sat, 24 Aug 2013 20:44:23 +0400 (MSK) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 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: Sat, 24 Aug 2013 16:44:25 -0000 On Sat, 24 Aug 2013, Alexander Motin wrote: > > > > > Remove locking from taskqueue_member(). The list of threads is > > > > Ah, so IIUC, it does not depend on other changes and could be tested on > > stock stable/9 right now? If so, I'd merge this to our tinderbox/poudriere > > builder and test. > > Yes, it is completely independent. Please, welcome. I am just not sure whether > you notice much difference unless you have really a lot of IOPS. On my tests > doing 90K IOPS over bunch of SSDs I've measured difference of several > percents. Yes, this is not a case of a builder, as most of IOSP is done in tmpfs (BTW, doesn't this lock appliy there as well?) OTOH, we have a couple of iSCSI storages where this could be useable too... -- Sincerely, D.Marck [DM5020, MCK-RIPE, DM3-RIPN] [ FreeBSD committer: marck@FreeBSD.org ] ------------------------------------------------------------------------ *** Dmitry Morozovsky --- D.Marck --- Wild Woozle --- marck@rinet.ru *** ------------------------------------------------------------------------