From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.1 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 44DA2C4363D for ; Wed, 30 Sep 2020 23:35:53 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id AD0AA20B1F for ; Wed, 30 Sep 2020 23:35:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="diZHKDET" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org AD0AA20B1F Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 0D9128E0001; Wed, 30 Sep 2020 19:35:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 08B106B0062; Wed, 30 Sep 2020 19:35:52 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F096A8E0001; Wed, 30 Sep 2020 19:35:51 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0027.hostedemail.com [216.40.44.27]) by kanga.kvack.org (Postfix) with ESMTP id D85416B005C for ; Wed, 30 Sep 2020 19:35:51 -0400 (EDT) Received: from smtpin05.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id 9A47A1F10 for ; Wed, 30 Sep 2020 23:35:51 +0000 (UTC) X-FDA: 77321337702.05.tin44_2c1834a27196 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin05.hostedemail.com (Postfix) with ESMTP id 790F61815A4AF for ; Wed, 30 Sep 2020 23:35:51 +0000 (UTC) X-HE-Tag: tin44_2c1834a27196 X-Filterd-Recvd-Size: 6382 Received: from mail-lf1-f67.google.com (mail-lf1-f67.google.com [209.85.167.67]) by imf01.hostedemail.com (Postfix) with ESMTP for ; Wed, 30 Sep 2020 23:35:50 +0000 (UTC) Received: by mail-lf1-f67.google.com with SMTP id b12so4242636lfp.9 for ; Wed, 30 Sep 2020 16:35:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=rC8CI2ke9jAFUpO6Co2LbsAK2GppzR0D9YtgZMZXg44=; b=diZHKDET5rO8jvr/fdB7nmT7XwckVn/UpajXpihkv/H9QFyJGVYPljBcgJCkMQ/YD3 bo72M6bQKvjWyTIpXkG6dePz+G9mAZSVeCud+dbcWLaGRxa+K7awIkm4VRotazC1r0Xf ppyQiii6NVGDeiTrwFy7yM9KMtH6vANz7Yw4pEnD69fHVqntTwc6zK19m9SgOTRtzn0O d4U4fPtk7de+Y9CT4SJatuZXp6LvnCEU+wwaMVymqe+StnAm6z+Otbp5zRyRw5GcbVJt lAZtyO4tjoa6fk+rBeMx/4piZah+aSm8TOa7dyeTziYL4GSjDL9z6dqdfHiQFlOrj/Hq Tr/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=rC8CI2ke9jAFUpO6Co2LbsAK2GppzR0D9YtgZMZXg44=; b=N21PCxXFSQeWmi3mOPZdH525e15K3C1d2WcoE3DKCN9WrbwM0gZgnV00Quk3VOQzTl NGgnGx7WzXmx6O+3rIHdWM0I3PxoJCjN8JQX+xL/mNiE/I0LyL3mnTGPPGnqV8crhXra vrsIfwYCBbM4HLM5MIgsKO62NnTd4cbNeoxFU4YM1++g3YJoT75nzbSPEMd3yLYZfwNC woOoIYFNEtmGNiAz6v/CIX8NMtPses+QWMfrzXihni3+Dt1wwSDVMjWzQ8NdIf2PUQC3 lM6LL81goEK3MO1qkwHboZTnHTvGFMbFCEmGpvH0d9ULnCJ7wetf8uEV+psPGitorxz3 znBg== X-Gm-Message-State: AOAM532+R1sT/9qJsnFDxyd1oIrtbVLi3SceonX7xGp3Pk9Ohkz7K7IO GQhgDW5jJ4gy3j8C6FPpUAkw2Gg2gE6bG9sqpw5zaA== X-Google-Smtp-Source: ABdhPJz7aaNnBs7Sg579FwOOrYFrkhMrAPLz6RJ3pfRUQKrzLvH0tfQc5JeVzfiwHWrBEP3pU+2BurqrBd/7DkdkbyY= X-Received: by 2002:a19:604e:: with SMTP id p14mr1486668lfk.385.1601508949201; Wed, 30 Sep 2020 16:35:49 -0700 (PDT) MIME-Version: 1.0 References: <20200929235920.537849-1-guro@fb.com> <20200929235920.537849-4-guro@fb.com> In-Reply-To: <20200929235920.537849-4-guro@fb.com> From: Shakeel Butt Date: Wed, 30 Sep 2020 16:35:37 -0700 Message-ID: Subject: Re: [PATCH v3 3/4] mm: introduce page memcg flags To: Roman Gushchin Cc: Andrew Morton , Johannes Weiner , Michal Hocko , LKML , Linux MM , Kernel Team Content-Type: text/plain; charset="UTF-8" X-Bogosity: Ham, tests=bogofilter, spamicity=0.003273, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue, Sep 29, 2020 at 4:59 PM Roman Gushchin wrote: > > The lowest bit in page->memcg_data is used to distinguish between > struct memory_cgroup pointer and a pointer to a objcgs array. > All checks and modifications of this bit are open-coded. > > Let's formalize it using page memcg flags, defined in enum > page_memcg_data_flags. > > Additional flags might be added later. > > Signed-off-by: Roman Gushchin One nit below: Reviewed-by: Shakeel Butt > --- > include/linux/memcontrol.h | 35 ++++++++++++++++++++++++----------- > 1 file changed, 24 insertions(+), 11 deletions(-) > > diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h > index d4d15c04bbaf..35f846c6b89b 100644 > --- a/include/linux/memcontrol.h > +++ b/include/linux/memcontrol.h > @@ -343,6 +343,15 @@ struct mem_cgroup { > > extern struct mem_cgroup *root_mem_cgroup; > > +enum page_memcg_data_flags { > + /* page->memcg_data is a pointer to an objcgs vector */ > + MEMCG_DATA_OBJCGS, > + /* the next bit after the last actual flag */ > + __NR_MEMCG_DATA_FLAGS, > +}; > + > +#define MEMCG_DATA_FLAGS_MASK ((0x1UL << __NR_MEMCG_DATA_FLAGS) - 1) > + Nit: #define PG_MEMCG_OBJCGS (0x1UL << MEMCG_DATA_OBJCGS) and use this macro below. > /* > * page_memcg - get the memory cgroup associated with a page > * @page: a pointer to the page struct > @@ -404,13 +413,7 @@ static inline struct mem_cgroup *page_memcg_check(struct page *page) > */ > unsigned long memcg_data = READ_ONCE(page->memcg_data); > > - /* > - * The lowest bit set means that memcg isn't a valid > - * memcg pointer, but a obj_cgroups pointer. > - * In this case the page is shared and doesn't belong > - * to any specific memory cgroup. > - */ > - if (memcg_data & 0x1UL) > + if (memcg_data & (0x1UL << MEMCG_DATA_OBJCGS)) > return NULL; > > return (struct mem_cgroup *)memcg_data; > @@ -459,7 +462,12 @@ static inline void clear_page_memcg(struct page *page) > */ > static inline struct obj_cgroup **page_objcgs(struct page *page) > { > - return (struct obj_cgroup **)(READ_ONCE(page->memcg_data) & ~0x1UL); > + unsigned long memcg_data = READ_ONCE(page->memcg_data); > + > + VM_BUG_ON_PAGE(memcg_data && > + !(memcg_data & (0x1UL << MEMCG_DATA_OBJCGS)), page); > + > + return (struct obj_cgroup **)(memcg_data & ~MEMCG_DATA_FLAGS_MASK); > } > > /* > @@ -474,8 +482,9 @@ static inline struct obj_cgroup **page_objcgs_check(struct page *page) > { > unsigned long memcg_data = READ_ONCE(page->memcg_data); > > - if (memcg_data && (memcg_data & 0x1UL)) > - return (struct obj_cgroup **)(memcg_data & ~0x1UL); > + if (memcg_data && (memcg_data & (0x1UL << MEMCG_DATA_OBJCGS))) > + return (struct obj_cgroup **) > + (memcg_data & ~MEMCG_DATA_FLAGS_MASK); > > return NULL; > } > @@ -490,7 +499,11 @@ static inline struct obj_cgroup **page_objcgs_check(struct page *page) > static inline bool set_page_objcgs(struct page *page, > struct obj_cgroup **objcgs) > { > - return !cmpxchg(&page->memcg_data, 0, (unsigned long)objcgs | 0x1UL); > + unsigned long memcg_data = (unsigned long)objcgs; > + > + memcg_data |= (0x1UL << MEMCG_DATA_OBJCGS); > + > + return !cmpxchg(&page->memcg_data, 0, memcg_data); > } > > /* > -- > 2.26.2 >