From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [PATCH] vma limited swapin readahead References: <20010131162424.E9053@archimedes.oak.suse.com> From: ebiederm@xmission.com (Eric W. Biederman) Date: 01 Feb 2001 00:41:12 -0700 In-Reply-To: David Gould's message of "Wed, 31 Jan 2001 16:24:24 -0800" Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: owner-linux-mm@kvack.org Return-Path: To: David Gould Cc: Marcelo Tosatti , "Stephen C. Tweedie" , lkml , linux-mm@kvack.org List-ID: David Gould writes: > Hmmm, arguably reading pages we do not want is a mistake. I should think that > if a big performance win is required to justify a design choice, it should > be especially required to show such a win for doing something that on its > face is wrong. The case for files and has already been justified. The performance gain of reading pages that are contiguous on disk has been justified. The only problem thing that has not been shown is that swap pages that are used together are located near each other in swap. As for design choices simplicity, maintainability and comprehensiblility, tend to be more important than absolute performance. This lets bugs be fixed, and the big changes that tend to be the biggest wins happen. > I am skeptical of the argument that we can win by replacing "the least > desirable" pages with pages were even less desireable and that we have > no recent indication of any need for. It seems possible under heavy swap > to discard quite a portion of the useful pages in favor of junk that just > happenned to have a lucky disk address. I won't argue that. My gut just says we should work to improve the disk addresses, so it isn't luck. ;) And only if we fail in that hack up the efficient simple policy, that we have for reading disk data in. Of course since I'm not actually writing the code at the moment this is all hot air :) Eric -- 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/