On Fri, Dec 03, 2021 at 03:53:56PM -0800, Matthew Brost wrote: > 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.chang@xxxxxxxxx> > > Cc: Matthew Brost <matthew.brost@xxxxxxxxx> > > Cc: John Harrison <John.C.Harrison@xxxxxxxxx> > > --- > > 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.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx > > s/10*HZ/10 * HZ/ > > With that and an agreed upon audit of selftests / IGTs: > Reviewed-by: Matthew Brost <matthew.brost@xxxxxxxxx> > ^^^ Copy paste malfunction.... > > pr_err("Failed to populated %d contexts\n", nctx); > > intel_gt_set_wedged(&i915->gt); > > i915_request_put(rq); > > -- > > 2.21.3 > >
