All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rajiv Andrade <srajiv@linux.vnet.ibm.com>
To: Jiri Kosina <jkosina@suse.cz>
Cc: Debora Velarde <debora@linux.vnet.ibm.com>,
	Marcel Selhorst <m.selhorst@sirrix.com>,
	linux-kernel@vger.kernel.org, tpmdd-devel@lists.sourceforge.net,
	"Rafael J. Wysocki" <rjw@suse.com>,
	jmorris@namei.org, mjg@redhat.com
Subject: Re: [REGRESSION] Suspend fails because of TPM modules
Date: Mon, 29 Nov 2010 13:22:47 -0200	[thread overview]
Message-ID: <4CF3C547.6020302@linux.vnet.ibm.com> (raw)
In-Reply-To: <alpine.LNX.2.00.1011291614060.981@pobox.suse.cz>

On 11/29/2010 01:15 PM, Jiri Kosina wrote:
> On Mon, 29 Nov 2010, Jiri Kosina wrote:
>
>> Hi,
>>
>> on my thinkpad x200s (and I have seen reports on different HW as well),
>> suspend fails when TPM modules are loaded.
>>
>> 	tpm_tis 00:0a: tpm_transmit: tpm_send: error -5
>> 	legacy_suspend(): pnp_bus_suspend+0x0/0xa0 returns -5
>> 	PM: Device 00:0a failed to suspend: error -5
>> 	PM: Some devices failed to suspend
>>
>> Once tpm, tpm_bios, tpm_tis and tpm modules are unloaded, suspend/resume
>> works.
>>
>> This is a regression. It definitely worked on this very same hardware on
>> 2.6.34. Any kernel between .34 and .37 wasn't booted there, so I don't
>> have any data of that kind.
>>
>> I can try bisecting it, but if anyone sees immediately what the culprit
>> might be, that'd be helpful.
> I just found out, that if I modprobe tpm_tis module with
>
> 	itpm=1
>
> parameter, the problem doesn't happen any more and suspend works fine.
>
> This definitely wasn't needed on older kernels though, so I'd consider
> that still a regression.
>
> Also, can't we make the module automatically detect the machines on which
> to apply the workaround? Let's say, based on DMI?
>
There's a patch already submitted that solves this:
http://marc.info/?l=linux-kernel&m=128769741101534&w=2 
<http://marc.info/?l=linux-kernel&m=128769741101534&w=2>

This side effect (to solve the suspend issue) should increase its 
urgency I think.
James, any thoughts?

Rajiv





  parent reply	other threads:[~2010-11-29 15:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-29 15:00 [REGRESSION] Suspend fails because of TPM modules Jiri Kosina
2010-11-29 15:15 ` Jiri Kosina
2010-11-29 15:19   ` Matthew Garrett
2010-11-29 15:22   ` Rajiv Andrade [this message]
2010-11-29 15:26     ` Jiri Kosina
2010-11-29 15:32       ` Matthew Garrett
2010-11-29 15:46         ` Rajiv Andrade
2010-11-29 15:41       ` Rajiv Andrade
2010-11-29 16:14   ` [tpmdd-devel] " Michael Doube
2010-11-29 16:22     ` Jiri Kosina

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=4CF3C547.6020302@linux.vnet.ibm.com \
    --to=srajiv@linux.vnet.ibm.com \
    --cc=debora@linux.vnet.ibm.com \
    --cc=jkosina@suse.cz \
    --cc=jmorris@namei.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m.selhorst@sirrix.com \
    --cc=mjg@redhat.com \
    --cc=rjw@suse.com \
    --cc=tpmdd-devel@lists.sourceforge.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.