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

Re: [Xen-devel] [PATCH] kexec: don't disable interrupts when acquiring load/unload lock



>>> On 06.11.13 at 14:20, David Vrabel <david.vrabel@xxxxxxxxxx> wrote:
> On 06/11/13 12:22, Jan Beulich wrote:
>> This doesn't appear to have served any purpose other than causing
>> map_pages_to_xen() to be (incorrectly) invoked with interrupts
>> disabled. In particular, serialization against actual kexec-ing is done
>> without this lock being involved. Clarify the scope of the lock at once
>> by making it local to do_kexec_op_internal().
>> 
>> Once at it, also drop a pointless initializer.
> 
> I've already dropped the irqsave from this lock in my kexec series and
> can fold the other part in as well (which would now be in
> kexec_swap_images()).

That would generally be fine, but I'd like to get this fixed no matter
whether your patches make 4.4.

Jan


_______________________________________________
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®.