diff options
author | Chris Wilson <[email protected]> | 2017-03-15 21:07:26 +0000 |
---|---|---|
committer | Chris Wilson <[email protected]> | 2017-03-15 21:45:41 +0000 |
commit | db93991bf515916e181aa3ddb5a6cbc8b86bdc42 (patch) | |
tree | 4fa5df16ad82773695fbc65ffea8f14ae886c10f /scripts/gcc-plugins/cyc_complexity_plugin.c | |
parent | 908a6cbf84db77f33d8522f8ff9d9bf34dd7441f (diff) |
drm/i915: Only attempt to signal the request once from the interrupt handler
Check that request has not been signaled before acquiring a reference to
the request for signaling later in the interrupt handler.
The loading of the cacheline (for request->fence.flags) should be "free"
when followed by the locked increment of the request->fence.refcount
(which then sets the cacheline to exclusive mode), i.e. the cost of
test_bit prior to an atomic_inc should be negligible. This should
benefit us when we have a pile of bare breadcrumbs (interrupted execbuf)
where we may get interrupts faster than we can get rid of the
intel_wait, or if the device is too slow to run the bottom-half between
interrupts.
Signed-off-by: Chris Wilson <[email protected]>
Cc: Mika Kuoppala <[email protected]>
Cc: Tvrtko Ursulin <[email protected]>
Reviewed-by: Tvrtko Ursulin <[email protected]>
Link: http://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'scripts/gcc-plugins/cyc_complexity_plugin.c')
0 files changed, 0 insertions, 0 deletions