linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
To: Zhang Rui <rui.zhang@intel.com>
Cc: Linux PM list <linux-pm@vger.kernel.org>
Subject: Re: Possible regression caused by commit 1bd047be37d9 ("tpm_crb: Use devm_ioremap_resource")
Date: Wed, 3 Aug 2016 12:37:25 -0600	[thread overview]
Message-ID: <20160803183725.GA4308@obsidianresearch.com> (raw)
In-Reply-To: <1469933246.2361.7.camel@intel.com>

On Sun, Jul 31, 2016 at 10:47:26AM +0800, Zhang Rui wrote:
> Hi, Jason,
> 
> we got a bug report complaining that the system can not enter deeper
> package c-states than pc2, and the git bisect result points to commit
> 1bd047be37d9 ("tpm_crb: Use devm_ioremap_resource").
> 
> can you take a look at this issue?
> 
> Bugzilla link: https://bugzilla.kernel.org/show_bug.cgi?id=118451

The tagged commit basically swiches devm_ioremap_nocache for
devm_ioremap_resource - AFAIK the only external difference being that the
TPM driver is now part of the resource tree.

I didn't think resource tree interacted with Intel PM at all?

One thing we know about is that the TPM driver fails to load on some
systems due to resource conflicts. Is it possible Yrjan's is seeing
TPM load failures in one kernel? Presumably not loading the tpm driver
would impact PM somehow??

I recommend checking for the issue solved in this patch:

commit 422eac3f7deae34dbaffd08e03e27f37a5394a56
Author: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Date:   Tue Apr 19 12:54:18 2016 +0300

    tpm_crb: fix mapping of the buffers
    
    On my Lenovo x250 the following situation occurs:
    
    [18697.813871] tpm_crb MSFT0101:00: can't request region for resource
    [mem 0xacdff080-0xacdfffff]

Jason

      reply	other threads:[~2016-08-03 18:38 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-31  2:47 Possible regression caused by commit 1bd047be37d9 ("tpm_crb: Use devm_ioremap_resource") Zhang Rui
2016-08-03 18:37 ` Jason Gunthorpe [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=20160803183725.GA4308@obsidianresearch.com \
    --to=jgunthorpe@obsidianresearch.com \
    --cc=linux-pm@vger.kernel.org \
    --cc=rui.zhang@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;
as well as URLs for NNTP newsgroup(s).