[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <639d2a00-3094-951a-3665-b0f1cb6dec91@molgen.mpg.de>
Date: Wed, 11 Jan 2017 18:16:12 +0100
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: Tomas Winkler <tomas.winkler@...el.com>,
Thorsten Leemhuis <linux@...mhuis.info>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Jan Niehusmann <jan@...dor.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Daniel Blueman <daniel@...ra.org>,
Len Brown <len.brown@...el.com>,
"Usyskin, Alexander" <alexander.usyskin@...el.com>
Subject: Re: Dell XPS13 does not suspend with Linux 4.10-rc3
On 01/11/17 10:49, Winkler, Tomas wrote:
>> Subject: Re: Dell XPS13 does not suspend with Linux 4.10-rc3
>>
>> On 01/11/17 10:36, Greg Kroah-Hartman wrote:
>>> On Wed, Jan 11, 2017 at 09:53:38AM +0100, Paul Menzel wrote:
>>
>>>> On 01/10/17 23:24, Jan Niehusmann wrote:
>>>>> On Tue, Jan 10, 2017 at 09:43:31PM +0100, Jan Niehusmann wrote:
>>>>>> And I bisected the issue to commit 7279b238ba, "mei: send OS type
>>>>>> to the FW"
>>>>>
>>>>> Indeed, just disabling the FIXUP implemented by that commit fixes
>>>>> suspend for me, with 4.10.0-rc3. Btw, this is on a Thinkpad x201s.
>>>>>
>>>>> (For the addressees newly added to cc, it seems like the mentioned
>>>>> commit breaks suspend to ram on Dell XPS13 and Thinkpad x201s)
>>>>>
>>>>> --- a/drivers/misc/mei/bus-fixup.c
>>>>> +++ b/drivers/misc/mei/bus-fixup.c
>>>>> @@ -406,3 +406,3 @@ static struct mei_fixup {
>>>>> MEI_FIXUP(MEI_UUID_WD, mei_wd),
>>>>> - MEI_FIXUP(MEI_UUID_MKHIF_FIX, mei_mkhi_fix),
>>>>> +// MEI_FIXUP(MEI_UUID_MKHIF_FIX, mei_mkhi_fix),
>>>>> };
>>>>>
>>>>>
>>>>> I don't know what this may break, as the description of 7279b23 just
>>>>> says:
>>>>> "This workaround was added to support other broken OS and we need to
>>>>> follow here."
>>>>>
>>>>> Unless there is a better justification, I think the commit should be
>>>>> reverted, for now.
>>>>
>>>> Somebody else already created the bug 192051 in the Linux Kernel Bug
>>>> Tracker [1].
>>>
>>> There's a fix for this in my tree to go to Linus for 4.10-rc4.
>>
>> Yes, Tomas referenced it in the bug report. I test it on the Dell XPS13
>> (9360) now.
As written elsewhere, the issue on the Dell XPS13 seems to be a
different issue. (No idea, which message I should have replied to.) I
have to run, and won’t have access to the machine until Monday, so here
is as far as I got.
```
$ git bisect log
git bisect start
# good: [69973b830859bc6529a7a0468ba0d80ee5117826] Linux 4.9
git bisect good 69973b830859bc6529a7a0468ba0d80ee5117826
# good: [69973b830859bc6529a7a0468ba0d80ee5117826] Linux 4.9
git bisect good 69973b830859bc6529a7a0468ba0d80ee5117826
# bad: [a121103c922847ba5010819a3f250f1f7fc84ab8] Linux 4.10-rc3
git bisect bad a121103c922847ba5010819a3f250f1f7fc84ab8
# bad: [72cca7baf4fba777b8ab770b902cf2e08941773f] Merge tag
'staging-4.10-rc1' of
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging
git bisect bad 72cca7baf4fba777b8ab770b902cf2e08941773f
# good: [b8d2798f32785398fcd1c48ea80c0c6c5ab88537] Merge tag
'clk-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/clk/linux
git bisect good b8d2798f32785398fcd1c48ea80c0c6c5ab88537
# good: [9439b3710df688d853eb6cb4851256f2c92b1797] Merge tag
'drm-for-v4.10' of git://people.freedesktop.org/~airlied/linux
git bisect good 9439b3710df688d853eb6cb4851256f2c92b1797
```
[…]
Kind regards,
Paul
Powered by blists - more mailing lists