On 06/09/2017 10:16 AM, Konrad Rzeszutek Wilk 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.lagerw...@citrix.com> > CC: Boris Ostrovsky <boris.ostrov...@oracle.com> > CC: Jan Beulich <jbeul...@suse.com> > CC: Andrew Cooper <andrew.coop...@citrix.com> > CC: Julien Grall <julien.gr...@arm.com> > Signed-off-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Reviewed-and-tested-by: Boris Ostrovsky <boris.ostrov...@oracle.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel