diff options
author | Gerald Schaefer <gerald.schaefer@de.ibm.com> | 2015-04-14 18:42:30 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-04-14 19:48:56 -0400 |
commit | 97534127012f0e396eddea4691f4c9b170aed74b (patch) | |
tree | 123663a18882ffe8ab4a4699013fd5ddee329f2e /mm/hugetlb.c | |
parent | 31f7dc796998d2967e999a0f9229d8a50c7b348d (diff) |
mm/hugetlb: use pmd_page() in follow_huge_pmd()
Commit 61f77eda9bbf ("mm/hugetlb: reduce arch dependent code around
follow_huge_*") broke follow_huge_pmd() on s390, where pmd and pte
layout differ and using pte_page() on a huge pmd will return wrong
results. Using pmd_page() instead fixes this.
All architectures that were touched by that commit have pmd_page()
defined, so this should not break anything on other architectures.
Fixes: 61f77eda "mm/hugetlb: reduce arch dependent code around follow_huge_*"
Signed-off-by: Gerald Schaefer <gerald.schaefer@de.ibm.com>
Acked-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Cc: Hugh Dickins <hughd@google.com>
Cc: Michal Hocko <mhocko@suse.cz>, Andrea Arcangeli <aarcange@redhat.com>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Acked-by: David Rientjes <rientjes@google.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/hugetlb.c')
-rw-r--r-- | mm/hugetlb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mm/hugetlb.c b/mm/hugetlb.c index c41b2a0ee273..caad3c5a926f 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c | |||
@@ -3735,8 +3735,7 @@ retry: | |||
3735 | if (!pmd_huge(*pmd)) | 3735 | if (!pmd_huge(*pmd)) |
3736 | goto out; | 3736 | goto out; |
3737 | if (pmd_present(*pmd)) { | 3737 | if (pmd_present(*pmd)) { |
3738 | page = pte_page(*(pte_t *)pmd) + | 3738 | page = pmd_page(*pmd) + ((address & ~PMD_MASK) >> PAGE_SHIFT); |
3739 | ((address & ~PMD_MASK) >> PAGE_SHIFT); | ||
3740 | if (flags & FOLL_GET) | 3739 | if (flags & FOLL_GET) |
3741 | get_page(page); | 3740 | get_page(page); |
3742 | } else { | 3741 | } else { |