On Fri, 10 Mar 2000, Linus Torvalds wrote:

On Fri, 10 Mar 2000, Chuck Lever wrote:

quoted

ja, that's exactly what my patch does.  i'm still not sure what you don't
like about it.  i'm happy to make it do anything you want, but it sounds
like we are in agreement here.

Ok. I may not have seen (or noticed) your patch, I've seen earlier patches
that didn't do that.

quoted

my only sticking point is that i'm not sure what community consensus there
is about MADV_DONTNEED.  i'd like to create a patch that implements
everything except MADV_DONTNEED, then maybe we should have more discussion
about exactly how that will work and add it later.

I'd like MADV_DONTNEED to just clear the page tables. If it was a private
mapping, all the modifications get lost. If it was a shared mapping,
modifications since the last msync() may or may not get lost. 

i'll create a patch against 2.3.51-pre3 for madvise() and post it on the
mm list before sunday.  then we can argue about something we've both seen
:)

let me think some more about DONTNEED.  at the face of it, i agree with
your suggestion, but i may just exclude it from the patch until there is
more discussion here.

	- Chuck Lever
--
corporate:	[off-list ref]
personal:	[off-list ref] or [off-list ref]

The Linux Scalability project:
	http://www.citi.umich.edu/projects/linux-scalability/

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux.eu.org/Linux-MM/