Re: [Intel-gfx] [PATCH 1/2] drm/i915: Kill intel_crtc->cursor_bo

2015-12-14 Thread Ville Syrjälä
On Mon, Dec 14, 2015 at 11:32:23AM +, Chris Wilson wrote: > On Mon, Dec 14, 2015 at 01:16:47PM +0200, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > The vma may have been rebound between the last time the cursor was > > enabled and now, so skipping the cursor gtt offset

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Kill intel_crtc->cursor_bo

2015-12-14 Thread Chris Wilson
On Mon, Dec 14, 2015 at 01:16:47PM +0200, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > 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 whe

[Intel-gfx] [PATCH 1/2] drm/i915: Kill intel_crtc->cursor_bo

2015-12-14 Thread ville . syrjala
From: Ville Syrjälä 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