lkml.org 
[lkml]   [2019]   [Nov]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[RFC v1 03/19] mm, oom_kill: use for_each_node in constrained_alloc()
Date
In constrained_alloc(), we want to traverse node instead of zone, so
use for_each_node instead of for_each_zone.

Signed-off-by: Pengfei Li <fly@kernel.page>
---
mm/oom_kill.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/mm/oom_kill.c b/mm/oom_kill.c
index f44c79db0cd6..db509d5e4db3 100644
--- a/mm/oom_kill.c
+++ b/mm/oom_kill.c
@@ -254,7 +254,6 @@ static enum oom_constraint constrained_alloc(struct oom_control *oc)
enum zone_type high_zoneidx = gfp_zone(oc->gfp_mask);
bool cpuset_limited = false;
struct nlist_traverser t;
- struct zone *zone;
int nid;

if (is_memcg_oom(oc)) {
@@ -292,10 +291,13 @@ static enum oom_constraint constrained_alloc(struct oom_control *oc)
}

/* Check this allocation failure is caused by cpuset's wall function */
- for_each_zone_nlist_nodemask(zone, &t, oc->nodelist,
- high_zoneidx, oc->nodemask)
- if (!cpuset_zone_allowed(zone, oc->gfp_mask))
+ for_each_node_nlist_nodemask(nid, &t, oc->nodelist,
+ high_zoneidx, oc->nodemask) {
+ if (!cpuset_node_allowed(nid, oc->gfp_mask)) {
cpuset_limited = true;
+ break;
+ }
+ }

if (cpuset_limited) {
oc->totalpages = total_swap_pages;
--
2.23.0
\
 
 \ /
  Last update: 2019-11-21 16:20    [W:0.260 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site