[Intel-gfx] [PATCH 4/8] drm/i915: trace vm eviction instead of everything

2013-09-11 Thread Ben Widawsky
Tracing vm eviction is really the event we care about. For the cases we evict everything, we still will get the trace. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_gem_evict.c | 2 ++ drivers/gpu/drm/i915/i915_trace.h | 15 +++ 2 files changed, 17 insertions(+) dif

Re: [Intel-gfx] [PATCH 4/8] drm/i915: trace vm eviction instead of everything

2013-09-11 Thread Chris Wilson
On Wed, Sep 11, 2013 at 02:57:51PM -0700, Ben Widawsky wrote: > Tracing vm eviction is really the event we care about. For the cases we > evict everything, we still will get the trace. > > Signed-off-by: Ben Widawsky > --- > drivers/gpu/drm/i915/i915_gem_evict.c | 2 ++ > drivers/gpu/drm/i915/i

Re: [Intel-gfx] [PATCH 4/8] drm/i915: trace vm eviction instead of everything

2013-08-30 Thread Ben Widawsky
On Sat, Aug 31, 2013 at 01:06:02AM +0100, Chris Wilson wrote: > On Fri, Aug 30, 2013 at 04:43:57PM -0700, Ben Widawsky wrote: > > Tracing vm eviction is really the event we care about. For the cases we > > evict everything, we still will get the trace. > > Keep both until you retire evict_everythi

Re: [Intel-gfx] [PATCH 4/8] drm/i915: trace vm eviction instead of everything

2013-08-30 Thread Chris Wilson
On Fri, Aug 30, 2013 at 04:43:57PM -0700, Ben Widawsky wrote: > Tracing vm eviction is really the event we care about. For the cases we > evict everything, we still will get the trace. Keep both until you retire evict_everything for good. Sometimes you need flashing neon lights to explain things i

[Intel-gfx] [PATCH 4/8] drm/i915: trace vm eviction instead of everything

2013-08-30 Thread Ben Widawsky
Tracing vm eviction is really the event we care about. For the cases we evict everything, we still will get the trace. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_gem_evict.c | 4 ++-- drivers/gpu/drm/i915/i915_trace.h | 12 ++-- 2 files changed, 8 insertions(+), 8 del