Commit 89dc6a96 authored by Yanfei Xu's avatar Yanfei Xu Committed by Linus Torvalds

mm/khugepaged.c: replace barrier() with READ_ONCE() for a selective variable

READ_ONCE() is more selective and lightweight.  It is more appropriate
that using a READ_ONCE() for the certain variable to prevent the
compiler from reordering.

Link: https://lkml.kernel.org/r/20210323092730.247583-1-yanfei.xu@windriver.comSigned-off-by: default avatarYanfei Xu <yanfei.xu@windriver.com>
Acked-by: default avatarKirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Mike Kravetz <mike.kravetz@oracle.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent a44f89dc
...@@ -2202,11 +2202,9 @@ static void khugepaged_do_scan(void) ...@@ -2202,11 +2202,9 @@ static void khugepaged_do_scan(void)
{ {
struct page *hpage = NULL; struct page *hpage = NULL;
unsigned int progress = 0, pass_through_head = 0; unsigned int progress = 0, pass_through_head = 0;
unsigned int pages = khugepaged_pages_to_scan; unsigned int pages = READ_ONCE(khugepaged_pages_to_scan);
bool wait = true; bool wait = true;
barrier(); /* write khugepaged_pages_to_scan to local stack */
lru_add_drain_all(); lru_add_drain_all();
while (progress < pages) { while (progress < pages) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment