On Mon, Mar 06, 2017 at 09:29:15AM +0000, Chris Wilson wrote:
>  void intel_engine_disarm_breadcrumbs(struct intel_engine_cs *engine)
>  {
>       struct intel_breadcrumbs *b = &engine->breadcrumbs;
> -     unsigned long flags;
> +     struct intel_wait *wait, *n;
>  
>       if (!b->irq_armed)
>               return;
>  
> -     spin_lock_irqsave(&b->irq_lock, flags);
> -
>       /* We only disarm the irq when we are idle (all requests completed),
>        * so if there remains a sleeping waiter, it missed the request

*so if the bottom-half remains asleep, it missed the request

Reviewed-by: Mika Kuoppala <[email protected]>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to