Re: [PATCH] mm: make fault_around_bytes configurable

From	"Kirill A. Shutemov" <kirill@shutemov.name>
Newsgroups	linux.kernel
Subject	Re: [PATCH] mm: make fault_around_bytes configurable
Date	2016-05-17 14:40 +0200
Message-ID	<rzMs1-4zK-1@gated-at.bofh.it> (permalink)
References	(5 earlier) <rwNXk-n9-25@gated-at.bofh.it> <rx6wW-2kR-5@gated-at.bofh.it> <rzrxg-85I-23@gated-at.bofh.it> <rzrGW-892-3@gated-at.bofh.it> <rzs9Y-8iT-35@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, May 16, 2016 at 11:56:32PM +0900, Minchan Kim wrote:
> On Mon, May 16, 2016 at 05:29:00PM +0300, Kirill A. Shutemov wrote:
> > > Kirill,
> > > You wanted to test non-HW access bit system and I did.
> > > What's your opinion?
> > 
> > Sorry, for late response.
> > 
> > My patch is incomlete: we need to find a way to not mark pte as old if we
> > handle page fault for the address the pte represents.
> 
> I'm sure you can handle it but my point is there wouldn't be a big gain
> although you can handle it in non-HW access bit system. Okay, let's be
> more clear because I don't have every non-HW access bit architecture.
> At least, current mobile workload in ARM which I have wouldn't be huge
> benefit.
> I will say one more.
> I tested the workload on quad-core system and core speed is not so slow
> compared to recent other mobile phone SoC. Even when I tested the benchmark
> without pte_mkold, the benefit is within noise because storage is really
> slow so major fault is dominant factor. So, I decide test storage from eMMC
> to eSATA. And then finally, I manage to see the a little beneift with
> fault_around without pte_mkold.
> 
> However, let's consider side-effect aspect from fault_around.
> 
> 1. Increase slab shrinking compard to old
> 2. high level vmpressure compared to old
> 
> With considering that regressions on my system, it's really not worth to
> try at the moment.
> That's why I wanted to disable fault_around as default in non-HW access
> bit system.

Feel free to post such patch. I guess it's reasonable.

> > Once this will be done, the number of page faults shouldn't be higher with
> > fault-around enabled even on machines without hardware accessed bit. This
> > will address performance regression with the patch on such machines.
> 
> Although you solves that, I guess the benefit would be marginal in
> some architectures but we should solve above side-effects.
> 
> > 
> > I'll try to find time to update the patch soon.
> 
> I hope you can solve above those regressions as well.

The patch is posted. Please test.

-- 
 Kirill A. Shutemov

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] mm: make fault_around_bytes configurable Minchan Kim <minchan@kernel.org> - 2016-05-16 16:20 +0200
  Re: [PATCH] mm: make fault_around_bytes configurable "Kirill A. Shutemov" <kirill@shutemov.name> - 2016-05-16 16:30 +0200
    Re: [PATCH] mm: make fault_around_bytes configurable Minchan Kim <minchan@kernel.org> - 2016-05-16 17:00 +0200
      Re: [PATCH] mm: make fault_around_bytes configurable "Kirill A. Shutemov" <kirill@shutemov.name> - 2016-05-17 14:40 +0200

csiph-web