Re: [PATCH] drm/i915: tune down hangcheck noise

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

2013/8/28 Daniel Vetter <daniel.vetter@xxxxxxxx>:
> We already have a big splashing *ERROR* for all the relevant cases of
> hangs, so this one here is redudant. And it results in an unclean
> dmesg when running with simulated hangs. Regression has been
> introduced in

Bug https://bugs.freedesktop.org/show_bug.cgi?id=69747 has a case
where despite the messages, there's no splashing *ERROR* (except for
an unclaimed-register message which should be ignored in this specific
case since it can be fixed without fixing the GPU hang). As a
consequence, QA reported the bug mentioning just the "unclaimed
regsiter" message, not the real GPU hang, while the unclaimed register
is just a consequence of the real GPU hang. ZZ_check_dmesg wouldn't
catch this specific bug if we didn't have the unclaimed register
problem. If the same bug happens on IVB we'll never know because we
won't have the "unclaimed register" message to point it.

>
> commit 05407ff889ceebe383aa5907219f86582ef96b72
> Author: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx>
> Date:   Thu May 30 09:04:29 2013 +0300
>
>     drm/i915: detect hang using per ring hangcheck_score
>
> Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68641
> Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> ---
>  drivers/gpu/drm/i915/i915_irq.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index a03b445..4c6853f 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -2027,9 +2027,9 @@ static void i915_hangcheck_elapsed(unsigned long data)
>
>         for_each_ring(ring, dev_priv, i) {
>                 if (ring->hangcheck.score > FIRE) {
> -                       DRM_ERROR("%s on %s\n",
> -                                 stuck[i] ? "stuck" : "no progress",
> -                                 ring->name);
> +                       DRM_INFO("%s on %s\n",
> +                                stuck[i] ? "stuck" : "no progress",
> +                                ring->name);
>                         rings_hung++;
>                 }
>         }
> --
> 1.8.4.rc3
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Paulo Zanoni
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux