From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Hocko Subject: Re: [PATCH -mm v9 0/8] idle memory tracking Date: Wed, 29 Jul 2015 14:36:30 +0200 Message-ID: <20150729123629.GI15801@dhcp22.suse.cz> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-doc-owner@vger.kernel.org To: Vladimir Davydov Cc: Andrew Morton , Andres Lagar-Cavilla , Minchan Kim , Raghavendra K T , Johannes Weiner , Greg Thelen , Michel Lespinasse , David Rientjes , Pavel Emelyanov , Cyrill Gorcunov , Jonathan Corbet , linux-api@vger.kernel.org, linux-doc@vger.kernel.org, linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-api@vger.kernel.org On Sun 19-07-15 15:31:09, Vladimir Davydov wrote: [...] > ---- USER API ---- > > The user API consists of two new proc files: I was thinking about this for a while. I dislike the interface. It is quite awkward to use - e.g. you have to read the full memory to check a single memcg idleness. This might turn out being a problem especially on large machines. It also provides a very low level information (per-pfn idleness) which is inherently racy. Does anybody really require this level of detail? I would assume that most users are interested only in a single number which tells the idleness of the system/memcg. Well, you have mentioned a per-process reclaim but I am quite skeptical about this. I guess the primary reason to rely on the pfn rather than the LRU walk, which would be more targeted (especially for memcg cases), is that we cannot hold lru lock for the whole LRU walk and we cannot continue walking after the lock is dropped. Maybe we can try to address that instead? I do not think this is easy to achieve but have you considered that as an option? -- Michal Hocko SUSE Labs