From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753280Ab2F2Wco (ORCPT ); Fri, 29 Jun 2012 18:32:44 -0400 Received: from mail-pz0-f46.google.com ([209.85.210.46]:39499 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751789Ab2F2Wcn (ORCPT ); Fri, 29 Jun 2012 18:32:43 -0400 Date: Sat, 30 Jun 2012 07:32:35 +0900 From: Minchan Kim To: Cong Wang Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] vmscan: remove obsolete comment of shrinker Message-ID: <20120629223235.GB2079@barrios> References: <1340945500-14566-1-git-send-email-minchan@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jun 29, 2012 at 01:15:43PM +0000, Cong Wang wrote: > > On Fri, 29 Jun 2012 at 04:51 GMT, Minchan Kim wrote: > > 09f363c7 fixed shrinker callback returns -1 when nr_to_scan is zero > > for preventing excessive the slab scanning. But 635697c6 fixed the > > problem, again so we can freely return -1 although nr_to_scan is zero. > > So let's revert 09f363c7 because the comment added in 09f363c7 made a > > unnecessary rule shrinker user should be aware of. > > > > Please also include the subject of the commit, not just raw hash number. ;) > > For example, > > 09f363c7("vmscan: fix shrinker callback bug in fs/super.c") > 635697c6("vmscan: fix initial shrinker size handling") Yeb. It seems akpm handled it by himself. But I will keep in mind. Thanks, Cong!