drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-25 Thread Daniel Vetter
On Thu, May 24, 2012 at 07:27:04PM -0700, Linus Torvalds wrote:
> These guys seem to be recently introduced:
> 
>   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
> GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206
>   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
> GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700
> 
> This is on my SNB Macbook Air.
> 
> Everything seems to *work*, which makes me think:
> 
>  - that error isn't really so big a deal that you have to *SHOUT* about it.
> 
>  - I wonder how valid the discrepancy checking code is to begin with.
> 
> Hmm?

On snb/ivb we have a problem that the gpu turbo can get stuck, and most
often it gets stuck at the lowest frequency, resulting in unhappy users.
Chris noticed that this happens when the RP irq limit register reads a
bogus value (and eventually gets stuck). Although your machine looks
interesting, because usually we just read back 0.

I've already sent a pull request to Dave with our current best effort to
duct-tape over the problem. If that doesn't cut out the dmesg flood, I
guess we need to tune down that message a bit. But we've added it in that
obnoxious way in the first place to get bug reports, so that we actually
know how bad the problem is. Hence I'd like to keep it around a bit.

Cheers, Daniel
-- 
Daniel Vetter
Mail: daniel at ffwll.ch
Mobile: +41 (0)79 365 57 48


drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-25 Thread Chris Wilson
On Thu, 24 May 2012 19:27:04 -0700, Linus Torvalds  wrote:
> These guys seem to be recently introduced:
> 
>   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
> GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206
>   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
> GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700
> 
> This is on my SNB Macbook Air.
> 
> Everything seems to *work*, which makes me think:
> 
>  - that error isn't really so big a deal that you have to *SHOUT* about it.
> 
>  - I wonder how valid the discrepancy checking code is to begin with.

Extremely, it is accurately telling us that only half of the patches
were applied...

After that when it does shout, it just saved your system. However, even
after fixing it to the best of our knowledge, it is still a far too
regular occurrence (playing video seems to have just the right
combination of turbo and rc6 to trigger it reliably) and even though it is
a calamity we do now expect it to be seen in the wild and so will reduce
the severity of the warning.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


Re: drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-25 Thread Chris Wilson
On Thu, 24 May 2012 19:27:04 -0700, Linus Torvalds 
torva...@linux-foundation.org wrote:
 These guys seem to be recently introduced:
 
   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
 GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206
   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
 GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700
 
 This is on my SNB Macbook Air.
 
 Everything seems to *work*, which makes me think:
 
  - that error isn't really so big a deal that you have to *SHOUT* about it.
 
  - I wonder how valid the discrepancy checking code is to begin with.

Extremely, it is accurately telling us that only half of the patches
were applied...

After that when it does shout, it just saved your system. However, even
after fixing it to the best of our knowledge, it is still a far too
regular occurrence (playing video seems to have just the right
combination of turbo and rc6 to trigger it reliably) and even though it is
a calamity we do now expect it to be seen in the wild and so will reduce
the severity of the warning.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-25 Thread Daniel Vetter
On Thu, May 24, 2012 at 07:27:04PM -0700, Linus Torvalds wrote:
 These guys seem to be recently introduced:
 
   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
 GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206
   [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
 GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700
 
 This is on my SNB Macbook Air.
 
 Everything seems to *work*, which makes me think:
 
  - that error isn't really so big a deal that you have to *SHOUT* about it.
 
  - I wonder how valid the discrepancy checking code is to begin with.
 
 Hmm?

On snb/ivb we have a problem that the gpu turbo can get stuck, and most
often it gets stuck at the lowest frequency, resulting in unhappy users.
Chris noticed that this happens when the RP irq limit register reads a
bogus value (and eventually gets stuck). Although your machine looks
interesting, because usually we just read back 0.

I've already sent a pull request to Dave with our current best effort to
duct-tape over the problem. If that doesn't cut out the dmesg flood, I
guess we need to tune down that message a bit. But we've added it in that
obnoxious way in the first place to get bug reports, so that we actually
know how bad the problem is. Hence I'd like to keep it around a bit.

Cheers, Daniel
-- 
Daniel Vetter
Mail: dan...@ffwll.ch
Mobile: +41 (0)79 365 57 48
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-24 Thread Linus Torvalds
These guys seem to be recently introduced:

  [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206
  [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700

This is on my SNB Macbook Air.

Everything seems to *work*, which makes me think:

 - that error isn't really so big a deal that you have to *SHOUT* about it.

 - I wonder how valid the discrepancy checking code is to begin with.

Hmm?

  Linus


drm/i915 3.5 merge window: gen6_sanitize_pm errors

2012-05-24 Thread Linus Torvalds
These guys seem to be recently introduced:

  [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
GEN6_RP_INTERRUPT_LIMITS expected 1700, was 1206
  [drm:gen6_sanitize_pm] *ERROR* Power management discrepancy:
GEN6_RP_INTERRUPT_LIMITS expected 1707, was 1700

This is on my SNB Macbook Air.

Everything seems to *work*, which makes me think:

 - that error isn't really so big a deal that you have to *SHOUT* about it.

 - I wonder how valid the discrepancy checking code is to begin with.

Hmm?

  Linus
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel