On Tue, Jun 10, 2014 at 11:32:19PM -0400, Peter Hurley wrote: > PF: none (google.com: pe...@hurleysoftware.com does not designate permitted > sender hosts) client-ip=216.70.64.70; > Received: from h96-61-95-138.cntcnh.dsl.dynamic.tds.net ([96.61.95.138]:55986 > helo=[192.168.1.139]) > by n23.mail01.mtsvc.net with esmtpsa (TLSv1:AES128-SHA:128) > (Exim 4.72) > (envelope-from <pe...@hurleysoftware.com>) > id 1WuZGw-00064f-2L; Tue, 10 Jun 2014 23:32:22 -0400 > Message-ID: <5397cdc3.1050...@hurleysoftware.com> > Date: Tue, 10 Jun 2014 23:32:19 -0400 > From: Peter Hurley <pe...@hurleysoftware.com> > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 > Thunderbird/24.5.0 > MIME-Version: 1.0 > To: Joonsoo Kim <iamjoonsoo....@lge.com>, Andrew Morton > <a...@linux-foundation.org> > CC: Zhang Yanfei <zhangyanfei....@gmail.com>, Johannes Weiner > <han...@cmpxchg.org>, > Andi Kleen <a...@firstfloor.org>, linux...@kvack.org, > linux-kernel@vger.kernel.org, Richard Yao <r...@gentoo.org>, Eric > Dumazet <eric.duma...@gmail.com> > Subject: Re: [PATCH v2] vmalloc: use rcu list iterator to reduce > vmap_area_lock > contention > References: <1402453146-10057-1-git-send-email-iamjoonsoo....@lge.com> > In-Reply-To: <1402453146-10057-1-git-send-email-iamjoonsoo....@lge.com> > Content-Type: text/plain; charset=UTF-8; format=flowed > Content-Transfer-Encoding: 7bit > X-Authenticated-User: 990527 pe...@hurleysoftware.com > X-MT-ID: 8FA290C2A27252AACF65DBC4A42F3CE3735FB2A4 > X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 > Sender: owner-linux...@kvack.org > Precedence: bulk > X-Loop: owner-majord...@kvack.org > List-ID: <linux-mm.kvack.org> > Status: O > Content-Length: 3338 > Lines: 96 > > On 06/10/2014 10:19 PM, Joonsoo Kim wrote: > >Richard Yao reported a month ago that his system have a trouble > >with vmap_area_lock contention during performance analysis > >by /proc/meminfo. Andrew asked why his analysis checks /proc/meminfo > >stressfully, but he didn't answer it. > > > >https://lkml.org/lkml/2014/4/10/416 > > > >Although I'm not sure that this is right usage or not, there is a solution > >reducing vmap_area_lock contention with no side-effect. That is just > >to use rcu list iterator in get_vmalloc_info(). > > > >rcu can be used in this function because all RCU protocol is already > >respected by writers, since Nick Piggin commit db64fe02258f1507e13fe5 > >("mm: rewrite vmap layer") back in linux-2.6.28 > > While rcu list traversal over the vmap_area_list is safe, this may > arrive at different results than the spinlocked version. The rcu list > traversal version will not be a 'snapshot' of a single, valid instant > of the entire vmap_area_list, but rather a potential amalgam of > different list states.
Hello, Yes, you are right, but I don't think that we should be strict here. Meminfo is already not a 'snapshot' at specific time. While we try to get certain stats, the other stats can change. And, although we may arrive at different results than the spinlocked version, the difference would not be large and would not make serious side-effect. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
