From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: Dirty/Access bits vs. page content Date: Fri, 25 Apr 2014 12:41:40 +1000 Message-ID: <1398393700.8437.22.camel@pasglop> References: <53558507.9050703@zytor.com> <53559F48.8040808@intel.com> <20140422075459.GD11182@twins.programming.kicks-ass.net> <20140423184145.GH17824@quack.suse.cz> <20140424065133.GX26782@laptop.programming.kicks-ass.net> <1398389846.8437.6.camel@pasglop> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1398389846.8437.6.camel@pasglop> Sender: linux-arch-owner@vger.kernel.org To: Linus Torvalds Cc: Hugh Dickins , Peter Zijlstra , Jan Kara , Dave Hansen , "H. Peter Anvin" , "linux-arch@vger.kernel.org" , linux-mm , Russell King - ARM Linux , Tony Luck List-Id: linux-mm.kvack.org On Fri, 2014-04-25 at 11:37 +1000, Benjamin Herrenschmidt wrote: > On Thu, 2014-04-24 at 16:46 -0700, Linus Torvalds wrote: > > - we do the TLB shootdown holding the page table lock (but that's not > > new - ptep_get_and_flush does the same > > > > The flip side is that we do a lot more IPIs for large invalidates, > since we drop the PTL on every page table page. Oh I missed that your patch was smart enough to only do that in the presence of non-anonymous dirty pages. That should take care of the common case of short lived programs, those should still fit in a single big batch. Cheers, Ben.