From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx203.postini.com [74.125.245.203]) by kanga.kvack.org (Postfix) with SMTP id BC3276B002B for ; Fri, 21 Sep 2012 03:24:06 -0400 (EDT) Date: Fri, 21 Sep 2012 10:23:59 +0300 From: Dan Carpenter Subject: Re: mm: frontswap: fix a wrong if condition in frontswap_shrink Message-ID: <20120921072359.GB13767@mwanda> References: <505BDF34.3080905@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <505BDF34.3080905@oracle.com> Sender: owner-linux-mm@kvack.org List-ID: To: Zhenzhong Duan Cc: "linux-kernel@vger.kernel.org" , linux-mm@kvack.org, Dan Magenheimer , Konrad Rzeszutek Wilk , levinsasha928@gmail.com, Feng Jin On Fri, Sep 21, 2012 at 11:29:56AM +0800, Zhenzhong Duan wrote: > pages_to_unuse is set to 0 to unuse all frontswap pages > But that doesn't happen since a wrong condition in frontswap_shrink > cancels it. > > Signed-off-by: Zhenzhong Duan > --- > mm/frontswap.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/mm/frontswap.c b/mm/frontswap.c > index 6b3e71a..db2a86f 100644 > --- a/mm/frontswap.c > +++ b/mm/frontswap.c > @@ -275,7 +275,7 @@ static int __frontswap_shrink(unsigned long target_pages, > if (total_pages <= target_pages) { > /* Nothing to do */ > *pages_to_unuse = 0; > - return 0; > + return 1; > } This function used to return 0 or an error code. Could we add a comment at the top saying what the return values mean. regards, dan carpenter -- 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