[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v8 07/10] xen/arm: remove workaround to inject evtchn_irq on irq enable



On Wed, 23 Jul 2014, Ian Campbell wrote:
> On Wed, 2014-07-23 at 17:12 +0100, Stefano Stabellini wrote:
> > On Wed, 23 Jul 2014, Ian Campbell wrote:
> > > On Wed, 2014-07-23 at 17:09 +0100, Stefano Stabellini wrote:
> > > > On Wed, 23 Jul 2014, Stefano Stabellini wrote:
> > > > > On Thu, 17 Jul 2014, Ian Campbell wrote:
> > > > > > On Thu, 2014-07-10 at 19:13 +0100, Stefano Stabellini wrote:
> > > > > > > evtchn_upcall_pending is already set by common code at vcpu 
> > > > > > > creation,
> > > > > > > therefore on ARM we also need to call vgic_vcpu_inject_irq for it.
> > > > > > > Currently we do that from vgic_enable_irqs as a workaround.
> > > > > > 
> > > > > > Perhaps we should gate these on evtchn_upcall_pending then? That 
> > > > > > would
> > > > > > make it pretty obvious in most places what it was for.
> > > > > > 
> > > > > > Other than that suggestion and Julien's request for a comment this 
> > > > > > looks
> > > > > > good to me.
> > > > > 
> > > > > Makes sense, I'll make the changes
> > > > 
> > > > I take it back: checking evtchn_upcall_pending wouldn't work because it
> > > > hasn't been set yet. I think it's best not to introduce a dependency on
> > > > the order of the calls.
> > > 
> > > Then shouldn't whatever is setting evtchn_upcall_pending be doing the
> > > inject?
> > 
> > That is common code, hence the reason for this patch...
> 
> Some sort of arch callback at that point doesn't seem unreasonable.

All right, that might be better. I'll do that.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.