<div class="gmail_quote">On Sat, Feb 11, 2012 at 10:59, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Sat, 11 Feb 2012 10:34:15 -0200, Eugeni Dodonov <<a href="mailto:eugeni.dodonov@intel.com">eugeni.dodonov@intel.com</a>> wrote:<br>
> This is yet another chapter in the ongoing saga of bringing RC6 to Sandy<br>
> Bridge machines by default.<br>
><br>
> Now that we have discovered that RC6 issues are triggered by RC6+ state,<br>
> let's try to disable it by default. Plain RC6 is the one responsible for<br>
> most energy savings, and so far it haven't given any problems - at least,<br>
> none we are aware of.<br>
><br>
> So with this, when i915_enable_rc6=-1 (e.g., the default value), we'll<br>
> attempt to enable plain RC6 only on SNB. For Ivy Bridge, the behavior<br>
> stays the same as always - we enable both RC6 and deep RC6.<br>
<br>
</div>We appear to be lacking a bunch of tested-bys for what we proclaim to<br>
have been tested.<br></blockquote><div><br>So far I had 2 reports saying that RC6 solved the problems which were present with RC6+RC6p (previous default) on SNB.<br><br>I'll wait a bit more and send a v2 version of this patch with all the Tested-by's. Indeed, it makes sense to have those on this patch before picking it into any kernel tree. Meanwhile, if there are any other volunteers who were affected by SNB RC6 voodoo in the past and are willing to give these patches a try, please, do so!<br>

<br>(Not that it wouldn't prevent all the possible issues, but it would certify that it is not a placebo effect at least for someone...) <br clear="all"></div></div><br>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br>

</a><br>