[Intel-gfx] [PATCH 2/2] drm/i915: enable plain RC6 on Sandy Bridge by default

Keith Packard keithp at keithp.com
Tue Feb 14 05:01:30 CET 2012


<#part sign=pgpmime>
On Sun, 12 Feb 2012 22:34:48 -0200, Eugeni Dodonov <eugeni at dodonov.net> wrote:

> what if we pick only the 1st patch in this series for -fixes? It won't
> change the defaults in any way, but it will allow the ones willing to
> enable it manually on SNB to prevent issues.

That seems like too big a change at this point in the release cycle. We
should be focused purely on fixing functionality bugs, not adding new
features. So, instead of adding the ability to individually control RC6
levels, we should just change what RC6 does on SNB when it is enabled,
then encourage people to test that and see if the shallowest RC6 states
actually work. That will provide sufficient information to know whether
it will be safe to turn that on by default in the next release, which is
what we really want.

The smallest change that gives us the data we need is what I'd like to have.

-- 
keith.packard at intel.com



More information about the Intel-gfx mailing list