Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757601AbYFSITw (ORCPT ); Thu, 19 Jun 2008 04:19:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756413AbYFSITe (ORCPT ); Thu, 19 Jun 2008 04:19:34 -0400 Received: from fgwmail5.fujitsu.co.jp ([192.51.44.35]:36664 "EHLO fgwmail5.fujitsu.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751545AbYFSITc (ORCPT ); Thu, 19 Jun 2008 04:19:32 -0400 Date: Thu, 19 Jun 2008 17:24:09 +0900 From: KAMEZAWA Hiroyuki To: Daisuke Nishimura Cc: KOSAKI Motohiro , Andrew Morton , Rik van Riel , Lee Schermerhorn , Nick Piggin , linux-mm@kvack.org, linux-kernel@vger.kernel.org, kernel-testers@vger.kernel.org Subject: Re: [Experimental][PATCH] putback_lru_page rework Message-Id: <20080619172409.9fd80838.kamezawa.hiroyu@jp.fujitsu.com> In-Reply-To: <20080619170059.60b42e73.nishimura@mxp.nes.nec.co.jp> References: <20080617164709.de4db070.nishimura@mxp.nes.nec.co.jp> <20080618184000.a855dfe0.kamezawa.hiroyu@jp.fujitsu.com> <20080618195009.37BF.KOSAKI.MOTOHIRO@jp.fujitsu.com> <20080618205540.11a1644b.kamezawa.hiroyu@jp.fujitsu.com> <20080619170059.60b42e73.nishimura@mxp.nes.nec.co.jp> Organization: Fujitsu X-Mailer: Sylpheed 2.4.2 (GTK+ 2.10.11; i686-pc-mingw32) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2043 Lines: 66 On Thu, 19 Jun 2008 17:00:59 +0900 Daisuke Nishimura wrote: > > > > - unlock = putback_lru_page(newpage); > > > > + putback_lru_page(newpage); > > > > } else > > > > newpage->mapping = NULL; > > > > > > originally move_to_lru() called in unmap_and_move(). > > > unevictable infrastructure patch move to this point for > > > calling putback_lru_page() under page locked. > > > > > > So, your patch remove page locked dependency. > > > move to unmap_and_move() again is better. > > > > > > it become page lock holding time reducing. > > > > > ok, will look into again. > > > > I agree with Kosaki-san. > > And VM_BUG_ON(page_count(newpage) != 1) in unmap_and_move() > is not correct again, IMHO. > I got this BUG actually when testing this patch(with > migratin_entry_wait fix). > > unmap_and_move() > move_to_new_page() > migrate_page() > remove_migration_ptes() > putback_lru_page() (*1) > : > if (!newpage->mapping) (*2) > VM_BUG_ON(page_count(newpage) != 1) > > If a anonymous page(without mapping) is migrated successfully, > this page is moved back to lru by putback_lru_page()(*1), > and the page count becomes 1(pte only). > yes. > At the same time(between *1 and *2), if the process > that owns this page are freeing this page, the page count > becomes 0 and ->mapping becomes NULL by free_hot_cold_page(), > so this BUG is caused. > Agree, I see. > I've not seen this BUG on real HW yet(seen twice on fake-numa > hvm guest of Xen), but I think it can happen theoretically. > That's (maybe) because page->mapping is not cleared when it's removed from rmap. (and there is pagevec to dealy freeing....) But ok, I see your point. KOSAKI-san is now writing patch set to fix the whole. please see it. Thanks, -Kame -- 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/