Re: [PATCH 0/3] irqsave/restore improvements

[Keir Fraser <keir.xen@gmail.com>]

On 21/10/2013 14:41, "Andrew Cooper" <andrew.cooper3@citrix.com> wrote:

> This series contains two independent but related fixes to do with
> irqsave/restore semantics.
> 
> The first patch prevents local_irq_restore() from corrupting system flags.
> 
> The second was constructed using the compile errors from the third patch,
> separated for clarity and reordered to prevent bisection problems.
> 
> The third patch uses a BUILD_BUG_ON() to ensure that the flags parameter to
> spin_lock_irqsave is wide enough to not trucate the result from
> local_irq_save().
> 
> I have not compile tested on arm, so request that one of the maintainers
> explicitly acks the final patch.  Any compile failures will almost certainly
> be fixed with an int->unsigned long conversion on the affected variable(s).
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> CC: Keir Fraser <keir@xen.org>
> CC: Jan Beulich <JBeulich@suse.com>
> CC: Ian Campbell <ian.campbell@citrix.com>
> CC: Stefano Stabellini <stefano.stabellini@citrix.com>
> CC: Tim Deegan <tim@xen.org>
> CC: George Dunlap <george.dunlap@eu.citrix.com>

Acked-by: Keir Fraser <keir@xen.org>

> --
> 1.7.10.4
>