From mboxrd@z Thu Jan 1 00:00:00 1970 From: KOSAKI Motohiro Subject: Re: [PATCH 02/12] vmscan: shrinker->nr updates race and go wrong Date: Mon, 20 Jun 2011 13:30:29 +0900 Message-ID: <4DFECCE5.5030409@jp.fujitsu.com> References: <1306998067-27659-1-git-send-email-david@fromorbit.com> <1306998067-27659-3-git-send-email-david@fromorbit.com> <4DFE987E.1070900@jp.fujitsu.com> <20110620012531.GN561@dastard> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, xfs@oss.sgi.com To: david@fromorbit.com Return-path: Received: from fgwmail5.fujitsu.co.jp ([192.51.44.35]:35940 "EHLO fgwmail5.fujitsu.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751139Ab1FTEai (ORCPT ); Mon, 20 Jun 2011 00:30:38 -0400 In-Reply-To: <20110620012531.GN561@dastard> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: >> Looks great fix. Please remove tracepoint change from this patch and send it >> to -stable. iow, I expect I'll ack your next spin. > > I don't believe such a change belongs in -stable. This code has been > buggy for many years and as I mentioned it actually makes existing > bad shrinker behaviour worse. I don't test stable kernels, so I've > got no idea what side effects it will have outside of this series. > I'm extremely hesitant to change VM behaviour in stable kernels > without having tested first, so I'm not going to push it for stable > kernels. Ok, I have no strong opinion. > > If you want it in stable kernels, then you can always let > stable@kernel.org know once the commits are in the mainline tree and > you've tested them...