From: "Sun, Yi" <yi.sun@intel.com>
To: Daniel J Blueman <daniel.blueman@gmail.com>,
Eric Anholt <eric@anholt.net>,
Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: Dave Airlie <airlied@redhat.com>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
"Meng, Mengmeng" <mengmeng.meng@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 3.0-rc3] i915: Fix gen6 (SNB) GPU stalling
Date: Fri, 17 Jun 2011 15:52:58 +0800 [thread overview]
Message-ID: <6E3BC7F7C9A4BF4286DD4C043110F30B568EC644CB@shsmsx502.ccr.corp.intel.com> (raw)
In-Reply-To: <1308070307-2630-1-git-send-email-daniel.blueman@gmail.com>
[-- Attachment #1.1: Type: text/plain, Size: 3865 bytes --]
All,
Thank for Mengmeng’s testing work, now the status is as following:
The bug33394(performance regression: screen stuttered when running the demo of 3D games with compiz enabled without GPU semaphores) is fixed. The two issue(stutter and hangcheck) is gone, now.
The issue described as bug 36407 isn’t able to be reproduced.
The bug 36653 is still there.
As to the performance, the detail is listed as the table.
From the table, we can get the information that the patch make little effect to the 2D performance, but it improve 3D performance much.
without patch
with patch
dis-semaphores
en-semaphores
dis-semaphores
en-semaphores
2D-aa10text
1790k
2650k
1640k
2550k
2D-rgb10text
1380k
2380k
1100k
2320k
openarena
11
86.2
98.9
103.9 fps
urbanterror
10.5
71.4
68.5
70.9 fps
padman
12.1
100.7
92
100.3 fps
nexuiz
6
20
19.5
20 fps
Thanks
--Yi,Sun
-----Original Message-----
From: intel-gfx-bounces+yi.sun=intel.com@lists.freedesktop.org [mailto:intel-gfx-bounces+yi.sun=intel.com@lists.freedesktop.org] On Behalf Of Daniel J Blueman
Sent: Wednesday, June 15, 2011 12:52 AM
To: Eric Anholt
Cc: Daniel J Blueman; intel-gfx@lists.freedesktop.org; linux-kernel@vger.kernel.org; Dave Airlie
Subject: [Intel-gfx] [PATCH 3.0-rc3] i915: Fix gen6 (SNB) GPU stalling
On 14 June 2011 13:23, Eric Anholt <eric@anholt.net> wrote:
> On Tue, 14 Jun 2011 12:18:36 +0800, Daniel J Blueman <daniel.blueman@gmail.com> wrote:
>> Hi Eric,
>>
>> The frequent ~1.5s pauses I hit with SNB hardware in the gnome3 UI (eg
>> whenever you hit the top-left of the screen to show all windows) are
>> nicely addressed by your recent wake patch [1] (ported to -rc3). Thus
>> I see no 'missed IRQ' kernel messages.
>>
>> As this addresses a significant usability regression, are you happy to
>> add it to the 3.0-rc queue? I think it has very good value in -stable
>> also (assuming correctness). What do you think?
>
> This one had significant performance impacts, and later hacks in this
> series worked around the problem to approximately the same level of
> success with less impact, and we don't actually have a justification of
> why any of them work. We were still hoping to come up with some clue,
> and haven't yet.
True; that is quite heavy handed delay looping.
It's a pity the usual Intel font didn't make it to the programmer's
reference manuals. Anyway, unmasking the blitter user interrupt in the hardware
status mask register addresses the root cause. Out of reset it's FFFFFFFFh,
so we don't need to read it here.
It would be good to get this into -rc4. -stable probably needs some additional
tweaks.
Signed-off-by: Daniel J Blueman <daniel.blueman@gmail.com>
---
drivers/gpu/drm/i915/i915_irq.c | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index b9fafe3..9a98c1b 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -1827,6 +1827,12 @@ int ironlake_irq_postinstall(struct drm_device *dev)
ironlake_enable_display_irq(dev_priv, DE_PCU_EVENT);
}
+ if (IS_GEN6(dev))
+ /* allow blitter user interrupt to generate a MSI write from
+ the ISR */
+ I915_WRITE(GEN6_BLITTER_HWSTAM,
+ 0xffffffff & ~GEN6_BLITTER_USER_INTERRUPT);
+
return 0;
}
--
1.7.4.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
[-- Attachment #1.2: Type: text/html, Size: 22836 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
next prev parent reply other threads:[~2011-06-17 7:54 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-14 16:51 [PATCH 3.0-rc3] i915: Fix gen6 (SNB) GPU stalling Daniel J Blueman
2011-06-14 17:23 ` Chris Wilson
2011-06-14 17:23 ` Chris Wilson
2011-06-15 2:06 ` Eric Anholt
2011-06-15 2:06 ` Eric Anholt
2011-06-15 3:24 ` Daniel J Blueman
2011-06-15 4:43 ` [Intel-gfx] " Ben Widawsky
2011-06-15 4:43 ` Ben Widawsky
2011-06-15 5:04 ` [Intel-gfx] " Daniel J Blueman
2011-06-15 5:04 ` Daniel J Blueman
2011-06-15 15:16 ` [Intel-gfx] " Ben Widawsky
2011-06-16 2:45 ` Daniel J Blueman
2011-06-16 2:45 ` Daniel J Blueman
2011-06-16 18:36 ` [Intel-gfx] " Eric Anholt
2011-06-16 18:36 ` Eric Anholt
2011-06-15 16:38 ` [Intel-gfx] " Eric Anholt
2011-06-16 3:45 ` Daniel J Blueman
2011-06-16 3:45 ` Daniel J Blueman
2011-06-15 17:11 ` [Intel-gfx] " Kenneth Graunke
2011-06-15 17:11 ` Kenneth Graunke
2011-06-17 7:52 ` Sun, Yi [this message]
2011-06-17 14:12 ` [Intel-gfx] " Jesse Barnes
2011-06-17 14:12 ` Jesse Barnes
2011-06-17 15:29 ` [Intel-gfx] " Robert Hooker
2011-06-17 15:29 ` Robert Hooker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6E3BC7F7C9A4BF4286DD4C043110F30B568EC644CB@shsmsx502.ccr.corp.intel.com \
--to=yi.sun@intel.com \
--cc=airlied@redhat.com \
--cc=daniel.blueman@gmail.com \
--cc=eric@anholt.net \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jbarnes@virtuousgeek.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mengmeng.meng@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.