Re: [Intel-gfx] [PATCH 11/11] drm/i915: Move active to vma

2013-07-10 Thread Daniel Vetter
On Wed, Jul 10, 2013 at 09:39:30AM -0700, Ben Widawsky wrote: > On Tue, Jul 09, 2013 at 09:45:09AM +0200, Daniel Vetter wrote: > > On Mon, Jul 08, 2013 at 11:08:42PM -0700, Ben Widawsky wrote: > > > Probably need to squash whole thing, or just the inactive part, tbd... > > > > > > Signed-off-by: B

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Move active to vma

2013-07-10 Thread Ben Widawsky
On Tue, Jul 09, 2013 at 09:45:09AM +0200, Daniel Vetter wrote: > On Mon, Jul 08, 2013 at 11:08:42PM -0700, Ben Widawsky wrote: > > Probably need to squash whole thing, or just the inactive part, tbd... > > > > Signed-off-by: Ben Widawsky > > I agree that we need vma->active, but I'm not sold on

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Move active to vma

2013-07-09 Thread Daniel Vetter
On Mon, Jul 08, 2013 at 11:08:42PM -0700, Ben Widawsky wrote: > Probably need to squash whole thing, or just the inactive part, tbd... > > Signed-off-by: Ben Widawsky I agree that we need vma->active, but I'm not sold on removing obj->active. Atm we have to use-cases for checking obj->active: -

[Intel-gfx] [PATCH 11/11] drm/i915: Move active to vma

2013-07-08 Thread Ben Widawsky
Probably need to squash whole thing, or just the inactive part, tbd... Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_drv.h | 14 ++-- drivers/gpu/drm/i915/i915_gem.c | 47 - 2 files changed, 35 insertions(+), 26 deletions(-) diff --git