aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2017-02-27 20:58:47 +0000
committerChris Wilson <chris@chris-wilson.co.uk>2017-02-27 21:57:19 +0000
commit8d769ea7bc16c34c9dc5143be021e943014c4cd1 (patch)
tree84b0f782b37911dcaad138f0078eecb703dfabde /net
parentdrm/i915: Distinguish between timeout and error in sideband transactions (diff)
downloadlinux-dev-8d769ea7bc16c34c9dc5143be021e943014c4cd1.tar.xz
linux-dev-8d769ea7bc16c34c9dc5143be021e943014c4cd1.zip
drm/i915: Report both waiters and success from intel_engine_wakeup()
The two users of the return value from intel_engine_wakeup() are expecting different results. In the breadcrumbs hangcheck, we are using it to determine whether wake_up_process() detected the waiter was currently running (and if so we presume that it hasn't yet missed the interrupt). However, in the fake_irq path, we are using the return value as a check as to whether there are any waiters, and so we may incorrectly stop the fake-irq if that waiter was currently running. To handle the two different needs, return both bits of information! We uninline it from the irq path in preparation for the next patch which makes the irq hotpath special and relegates intel_engine_wakeup() to the slow fixup paths. v2: s/ret/result/ Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170227205850.2828-1-chris@chris-wilson.co.uk
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions