Re: [Intel-gfx] [PATCH 2/4] [v2] drm/i915: Unbind the fb

2013-05-31 Thread Ben Widawsky
On Fri, May 31, 2013 at 08:39:32PM +0100, Chris Wilson wrote: > On Fri, May 31, 2013 at 11:45:29AM -0700, Ben Widawsky wrote: > > This is the only place I could find where we don't already unbind the > > object before freeing it. This change will allow us to have a nice > > assertion on free about

Re: [Intel-gfx] [PATCH 2/4] [v2] drm/i915: Unbind the fb

2013-05-31 Thread Chris Wilson
On Fri, May 31, 2013 at 11:45:29AM -0700, Ben Widawsky wrote: > This is the only place I could find where we don't already unbind the > object before freeing it. This change will allow us to have a nice > assertion on free about whether or not an object still has pinned pages. > > v2: Just use the

[Intel-gfx] [PATCH 2/4] [v2] drm/i915: Unbind the fb

2013-05-31 Thread Ben Widawsky
This is the only place I could find where we don't already unbind the object before freeing it. This change will allow us to have a nice assertion on free about whether or not an object still has pinned pages. v2: Just use the i915 object instead of casting Signed-off-by: Ben Widawsky --- drive