On Thu, 06 Jun 2013 11:35:47 +0530 "Aneesh Kumar K.V"
<aneesh.ku...@linux.vnet.ibm.com> wrote:
> Andrew Morton <a...@linux-foundation.org> writes:
>
> > On Thu, 06 Jun 2013 09:31:06 +1000 Benjamin Herrenschmidt
> > <b...@kernel.crashing.org> wrote:
> >
> >> On Wed, 2013-06-05 at 20:58 +0530, Aneesh Kumar K.V wrote:
> >> >
> >> > This is the second patchset needed to support THP on ppc64. Some of the
> >> > changes
> >> > included in this series are tricky in that it changes the powerpc linux
> >> > page table
> >> > walk subtly. We also overload few of the pte flags for ptes at PMD level
> >> > (huge
> >> > page PTEs).
> >> >
> >> > The related mm/ changes are already merged to Andrew's -mm tree.
> >>
> >> If I am to put that into powerpc-next, I need the dependent mm/ changes as
> >> well.
> >>
> >> Do you have them in the form of a separate git tree that is *exactly*
> >> (same SHA1s)
> >> what is expected to go upstream via Andrew ?
> >>
> >> Andrew, are they fully acked on your side and ready to go ?
> >
> > Not being on linuxppc-dev I'm at a bit of a loss here.
> >
> > I assume we're referring to
> >
> > mm-thp-add-pmd-args-to-pgtable-deposit-and-withdraw-apis.patch
> > mm-thp-withdraw-the-pgtable-after-pmdp-related-operations.patch
> > mm-thp-withdraw-the-pgtable-after-pmdp-related-operations-fix.patch
> > mm-thp-dont-use-hpage_shift-in-transparent-hugepage-code.patch
> > mm-thp-deposit-the-transpare-huge-pgtable-before-set_pmd.patch
> >
>
> There is one more:
>
> mm/THP: Use the right function when updating access flags
>
> mm-thp-use-the-right-function-when-updating-access-flags.patc
Hereunder. This actually precedes the above four(+fix) patches.
From: "Aneesh Kumar K.V" <aneesh.ku...@linux.vnet.ibm.com>
Subject: mm/thp: use the correct function when updating access flags
We should use pmdp_set_access_flags to update access flags. Archs like
powerpc use extra checks(_PAGE_BUSY) when updating a hugepage PTE. A
set_pmd_at doesn't do those checks. We should use set_pmd_at only when
updating a none hugepage PTE.
Signed-off-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com>
Cc: Andrea Arcangeli <aarca...@redhat.com>a
Signed-off-by: Andrew Morton <a...@linux-foundation.org>
---
mm/huge_memory.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff -puN
mm/huge_memory.c~mm-thp-use-the-right-function-when-updating-access-flags
mm/huge_memory.c
--- a/mm/huge_memory.c~mm-thp-use-the-right-function-when-updating-access-flags
+++ a/mm/huge_memory.c
@@ -1265,7 +1265,9 @@ struct page *follow_trans_huge_pmd(struc
* young bit, instead of the current set_pmd_at.
*/
_pmd = pmd_mkyoung(pmd_mkdirty(*pmd));
- set_pmd_at(mm, addr & HPAGE_PMD_MASK, pmd, _pmd);
+ if (pmdp_set_access_flags(vma, addr & HPAGE_PMD_MASK,
+ pmd, _pmd, 1))
+ update_mmu_cache_pmd(vma, addr, pmd);
}
if ((flags & FOLL_MLOCK) && (vma->vm_flags & VM_LOCKED)) {
if (page->mapping && trylock_page(page)) {
_
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
