From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Stabellini Subject: Re: [PATCH v8 05/10] xen/arm: physical irq follow virtual irq Date: Wed, 23 Jul 2014 16:00:52 +0100 Message-ID: References: <1405016003-19131-5-git-send-email-stefano.stabellini@eu.citrix.com> <53BFE18D.80000@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <53BFE18D.80000@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall Cc: julien.grall@citrix.com, xen-devel@lists.xensource.com, Ian.Campbell@citrix.com, Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On Fri, 11 Jul 2014, Julien Grall wrote: > On 07/10/2014 07:13 PM, Stefano Stabellini wrote: > > void vgic_disable_irqs(struct vcpu *v, uint32_t r, int n) > > { > > const unsigned long mask = r; > > @@ -277,6 +297,7 @@ void vgic_enable_irqs(struct vcpu *v, uint32_t r, int n) > > } > > if ( p->desc != NULL ) > > { > > + irq_set_affinity(p->desc, cpumask_of(v_target->processor)); > > spin_lock_irqsave(&p->desc->lock, flags); > > p->desc->handler->enable(p->desc); > > spin_unlock_irqrestore(&p->desc->lock, flags); > > diff --git a/xen/include/asm-arm/gic.h b/xen/include/asm-arm/gic.h > > index 6410280..8d93ccd 100644 > > --- a/xen/include/asm-arm/gic.h > > +++ b/xen/include/asm-arm/gic.h > > @@ -321,6 +321,7 @@ struct gic_hw_operations { > > void register_gic_ops(const struct gic_hw_operations *ops); > > > > struct vcpu *vgic_get_target_vcpu(struct vcpu *v, unsigned int irq); > > +void arch_move_irqs(struct vcpu *v); > > I would stay consistent with x86 by moving this declaration in irq.h. OK