powerpc/mm: don't do tlbie for updatepp request with NO HPTE fault
authorAneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Thu, 4 Dec 2014 05:30:14 +0000 (11:00 +0530)
committerMichael Ellerman <mpe@ellerman.id.au>
Fri, 5 Dec 2014 05:26:15 +0000 (16:26 +1100)
upatepp can get called for a nohpte fault when we find from the linux
page table that the translation was hashed before. In that case
we are sure that there is no existing translation, hence we could
avoid doing tlbie.

We could possibly race with a parallel fault filling the TLB. But
that should be ok because updatepp is only ever relaxing permissions.
We also look at linux pte permission bits when filling hash pte
permission bits. We also hold the linux pte busy bits while
inserting/updating a hashpte entry, hence a paralle update of
linux pte is not possible. On the other hand mprotect involves
ptep_modify_prot_start which cause a hpte invalidate and not updatepp.

Performance number:
We use randbox_access_bench written by Anton.

Kernel with THP disabled and smaller hash page table size.

    86.60%  random_access_b  [kernel.kallsyms]                [k] .native_hpte_updatepp
     2.10%  random_access_b  random_access_bench              [.] doit
     1.99%  random_access_b  [kernel.kallsyms]                [k] .do_raw_spin_lock
     1.85%  random_access_b  [kernel.kallsyms]                [k] .native_hpte_insert
     1.26%  random_access_b  [kernel.kallsyms]                [k] .native_flush_hash_range
     1.18%  random_access_b  [kernel.kallsyms]                [k] .__delay
     0.69%  random_access_b  [kernel.kallsyms]                [k] .native_hpte_remove
     0.37%  random_access_b  [kernel.kallsyms]                [k] .clear_user_page
     0.34%  random_access_b  [kernel.kallsyms]                [k] .__hash_page_64K
     0.32%  random_access_b  [kernel.kallsyms]                [k] fast_exception_return
     0.30%  random_access_b  [kernel.kallsyms]                [k] .hash_page_mm

With Fix:

    27.54%  random_access_b  random_access_bench              [.] doit
    22.90%  random_access_b  [kernel.kallsyms]                [k] .native_hpte_insert
     5.76%  random_access_b  [kernel.kallsyms]                [k] .native_hpte_remove
     5.20%  random_access_b  [kernel.kallsyms]                [k] fast_exception_return
     5.12%  random_access_b  [kernel.kallsyms]                [k] .__hash_page_64K
     4.80%  random_access_b  [kernel.kallsyms]                [k] .hash_page_mm
     3.31%  random_access_b  [kernel.kallsyms]                [k] data_access_common
     1.84%  random_access_b  [kernel.kallsyms]                [k] .trace_hardirqs_on_caller

Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
16 files changed:
arch/powerpc/include/asm/machdep.h
arch/powerpc/include/asm/mmu-hash64.h
arch/powerpc/include/asm/tlbflush.h
arch/powerpc/kernel/exceptions-64s.S
arch/powerpc/mm/hash_low_64.S
arch/powerpc/mm/hash_native_64.c
arch/powerpc/mm/hash_utils_64.c
arch/powerpc/mm/hugepage-hash64.c
arch/powerpc/mm/hugetlbpage-hash64.c
arch/powerpc/mm/pgtable_64.c
arch/powerpc/platforms/cell/beat_htab.c
arch/powerpc/platforms/cell/spu_base.c
arch/powerpc/platforms/cell/spufs/fault.c
arch/powerpc/platforms/ps3/htab.c
arch/powerpc/platforms/pseries/lpar.c
drivers/misc/cxl/fault.c

index e5c0919acca470536007c985e0f6cbd6f7c7a9bd..c8175a3fe5601d0e37192c0ae9dbe95f7bb618f9 100644 (file)
@@ -42,7 +42,7 @@ struct machdep_calls {
                                         unsigned long newpp, 
                                         unsigned long vpn,
                                         int bpsize, int apsize,
-                                        int ssize, int local);
+                                        int ssize, unsigned long flags);
        void            (*hpte_updateboltedpp)(unsigned long newpp, 
                                               unsigned long ea,
                                               int psize, int ssize);
index aeebc94b2bced66d4d5762077629362ece591103..4f13c3ed7acf923fe4d9d0b4fac4568853b3ef35 100644 (file)
@@ -316,27 +316,33 @@ static inline unsigned long hpt_hash(unsigned long vpn,
        return hash & 0x7fffffffffUL;
 }
 
+#define HPTE_LOCAL_UPDATE      0x1
+#define HPTE_NOHPTE_UPDATE     0x2
+
 extern int __hash_page_4K(unsigned long ea, unsigned long access,
                          unsigned long vsid, pte_t *ptep, unsigned long trap,
-                         unsigned int local, int ssize, int subpage_prot);
+                         unsigned long flags, int ssize, int subpage_prot);
 extern int __hash_page_64K(unsigned long ea, unsigned long access,
                           unsigned long vsid, pte_t *ptep, unsigned long trap,
-                          unsigned int local, int ssize);
+                          unsigned long flags, int ssize);
 struct mm_struct;
 unsigned int hash_page_do_lazy_icache(unsigned int pp, pte_t pte, int trap);
-extern int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, unsigned long trap);
-extern int hash_page(unsigned long ea, unsigned long access, unsigned long trap);
+extern int hash_page_mm(struct mm_struct *mm, unsigned long ea,
+                       unsigned long access, unsigned long trap,
+                       unsigned long flags);
+extern int hash_page(unsigned long ea, unsigned long access, unsigned long trap,
+                    unsigned long dsisr);
 int __hash_page_huge(unsigned long ea, unsigned long access, unsigned long vsid,
-                    pte_t *ptep, unsigned long trap, int local, int ssize,
-                    unsigned int shift, unsigned int mmu_psize);
+                    pte_t *ptep, unsigned long trap, unsigned long flags,
+                    int ssize, unsigned int shift, unsigned int mmu_psize);
 #ifdef CONFIG_TRANSPARENT_HUGEPAGE
 extern int __hash_page_thp(unsigned long ea, unsigned long access,
                           unsigned long vsid, pmd_t *pmdp, unsigned long trap,
-                          int local, int ssize, unsigned int psize);
+                          unsigned long flags, int ssize, unsigned int psize);
 #else
 static inline int __hash_page_thp(unsigned long ea, unsigned long access,
                                  unsigned long vsid, pmd_t *pmdp,
-                                 unsigned long trap, int local,
+                                 unsigned long trap, unsigned long flags,
                                  int ssize, unsigned int psize)
 {
        BUG();
index 4d3ecd8d8929c1bd26e799b4f94413a0cbafd396..23d351ca0303492f699f5e630668339a6b28effb 100644 (file)
@@ -125,11 +125,11 @@ static inline void arch_leave_lazy_mmu_mode(void)
 
 
 extern void flush_hash_page(unsigned long vpn, real_pte_t pte, int psize,
-                           int ssize, int local);
+                           int ssize, unsigned long flags);
 extern void flush_hash_range(unsigned long number, int local);
 extern void flush_hash_hugepage(unsigned long vsid, unsigned long addr,
                                pmd_t *pmdp, unsigned int psize, int ssize,
-                               int local);
+                               unsigned long flags);
 
 static inline void local_flush_tlb_mm(struct mm_struct *mm)
 {
index ad62f4d6ce31c4830cc543da15ad344767325b7f..6213f494f40bea87736994d53c981c1f563853f8 100644 (file)
@@ -1565,9 +1565,11 @@ do_hash_page:
         * r3 contains the faulting address
         * r4 contains the required access permissions
         * r5 contains the trap number
+        * r6 contains dsisr
         *
         * at return r3 = 0 for success, 1 for page fault, negative for error
         */
+       ld      r6,_DSISR(r1)
        bl      hash_page               /* build HPTE if possible */
        cmpdi   r3,0                    /* see if hash_page succeeded */
 
index 5094f32b706e8b6ef16435c2a69d0ea3b2ff87bf..463174a4a64755e2d8d06d2f2b7e174d7297878d 100644 (file)
@@ -46,7 +46,8 @@
 
 /*
  * _hash_page_4K(unsigned long ea, unsigned long access, unsigned long vsid,
- *              pte_t *ptep, unsigned long trap, int local, int ssize)
+ *              pte_t *ptep, unsigned long trap, unsigned long flags,
+ *              int ssize)
  *
  * Adds a 4K page to the hash table in a segment of 4K pages only
  */
@@ -298,7 +299,7 @@ htab_modify_pte:
        li      r6,MMU_PAGE_4K          /* base page size */
        li      r7,MMU_PAGE_4K          /* actual page size */
        ld      r8,STK_PARAM(R9)(r1)    /* segment size */
-       ld      r9,STK_PARAM(R8)(r1)    /* get "local" param */
+       ld      r9,STK_PARAM(R8)(r1)    /* get "flags" param */
 .globl htab_call_hpte_updatepp
 htab_call_hpte_updatepp:
        bl      .                       /* Patched by htab_finish_init() */
@@ -338,8 +339,8 @@ htab_pte_insert_failure:
  *****************************************************************************/
 
 /* _hash_page_4K(unsigned long ea, unsigned long access, unsigned long vsid,
- *              pte_t *ptep, unsigned long trap, int local, int ssize,
- *              int subpg_prot)
+ *              pte_t *ptep, unsigned long trap, unsigned local flags,
+ *              int ssize, int subpg_prot)
  */
 
 /*
@@ -594,7 +595,7 @@ htab_inval_old_hpte:
        li      r5,0                    /* PTE.hidx */
        li      r6,MMU_PAGE_64K         /* psize */
        ld      r7,STK_PARAM(R9)(r1)    /* ssize */
-       ld      r8,STK_PARAM(R8)(r1)    /* local */
+       ld      r8,STK_PARAM(R8)(r1)    /* flags */
        bl      flush_hash_page
        /* Clear out _PAGE_HPTE_SUB bits in the new linux PTE */
        lis     r0,_PAGE_HPTE_SUB@h
@@ -666,7 +667,7 @@ htab_modify_pte:
        li      r6,MMU_PAGE_4K          /* base page size */
        li      r7,MMU_PAGE_4K          /* actual page size */
        ld      r8,STK_PARAM(R9)(r1)    /* segment size */
-       ld      r9,STK_PARAM(R8)(r1)    /* get "local" param */
+       ld      r9,STK_PARAM(R8)(r1)    /* get "flags" param */
 .globl htab_call_hpte_updatepp
 htab_call_hpte_updatepp:
        bl      .                       /* patched by htab_finish_init() */
@@ -962,7 +963,7 @@ ht64_modify_pte:
        li      r6,MMU_PAGE_64K         /* base page size */
        li      r7,MMU_PAGE_64K         /* actual page size */
        ld      r8,STK_PARAM(R9)(r1)    /* segment size */
-       ld      r9,STK_PARAM(R8)(r1)    /* get "local" param */
+       ld      r9,STK_PARAM(R8)(r1)    /* get "flags" param */
 .globl ht64_call_hpte_updatepp
 ht64_call_hpte_updatepp:
        bl      .                       /* patched by htab_finish_init() */
index 13700911b522b1cfdc8518c4566fd3b7ab859f4e..9c4880ddecd63f06e0f4b2b45aded0b57b47e940 100644 (file)
@@ -283,11 +283,11 @@ static long native_hpte_remove(unsigned long hpte_group)
 
 static long native_hpte_updatepp(unsigned long slot, unsigned long newpp,
                                 unsigned long vpn, int bpsize,
-                                int apsize, int ssize, int local)
+                                int apsize, int ssize, unsigned long flags)
 {
        struct hash_pte *hptep = htab_address + slot;
        unsigned long hpte_v, want_v;
-       int ret = 0;
+       int ret = 0, local = 0;
 
        want_v = hpte_encode_avpn(vpn, bpsize, ssize);
 
@@ -322,8 +322,15 @@ static long native_hpte_updatepp(unsigned long slot, unsigned long newpp,
                }
                native_unlock_hpte(hptep);
        }
-       /* Ensure it is out of the tlb too. */
-       tlbie(vpn, bpsize, apsize, ssize, local);
+
+       if (flags & HPTE_LOCAL_UPDATE)
+               local = 1;
+       /*
+        * Ensure it is out of the tlb too if it is not a nohpte fault
+        */
+       if (!(flags & HPTE_NOHPTE_UPDATE))
+               tlbie(vpn, bpsize, apsize, ssize, local);
+
        return ret;
 }
 
index 68211d398fdb18021bd06e5f3c0965ec0bd3cf4f..e56a307bc676b392ec9d41a9d41fbd16c9925ca3 100644 (file)
@@ -989,7 +989,9 @@ static void check_paca_psize(unsigned long ea, struct mm_struct *mm,
  * -1 - critical hash insertion error
  * -2 - access not permitted by subpage protection mechanism
  */
-int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, unsigned long trap)
+int hash_page_mm(struct mm_struct *mm, unsigned long ea,
+                unsigned long access, unsigned long trap,
+                unsigned long flags)
 {
        enum ctx_state prev_state = exception_enter();
        pgd_t *pgdir;
@@ -997,7 +999,7 @@ int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, u
        pte_t *ptep;
        unsigned hugeshift;
        const struct cpumask *tmp;
-       int rc, user_region = 0, local = 0;
+       int rc, user_region = 0;
        int psize, ssize;
 
        DBG_LOW("hash_page(ea=%016lx, access=%lx, trap=%lx\n",
@@ -1049,7 +1051,7 @@ int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, u
        /* Check CPU locality */
        tmp = cpumask_of(smp_processor_id());
        if (user_region && cpumask_equal(mm_cpumask(mm), tmp))
-               local = 1;
+               flags |= HPTE_LOCAL_UPDATE;
 
 #ifndef CONFIG_PPC_64K_PAGES
        /* If we use 4K pages and our psize is not 4K, then we might
@@ -1086,11 +1088,11 @@ int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, u
        if (hugeshift) {
                if (pmd_trans_huge(*(pmd_t *)ptep))
                        rc = __hash_page_thp(ea, access, vsid, (pmd_t *)ptep,
-                                            trap, local, ssize, psize);
+                                            trap, flags, ssize, psize);
 #ifdef CONFIG_HUGETLB_PAGE
                else
                        rc = __hash_page_huge(ea, access, vsid, ptep, trap,
-                                             local, ssize, hugeshift, psize);
+                                             flags, ssize, hugeshift, psize);
 #else
                else {
                        /*
@@ -1149,7 +1151,8 @@ int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, u
 
 #ifdef CONFIG_PPC_HAS_HASH_64K
        if (psize == MMU_PAGE_64K)
-               rc = __hash_page_64K(ea, access, vsid, ptep, trap, local, ssize);
+               rc = __hash_page_64K(ea, access, vsid, ptep, trap,
+                                    flags, ssize);
        else
 #endif /* CONFIG_PPC_HAS_HASH_64K */
        {
@@ -1158,7 +1161,7 @@ int hash_page_mm(struct mm_struct *mm, unsigned long ea, unsigned long access, u
                        rc = -2;
                else
                        rc = __hash_page_4K(ea, access, vsid, ptep, trap,
-                                           local, ssize, spp);
+                                           flags, ssize, spp);
        }
 
        /* Dump some info in case of hash insertion failure, they should
@@ -1181,14 +1184,19 @@ bail:
 }
 EXPORT_SYMBOL_GPL(hash_page_mm);
 
-int hash_page(unsigned long ea, unsigned long access, unsigned long trap)
+int hash_page(unsigned long ea, unsigned long access, unsigned long trap,
+             unsigned long dsisr)
 {
+       unsigned long flags = 0;
        struct mm_struct *mm = current->mm;
 
        if (REGION_ID(ea) == VMALLOC_REGION_ID)
                mm = &init_mm;
 
-       return hash_page_mm(mm, ea, access, trap);
+       if (dsisr & DSISR_NOHPTE)
+               flags |= HPTE_NOHPTE_UPDATE;
+
+       return hash_page_mm(mm, ea, access, trap, flags);
 }
 EXPORT_SYMBOL_GPL(hash_page);
 
@@ -1200,7 +1208,7 @@ void hash_preload(struct mm_struct *mm, unsigned long ea,
        pgd_t *pgdir;
        pte_t *ptep;
        unsigned long flags;
-       int rc, ssize, local = 0;
+       int rc, ssize, update_flags = 0;
 
        BUG_ON(REGION_ID(ea) != USER_REGION_ID);
 
@@ -1251,16 +1259,17 @@ void hash_preload(struct mm_struct *mm, unsigned long ea,
 
        /* Is that local to this CPU ? */
        if (cpumask_equal(mm_cpumask(mm), cpumask_of(smp_processor_id())))
-               local = 1;
+               update_flags |= HPTE_LOCAL_UPDATE;
 
        /* Hash it in */
 #ifdef CONFIG_PPC_HAS_HASH_64K
        if (mm->context.user_psize == MMU_PAGE_64K)
-               rc = __hash_page_64K(ea, access, vsid, ptep, trap, local, ssize);
+               rc = __hash_page_64K(ea, access, vsid, ptep, trap,
+                                    update_flags, ssize);
        else
 #endif /* CONFIG_PPC_HAS_HASH_64K */
-               rc = __hash_page_4K(ea, access, vsid, ptep, trap, local, ssize,
-                                   subpage_protection(mm, ea));
+               rc = __hash_page_4K(ea, access, vsid, ptep, trap, update_flags,
+                                   ssize, subpage_protection(mm, ea));
 
        /* Dump some info in case of hash insertion failure, they should
         * never happen so it is really useful to know if/when they do
@@ -1278,9 +1287,10 @@ out_exit:
  *          do not forget to update the assembly call site !
  */
 void flush_hash_page(unsigned long vpn, real_pte_t pte, int psize, int ssize,
-                    int local)
+                    unsigned long flags)
 {
        unsigned long hash, index, shift, hidx, slot;
+       int local = flags & HPTE_LOCAL_UPDATE;
 
        DBG_LOW("flush_hash_page(vpn=%016lx)\n", vpn);
        pte_iterate_hashed_subpages(pte, psize, vpn, index, shift) {
@@ -1317,12 +1327,14 @@ void flush_hash_page(unsigned long vpn, real_pte_t pte, int psize, int ssize,
 
 #ifdef CONFIG_TRANSPARENT_HUGEPAGE
 void flush_hash_hugepage(unsigned long vsid, unsigned long addr,
-                        pmd_t *pmdp, unsigned int psize, int ssize, int local)
+                        pmd_t *pmdp, unsigned int psize, int ssize,
+                        unsigned long flags)
 {
        int i, max_hpte_count, valid;
        unsigned long s_addr;
        unsigned char *hpte_slot_array;
        unsigned long hidx, shift, vpn, hash, slot;
+       int local = flags & HPTE_LOCAL_UPDATE;
 
        s_addr = addr & HPAGE_PMD_MASK;
        hpte_slot_array = get_hpte_slot_array(pmdp);
index 3a648cd363ae84c5f39352a64e25019fefd0d01c..86686514ae1359742219d8e873d1910c74935e4b 100644 (file)
@@ -19,8 +19,8 @@
 #include <asm/machdep.h>
 
 int __hash_page_thp(unsigned long ea, unsigned long access, unsigned long vsid,
-                   pmd_t *pmdp, unsigned long trap, int local, int ssize,
-                   unsigned int psize)
+                   pmd_t *pmdp, unsigned long trap, unsigned long flags,
+                   int ssize, unsigned int psize)
 {
        unsigned int index, valid;
        unsigned char *hpte_slot_array;
@@ -95,7 +95,7 @@ int __hash_page_thp(unsigned long ea, unsigned long access, unsigned long vsid,
                 */
                if ((old_pmd & _PAGE_HASHPTE) && !(old_pmd & _PAGE_COMBO))
                        flush_hash_hugepage(vsid, ea, pmdp, MMU_PAGE_64K,
-                                           ssize, local);
+                                           ssize, flags);
        }
 
        valid = hpte_valid(hpte_slot_array, index);
@@ -108,7 +108,7 @@ int __hash_page_thp(unsigned long ea, unsigned long access, unsigned long vsid,
                slot += hidx & _PTEIDX_GROUP_IX;
 
                ret = ppc_md.hpte_updatepp(slot, rflags, vpn,
-                                          psize, lpsize, ssize, local);
+                                          psize, lpsize, ssize, flags);
                /*
                 * We failed to update, try to insert a new entry.
                 */
index a5bcf930119691ad01c4116faaece823292669aa..d94b1af53a93620bb7888902dd3f0cd40b46e384 100644 (file)
@@ -19,8 +19,8 @@ extern long hpte_insert_repeating(unsigned long hash, unsigned long vpn,
                                  unsigned long vflags, int psize, int ssize);
 
 int __hash_page_huge(unsigned long ea, unsigned long access, unsigned long vsid,
-                    pte_t *ptep, unsigned long trap, int local, int ssize,
-                    unsigned int shift, unsigned int mmu_psize)
+                    pte_t *ptep, unsigned long trap, unsigned long flags,
+                    int ssize, unsigned int shift, unsigned int mmu_psize)
 {
        unsigned long vpn;
        unsigned long old_pte, new_pte;
@@ -81,7 +81,7 @@ int __hash_page_huge(unsigned long ea, unsigned long access, unsigned long vsid,
                slot += (old_pte & _PAGE_F_GIX) >> 12;
 
                if (ppc_md.hpte_updatepp(slot, rflags, vpn, mmu_psize,
-                                        mmu_psize, ssize, local) == -1)
+                                        mmu_psize, ssize, flags) == -1)
                        old_pte &= ~_PAGE_HPTEFLAGS;
        }
 
index eea9fa1f8ae79b37fccc6b0b8a2f620cacd7347c..4fe5f64cc1793e693693dc0ca92420ce99dc6209 100644 (file)
@@ -739,9 +739,10 @@ void pmdp_invalidate(struct vm_area_struct *vma, unsigned long address,
 void hpte_do_hugepage_flush(struct mm_struct *mm, unsigned long addr,
                            pmd_t *pmdp, unsigned long old_pmd)
 {
-       int ssize, local = 0;
+       int ssize;
        unsigned int psize;
        unsigned long vsid;
+       unsigned long flags = 0;
        const struct cpumask *tmp;
 
        /* get the base page size,vsid and segment size */
@@ -765,9 +766,9 @@ void hpte_do_hugepage_flush(struct mm_struct *mm, unsigned long addr,
 
        tmp = cpumask_of(smp_processor_id());
        if (cpumask_equal(mm_cpumask(mm), tmp))
-               local = 1;
+               flags |= HPTE_LOCAL_UPDATE;
 
-       return flush_hash_hugepage(vsid, addr, pmdp, psize, ssize, local);
+       return flush_hash_hugepage(vsid, addr, pmdp, psize, ssize, flags);
 }
 
 static pmd_t pmd_set_protbits(pmd_t pmd, pgprot_t pgprot)
index d4d245c0d78799a6f48e65aae52560b75daac6e7..bee9232fe619240179e3100676bd7ddeab750911 100644 (file)
@@ -186,7 +186,7 @@ static long beat_lpar_hpte_updatepp(unsigned long slot,
                                    unsigned long newpp,
                                    unsigned long vpn,
                                    int psize, int apsize,
-                                   int ssize, int local)
+                                   int ssize, unsigned long flags)
 {
        unsigned long lpar_rc;
        u64 dummy0, dummy1;
@@ -369,7 +369,7 @@ static long beat_lpar_hpte_updatepp_v3(unsigned long slot,
                                       unsigned long newpp,
                                       unsigned long vpn,
                                       int psize, int apsize,
-                                      int ssize, int local)
+                                      int ssize, unsigned long flags)
 {
        unsigned long lpar_rc;
        unsigned long want_v;
index ffcbd242e6693de80a632be5100f1c8d7b1fd072..f7af74f836934575cbe491d70a3409ca350a6e25 100644 (file)
@@ -181,7 +181,8 @@ static int __spu_trap_data_seg(struct spu *spu, unsigned long ea)
        return 0;
 }
 
-extern int hash_page(unsigned long ea, unsigned long access, unsigned long trap); //XXX
+extern int hash_page(unsigned long ea, unsigned long access,
+                    unsigned long trap, unsigned long dsisr); //XXX
 static int __spu_trap_data_map(struct spu *spu, unsigned long ea, u64 dsisr)
 {
        int ret;
@@ -196,7 +197,7 @@ static int __spu_trap_data_map(struct spu *spu, unsigned long ea, u64 dsisr)
            (REGION_ID(ea) != USER_REGION_ID)) {
 
                spin_unlock(&spu->register_lock);
-               ret = hash_page(ea, _PAGE_PRESENT, 0x300);
+               ret = hash_page(ea, _PAGE_PRESENT, 0x300, dsisr);
                spin_lock(&spu->register_lock);
 
                if (!ret) {
index e45894a081185bc92142ad4b75a2eec9138e5fe0..d98f845ac7771b8f198c44b24824ec89faf07b5c 100644 (file)
@@ -144,7 +144,7 @@ int spufs_handle_class1(struct spu_context *ctx)
        access = (_PAGE_PRESENT | _PAGE_USER);
        access |= (dsisr & MFC_DSISR_ACCESS_PUT) ? _PAGE_RW : 0UL;
        local_irq_save(flags);
-       ret = hash_page(ea, access, 0x300);
+       ret = hash_page(ea, access, 0x300, dsisr);
        local_irq_restore(flags);
 
        /* hashing failed, so try the actual fault handler */
index 3e270e3412ae69fed007e68cd9fd008b20eb24b9..2f95d33cf34a491d5970dd14ff5b88f757b31181 100644 (file)
@@ -110,7 +110,7 @@ static long ps3_hpte_remove(unsigned long hpte_group)
 
 static long ps3_hpte_updatepp(unsigned long slot, unsigned long newpp,
                              unsigned long vpn, int psize, int apsize,
-                             int ssize, int local)
+                             int ssize, unsigned long inv_flags)
 {
        int result;
        u64 hpte_v, want_v, hpte_rs;
index 832f221840f2fe61fbe59b1e8c222739274eff8c..469751d9200469c5220be3da3ec573207d9c1100 100644 (file)
@@ -284,7 +284,7 @@ static long pSeries_lpar_hpte_updatepp(unsigned long slot,
                                       unsigned long newpp,
                                       unsigned long vpn,
                                       int psize, int apsize,
-                                      int ssize, int local)
+                                      int ssize, unsigned long inv_flags)
 {
        unsigned long lpar_rc;
        unsigned long flags = (newpp & 7) | H_AVPN;
index c99e896604ee35909e0a2d55bbd95fd31caddbea..f8684bca2d79ddb5be7af1621fd6f72bf779c8af 100644 (file)
@@ -133,7 +133,7 @@ static void cxl_handle_page_fault(struct cxl_context *ctx,
 {
        unsigned flt = 0;
        int result;
-       unsigned long access, flags;
+       unsigned long access, flags, inv_flags = 0;
 
        if ((result = copro_handle_mm_fault(mm, dar, dsisr, &flt))) {
                pr_devel("copro_handle_mm_fault failed: %#x\n", result);
@@ -149,8 +149,12 @@ static void cxl_handle_page_fault(struct cxl_context *ctx,
                access |= _PAGE_RW;
        if ((!ctx->kernel) || ~(dar & (1ULL << 63)))
                access |= _PAGE_USER;
+
+       if (dsisr & DSISR_NOHPTE)
+               inv_flags |= HPTE_NOHPTE_UPDATE;
+
        local_irq_save(flags);
-       hash_page_mm(mm, dar, access, 0x300);
+       hash_page_mm(mm, dar, access, 0x300, inv_flags);
        local_irq_restore(flags);
 
        pr_devel("Page fault successfully handled for pe: %i!\n", ctx->pe);