46 lines
1.7 KiB
Diff
46 lines
1.7 KiB
Diff
From d3ca6725643c02176fc3d31b1441c1c1b4a62740 Mon Sep 17 00:00:00 2001
|
|
Message-Id: <d3ca6725643c02176fc3d31b1441c1c1b4a62740.1590093525.git.zanussi@kernel.org>
|
|
In-Reply-To: <4f310e18bbb62e33196484e72d1a7d0416189d63.1590093525.git.zanussi@kernel.org>
|
|
References: <4f310e18bbb62e33196484e72d1a7d0416189d63.1590093525.git.zanussi@kernel.org>
|
|
From: Matt Fleming <matt@codeblueprint.co.uk>
|
|
Date: Sun, 26 Jan 2020 21:19:45 +0000
|
|
Subject: [PATCH 319/327] mm/memcontrol: Move misplaced
|
|
local_unlock_irqrestore()
|
|
Origin: https://www.kernel.org/pub/linux/kernel/projects/rt/4.19/older/patches-4.19.124-rt53.tar.xz
|
|
|
|
[ Upstream commit 071a1d6a6e14d0dec240a8c67b425140d7f92f6a ]
|
|
|
|
The comment about local_lock_irqsave() mentions just the counters and
|
|
css_put_many()'s callback just invokes a worker so it is safe to move the
|
|
unlock function after memcg_check_events() so css_put_many() can be invoked
|
|
without the lock acquired.
|
|
|
|
Cc: Daniel Wagner <wagi@monom.org>
|
|
Signed-off-by: Matt Fleming <matt@codeblueprint.co.uk>
|
|
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
|
|
[bigeasy: rewrote the patch description]
|
|
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
|
|
---
|
|
mm/memcontrol.c | 2 +-
|
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
|
diff --git a/mm/memcontrol.c b/mm/memcontrol.c
|
|
index 4156a4e766fb..d059e2a99441 100644
|
|
--- a/mm/memcontrol.c
|
|
+++ b/mm/memcontrol.c
|
|
@@ -6538,10 +6538,10 @@ void mem_cgroup_swapout(struct page *page, swp_entry_t entry)
|
|
mem_cgroup_charge_statistics(memcg, page, PageTransHuge(page),
|
|
-nr_entries);
|
|
memcg_check_events(memcg, page);
|
|
+ local_unlock_irqrestore(event_lock, flags);
|
|
|
|
if (!mem_cgroup_is_root(memcg))
|
|
css_put_many(&memcg->css, nr_entries);
|
|
- local_unlock_irqrestore(event_lock, flags);
|
|
}
|
|
|
|
/**
|
|
--
|
|
2.17.1
|
|
|