From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [RFC] Making memcg track ownership per address_space or anon_vma Date: Wed, 11 Feb 2015 17:30:30 -0500 Message-ID: <20150211223030.GB12728@htj.duckdns.org> References: <20150207143839.GA9926@htj.dyndns.org> <20150211021906.GA21356@htj.duckdns.org> <20150211203359.GF21356@htj.duckdns.org> <20150211214650.GA11920@htj.duckdns.org> <20150211220530.GA12728@htj.duckdns.org> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=ZN8k+5eb38B6zfXDxf5+A24nhopFVyP5jBFDupM/JMQ=; b=CUwKe10ihzTrSGkcWov+2hbXCKfScRTT8YsozctgvN5PAcTfmFXSIogLpejSPLfI1Z nbjeV9OAC5ASy5QfPAzXmFyrzxBx55M7Ya/qiI6+aPErZStLb8PvVRo71dIw8NmIXlPy 4iwQICmHJZ95v//9mj+oN+WodqFl8pIjtJypZoCev1sxrRu6VpT+aFDxYIGDKbCvs3nF t07xTyok43X8MBuUGnPO6zRQ+dIrtovG+QYHul8zA1V57dGzedsCxsPzvp6CrCVgcPaK 7XjX50GK2S/KvbJ+t155sSJ+M1ZU3cIBIERkyBQTCu6EeUJhBVdOW4r6cuYMLxXtTFRz S1tQ== Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Konstantin Khlebnikov Cc: Greg Thelen , Konstantin Khlebnikov , Johannes Weiner , Michal Hocko , Cgroups , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" , Jan Kara , Dave Chinner , Jens Axboe , Christoph Hellwig , Li Zefan , Hugh Dickins Hello, On Thu, Feb 12, 2015 at 02:15:29AM +0400, Konstantin Khlebnikov wrote: > Well, ok. Even if shared writes are rare whey should be handled somehow > without relying on kupdate-like writeback. If memcg has a lot of dirty pages This only works iff we consider those cases to be marginal enough to be handle them in a pretty ghetto way. > but their inodes are accidentially belong to wrong wb queues when tasks in > that memcg shouldn't stuck in balance-dirty-pages until somebody outside > acidentially writes this data. That's all what I wanted to say. But, right, yeah, corner cases around this could be nasty if writeout interval is set really high. I don't think it matters for the default 5s interval at all. Maybe what we need is queueing a delayed per-wb work w/ the default writeout interval when dirtying a foreign inode. I'll think more about it. Thanks. -- tejun