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

Re: [Xen-devel] [PATCH-4.5 v2 02/10] xen/arm: support HW interrupts in gic_set_lr

On Tue, 18 Mar 2014, Ian Campbell wrote:
> On Fri, 2014-02-14 at 17:49 +0000, Julien Grall wrote:
> > On 02/14/2014 03:51 PM, Stefano Stabellini wrote:
> > > If the irq to be injected is an hardware irq (p->desc != NULL), set
> > > GICH_LR_HW.
> > > 
> > > Remove the code to EOI a physical interrupt on behalf of the guest
> > > because it has become unnecessary.
> > > 
> > > Also add a struct vcpu* parameter to gic_set_lr.
> > > 
> > > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > 
> > IRL you told me that this patch as dependency on another. It would be
> > nice to mention this dependency in the commit message for bisection.
> As in bisection is broken and needs manual intervention? Please don't do
> that, reorder it or resplit things to make the issue go away.
> The patch itself looks ok to me, but I won't ack until this is explained
> or resolved.

Patch #2 and #3 should really be applied together, I separated them out
just for the sake of readibility. The reason is that you can't receive
maintenance interrupts for hw interrupts, so in order for this to work
properly we need the following patch "do not request

I can send them as a single patch next time, but it would be harder to

Xen-devel mailing list



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