summaryrefslogtreecommitdiff
path: root/kernel/latencytop.c
diff options
context:
space:
mode:
authorRobert Bragg <robert@sixbynine.org>2017-05-11 16:43:25 +0100
committerChris Wilson <chris@chris-wilson.co.uk>2017-05-13 10:59:07 +0100
commit26ebd9c734d5b807055c333b29504b05fe971185 (patch)
treebec5aacb0804270f94a47bd28125ca9173da0d94 /kernel/latencytop.c
parente81b3a555f27cae5381ab148df3fa543e1b93ea2 (diff)
downloadlwn-26ebd9c734d5b807055c333b29504b05fe971185.tar.gz
lwn-26ebd9c734d5b807055c333b29504b05fe971185.zip
drm/i915/perf: avoid poll, read, EAGAIN busy loops
If the function for checking whether there is OA buffer data available (during a poll or blocking read) has false positives then we want to avoid a situation where the subsequent read() returns EAGAIN (after a more accurate check) followed by a poll() immediately reporting the same false positive POLLIN event and effectively maintaining a busy loop until there really is data. This makes sure that we clear the .pollin event status whenever we return EAGAIN to userspace which will throttle subsequent POLLIN events and repeated attempts to read to the 5ms intervals of the hrtimer callback we have. Signed-off-by: Robert Bragg <robert@sixbynine.org> Reviewed-by: Matthew Auld <matthew.auld@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170511154345.962-3-lionel.g.landwerlin@intel.com Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'kernel/latencytop.c')
0 files changed, 0 insertions, 0 deletions