From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754116Ab2ARMao (ORCPT ); Wed, 18 Jan 2012 07:30:44 -0500 Received: from mail-wi0-f174.google.com ([209.85.212.174]:48932 "EHLO mail-wi0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752156Ab2ARMan (ORCPT ); Wed, 18 Jan 2012 07:30:43 -0500 MIME-Version: 1.0 In-Reply-To: <20120117140712.GC14907@tiehlicka.suse.cz> References: <20120117131601.GB14907@tiehlicka.suse.cz> <20120117140712.GC14907@tiehlicka.suse.cz> Date: Wed, 18 Jan 2012 20:30:41 +0800 Message-ID: Subject: Re: [PATCH] mm: memcg: remove checking reclaim order in soft limit reclaim From: Hillf Danton To: Michal Hocko Cc: linux-mm@kvack.org, Johannes Weiner , KAMEZAWA Hiroyuki , Hugh Dickins , Andrew Morton , LKML Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 17, 2012 at 10:07 PM, Michal Hocko wrote: > On Tue 17-01-12 21:29:52, Hillf Danton wrote: >> On Tue, Jan 17, 2012 at 9:16 PM, Michal Hocko wrote: >> > Hi, >> > >> > On Tue 17-01-12 20:47:59, Hillf Danton wrote: >> >> If async order-O reclaim expected here, it is settled down when setting up scan >> >> control, with scan priority hacked to be zero. Other than that, deny of reclaim >> >> should be removed. >> > >> > Maybe I have misunderstood you but this is not right. The check is to >> > protect from the _global_ reclaim with order > 0 when we prevent from >> > memcg soft reclaim. >> > >> need to bear mm hog in this way? > > Could you be more specific? Are you trying to fix any particular > problem? > My thought is simple, the outcome of softlimit reclaim depends little on the value of reclaim order, zero or not, and only exceeding is reclaimed, so selective response to swapd's request is incorrect. > Global reclaim should take are of the global memory pressure. Soft > reclaim is intended just to make its job easier. Btw. softlimit reclaim > is on its way out of the kernel but this will not happen in 3.3. > I will check it in 3.3 if too late for 3.2. Thanks Hillf