linux-kernel - Re: [ 052/100] drm/i915: Increase the RC6p threshold.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <20130318195046.GA32067@kroah.com>
Date:	Mon, 18 Mar 2013 12:50:46 -0700
From:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:	Stéphane Marchesin <marcheu@...omium.org>
Cc:	Ben Hutchings <ben@...adent.org.uk>, linux-kernel@...r.kernel.org,
	stable@...r.kernel.org, Jesse Barnes <jbarnes@...tuousgeek.org>,
	Daniel Vetter <daniel.vetter@...ll.ch>
Subject: Re: [ 052/100] drm/i915: Increase the RC6p threshold.

On Mon, Mar 18, 2013 at 12:40:35PM -0700, Stéphane Marchesin wrote:
> 
> 
> 
> On Mon, Mar 18, 2013 at 12:38 PM, Greg Kroah-Hartman <
> gregkh@...uxfoundation.org> wrote:
> 
>     On Mon, Mar 18, 2013 at 01:11:09AM +0000, Ben Hutchings wrote:
>     > On Tue, 2013-03-12 at 15:31 -0700, Greg Kroah-Hartman wrote:
>     > > 3.8-stable review patch.  If anyone has any objections, please let me
>     know.
>     > >
>     > > ------------------
>     > >
>     > > From: Stéphane Marchesin <marcheu@...omium.org>
>     > >
>     > > commit 0920a48719f1ceefc909387a64f97563848c7854 upstream.
>     > >
>     > > This increases GEN6_RC6p_THRESHOLD from 100000 to 150000. For some
>     > > reason this avoids the gen6_gt_check_fifodbg.isra warnings and
>     > > associated GPU lockups, which makes my ivy bridge machine stable.
>     > >
>     > > Signed-off-by: Stéphane Marchesin <marcheu@...omium.org>
>     > > Acked-by: Jesse Barnes <jbarnes@...tuousgeek.org>
>     > > Signed-off-by: Daniel Vetter <daniel.vetter@...ll.ch>
>     > > Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
>     > >
>     > > ---
>     > >  drivers/gpu/drm/i915/intel_pm.c |    2 +-
>     > >  1 file changed, 1 insertion(+), 1 deletion(-)
>     > >
>     > > --- a/drivers/gpu/drm/i915/intel_pm.c
>     > > +++ b/drivers/gpu/drm/i915/intel_pm.c
>     > > @@ -2572,7 +2572,7 @@ static void gen6_enable_rps(struct drm_d
>     > >     I915_WRITE(GEN6_RC_SLEEP, 0);
>     > >     I915_WRITE(GEN6_RC1e_THRESHOLD, 1000);
>     > >     I915_WRITE(GEN6_RC6_THRESHOLD, 50000);
>     > > -   I915_WRITE(GEN6_RC6p_THRESHOLD, 100000);
>     > > +   I915_WRITE(GEN6_RC6p_THRESHOLD, 150000);
>     > >     I915_WRITE(GEN6_RC6pp_THRESHOLD, 64000); /* unused */
>     > >
>     > >     /* Check if we are enabling RC6 */
>     >
>     > Is there any reason why this shouldn't be applied to 3.2.y and 3.4.y?
>     > The same function and writes are present, only in intel_display.c rather
>     > than intel_pm.c.
> 
>     Ah, missed the fact that the file was renamed, nice catch.
> 
>     I'll apply it to older kernels if the authors and maintainers say it's
>     safe to do so.
> 
> 
> 
> It should be safe; I actually wrote this patch against Chrome OS's 3.4 kernel
> initially.

Ok, now applied to the 3.4-stable queue.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/