All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/i915: apply HWSTAM writes to Ivy Bridge as well
@ 2011-07-01 18:08 Jesse Barnes
  2011-07-01 20:01 ` Kenneth Graunke
  0 siblings, 1 reply; 3+ messages in thread
From: Jesse Barnes @ 2011-07-01 18:08 UTC (permalink / raw)
  To: intel-gfx

In an attempt to fix 38862 and 38863.

Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
---
 drivers/gpu/drm/i915/i915_irq.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 2e18161..23d1ae6 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -1745,7 +1745,7 @@ static void ironlake_irq_preinstall(struct drm_device *dev)
 		INIT_WORK(&dev_priv->rps_work, gen6_pm_rps_work);
 
 	I915_WRITE(HWSTAM, 0xeffe);
-	if (IS_GEN6(dev)) {
+	if (IS_GEN6(dev) || IS_GEN7(dev)) {
 		/* Workaround stalls observed on Sandy Bridge GPUs by
 		 * making the blitter command streamer generate a
 		 * write to the Hardware Status Page for
-- 
1.7.4.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/i915: apply HWSTAM writes to Ivy Bridge as well
  2011-07-01 18:08 [PATCH] drm/i915: apply HWSTAM writes to Ivy Bridge as well Jesse Barnes
@ 2011-07-01 20:01 ` Kenneth Graunke
  2011-07-01 20:28   ` Keith Packard
  0 siblings, 1 reply; 3+ messages in thread
From: Kenneth Graunke @ 2011-07-01 20:01 UTC (permalink / raw)
  To: intel-gfx

On 07/01/2011 11:08 AM, Jesse Barnes wrote:
> In an attempt to fix 38862 and 38863.
>
> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

Tested-by: Kenneth Graunke <kenneth@whitecape.org>

Prior to this patch, running the piglit test suite would randomly hang
the GPU unless I added msleeps between each test.  (Individual tests ran
fine.)

This patch fixes the issue.  I am now able to run piglit unmodified
without hanging the GPU.

Keith, could this go into -fixes?

Thanks for finding this, Chris and Jesse!

--Kenneth

> ---
>  drivers/gpu/drm/i915/i915_irq.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index 2e18161..23d1ae6 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -1745,7 +1745,7 @@ static void ironlake_irq_preinstall(struct drm_device *dev)
>  		INIT_WORK(&dev_priv->rps_work, gen6_pm_rps_work);
>  
>  	I915_WRITE(HWSTAM, 0xeffe);
> -	if (IS_GEN6(dev)) {
> +	if (IS_GEN6(dev) || IS_GEN7(dev)) {
>  		/* Workaround stalls observed on Sandy Bridge GPUs by
>  		 * making the blitter command streamer generate a
>  		 * write to the Hardware Status Page for

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/i915: apply HWSTAM writes to Ivy Bridge as well
  2011-07-01 20:01 ` Kenneth Graunke
@ 2011-07-01 20:28   ` Keith Packard
  0 siblings, 0 replies; 3+ messages in thread
From: Keith Packard @ 2011-07-01 20:28 UTC (permalink / raw)
  To: Kenneth Graunke, intel-gfx


[-- Attachment #1.1: Type: text/plain, Size: 308 bytes --]

On Fri, 01 Jul 2011 13:01:50 -0700, Kenneth Graunke <kenneth@whitecape.org> wrote:

> Keith, could this go into -fixes?

Yes. Would Chris and/or Jesse be willing to review the remaining
GEN6-specific code and make sure the appropriate bits are also turned on
for GEN7?

-- 
keith.packard@intel.com

[-- Attachment #1.2: Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-07-01 20:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-01 18:08 [PATCH] drm/i915: apply HWSTAM writes to Ivy Bridge as well Jesse Barnes
2011-07-01 20:01 ` Kenneth Graunke
2011-07-01 20:28   ` Keith Packard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.