On to, 2016-10-27 at 17:10 +0100, Chris Wilson wrote: > @@ -473,14 +477,14 @@ static int intel_breadcrumbs_signaler(void *arg) > * we just completed - so double check we are still > * the oldest before picking the next one. > */ > - spin_lock(&b->lock); > + spin_lock_irqsave(&b->lock, flags); With the convention discussed in IRC, this should be spin_lock_irq(). With that changed; Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
