Re: [PATCH] powerpc/ptdump: Fix DEBUG_WX since generic ptdump conversion

[Christophe Leroy <christophe.leroy@csgroup.eu>]

Le 03/12/2021 à 13:41, Michael Ellerman a écrit :
> In note_prot_wx() we bail out without reporting anything if
> CONFIG_PPC_DEBUG_WX is disabled.
> 
> But CONFIG_PPC_DEBUG_WX was removed in the conversion to generic ptdump,
> we now need to use CONFIG_DEBUG_WX instead.
> 
> Fixes: e084728393a5 ("powerpc/ptdump: Convert powerpc to GENERIC_PTDUMP")
> Cc: stable@vger.kernel.org # v5.15+
> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
> ---

Oops ... my mistake.

Reviewed-by: Christophe Leroy <christophe.leroy@csgroup.eu>

>   arch/powerpc/mm/ptdump/ptdump.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/powerpc/mm/ptdump/ptdump.c b/arch/powerpc/mm/ptdump/ptdump.c
> index 031956d0ee84..be120e09aa3e 100644
> --- a/arch/powerpc/mm/ptdump/ptdump.c
> +++ b/arch/powerpc/mm/ptdump/ptdump.c
> @@ -183,7 +183,7 @@ static void note_prot_wx(struct pg_state *st, unsigned long addr)
>   {
>   	pte_t pte = __pte(st->current_flags);
>   
> -	if (!IS_ENABLED(CONFIG_PPC_DEBUG_WX) || !st->check_wx)
> +	if (!IS_ENABLED(CONFIG_DEBUG_WX) || !st->check_wx)
>   		return;
>   
>   	if (!pte_write(pte) || !pte_exec(pte))
>