On Mon, Jun 27, 2016 at 12:38:37PM +0100, Tvrtko Ursulin wrote:
> Looks good to me.
> 
> Only thing I would possibly change is the thread name to not mislead
> anyone into thinking this is the main and only i915 irq activity.
> Looks to me it should just be called something like
> i915/signaller:%u or similar.

I wanted it fairly short for top (about 10 characters on a normal
terminal iirc). There is nothing preventing us from having multiple irq
threads (and preceding with irq seems to fit in with others).

If we need more, I would suggest irq/i915:%u-signal
-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