linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mm: mlock: remove lru_add_drain_all()
@ 2017-10-18 23:17 Shakeel Butt
  2017-10-19  3:18 ` Balbir Singh
                   ` (3 more replies)
  0 siblings, 4 replies; 14+ messages in thread
From: Shakeel Butt @ 2017-10-18 23:17 UTC (permalink / raw)
  To: Andrew Morton, Kirill A. Shutemov, Vlastimil Babka, Michal Hocko,
	Joonsoo Kim, Minchan Kim, Yisheng Xie, Ingo Molnar, Greg Thelen,
	Hugh Dickins
  Cc: linux-mm, linux-kernel, Shakeel Butt

Recently we have observed high latency in mlock() in our generic
library and noticed that users have started using tmpfs files even
without swap and the latency was due to expensive remote LRU cache
draining.

Is lru_add_drain_all() required by mlock()? The answer is no and the
reason it is still in mlock() is to rapidly move mlocked pages to
unevictable LRU. Without lru_add_drain_all() the mlocked pages which
were on pagevec at mlock() time will be moved to evictable LRUs but
will eventually be moved back to unevictable LRU by reclaim. So, we
can safely remove lru_add_drain_all() from mlock(). Also there is no
need for local lru_add_drain() as it will be called deep inside
__mm_populate() (in follow_page_pte()).

Signed-off-by: Shakeel Butt <shakeelb@google.com>
---
 mm/mlock.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/mm/mlock.c b/mm/mlock.c
index dfc6f1912176..3ceb2935d1e0 100644
--- a/mm/mlock.c
+++ b/mm/mlock.c
@@ -669,8 +669,6 @@ static __must_check int do_mlock(unsigned long start, size_t len, vm_flags_t fla
 	if (!can_do_mlock())
 		return -EPERM;
 
-	lru_add_drain_all();	/* flush pagevec */
-
 	len = PAGE_ALIGN(len + (offset_in_page(start)));
 	start &= PAGE_MASK;
 
@@ -797,9 +795,6 @@ SYSCALL_DEFINE1(mlockall, int, flags)
 	if (!can_do_mlock())
 		return -EPERM;
 
-	if (flags & MCL_CURRENT)
-		lru_add_drain_all();	/* flush pagevec */
-
 	lock_limit = rlimit(RLIMIT_MEMLOCK);
 	lock_limit >>= PAGE_SHIFT;
 
-- 
2.15.0.rc1.287.g2b38de12cc-goog

--
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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply related	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2017-10-19 20:53 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-18 23:17 [PATCH] mm: mlock: remove lru_add_drain_all() Shakeel Butt
2017-10-19  3:18 ` Balbir Singh
2017-10-19 20:12   ` Shakeel Butt
2017-10-19  6:24 ` Anshuman Khandual
2017-10-19 19:19   ` Shakeel Butt
2017-10-19 10:18 ` Kirill A. Shutemov
2017-10-19 19:19   ` Shakeel Butt
2017-10-19 12:32 ` Michal Hocko
2017-10-19 19:19   ` Shakeel Butt
2017-10-19 19:35     ` Michal Hocko
2017-10-19 19:46       ` Shakeel Butt
2017-10-19 20:13         ` Michal Hocko
2017-10-19 20:14           ` Shakeel Butt
2017-10-19 20:53             ` Michal Hocko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).