[Intel-gfx] [PATCH] drm/i915: Tune down reset_stat output from ERROR to debug

[Daniel Vetter]

On Tue, Jan 14, 2014 at 01:36:32PM +0200, Mika Kuoppala wrote:
> Daniel Vetter <daniel.vetter at ffwll.ch> writes:
> 
> > This is user-triggerable and hence we should not allow it to spam
> > dmesg. Also, it upsets the nice dmesg tracking piglit does.
> >
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72740
> > Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> 
> Reviewed-by: Mika Kuoppala <mika.kuoppala at intel.com>

Chris note added and patch merged, thanks for the review.
-Daniel

> 
> > ---
> >  drivers/gpu/drm/i915/i915_gem.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
> > index af6b3152308b..bd9353402f02 100644
> > --- a/drivers/gpu/drm/i915/i915_gem.c
> > +++ b/drivers/gpu/drm/i915/i915_gem.c
> > @@ -2337,7 +2337,7 @@ static void i915_set_reset_status(struct intel_ring_buffer *ring,
> >  
> >  	if (ring->hangcheck.action != HANGCHECK_WAIT &&
> >  	    i915_request_guilty(request, acthd, &inside)) {
> > -		DRM_ERROR("%s hung %s bo (0x%lx ctx %d) at 0x%x\n",
> > +		DRM_DEBUG("%s hung %s bo (0x%lx ctx %d) at 0x%x\n",
> >  			  ring->name,
> >  			  inside ? "inside" : "flushing",
> >  			  offset,
> > -- 
> > 1.8.1.4

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch