On 06/09/2017 03:16 PM, 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-by: Ross Lagerwall <ross.lagerw...@citrix.com>

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

Reply via email to