Re: [PATCH v8 05/10] xen/arm: physical irq follow virtual irq

[Julien Grall <julien.grall@linaro.org>]

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.

-- 
Julien Grall