Re: [Intel-gfx] [PATCH 3/5] drm/i915: Track pinned vma in intel_plane_state

2016-11-15 Thread Daniel Vetter
On Tue, Nov 15, 2016 at 08:58:15AM +, Chris Wilson wrote: > @@ -12340,7 +12325,8 @@ static int intel_crtc_page_flip(struct drm_crtc *crtc, > cleanup_request: > i915_add_request_no_flush(request); > cleanup_unpin: > - intel_unpin_fb_obj(fb, crtc->primary->state->rotation); > + to

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Track pinned vma in intel_plane_state

2016-11-15 Thread Daniel Vetter
On Tue, Nov 15, 2016 at 10:02:23AM +, Chris Wilson wrote: > On Tue, Nov 15, 2016 at 10:52:00AM +0100, Daniel Vetter wrote: > > On Tue, Nov 15, 2016 at 08:58:15AM +, Chris Wilson wrote: > > > With atomic plane states we are able to track an allocation right from > > > preparation, during use

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Track pinned vma in intel_plane_state

2016-11-15 Thread Chris Wilson
On Tue, Nov 15, 2016 at 10:52:00AM +0100, Daniel Vetter wrote: > On Tue, Nov 15, 2016 at 08:58:15AM +, Chris Wilson wrote: > > With atomic plane states we are able to track an allocation right from > > preparation, during use and through to the final free after being > > swapped out for a new p

Re: [Intel-gfx] [PATCH 3/5] drm/i915: Track pinned vma in intel_plane_state

2016-11-15 Thread Daniel Vetter
On Tue, Nov 15, 2016 at 08:58:15AM +, Chris Wilson wrote: > With atomic plane states we are able to track an allocation right from > preparation, during use and through to the final free after being > swapped out for a new plane. We can couple the VMA we pin for the > framebuffer (and its rotat

[Intel-gfx] [PATCH 3/5] drm/i915: Track pinned vma in intel_plane_state

2016-11-15 Thread Chris Wilson
With atomic plane states we are able to track an allocation right from preparation, during use and through to the final free after being swapped out for a new plane. We can couple the VMA we pin for the framebuffer (and its rotation) to this lifetime and avoid all the clumsy lookups in between. Si

[Intel-gfx] [PATCH 3/5] drm/i915: Track pinned vma in intel_plane_state

2016-10-29 Thread Chris Wilson
With atomic plane states we are able to track an allocation right from preparation, during use and through to the final free after being swapped out for a new plane. We can couple the VMA we pin for the framebuffer (and its rotation) to this lifetime and avoid all the clumsy lookups in between. Si