On 23/08/16 20:57, Aneesh Kumar K.V wrote:
> Signed-off-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com>
> ---
> .../powerpc/include/asm/book3s/64/tlbflush-radix.h | 1 +
> arch/powerpc/mm/tlb-radix.c | 24
> ++++++++++++++++++++++
> 2 files changed, 25 insertions(+)
>
> diff --git a/arch/powerpc/include/asm/book3s/64/tlbflush-radix.h
> b/arch/powerpc/include/asm/book3s/64/tlbflush-radix.h
> index 65037762b120..a9e19cb2f7c5 100644
> --- a/arch/powerpc/include/asm/book3s/64/tlbflush-radix.h
> +++ b/arch/powerpc/include/asm/book3s/64/tlbflush-radix.h
> @@ -41,4 +41,5 @@ extern void radix__flush_tlb_page_psize(struct mm_struct
> *mm, unsigned long vmad
> extern void radix__flush_tlb_lpid_va(unsigned long lpid, unsigned long gpa,
> unsigned long page_size);
> extern void radix__flush_tlb_lpid(unsigned long lpid);
> +extern void radix__flush_tlb_all(void);
> #endif
> diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c
> index 48df05ef5231..0e49ec541ab5 100644
> --- a/arch/powerpc/mm/tlb-radix.c
> +++ b/arch/powerpc/mm/tlb-radix.c
> @@ -400,3 +400,27 @@ void radix__flush_pmd_tlb_range(struct vm_area_struct
> *vma,
> radix__flush_tlb_range_psize(vma->vm_mm, start, end, MMU_PAGE_2M);
> }
> EXPORT_SYMBOL(radix__flush_pmd_tlb_range);
> +
> +void radix__flush_tlb_all(void)
> +{
> + unsigned long rb,prs,r,rs;
> + unsigned long ric = RIC_FLUSH_ALL;
> +
> + rb = 0x3 << PPC_BITLSHIFT(53); /* IS = 3 */
> + prs = 0; /* partition scoped */
> + r = 1; /* raidx format */
^^ typo - radix
> + rs = 1 & ((1UL << 32) - 1); /* any LPID value to flush guest mappings */
> +
For RIC=2 and IS=3, I think RS is ignored.. I don't think we need to
set RS to anything other than 0, please double check
> + asm volatile("ptesync": : :"memory");
> + /*
> + * now flush guest entries by passing PRS = 1 and LPID != 0
> + */
> + asm volatile(PPC_TLBIE_5(%0, %4, %3, %2, %1)
> + : : "r"(rb), "i"(r), "i"(1), "i"(ric), "r"(rs) : "memory");
> + /*
> + * now flush host entires by passing PRS = 0 and LPID == 0
> + */
> + asm volatile(PPC_TLBIE_5(%0, %4, %3, %2, %1)
> + : : "r"(rb), "i"(r), "i"(prs), "i"(ric), "r"(0) :
> "memory");
> + asm volatile("eieio; tlbsync; ptesync": : :"memory");
> +}
>
Otherwise looks good
Balbir Singh.
