Re: [PATCH 6/6] add documentation on proc.txt

[Minchan Kim]

Hello Dave,

On Tue, Apr 23, 2013 at 07:33:53AM -0700, Dave Hansen wrote:
> On 04/22/2013 06:53 PM, Minchan Kim wrote:
> > echo 'file' > /proc/PID/reclaim
> > echo 'anon' > /proc/PID/reclaim
> > echo 'both' > /proc/PID/reclaim
> > 
> > For range reclaim,
> > 
> > echo $((1<<20)) 8192 > /proc/PID/reclaim.
> > 
> > IOW, we don't need any type for range reclaim because only thing
> > user takes care is address range which has mapped page regardless
> > of that it's anon or file.
> > 
> > Does it make sense to you?
> 
> That looks very nice!  Although, I'd probably use 'all' instead of
> 'both'.  It leaves you more wiggle room to add more types in the future,
> like volatile pages.

Fair enough.
Thanks for the review.
I just sent a new. 

-- 
Kind regards,
Minchan Kim
--
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/