* 2.6.35-rc6+: i915: Bisected regression
@ 2010-08-01 9:01 Thomas Meyer
2010-08-01 11:42 ` Stefan Richter
0 siblings, 1 reply; 10+ messages in thread
From: Thomas Meyer @ 2010-08-01 9:01 UTC (permalink / raw)
To: Linux Kernel Mailing List
This commit makes suspend to ram unreliable for me:
commit d1b851fc0d105caa6b6e3e7c92d2987dfb52cbe0
Author: Zou Nan hai <nanhai.zou@intel.com>
Date: Fri May 21 09:08:57 2010 +0800
drm/i915: implement BSD ring buffer V2
The BSD (bit stream decoder) ring is used for accessing the BSD engine
which decodes video bitstream for H.264 and VC1 on G45+. It is
asynchronous with the render ring and has access to separate parts of
the GPU from it, though the render cache is coherent between the two.
Signed-off-by: Zou Nan hai <nanhai.zou@intel.com>
Signed-off-by: Xiang Hai hao <haihao.xiang@intel.com>
Signed-off-by: Eric Anholt <eric@anholt.net>
git revert didn't work on 2.6.35-rc6 so I just changed the HAS_BSD thing to:
#define HAS_BSD(dev) (0)
with this change applied suspend to ram is back to the reliability of 2.6.34.y
with kind regards
thomas
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-01 11:42 ` Stefan Richter
@ 2010-08-01 10:51 ` Thomas Meyer
2010-08-01 13:14 ` Paul Rolland
2010-08-05 5:43 ` Shuang He
0 siblings, 2 replies; 10+ messages in thread
From: Thomas Meyer @ 2010-08-01 10:51 UTC (permalink / raw)
To: Stefan Richter
Cc: Linux Kernel Mailing List, Zou Nan hai, Xiang Hai hao,
Eric Anholt, David Airlie, dri-devel
Am 01.08.2010 um 13:42 schrieb Stefan Richter <stefanr@s5r6.in-berlin.de>:
> Thomas Meyer wrote:
>> This commit makes suspend to ram unreliable for me:
>>
>> commit d1b851fc0d105caa6b6e3e7c92d2987dfb52cbe0
>> Author: Zou Nan hai <nanhai.zou@intel.com>
>> Date: Fri May 21 09:08:57 2010 +0800
>>
>> drm/i915: implement BSD ring buffer V2
>>
>> The BSD (bit stream decoder) ring is used for accessing the BSD engine
>> which decodes video bitstream for H.264 and VC1 on G45+. It is
>> asynchronous with the render ring and has access to separate parts of
>> the GPU from it, though the render cache is coherent between the two.
>>
>> Signed-off-by: Zou Nan hai <nanhai.zou@intel.com>
>> Signed-off-by: Xiang Hai hao <haihao.xiang@intel.com>
>> Signed-off-by: Eric Anholt <eric@anholt.net>
>>
>> git revert didn't work on 2.6.35-rc6 so I just changed the HAS_BSD thing to:
>>
>> #define HAS_BSD(dev) (0)
>>
>> with this change applied suspend to ram is back to the reliability of 2.6.34.y
>>
>> with kind regards
>> thomas
>
> Added CCs.
>
> Thomas, on what hardware did you observe the regression?
Is a Intel mobile express gm45 chipset.
> Is it suspend
> or resume that is unreliable?
suspend never finishes/hangs.
> What are the particular symptoms --- does
> not suspend, crashes during suspend, crashes during resume...?
Does not suspend, backlight is on.
> --
> Stefan Richter
> -=====-==-=- =--- ----=
> http://arcgraph.de/sr/
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-01 9:01 2.6.35-rc6+: i915: Bisected regression Thomas Meyer
@ 2010-08-01 11:42 ` Stefan Richter
2010-08-01 10:51 ` Thomas Meyer
0 siblings, 1 reply; 10+ messages in thread
From: Stefan Richter @ 2010-08-01 11:42 UTC (permalink / raw)
To: Thomas Meyer
Cc: Linux Kernel Mailing List, Zou Nan hai, Xiang Hai hao,
Eric Anholt, David Airlie, dri-devel
Thomas Meyer wrote:
> This commit makes suspend to ram unreliable for me:
>
> commit d1b851fc0d105caa6b6e3e7c92d2987dfb52cbe0
> Author: Zou Nan hai <nanhai.zou@intel.com>
> Date: Fri May 21 09:08:57 2010 +0800
>
> drm/i915: implement BSD ring buffer V2
>
> The BSD (bit stream decoder) ring is used for accessing the BSD engine
> which decodes video bitstream for H.264 and VC1 on G45+. It is
> asynchronous with the render ring and has access to separate parts of
> the GPU from it, though the render cache is coherent between the two.
>
> Signed-off-by: Zou Nan hai <nanhai.zou@intel.com>
> Signed-off-by: Xiang Hai hao <haihao.xiang@intel.com>
> Signed-off-by: Eric Anholt <eric@anholt.net>
>
> git revert didn't work on 2.6.35-rc6 so I just changed the HAS_BSD thing to:
>
> #define HAS_BSD(dev) (0)
>
> with this change applied suspend to ram is back to the reliability of 2.6.34.y
>
> with kind regards
> thomas
Added CCs.
Thomas, on what hardware did you observe the regression? Is it suspend
or resume that is unreliable? What are the particular symptoms --- does
not suspend, crashes during suspend, crashes during resume...?
--
Stefan Richter
-=====-==-=- =--- ----=
http://arcgraph.de/sr/
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-01 10:51 ` Thomas Meyer
@ 2010-08-01 13:14 ` Paul Rolland
2010-08-01 15:17 ` Roberto Oppedisano
2010-08-05 5:43 ` Shuang He
1 sibling, 1 reply; 10+ messages in thread
From: Paul Rolland @ 2010-08-01 13:14 UTC (permalink / raw)
To: Thomas Meyer
Cc: Stefan Richter, Linux Kernel Mailing List, Zou Nan hai,
Xiang Hai hao, Eric Anholt, David Airlie, dri-devel
Hello Thomas,
On Sun, 1 Aug 2010 12:51:36 +0200
Thomas Meyer <thomas@m3y3r.de> wrote:
> Am 01.08.2010 um 13:42 schrieb Stefan Richter <stefanr@s5r6.in-berlin.de>:
>
> > Thomas, on what hardware did you observe the regression?
>
> Is a Intel mobile express gm45 chipset.
My machine has :
00:02.0 0300: 8086:2a42 (rev 07)
00:02.1 0380: 8086:2a43 (rev 07)
00:02.0 VGA compatible controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07)
00:02.1 Display controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07)
(Dell Vostro 1520, BIOS Revision A04)
> > Is it suspend
> > or resume that is unreliable?
> suspend never finishes/hangs.
and is randomly suffering from exactly the same problems...
So, I've applied your hint (in i915_drv.h), rebuilt a kernel, booted, and
I've now successfully managed to go thru several suspend/resume cycles
without having the machine hanging while suspend !
Will check in the long term, but so far, it's a win !
Thx,
Paul
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-01 13:14 ` Paul Rolland
@ 2010-08-01 15:17 ` Roberto Oppedisano
2010-08-24 17:36 ` Thomas Meyer
0 siblings, 1 reply; 10+ messages in thread
From: Roberto Oppedisano @ 2010-08-01 15:17 UTC (permalink / raw)
To: Paul Rolland
Cc: Thomas Meyer, Stefan Richter, Linux Kernel Mailing List,
Zou Nan hai, Xiang Hai hao, Eric Anholt, David Airlie, dri-devel
On 01/08/2010 15:14, Paul Rolland wrote:
> My machine has :
> 00:02.0 0300: 8086:2a42 (rev 07)
> 00:02.1 0380: 8086:2a43 (rev 07)
>
> 00:02.0 VGA compatible controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07)
> 00:02.1 Display controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07)
>
> (Dell Vostro 1520, BIOS Revision A04)
>
>>> Is it suspend
>>> or resume that is unreliable?
>> suspend never finishes/hangs.
> and is randomly suffering from exactly the same problems...
>
> So, I've applied your hint (in i915_drv.h), rebuilt a kernel, booted, and
> I've now successfully managed to go thru several suspend/resume cycles
> without having the machine hanging while suspend !
I have to add a me too (HP 6730B, same PCI ids).
I was suffering from the same problem which seem completely gone with
the suggested hint.
R
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-01 10:51 ` Thomas Meyer
2010-08-01 13:14 ` Paul Rolland
@ 2010-08-05 5:43 ` Shuang He
2010-08-06 3:26 ` David John
1 sibling, 1 reply; 10+ messages in thread
From: Shuang He @ 2010-08-05 5:43 UTC (permalink / raw)
To: Thomas Meyer
Cc: Stefan Richter, Zou, Nanhai, Linux Kernel Mailing List, dri-devel
On 2010-8-1 18:51, Thomas Meyer wrote:
> Am 01.08.2010 um 13:42 schrieb Stefan Richter<stefanr@s5r6.in-berlin.de>:
>
>> Thomas Meyer wrote:
>>> This commit makes suspend to ram unreliable for me:
>>>
>>> commit d1b851fc0d105caa6b6e3e7c92d2987dfb52cbe0
>>> Author: Zou Nan hai<nanhai.zou@intel.com>
>>> Date: Fri May 21 09:08:57 2010 +0800
>>>
>>> drm/i915: implement BSD ring buffer V2
>>>
>>> The BSD (bit stream decoder) ring is used for accessing the BSD engine
>>> which decodes video bitstream for H.264 and VC1 on G45+. It is
>>> asynchronous with the render ring and has access to separate parts of
>>> the GPU from it, though the render cache is coherent between the two.
>>>
>>> Signed-off-by: Zou Nan hai<nanhai.zou@intel.com>
>>> Signed-off-by: Xiang Hai hao<haihao.xiang@intel.com>
>>> Signed-off-by: Eric Anholt<eric@anholt.net>
>>>
>>> git revert didn't work on 2.6.35-rc6 so I just changed the HAS_BSD thing to:
>>>
>>> #define HAS_BSD(dev) (0)
>>>
>>> with this change applied suspend to ram is back to the reliability of 2.6.34.y
>>>
>>> with kind regards
>>> thomas
>> Added CCs.
>>
>> Thomas, on what hardware did you observe the regression?
> Is a Intel mobile express gm45 chipset.
>
>> Is it suspend
>> or resume that is unreliable?
> suspend never finishes/hangs.
>
>> What are the particular symptoms --- does
>> not suspend, crashes during suspend, crashes during resume...?
> Does not suspend, backlight is on.
Could you help report a bug for this issue by following
http://intellinuxgraphics.org/how_to_report_bug.html
Thanks
--Shuang
>> --
>> Stefan Richter
>> -=====-==-=- =--- ----=
>> http://arcgraph.de/sr/
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-05 5:43 ` Shuang He
@ 2010-08-06 3:26 ` David John
0 siblings, 0 replies; 10+ messages in thread
From: David John @ 2010-08-06 3:26 UTC (permalink / raw)
To: Shuang He
Cc: Thomas Meyer, Stefan Richter, Zou, Nanhai,
Linux Kernel Mailing List, dri-devel, Rafael J. Wysocki
On 08/05/2010 11:13 AM, Shuang He wrote:
> Could you help report a bug for this issue by following
> http://intellinuxgraphics.org/how_to_report_bug.html
>
I have the same issue on a GM45 and Thomas's hack works in fixing it. It
is still present in 2.6.35.
Regards,
David.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-01 15:17 ` Roberto Oppedisano
@ 2010-08-24 17:36 ` Thomas Meyer
2010-08-25 3:02 ` Roberto Oppedisano
2010-08-25 13:41 ` Paul Rolland
0 siblings, 2 replies; 10+ messages in thread
From: Thomas Meyer @ 2010-08-24 17:36 UTC (permalink / raw)
To: Roberto Oppedisano
Cc: Paul Rolland, Stefan Richter, Linux Kernel Mailing List,
Zou Nan hai, Xiang Hai hao, Eric Anholt, David Airlie, dri-devel
On Sunday 01 August 2010 17:17:54 Roberto Oppedisano wrote:
> On 01/08/2010 15:14, Paul Rolland wrote:
> > My machine has :
> > 00:02.0 0300: 8086:2a42 (rev 07)
> > 00:02.1 0380: 8086:2a43 (rev 07)
> >
> > 00:02.0 VGA compatible controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07)
> > 00:02.1 Display controller: Intel Corporation Mobile 4 Series Chipset Integrated Graphics Controller (rev 07)
> >
> > (Dell Vostro 1520, BIOS Revision A04)
> >
> >>> Is it suspend
> >>> or resume that is unreliable?
> >> suspend never finishes/hangs.
Seems to be fixed in 2.6.36-rc2. Probably by merge commit
4238a417a91643e1162a98770288f630e37f0484.
Paul, Roberto: Could you please confirm this on your hardware?
kind regards
thomas
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-24 17:36 ` Thomas Meyer
@ 2010-08-25 3:02 ` Roberto Oppedisano
2010-08-25 13:41 ` Paul Rolland
1 sibling, 0 replies; 10+ messages in thread
From: Roberto Oppedisano @ 2010-08-25 3:02 UTC (permalink / raw)
To: Thomas Meyer
Cc: Paul Rolland, Stefan Richter, Linux Kernel Mailing List,
Zou Nan hai, Xiang Hai hao, Eric Anholt, David Airlie, dri-devel
2010/8/24, Thomas Meyer <thomas@m3y3r.de>:
> On Sunday 01 August 2010 17:17:54 Roberto Oppedisano wrote:
>> On 01/08/2010 15:14, Paul Rolland wrote:
>> > My machine has :
>> > 00:02.0 0300: 8086:2a42 (rev 07)
>> > 00:02.1 0380: 8086:2a43 (rev 07)
>> >
>> > 00:02.0 VGA compatible controller: Intel Corporation Mobile 4 Series
>> > Chipset Integrated Graphics Controller (rev 07)
>> > 00:02.1 Display controller: Intel Corporation Mobile 4 Series Chipset
>> > Integrated Graphics Controller (rev 07)
>> >
>> > (Dell Vostro 1520, BIOS Revision A04)
>> >
>> >>> Is it suspend
>> >>> or resume that is unreliable?
>> >> suspend never finishes/hangs.
>
> Seems to be fixed in 2.6.36-rc2. Probably by merge commit
> 4238a417a91643e1162a98770288f630e37f0484.
>
> Paul, Roberto: Could you please confirm this on your hardware?
The problem seems fixed for me too.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: 2.6.35-rc6+: i915: Bisected regression
2010-08-24 17:36 ` Thomas Meyer
2010-08-25 3:02 ` Roberto Oppedisano
@ 2010-08-25 13:41 ` Paul Rolland
1 sibling, 0 replies; 10+ messages in thread
From: Paul Rolland @ 2010-08-25 13:41 UTC (permalink / raw)
To: Thomas Meyer
Cc: Roberto Oppedisano, Stefan Richter, Linux Kernel Mailing List,
Zou Nan hai, Xiang Hai hao, Eric Anholt, David Airlie, dri-devel
Hello Thomas,
On Tue, 24 Aug 2010 19:36:18 +0200
Thomas Meyer <thomas@m3y3r.de> wrote:
> On Sunday 01 August 2010 17:17:54 Roberto Oppedisano wrote:
> > On 01/08/2010 15:14, Paul Rolland wrote:
> > > My machine has :
> > > 00:02.0 0300: 8086:2a42 (rev 07)
> > > 00:02.1 0380: 8086:2a43 (rev 07)
> > >
> > > 00:02.0 VGA compatible controller: Intel Corporation Mobile 4 Series
> > > Chipset Integrated Graphics Controller (rev 07) 00:02.1 Display
> > > controller: Intel Corporation Mobile 4 Series Chipset Integrated
> > > Graphics Controller (rev 07)
> > >
> > > (Dell Vostro 1520, BIOS Revision A04)
> > >
> > >>> Is it suspend
> > >>> or resume that is unreliable?
> > >> suspend never finishes/hangs.
>
> Seems to be fixed in 2.6.36-rc2. Probably by merge commit
> 4238a417a91643e1162a98770288f630e37f0484.
>
> Paul, Roberto: Could you please confirm this on your hardware?
I've just installed 2.6.36-rc2 on my machine, and I've tested a few
"Sleep", and so far, no problem any more...
So, I can't help you on the "is this the exact commit ?", but it is fixed
(or at least it looks as if ;)
Regards,
Paul
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-08-25 13:42 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-01 9:01 2.6.35-rc6+: i915: Bisected regression Thomas Meyer
2010-08-01 11:42 ` Stefan Richter
2010-08-01 10:51 ` Thomas Meyer
2010-08-01 13:14 ` Paul Rolland
2010-08-01 15:17 ` Roberto Oppedisano
2010-08-24 17:36 ` Thomas Meyer
2010-08-25 3:02 ` Roberto Oppedisano
2010-08-25 13:41 ` Paul Rolland
2010-08-05 5:43 ` Shuang He
2010-08-06 3:26 ` David John
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).