On Wed, Jul 30, 2014 at 07:15:05AM +0100, Chris Wilson wrote: > On Tue, Jul 29, 2014 at 01:14:29PM -0700, Ben Widawsky wrote: > > So don't bother checking it again. > > This was introduced: > > commit b361237bcc7cea1d99f770490120d8bc2aed7777 > > Author: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Date: Fri Aug 24 09:35:08 2012 +0100 > > > > drm/i915: Juggle code order to ease flow of the next patch > > > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx> > > Ah, it is not so much that the timeout may be unnormalized, but that it > *may* be negative. > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre >From looking at the code, I thought it didn't matter (ie. I thought it dtrt regardless of negative). However, I just wrote a little test, and you are correct. It is needed. Sorry about that. -- Ben Widawsky, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
