Messages in this thread | | | Date | Fri, 6 Jun 2008 23:43:14 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH -mm 07/25] second chance replacement for anonymous pages |
| |
On Sat, 07 Jun 2008 15:03:42 +0900 KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> wrote:
> > > + * pages. Ie. a ratio of 3 means 3:1 or 25% of the anonymous pages are > > > + * on the inactive list. > > > + * > > > + * total return max > > > + * memory value inactive anon > > > > This function doesn't "return" a "value". > > Sorry, my bad. > > in my country, "return value" is right word.
Not for a function which returns void!
> s/value/ratio/ is better?
Well it's referring to the value of zone->inactive_ratio. So replacing "return value" with "zone->inactive_ratio" would be ideal :)
> > > > > + * ------------------------------------- > > > + * 10MB 1 5MB > > > + * 100MB 1 50MB > > > + * 1GB 3 250MB > > > + * 10GB 10 0.9GB > > > + * 100GB 31 3GB > > > + * 1TB 101 10GB > > > + * 10TB 320 32GB > > > + */ >
| |