From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v3 6/8] xen: arm: add some helpers for assessing p2m pte Date: Thu, 26 Jun 2014 16:07:51 +0100 Message-ID: <53AC3747.1000509@linaro.org> References: <1403777793.16595.21.camel@kazak.uk.xensource.com> <1403777837-16779-6-git-send-email-ian.campbell@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1403777837-16779-6-git-send-email-ian.campbell@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , xen-devel@lists.xen.org Cc: tim@xen.org, stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On 06/26/2014 11:17 AM, Ian Campbell wrote: > diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c > index 8a6d295..b51dc1b 100644 > --- a/xen/arch/arm/p2m.c > +++ b/xen/arch/arm/p2m.c > @@ -14,6 +14,22 @@ > #define P2M_FIRST_ORDER 1 > #define P2M_FIRST_ENTRIES (LPAE_ENTRIES< > +static bool_t p2m_valid(lpae_t pte) { NIT: The { has to be on another line. > + return pte.p2m.valid; > +} > +/* These two can only be used on L0..L2 ptes because L3 mappings set > + * the table bit and therefore these would return the opposite to what > + * you would expect. */ > +static bool_t p2m_table(lpae_t pte) > +{ > + return p2m_valid(pte) && pte.p2m.table; > +} > +#if 0 > +static bool_t p2m_mapping(lpae_t pte) { Same here. > + return p2m_valid(pte) && !pte.p2m.table; > +} > +#endif > + With these small changes: Acked-by: Julien Grall -- Julien Grall