From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754520AbbHXMLd (ORCPT ); Mon, 24 Aug 2015 08:11:33 -0400 Received: from outbound-smtp03.blacknight.com ([81.17.249.16]:52685 "EHLO outbound-smtp03.blacknight.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754185AbbHXMJz (ORCPT ); Mon, 24 Aug 2015 08:09:55 -0400 From: Mel Gorman To: Andrew Morton Cc: Johannes Weiner , Rik van Riel , Vlastimil Babka , David Rientjes , Joonsoo Kim , Michal Hocko , Linux-MM , LKML , Mel Gorman Subject: [PATCH 04/12] mm, page_alloc: Only check cpusets when one exists that can be mem-controlled Date: Mon, 24 Aug 2015 13:09:43 +0100 Message-Id: <1440418191-10894-5-git-send-email-mgorman@techsingularity.net> X-Mailer: git-send-email 2.4.6 In-Reply-To: <1440418191-10894-1-git-send-email-mgorman@techsingularity.net> References: <1440418191-10894-1-git-send-email-mgorman@techsingularity.net> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org David Rientjes correctly pointed out that the "root cpuset may not exclude mems on the system so, even if mounted, there's no need to check or be worried about concurrent change when there is only one cpuset". The three checks for cpusets_enabled() care whether a cpuset exists that can limit memory, not that cpuset is enabled as such. This patch replaces cpusets_enabled() with cpusets_mems_enabled() which checks if at least one cpuset exists that can limit memory and updates the appropriate call sites. Signed-off-by: Mel Gorman --- include/linux/cpuset.h | 16 +++++++++------- mm/page_alloc.c | 2 +- 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/include/linux/cpuset.h b/include/linux/cpuset.h index 6eb27cb480b7..1e823870987e 100644 --- a/include/linux/cpuset.h +++ b/include/linux/cpuset.h @@ -17,10 +17,6 @@ #ifdef CONFIG_CPUSETS extern struct static_key cpusets_enabled_key; -static inline bool cpusets_enabled(void) -{ - return static_key_false(&cpusets_enabled_key); -} static inline int nr_cpusets(void) { @@ -28,6 +24,12 @@ static inline int nr_cpusets(void) return static_key_count(&cpusets_enabled_key) + 1; } +/* Returns true if a cpuset exists that can set cpuset.mems */ +static inline bool cpusets_mems_enabled(void) +{ + return nr_cpusets() > 1; +} + static inline void cpuset_inc(void) { static_key_slow_inc(&cpusets_enabled_key); @@ -104,7 +106,7 @@ extern void cpuset_print_task_mems_allowed(struct task_struct *p); */ static inline unsigned int read_mems_allowed_begin(void) { - if (!cpusets_enabled()) + if (!cpusets_mems_enabled()) return 0; return read_seqcount_begin(¤t->mems_allowed_seq); @@ -118,7 +120,7 @@ static inline unsigned int read_mems_allowed_begin(void) */ static inline bool read_mems_allowed_retry(unsigned int seq) { - if (!cpusets_enabled()) + if (!cpusets_mems_enabled()) return false; return read_seqcount_retry(¤t->mems_allowed_seq, seq); @@ -139,7 +141,7 @@ static inline void set_mems_allowed(nodemask_t nodemask) #else /* !CONFIG_CPUSETS */ -static inline bool cpusets_enabled(void) { return false; } +static inline bool cpusets_mems_enabled(void) { return false; } static inline int cpuset_init(void) { return 0; } static inline void cpuset_init_smp(void) {} diff --git a/mm/page_alloc.c b/mm/page_alloc.c index 62ae28d8ae8d..2c1c3bf54d15 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -2470,7 +2470,7 @@ get_page_from_freelist(gfp_t gfp_mask, unsigned int order, int alloc_flags, if (IS_ENABLED(CONFIG_NUMA) && zlc_active && !zlc_zone_worth_trying(zonelist, z, allowednodes)) continue; - if (cpusets_enabled() && + if (cpusets_mems_enabled() && (alloc_flags & ALLOC_CPUSET) && !cpuset_zone_allowed(zone, gfp_mask)) continue; -- 2.4.6