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

Re: [Xen-devel] [PATCH for v4.9] livepatch: Wrong usage of spinlock on debug console.



>>> On 09.06.17 at 16:16, <konrad.wilk@xxxxxxxxxx> wrote:
> If we have a large amount of livepatches and want to print them
> on the console using 'xl debug-keys x' we eventually hit
> the preemption check:
> 
>   if ( i && !(i % 64) )
>   {
>       spin_unlock(&payload_lock);
>       process_pending_softirqs();
>       if ( spin_trylock(&payload_lock) )
>               return
> 
> <facepalm> The effect is that we have just effectively
> taken the lock and returned without unlocking!
> 
> CC: Ross Lagerwall <ross.lagerwall@xxxxxxxxxx>
> CC: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
> CC: Jan Beulich <jbeulich@xxxxxxxx>
> CC: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> CC: Julien Grall <julien.grall@xxxxxxx>
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>

Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>


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

 


Rackspace

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