powerpc/book3s64/pkeys: Inherit correctly on fork.
authorAneesh Kumar K.V <aneesh.kumar@linux.ibm.com>
Fri, 27 Nov 2020 04:44:13 +0000 (10:14 +0530)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 3 Dec 2020 14:01:25 +0000 (01:01 +1100)
Child thread.kuap value is inherited from the parent in copy_thread_tls. We still
need to make sure when the child returns from a fork in the kernel we start with the kernel
default AMR value.

Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com>
Reviewed-by: Sandipan Das <sandipan@linux.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20201127044424.40686-12-aneesh.kumar@linux.ibm.com
arch/powerpc/kernel/process.c

index 3f0b6ad..2ec907a 100644 (file)
@@ -1760,6 +1760,16 @@ int copy_thread(unsigned long clone_flags, unsigned long usp,
                childregs->ppr = DEFAULT_PPR;
 
        p->thread.tidr = 0;
+#endif
+       /*
+        * Run with the current AMR value of the kernel
+        */
+#ifdef CONFIG_PPC_PKEY
+       if (mmu_has_feature(MMU_FTR_BOOK3S_KUAP))
+               kregs->amr = AMR_KUAP_BLOCKED;
+
+       if (mmu_has_feature(MMU_FTR_BOOK3S_KUEP))
+               kregs->iamr = AMR_KUEP_BLOCKED;
 #endif
        kregs->nip = ppc_function_entry(f);
        return 0;