diff options
| author | Chris Wilson <[email protected]> | 2018-08-17 08:24:05 +0000 |
|---|---|---|
| committer | Chris Wilson <[email protected]> | 2018-08-31 07:58:57 +0000 |
| commit | 4a477651033e48851386d12e773584c99a878670 (patch) | |
| tree | 5be4bbb51885637eee56d7d73587a9b1b3ae2ff7 /drivers/gpu/drm/i915/intel_dp_mst.c | |
| parent | drm/i915/ringbuffer: Delay after invalidating gen6+ xcs (diff) | |
| download | kernel-4a477651033e48851386d12e773584c99a878670.tar.gz kernel-4a477651033e48851386d12e773584c99a878670.zip | |
drm/i915: Keep physical cursors pinned while in use
The optimisation inherent in commit 6a2c4232ece1 ("drm/i915: Make the
physical object coherent with GTT") relies on that once we allocated a
cursor we would have coherent, zero overhead access to the scanout plane
holding the cursor. That is we could then do the very frequent cursor
updates X enjoys with no indirection or kernel involvement. However,
that all hinges on the GGTT mmap of the cursor being pinned and not
require refaulting on each access -- handling such a page fault likely
requires the busy GGTT to be rearranged causing a stall. A very simple
fix is then to handle the physical cursor exactly like other cursors and
keep its vma pinned while active.
References: https://bugs.freedesktop.org/show_bug.cgi?id=107600
References: 6a2c4232ece1 ("drm/i915: Make the physical object coherent with GTT")
Signed-off-by: Chris Wilson <[email protected]>
Cc: Ville Syrjälä <[email protected]>
Reviewed-by: Ville Syrjälä <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'drivers/gpu/drm/i915/intel_dp_mst.c')
0 files changed, 0 insertions, 0 deletions