From: Michel Thierry <michel.thierry@intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>,
intel-gfx@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@ffwll.ch>,
Oscar Mateo <oscar.mateo@intel.com>,
Damien Lespiau <damien.lespiau@intel.com>,
Nick Hoath <nicholas.hoath@intel.com>,
Tvrtko Ursulin <tvrtko.ursulin@intel.com>,
Arun Siluvery <arun.siluvery@linux.intel.com>,
John Harrison <John.C.Harrison@Intel.com>,
Thomas Daniel <thomas.daniel@intel.com>,
stable@vger.linux.kernel
Subject: Re: [PATCH] drm/i915: Disable execlists by default for gen8
Date: Tue, 7 Jul 2015 10:31:27 +0100 [thread overview]
Message-ID: <559B9C6F.1050106@intel.com> (raw)
In-Reply-To: <20150707083310.GU5312@nuc-i3427.alporthouse.com>
On 7/7/2015 9:33 AM, Chris Wilson wrote:
> On Sat, Apr 11, 2015 at 09:41:37AM +0100, Chris Wilson wrote:
>> Bug reports are still coming in for late 4.0-rcX that indicate that
>> execlists causes GPU hangs following resume.
>>
>> Fixes regression from
>> commit d7f621e50704306c348ccb192f17047f1499f9bc
>> Author: Oscar Mateo <oscar.mateo@intel.com>
>> Date: Thu Jul 24 17:04:49 2014 +0100
>>
>> drm/i915/bdw: Enable Logical Ring Contexts (hence, Execlists)
>>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89308
>
> Ping?
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89915
>
> Bugs are still being reported where execlists is the culprit in 4.0.6.
> We have no advantages for execlists in 4.0, only performance and
> stability regressions, so what was the justification for keeping it
> enabled?
> -Chris
>
Hi,
I still think it's the same problem fixed by Peter
(http://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/?id=364aece01a2dd748fc36a1e8bf52ef639b0857bd).
The issue was a race between enabling the interrupts and completing the
first batchbuffer, that's probably why we only saw it in chv, but it's
the same code bdw uses.
v4.0.6 didn't get the fix,
http://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/tree/drivers/gpu/drm/i915/i915_drv.c?id=v4.0.6
Only v4.0.7:
http://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/tree/drivers/gpu/drm/i915/i915_drv.c?id=v4.0.7
-Michel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2015-07-07 9:31 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-11 8:41 [PATCH] drm/i915: Disable execlists by default for gen8 Chris Wilson
2015-04-11 11:53 ` shuang.he
2015-04-13 15:37 ` Jesse Barnes
2015-07-07 8:33 ` Chris Wilson
2015-07-07 9:31 ` Michel Thierry [this message]
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=559B9C6F.1050106@intel.com \
--to=michel.thierry@intel.com \
--cc=John.C.Harrison@Intel.com \
--cc=arun.siluvery@linux.intel.com \
--cc=chris@chris-wilson.co.uk \
--cc=damien.lespiau@intel.com \
--cc=daniel.vetter@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=nicholas.hoath@intel.com \
--cc=oscar.mateo@intel.com \
--cc=stable@vger.linux.kernel \
--cc=thomas.daniel@intel.com \
--cc=tvrtko.ursulin@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox