[PATCH] drm/i915: Ensure that the wait ioctl return timeout is valid

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

 



On Thu, 29 Nov 2012 13:42:17 +0100
Daniel Vetter <daniel at ffwll.ch> wrote:

> On Wed, Nov 14, 2012 at 07:11:24PM +0000, Chris Wilson wrote:
> > Due to a discrepancy between the sleep time and the amount of time
> > we spend waiting inside the wait_event_timeout, it is possible that
> > we end up computing that negative time remains in the wait_ioctl.
> > This is obviously a bogus result to return to userspace, and
> > triggers a WARN, so we need to fix up the value before propagating
> > it back.
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Ben Widawsky <ben at bwidawsk.net>
> 
> Ben, can you pls take a look at this and smash an r-b on it if you're
> ok?
> 
> Thanks, Daniel

I want the real bug fixed! I do suspect it's a test bug, but I'd
prefer not to slap an r-b on until I can pass the original
problematic test. This just fixes an annoying WARN, which isn't super
important to me because it's not a regression (since it existed since
day 1), and no userspace really cares yet other than IGT.

I did review the patch previously, and it looks fine. So, the r-b is
pending.


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux