From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from pegase1.c-s.fr (pegase1.c-s.fr [93.17.236.30]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3zJCbk3NgjzF0mf for ; Sat, 13 Jan 2018 06:33:53 +1100 (AEDT) Subject: Re: [PATCH] powerpc/mm: Simplify _PAGE_RO handling in page table dump From: christophe leroy To: Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Scott Wood Cc: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org References: <20170509141626.F30E767954@pc13941vm.idsi0.si.c-s.fr> Message-ID: <8401f7ff-3bff-443e-3b87-42fa67a01cbd@c-s.fr> Date: Fri, 12 Jan 2018 20:33:46 +0100 MIME-Version: 1.0 In-Reply-To: <20170509141626.F30E767954@pc13941vm.idsi0.si.c-s.fr> Content-Type: text/plain; charset=utf-8; format=flowed List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Le 09/05/2017 à 16:16, Christophe Leroy a écrit : > Commit fd893fe56a130 ("powerpc/mm: Fix missing page attributes in > page table dump") added support of _PAGE_RO attribute. > > This patch makes it more simple Superseeded by https://patchwork.ozlabs.org/patch/859896/ Christophe > > Signed-off-by: Christophe Leroy > --- > arch/powerpc/mm/dump_linuxpagetables.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/arch/powerpc/mm/dump_linuxpagetables.c b/arch/powerpc/mm/dump_linuxpagetables.c > index d659345a98d6..eeef51107cff 100644 > --- a/arch/powerpc/mm/dump_linuxpagetables.c > +++ b/arch/powerpc/mm/dump_linuxpagetables.c > @@ -121,13 +121,8 @@ static const struct flag_info flag_array[] = { > .set = "user", > .clear = " ", > }, { > -#if _PAGE_RO == 0 > - .mask = _PAGE_RW, > + .mask = _PAGE_RW | _PAGE_RO, > .val = _PAGE_RW, > -#else > - .mask = _PAGE_RO, > - .val = 0, > -#endif > .set = "rw", > .clear = "ro", > }, { > --- L'absence de virus dans ce courrier électronique a été vérifiée par le logiciel antivirus Avast. https://www.avast.com/antivirus