Re: Gen8+ engine-reset

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

Hi,

We have had engine reset support in i915 for some time now. So lets call 
this work engine watchdog or something, just not engine reset.

By calling it engine reset we have confused sites like Phoronix and some 
our users.

Regards,

Tvrtko

On 05/01/2019 02:39, Carlos Santa wrote:
> This is a rebased on the original patch series from Michel Thierry
> that can be found here:
> 
> https://patchwork.freedesktop.org/series/21868
> 
> Note that this series is only limited to the GPU Watchdog timeout
> for execlists as it leaves out support
> for GuC based submission for a later time.
> 
> The series was also successfully tested from userspace through the
> the Intel i965 media driver that is readily found on some
> Linux based OS including Ubuntu OS and as well as Chromium OS. The
> changes on the i965 media userspace driver are currently under review at
> 
> https://github.com/intel/intel-vaapi-driver/pull/429/files
> 
> The testbed used on this series included a SKL-based NUC with
> 2 BSD rings as well as a KBL-based Chromebook with a 1 BSD ring.
> 
> Carlos Santa (1):
>    drm/i915: Only process VCS2 only when supported
> 
> Michel Thierry (7):
>    drm/i915: Add engine reset count in get-reset-stats ioctl
>    drm/i915: Watchdog timeout: IRQ handler for gen8+
>    drm/i915: Watchdog timeout: Ringbuffer command emission for gen8+
>    drm/i915: Watchdog timeout: DRM kernel interface to set the timeout
>    drm/i915: Watchdog timeout: Include threshold value in error state
>    drm/i915/watchdog: move emit_stop_watchdog until the very end of the
>      ring commands
>    drm/i915: Watchdog timeout: Blindly trust watchdog timeout for reset?
> 
>   drivers/gpu/drm/i915/i915_drv.h         |  56 +++++++
>   drivers/gpu/drm/i915/i915_gem_context.c | 103 +++++++++++-
>   drivers/gpu/drm/i915/i915_gem_context.h |   4 +
>   drivers/gpu/drm/i915/i915_gpu_error.c   |  12 +-
>   drivers/gpu/drm/i915/i915_gpu_error.h   |   5 +
>   drivers/gpu/drm/i915/i915_irq.c         |  17 +-
>   drivers/gpu/drm/i915/i915_reg.h         |   6 +
>   drivers/gpu/drm/i915/intel_hangcheck.c  |  20 ++-
>   drivers/gpu/drm/i915/intel_lrc.c        | 208 +++++++++++++++++++++++-
>   drivers/gpu/drm/i915/intel_ringbuffer.h |  10 ++
>   include/uapi/drm/i915_drm.h             |   7 +-
>   11 files changed, 428 insertions(+), 20 deletions(-)
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx