From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx166.postini.com [74.125.245.166]) by kanga.kvack.org (Postfix) with SMTP id 6035F6B0159 for ; Thu, 13 Sep 2012 12:04:39 -0400 (EDT) Date: Thu, 13 Sep 2012 18:04:32 +0200 From: Andrea Arcangeli Subject: Re: [patch 1/2 v2]compaction: abort compaction loop if lock is contended or run too long Message-ID: <20120913160432.GG3388@redhat.com> References: <20120910011830.GC3715@kernel.org> <20120911163455.bb249a3c.akpm@linux-foundation.org> <20120912004840.GI27078@redhat.com> <20120912142019.0e06bf52.akpm@linux-foundation.org> <20120912234808.GC3404@redhat.com> <20120913004722.GA5085@bbox> <20120913093826.GT11266@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120913093826.GT11266@suse.de> Sender: owner-linux-mm@kvack.org List-ID: To: Mel Gorman Cc: Minchan Kim , Andrew Morton , Shaohua Li , linux-mm@kvack.org On Thu, Sep 13, 2012 at 10:38:26AM +0100, Mel Gorman wrote: > I agree with Minchan. Andrea's patch ignores the fact that free page > isolation might have aborted due to lock contention. It's not necessarily > going to be isolating the pages it needs for migration. Actually I thought of calling putback_lru_pages first, but then I thought it was better to just complete the current slice. Note that putback_lru_pages can take the lru_lock immediately too when the pagevec gets full which won't work any better than if the cc->contended was set by the freepages isolation and we do migrate_pages. There's no way to abort lockless from that point, so I think it's better to take the last locks to finish the current slice of work and then abort if it's still contended (which confirms we're really trashing). Skipping isolated pages without rewinding low_pfn would also reduce compaction reliability so that should be evaluated as well. And rewinding with the putback_lru_pages would risk livelocks. I agree Minchan's patch would fix the problem too, and this should be a fairly uncommon path so either ways shouldn't make a noticeable difference. -- 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-mm.org/ . Don't email: email@kvack.org