Re: [PATCH] mm: MADV_WILLNEED implementation for anonymous memory

From: Peter Zijlstra
Date: Thu Jan 31 2008 - 04:53:53 EST



On Thu, 2008-01-31 at 01:47 -0800, Andrew Morton wrote:
> On Thu, 31 Jan 2008 10:35:18 +0100 Peter Zijlstra <a.p.zijlstra@xxxxxxxxx> wrote:
>
> >
> > On Thu, 2008-01-31 at 01:12 -0800, Andrew Morton wrote:
> >
> > > Implementation-wise: make_pages_present() _can_ be converted to do this.
> > > But it's a lot of patching, and the result will be a cleaner, faster and
> > > smaller core MM. Whereas your approach is easy, but adds more code and
> > > leaves the old stuff slow-and-dirty.
> > >
> > > Guess which approach is preferred? ;)
> >
> > Ok, I'll look at using make_pages_present().
>
> Am still curious to know what inspired this change. What are the use
> cases? Performance testing results, etc?

Ah, that is Lennarts Pulse Audio thing, he has samples in memory which
might not have been used for a while, and he wants to be able to
pre-fetch those when he suspects they might need to be played. So that
once the audio thread comes along and stuffs them down /dev/dsp its all
nice in memory.

Since its all soft real-time at best he feels its better to do a best
effort at not hitting swap than it is to strain the system with mlock
usage.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/