RE: Dell XPS13 does not suspend with Linux 4.10-rc3

From	"Winkler, Tomas" <tomas.winkler@intel.com>
Newsgroups	linux.kernel
Subject	RE: Dell XPS13 does not suspend with Linux 4.10-rc3
Date	2017-01-11 10:50 +0100
Message-ID	<sYnHz-5fJ-15@gated-at.bofh.it> (permalink)
References	(1 earlier) <sYc9s-6qU-33@gated-at.bofh.it> <sYd5v-74s-11@gated-at.bofh.it> <sYmVc-4JW-21@gated-at.bofh.it> <sYnxU-5ch-13@gated-at.bofh.it> <sYnHz-5fJ-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw


> 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.
> 
> I would have liked if it mentioned suspend issues, and the known broken
> devices in the commit message. That way, I would have found the commit
> more easily.

Yep,  I should probably add that info about the suspend issue, but looks like we broke the whole gen of older devices so we cannot list all of them :(
Tomas

Thread

Re: Dell XPS13 does not suspend with Linux 4.10-rc3 Jan Niehusmann <jan@gondor.com> - 2017-01-10 22:30 +0100
  Re: Dell XPS13 does not suspend with Linux 4.10-rc3 Jan Niehusmann <jan@gondor.com> - 2017-01-10 23:30 +0100
    Re: Dell XPS13 does not suspend with Linux 4.10-rc3 Paul Menzel <pmenzel@molgen.mpg.de> - 2017-01-11 10:00 +0100
      Re: Dell XPS13 does not suspend with Linux 4.10-rc3 Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-01-11 10:40 +0100
        RE: Dell XPS13 does not suspend with Linux 4.10-rc3 "Winkler, Tomas" <tomas.winkler@intel.com> - 2017-01-11 10:50 +0100
          Re: Dell XPS13 does not suspend with Linux 4.10-rc3 Paul Menzel <pmenzel@molgen.mpg.de> - 2017-01-11 18:20 +0100
            RE: Dell XPS13 does not suspend with Linux 4.10-rc3 "Winkler, Tomas" <tomas.winkler@intel.com> - 2017-01-11 23:20 +0100
        Re: Dell XPS13 does not suspend with Linux 4.10-rc3 Paul Menzel <pmenzel@molgen.mpg.de> - 2017-01-11 10:50 +0100

csiph-web