From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755388AbZHULTE (ORCPT ); Fri, 21 Aug 2009 07:19:04 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754694AbZHULTD (ORCPT ); Fri, 21 Aug 2009 07:19:03 -0400 Received: from mail-gx0-f205.google.com ([209.85.217.205]:59176 "EHLO mail-gx0-f205.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754501AbZHULTC (ORCPT ); Fri, 21 Aug 2009 07:19:02 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; b=sdq/BgssciVwMrmITO7ahHQ7ZngQ5hmtVz3f78qUzxJ7PbvcecWrM8NecA5PRHNSvJ bB3dJeMi1N5wTxxExbqr58i4Tm9/XI351hC4zGBIG0CoNXWqaeofgxV5vG5lVp8rVhLc RQ73JJuyNi4Uh+kdLPIODJn8dDanjSq0yvq0Q= MIME-Version: 1.0 In-Reply-To: <4A8C98DD.9030500@redhat.com> References: <20090820085544.faed1ca4.minchan.kim@barrios-desktop> <4A8C98DD.9030500@redhat.com> Date: Fri, 21 Aug 2009 20:19:02 +0900 X-Google-Sender-Auth: ed9c3569ffb1da2f Message-ID: <2f11576a0908210419u2feadeaxc40ffda1cd7f6b2d@mail.gmail.com> Subject: Re: [PATCH] mm: Fix to infinite churning of mlocked page From: KOSAKI Motohiro To: Rik van Riel Cc: Minchan Kim , linux-mm , lkml , Andrew Morton , Wu Fengguang , Lee Schermerhorn Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2009/8/20 Rik van Riel : > Minchan Kim wrote: >> >> Mlocked page might lost the isolatation race. >> It cause the page to clear PG_mlocked while it remains >> in VM_LOCKED vma. It means it can be put [in]active list. >> We can rescue it by try_to_unmap in shrink_page_list. >> >> But now, As Wu Fengguang pointed out, vmscan have a bug. >> If the page has PG_referenced, it can't reach try_to_unmap >> in shrink_page_list but put into active list. If the page >> is referenced repeatedly, it can remain [in]active list >> without moving unevictable list. >> >> This patch can fix it. >> >> Reported-by: Wu Fengguang >> Signed-off-by: Minchan Kim >> Cc: KOSAKI Motohiro < >> Cc: Lee Schermerhorn >> Cc: Rik van Riel > > Acked-by: Rik van Riel Reviewed-by: KOSAKI Motohiro