From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v4 05/10] xen/arm: keep track of the GICH_LR used for the irq in struct pending_irq Date: Wed, 19 Mar 2014 13:52:06 +0000 Message-ID: <5329A106.6000806@linaro.org> References: <1395232325-19226-5-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1395232325-19226-5-git-send-email-stefano.stabellini@eu.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: Stefano Stabellini Cc: julien.grall@citrix.com, jtd@galois.com, xen-devel@lists.xensource.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org Hi Stefano, On 03/19/2014 12:32 PM, Stefano Stabellini wrote: > diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h > index bc20a15..ea89057 100644 > --- a/xen/include/asm-arm/domain.h > +++ b/xen/include/asm-arm/domain.h > @@ -59,6 +59,7 @@ struct pending_irq > #define GIC_IRQ_GUEST_VISIBLE 1 > #define GIC_IRQ_GUEST_ENABLED 2 > unsigned long status; > + uint8_t lr; You are using uint8_t here but nr_lrs is defined as unsigned. Can you also change nr_lrs type to uint8_t? Regards, -- Julien Grall