Commit a85dfc30 authored by Yang Shi's avatar Yang Shi Committed by Linus Torvalds

mm: mempolicy: fix the wrong return value and potential pages leak of mbind

Commit d8835445 ("mm: mempolicy: make the behavior consistent when
MPOL_MF_MOVE* and MPOL_MF_STRICT were specified") fixed the return value
of mbind() for a couple of corner cases.  But, it altered the errno for
some other cases, for example, mbind() should return -EFAULT when part
or all of the memory range specified by nodemask and maxnode points
outside your accessible address space, or there was an unmapped hole in
the specified memory range specified by addr and len.

Fix this by preserving the errno returned by queue_pages_range().  And,
the pagelist may be not empty even though queue_pages_range() returns
error, put the pages back to LRU since mbind_range() is not called to
really apply the policy so those pages should not be migrated, this is
also the old behavior before the problematic commit.

Link: http://lkml.kernel.org/r/1572454731-3925-1-git-send-email-yang.shi@linux.alibaba.com
Fixes: d8835445 ("mm: mempolicy: make the behavior consistent when MPOL_MF_MOVE* and MPOL_MF_STRICT were specified")
Signed-off-by: default avatarYang Shi <yang.shi@linux.alibaba.com>
Reported-by: default avatarLi Xinhai <lixinhai.lxh@gmail.com>
Reviewed-by: default avatarLi Xinhai <lixinhai.lxh@gmail.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: Michal Hocko <mhocko@suse.com>
Cc: Mel Gorman <mgorman@techsingularity.net>
Cc: <stable@vger.kernel.org>	[4.19 and 5.2+]
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 875fef49
...@@ -672,7 +672,9 @@ static const struct mm_walk_ops queue_pages_walk_ops = { ...@@ -672,7 +672,9 @@ static const struct mm_walk_ops queue_pages_walk_ops = {
* 1 - there is unmovable page, but MPOL_MF_MOVE* & MPOL_MF_STRICT were * 1 - there is unmovable page, but MPOL_MF_MOVE* & MPOL_MF_STRICT were
* specified. * specified.
* 0 - queue pages successfully or no misplaced page. * 0 - queue pages successfully or no misplaced page.
* -EIO - there is misplaced page and only MPOL_MF_STRICT was specified. * errno - i.e. misplaced pages with MPOL_MF_STRICT specified (-EIO) or
* memory range specified by nodemask and maxnode points outside
* your accessible address space (-EFAULT)
*/ */
static int static int
queue_pages_range(struct mm_struct *mm, unsigned long start, unsigned long end, queue_pages_range(struct mm_struct *mm, unsigned long start, unsigned long end,
...@@ -1286,7 +1288,7 @@ static long do_mbind(unsigned long start, unsigned long len, ...@@ -1286,7 +1288,7 @@ static long do_mbind(unsigned long start, unsigned long len,
flags | MPOL_MF_INVERT, &pagelist); flags | MPOL_MF_INVERT, &pagelist);
if (ret < 0) { if (ret < 0) {
err = -EIO; err = ret;
goto up_out; goto up_out;
} }
...@@ -1305,10 +1307,12 @@ static long do_mbind(unsigned long start, unsigned long len, ...@@ -1305,10 +1307,12 @@ static long do_mbind(unsigned long start, unsigned long len,
if ((ret > 0) || (nr_failed && (flags & MPOL_MF_STRICT))) if ((ret > 0) || (nr_failed && (flags & MPOL_MF_STRICT)))
err = -EIO; err = -EIO;
} else } else {
up_out:
if (!list_empty(&pagelist))
putback_movable_pages(&pagelist); putback_movable_pages(&pagelist);
}
up_out:
up_write(&mm->mmap_sem); up_write(&mm->mmap_sem);
mpol_out: mpol_out:
mpol_put(new); mpol_put(new);
......
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