From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f69.google.com (mail-wm0-f69.google.com [74.125.82.69]) by kanga.kvack.org (Postfix) with ESMTP id 305C2828FF for ; Tue, 14 Jun 2016 03:01:34 -0400 (EDT) Received: by mail-wm0-f69.google.com with SMTP id 4so40244843wmz.1 for ; Tue, 14 Jun 2016 00:01:34 -0700 (PDT) Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com. [74.125.82.65]) by mx.google.com with ESMTPS id s20si2473908wmb.51.2016.06.14.00.01.32 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 14 Jun 2016 00:01:33 -0700 (PDT) Received: by mail-wm0-f65.google.com with SMTP id r5so19928318wmr.0 for ; Tue, 14 Jun 2016 00:01:32 -0700 (PDT) Date: Tue, 14 Jun 2016 09:01:31 +0200 From: Michal Hocko Subject: Re: [RFC 03/18] memcontrol: present maximum used memory also for cgroup-v2 Message-ID: <20160614070130.GB5681@dhcp22.suse.cz> References: <1465847065-3577-1-git-send-email-toiwoton@gmail.com> <1465847065-3577-4-git-send-email-toiwoton@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1465847065-3577-4-git-send-email-toiwoton@gmail.com> Sender: owner-linux-mm@kvack.org List-ID: To: Topi Miettinen Cc: linux-kernel@vger.kernel.org, Johannes Weiner , Vladimir Davydov , Andrew Morton , "open list:CONTROL GROUP - MEMORY RESOURCE CONTROLLER (MEMCG)" , "open list:CONTROL GROUP - MEMORY RESOURCE CONTROLLER (MEMCG)" On Mon 13-06-16 22:44:10, Topi Miettinen wrote: > Present maximum used memory in cgroup memory.current_max. It would be really much more preferable to present the usecase in the patch description. It is true that this information is presented in the v1 API but the current policy is to export new knobs only when there is a reasonable usecase for it. > Signed-off-by: Topi Miettinen > --- > include/linux/page_counter.h | 7 ++++++- > mm/memcontrol.c | 13 +++++++++++++ > 2 files changed, 19 insertions(+), 1 deletion(-) > > diff --git a/include/linux/page_counter.h b/include/linux/page_counter.h > index 7e62920..be4de17 100644 > --- a/include/linux/page_counter.h > +++ b/include/linux/page_counter.h > @@ -9,9 +9,9 @@ struct page_counter { > atomic_long_t count; > unsigned long limit; > struct page_counter *parent; > + unsigned long watermark; > > /* legacy */ > - unsigned long watermark; > unsigned long failcnt; > }; > > @@ -34,6 +34,11 @@ static inline unsigned long page_counter_read(struct page_counter *counter) > return atomic_long_read(&counter->count); > } > > +static inline unsigned long page_counter_read_watermark(struct page_counter *counter) > +{ > + return counter->watermark; > +} > + > void page_counter_cancel(struct page_counter *counter, unsigned long nr_pages); > void page_counter_charge(struct page_counter *counter, unsigned long nr_pages); > bool page_counter_try_charge(struct page_counter *counter, > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 75e7440..5513771 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -4966,6 +4966,14 @@ static u64 memory_current_read(struct cgroup_subsys_state *css, > return (u64)page_counter_read(&memcg->memory) * PAGE_SIZE; > } > > +static u64 memory_current_max_read(struct cgroup_subsys_state *css, > + struct cftype *cft) > +{ > + struct mem_cgroup *memcg = mem_cgroup_from_css(css); > + > + return (u64)page_counter_read_watermark(&memcg->memory) * PAGE_SIZE; > +} > + > static int memory_low_show(struct seq_file *m, void *v) > { > struct mem_cgroup *memcg = mem_cgroup_from_css(seq_css(m)); > @@ -5179,6 +5187,11 @@ static struct cftype memory_files[] = { > .read_u64 = memory_current_read, > }, > { > + .name = "current_max", > + .flags = CFTYPE_NOT_ON_ROOT, > + .read_u64 = memory_current_max_read, > + }, > + { > .name = "low", > .flags = CFTYPE_NOT_ON_ROOT, > .seq_show = memory_low_show, > -- > 2.8.1 -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org