[Intel-gfx] [PATCH 10/20] drm/i915: Watchdog timeout: DRM kernel interface enablement

[Daniel Vetter]

On Fri, Jan 22, 2016 at 02:08:32PM +0000, Arun Siluvery wrote:
> On 22/01/2016 13:59, Chris Wilson wrote:
> >On Fri, Oct 23, 2015 at 02:32:32AM +0100, Tomas Elf wrote:
> >>Final enablement patch for GPU hang recovery using watchdog timeout.
> >>Added execbuf flag for watchdog timeout in DRM kernel interface.
> >
> >Usual argument: per-batch or per-context flag? What's the use case being
> >consider that would favour one approach or another. That usecase should
> >be explained in the changelog so that we can be reminded of it again
> >later.
> 
> It is per-batch, user expects certain workloads to be completed before
> watchdog times out hence it is per-batch.
> This is mainly used for media workloads, don't know the exact usecase but I
> guess encode/decode work of frame to complete within a short time etc. I
> will add these details to commit msg.

Usual reminder: We need an open-source user to get these new interfaces
in. Looking at that should make it pretty clear whether the watchdog
should be per-ctx or per-batch.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch