From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Paul Menzel <pmenzel@molgen.mpg.de>
Cc: Tomas Winkler <tomas.winkler@intel.com>,
Jan Niehusmann <jan@gondor.com>,
Alexander Usyskin <alexander.usyskin@intel.com>,
linux-kernel@vger.kernel.org, Yu C Chen <yu.c.chen@intel.com>,
Tomi P Sarvela <tomi.p.sarvela@intel.com>,
Thorsten Leemhuis <linux@leemhuis.info>,
Daniel Blueman <daniel@quora.org>,
Len Brown <len.brown@intel.com>
Subject: Re: [char-misc for 4.10-rc4 V2] mei: bus: enable OS version only for SPT and newer
Date: Fri, 13 Jan 2017 14:00:18 +0100 [thread overview]
Message-ID: <20170113130018.GA896@kroah.com> (raw)
In-Reply-To: <b1ecfd53-e856-ce10-d56e-4fea27095a8f@molgen.mpg.de>
On Wed, Jan 11, 2017 at 03:26:06PM +0100, Paul Menzel wrote:
> Dear Tomas,
>
>
> On 01/11/17 15:12, Winkler, Tomas wrote:
>
> > > > On 01/11/17 10:24, Winkler, Tomas wrote:
> > > > > >
> > > > > > On Wed, Jan 11, 2017 at 01:27:21AM +0200, Tomas Winkler wrote:
> > > > > > > On older platforms the command should be just ignored by the
> > > > > > > firmware but some older platforms misbehave so it's safer to send
> > > > > > > the command only if required.
> > > > > >
> > > > > > Thanks! This fixes suspend-to-ram for me (on a Thinkpad x201s).
> > > > >
> > > > > What about Dell XPS13?
> > > >
> > > > With Linus' master branch from today, and Greg's char-misc-linus
> > > > merged
> > > > (Merge: 807b93e995d1 546cf3ef9c92), the regression is still there.
> > >
> > > Hmm, this should work on KBL....
> > >
> > > > I am now building a Linux kernel image with the two commits touching
> > > > `bus- fixup.c` reverted.
> > >
> > > Thanks for the effort.
> > >
> > > > Do you want me to open a separate bug report for that, or continue
> > > > debugging in the existing report [1], which is currently marked as resolved?
> > >
> > > Let's get some more data, shouldn't take long time.
> > > >
> > > > Do you have Kaby Lake devices sitting around for testing?
> > >
> > > We will of course try to reproduce the issue locally.
> >
> > Paul, currently we cannot reproduce this issue on Kaby Lake platforms on our side,
>
> It looks like it’s a different issue. Reverting the two commits touching
> `bus-fixup.c`, did not help.
>
> > we would be great for more debug data from your side.
> > You can get more info by enabling mode debug logs
> >
> > echo -n 'module mei +lfp' > /sys/kernel/debug/dynamic_debug/control
> > echo -n 'module mei_me +lfp' > /sys/kernel/debug/dynamic_debug/control
>
> I am currently bisecting to find the culprit. 13 steps will take some time
> though.
I can duplicate this on my laptop here as well :(
Did you get anywhere with your bisection?
thanks,
greg k-h
next prev parent reply other threads:[~2017-01-13 13:00 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-10 23:27 [char-misc for 4.10-rc4 V2] mei: bus: enable OS version only for SPT and newer Tomas Winkler
2017-01-10 22:49 ` Jan Niehusmann
2017-01-11 9:24 ` Winkler, Tomas
2017-01-11 10:20 ` Paul Menzel
2017-01-11 10:57 ` Winkler, Tomas
2017-01-11 14:12 ` Winkler, Tomas
2017-01-11 14:26 ` Paul Menzel
2017-01-11 16:10 ` Winkler, Tomas
2017-01-13 13:00 ` Greg Kroah-Hartman [this message]
2017-01-14 19:27 ` Paul Menzel
2017-01-14 19:39 ` Greg Kroah-Hartman
2017-01-15 7:19 ` Winkler, Tomas
2017-01-15 10:58 ` Greg Kroah-Hartman
2017-01-16 11:05 ` Greg Kroah-Hartman
2017-01-17 8:14 ` Thorsten Leemhuis
2017-01-17 14:34 ` Regression on Dell XPS13 (was: [char-misc for 4.10-rc4 V2] mei: bus: enable OS version only for SPT and newer) Paul Menzel
2017-01-17 16:57 ` Mario.Limonciello
2017-01-17 18:23 ` Greg KH
2017-01-17 18:38 ` Mario.Limonciello
2017-01-17 23:33 ` Darren Hart
2017-01-20 23:11 ` Mario.Limonciello
2017-01-21 9:11 ` Greg KH
2017-01-21 11:49 ` Rafael J. Wysocki
2017-01-22 11:25 ` Greg KH
2017-01-24 20:24 ` Mario.Limonciello
2017-01-22 9:45 ` Rafael J. Wysocki
2017-01-24 20:14 ` Mario.Limonciello
2017-01-18 2:18 ` Rafael J. Wysocki
2017-01-18 11:11 ` Regression on Dell XPS13 Paul Menzel
2017-01-18 11:30 ` Rafael J. Wysocki
2017-01-17 17:29 ` Thorsten Leemhuis
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=20170113130018.GA896@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=alexander.usyskin@intel.com \
--cc=daniel@quora.org \
--cc=jan@gondor.com \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@leemhuis.info \
--cc=pmenzel@molgen.mpg.de \
--cc=tomas.winkler@intel.com \
--cc=tomi.p.sarvela@intel.com \
--cc=yu.c.chen@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