On Fri, Dec 03, 2021 at 03:30:57PM -0800, Bruce Chang wrote: > Follow up on patch https://patchwork.freedesktop.org/patch/446832/ > > Different platforms will take a bit longer while GuC is enabled, so > increase the timeout and also add some margin in i915_gem_context > selftest. > > Signed-off-by: Bruce Chang <yu.bruce.ch...@intel.com> > Cc: Matthew Brost <matthew.br...@intel.com> > Cc: John Harrison <john.c.harri...@intel.com> > --- > drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c > b/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c > index b32f7fed2d9c..ae33e8c705da 100644 > --- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c > +++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c > @@ -88,7 +88,7 @@ static int live_nop_switch(void *arg) > rq = i915_request_get(this); > i915_request_add(this); > } > - if (i915_request_wait(rq, 0, HZ) < 0) { > + if (i915_request_wait(rq, 0, 10*HZ) < 0) {
s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com s/10*HZ/10 * HZ/ With that and an agreed upon audit of selftests / IGTs: Reviewed-by: Matthew Brost <matthew.br...@intel.com> > pr_err("Failed to populated %d contexts\n", nctx); > intel_gt_set_wedged(&i915->gt); > i915_request_put(rq); > -- > 2.21.3 >