[Intel-gfx] [PATCH 04/10] drm/i915/guc: Add a second client, to be used for preemption

[Chris Wilson]

Quoting Michał Winiarski (2017-10-05 10:13:43)
> From: Dave Gordon <david.s.gordon at intel.com>
> 
> This second client is created with priority KMD_HIGH, and marked
> as preemptive. This will allow us to request preemption using GuC actions.
> 
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Jeff Mcgee <jeff.mcgee at intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Oscar Mateo <oscar.mateo at intel.com>
> Signed-off-by: Dave Gordon <david.s.gordon at intel.com>
> Signed-off-by: Michał Winiarski <michal.winiarski at intel.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c        |  4 ++++
>  drivers/gpu/drm/i915/i915_guc_submission.c | 32 ++++++++++++++++++++++++++----
>  drivers/gpu/drm/i915/intel_uc.h            |  1 +
>  3 files changed, 33 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index b4a6ac60e7c6..1a963c13cab8 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -2488,6 +2488,10 @@ static int i915_guc_info(struct seq_file *m, void *data)
>  
>         seq_printf(m, "\nGuC execbuf client @ %p:\n", guc->execbuf_client);
>         i915_guc_client_info(m, dev_priv, guc->execbuf_client);
> +       if (guc->preempt_client) {

Hmm, when do we have guc->execbuf_client but not guc->preempt_client?
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris