Re: [PATCH i-g-t] i915/gem_mmap_gtt: Impose a timeout for "hang"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Mika Kuoppala (2019-02-12 10:56:13)
> Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes:
> 
> > Only wait a maximum of 2 more seconds for the hang children to complete,
> > to catch deadlocks in a timely fashion.
> >
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx>
> > ---
> >  tests/i915/gem_mmap_gtt.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/tests/i915/gem_mmap_gtt.c b/tests/i915/gem_mmap_gtt.c
> > index f9d811ad6..f6fbbe193 100644
> > --- a/tests/i915/gem_mmap_gtt.c
> > +++ b/tests/i915/gem_mmap_gtt.c
> > @@ -450,7 +450,7 @@ test_hang(int fd)
> >       igt_info("%lu resets\n", count);
> >  
> >       control->done = true;
> > -     igt_waitchildren();
> > +     igt_waitchildren_timeout(2, NULL);
> 
> Seems that no-one is using the reason argument. You could have
> been the first ever!

I resisted because I suggested Daniel could remove it as he's
overhauling igt_waitchildren_timeout().
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux