Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755697Ab0DUPcM (ORCPT ); Wed, 21 Apr 2010 11:32:12 -0400 Received: from nlpi129.sbcis.sbc.com ([207.115.36.143]:36891 "EHLO nlpi129.prodigy.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755554Ab0DUPcK (ORCPT ); Wed, 21 Apr 2010 11:32:10 -0400 Date: Wed, 21 Apr 2010 10:31:29 -0500 (CDT) From: Christoph Lameter X-X-Sender: cl@router.home To: Mel Gorman cc: Andrew Morton , Andrea Arcangeli , Adam Litke , Avi Kivity , David Rientjes , Minchan Kim , KAMEZAWA Hiroyuki , KOSAKI Motohiro , Rik van Riel , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH 04/14] mm,migration: Allow the migration of PageSwapCache pages In-Reply-To: <20100421151417.GK30306@csn.ul.ie> Message-ID: References: <1271797276-31358-1-git-send-email-mel@csn.ul.ie> <1271797276-31358-5-git-send-email-mel@csn.ul.ie> <20100421150037.GJ30306@csn.ul.ie> <20100421151417.GK30306@csn.ul.ie> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1450 Lines: 39 On Wed, 21 Apr 2010, Mel Gorman wrote: > On Wed, Apr 21, 2010 at 10:05:21AM -0500, Christoph Lameter wrote: > > On Wed, 21 Apr 2010, Mel Gorman wrote: > > > > > No, remap_swapcache could just be called "remap". If it's 0, it's > > > considered unsafe to remap the page. > > > > Call this "can_remap"? > > > > can_do - ba dum tisch. > > While you are looking though, maybe you can confirm something for me. > > 1. Is leaving a migration PTE like this behind reasonable? (I think yes > particularly as the page was already unmapped so it's not a new fault > incurred) The design of page migration only allows for the existence of these as long as the page is locked. Not sure what would happen if you leave this hanging around. Paths that are not prepared for a migration_pte may encounter one. > 2. Is the BUG_ON check in > include/linux/swapops.h#migration_entry_to_page() now wrong? (I > think yes, but I'm not sure and I'm having trouble verifying it) The bug check ensures that migration entries only occur when the page is locked. This patch changes that behavior. This is going too oops therefore in unmap_and_move() when you try to remove the migration_ptes from an unlocked page. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/