diff options
| author | Ville Syrjälä <[email protected]> | 2015-12-14 17:35:02 +0200 |
|---|---|---|
| committer | Jani Nikula <[email protected]> | 2015-12-22 12:59:06 +0200 |
| commit | 57a2af6bbc7a4f1b145cc216c34476402836f0b8 (patch) | |
| tree | 892aefe2ba755b0a838b0f32518dffe12712b1c3 /drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | |
| parent | ef8dd37af85a8f37ca3a29074647511e52c56181 (diff) | |
drm/i915: Kill intel_crtc->cursor_bo
The vma may have been rebound between the last time the cursor was
enabled and now, so skipping the cursor gtt offset deduction is not
safe unless we would also reset cursor_bo to NULL when disabling the
cursor. Just thow cursor_bo to the bin instead since it's lost all
other uses thanks to universal plane support.
Chris pointed out that cursor updates are currently too slow
via universal planes that micro optimizations like these wouldn't
even help.
v2: Add a note about futility of micro optimizations (Chris)
Cc: [email protected]
References: http://lists.freedesktop.org/archives/intel-gfx/2015-December/082976.html
Cc: Chris Wilson <[email protected]>
Cc: Takashi Iwai <[email protected]>
Cc: Jani Nikula <[email protected]>
Signed-off-by: Ville Syrjälä <[email protected]>
Link: http://patchwork.freedesktop.org/patch/msgid/[email protected]
Reviewed-by: Chris Wilson <[email protected]>
(cherry picked from commit 1264859d648c4bdc9f0a098efbff90cbf462a075)
Signed-off-by: Jani Nikula <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c')
0 files changed, 0 insertions, 0 deletions