Benjamin Herrenschmidt <b...@kernel.crashing.org> writes:

> The PWC flush only needs a single set call, just like the
> full (RIC=2) flush.
>
> Signed-off-by: Benjamin Herrenschmidt <b...@kernel.crashing.org>
> ---
>  arch/powerpc/mm/tlb-radix.c | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c
> index 02e7140..5403419 100644
> --- a/arch/powerpc/mm/tlb-radix.c
> +++ b/arch/powerpc/mm/tlb-radix.c
> @@ -52,12 +52,15 @@ static inline void _tlbiel_pid(unsigned long pid, 
> unsigned long ric)
>        */
>       __tlbiel_pid(pid, 0, ric);
>
> -     if (ric == RIC_FLUSH_ALL)
> -             /* For the remaining sets, just flush the TLB */
> -             ric = RIC_FLUSH_TLB;
> +     /* For PWC, only one flush is needed */
> +     if (ric == RIC_FLUSH_PWC) {
> +             asm volatile("ptesync": : :"memory");
> +             return;
> +     }
>
> +     /* For the remaining sets, just flush the TLB */
>       for (set = 1; set < POWER9_TLB_SETS_RADIX ; set++)
> -             __tlbiel_pid(pid, set, ric);
> +             __tlbiel_pid(pid, set, RIC_FLUSH_TLB);
>
>       asm volatile("ptesync": : :"memory");
>       asm volatile(PPC_INVALIDATE_ERAT "; isync" : : :"memory");


With the current code, we use tlbiel_pwc() for doing a pwc flush.
and that does what is done this patch. May be we can update this patch
such that we drop tlbiel_pwc and switch all those instance to
tlbiel_pid(pid, RIC_FLUSH_PWC) ?

You already do this in a later patch.

-aneesh

Reply via email to