From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932111AbbCWF13 (ORCPT ); Mon, 23 Mar 2015 01:27:29 -0400 Received: from mail-qc0-f182.google.com ([209.85.216.182]:36718 "EHLO mail-qc0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752382AbbCWF10 (ORCPT ); Mon, 23 Mar 2015 01:27:26 -0400 Date: Mon, 23 Mar 2015 01:27:22 -0400 From: Tejun Heo To: axboe@kernel.dk Cc: linux-kernel@vger.kernel.org, jack@suse.cz, hch@infradead.org, hannes@cmpxchg.org, linux-fsdevel@vger.kernel.org, vgoyal@redhat.com, lizefan@huawei.com, cgroups@vger.kernel.org, linux-mm@kvack.org, mhocko@suse.cz, clm@fb.com, fengguang.wu@intel.com, david@fromorbit.com, gthelen@google.com, Vladimir Davydov Subject: Re: [PATCH 18/18] mm: vmscan: remove memcg stalling on writeback pages during direct reclaim Message-ID: <20150323052722.GB8991@htj.duckdns.org> References: <1427087267-16592-1-git-send-email-tj@kernel.org> <1427087267-16592-19-git-send-email-tj@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1427087267-16592-19-git-send-email-tj@kernel.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 23, 2015 at 01:07:47AM -0400, Tejun Heo wrote: > Because writeback wasn't cgroup aware before, the usual dirty > throttling mechanism in balance_dirty_pages() didn't work for > processes under memcg limit. The writeback path didn't know how much > memory is available or how fast the dirty pages are being written out > for a given memcg and balance_dirty_pages() didn't have any measure of > IO back pressure for the memcg. > > To work around the issue, memcg implemented an ad-hoc dirty throttling > mechanism in the direct reclaim path by stalling on pages under > writeback which are encountered during direct reclaim scan. This is > rather ugly and crude - none of the configurability, fairness, or > bandwidth-proportional distribution of the normal path. > > The previous patches implemented proper memcg aware dirty throttling > and the ad-hoc mechanism is no longer necessary. Remove it. Oops, just realized that this can't be removed, at least yet. !unified path still depends on it. I'll update the patch to disable these checks only on the unified hierarchy. Thanks. -- tejun