All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Chua <jeff.chua.linux@gmail.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Len Brown <lenb@kernel.org>, LKML <linux-kernel@vger.kernel.org>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	Linux-pm mailing list <linux-pm@lists.linux-foundation.org>,
	Matthew Garrett <mjg59@srcf.ucam.org>
Subject: Re: [PATCH 0/11] ACPI: Fixes and cleanups related to iomaps management
Date: Mon, 24 Jan 2011 09:13:30 +0800	[thread overview]
Message-ID: <AANLkTi=YwD7kBZ6XsnUpCfqCmTuwwhkcAYvjXJL3Fj8O@mail.gmail.com> (raw)
In-Reply-To: <201101232128.03007.rjw@sisk.pl>

On Mon, Jan 24, 2011 at 4:28 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote:
> On Sunday, January 23, 2011, Jeff Chua wrote:
>> On Sun, Jan 23, 2011 at 3:12 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote:
>> > On Saturday, January 22, 2011, Jeff Chua wrote:
>> >> 2011/1/22 Rafael J. Wysocki <rjw@sisk.pl>:
>> >> > On Saturday, January 22, 2011, Jeff Chua wrote:
>> Strange. If I didn't offline, it hanged during suspend even without
>> thinkpad_acpi. I could see that the suspend code does the offline
>> automatically, but executing the offline before calling suspend makes
>> it no hanging.
>
> That means there's a problem in the CPU hotplug code that manifests itslef
> during suspend.  Is this 100% reproducible?  Did it happen with 2.6.37?

Quite reproducible. Once "something" trigger the hang,  no matter how
I reset the system, it'll still hang at suspend. And at other times,
it just worked. I tried many things, but it seems if I go back to use
an earlier version (2.6.37 works perfectly for suspend-to-disk/mem),
then switched back to the newer broken version, it'll work for a few
cycles before hanging at suspend-to-disk.

I guess it could be CPU hotplug, and may be that thinkpad-acpi is
triggering the broken cpu hotplug to fail.

Jeff
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: Jeff Chua <jeff.chua.linux@gmail.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Len Brown <lenb@kernel.org>, LKML <linux-kernel@vger.kernel.org>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	Linux-pm mailing list <linux-pm@lists.linux-foundation.org>,
	Matthew Garrett <mjg59@srcf.ucam.org>
Subject: Re: [PATCH 0/11] ACPI: Fixes and cleanups related to iomaps management
Date: Mon, 24 Jan 2011 09:13:30 +0800	[thread overview]
Message-ID: <AANLkTi=YwD7kBZ6XsnUpCfqCmTuwwhkcAYvjXJL3Fj8O@mail.gmail.com> (raw)
In-Reply-To: <201101232128.03007.rjw@sisk.pl>

On Mon, Jan 24, 2011 at 4:28 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote:
> On Sunday, January 23, 2011, Jeff Chua wrote:
>> On Sun, Jan 23, 2011 at 3:12 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote:
>> > On Saturday, January 22, 2011, Jeff Chua wrote:
>> >> 2011/1/22 Rafael J. Wysocki <rjw@sisk.pl>:
>> >> > On Saturday, January 22, 2011, Jeff Chua wrote:
>> Strange. If I didn't offline, it hanged during suspend even without
>> thinkpad_acpi. I could see that the suspend code does the offline
>> automatically, but executing the offline before calling suspend makes
>> it no hanging.
>
> That means there's a problem in the CPU hotplug code that manifests itslef
> during suspend.  Is this 100% reproducible?  Did it happen with 2.6.37?

Quite reproducible. Once "something" trigger the hang,  no matter how
I reset the system, it'll still hang at suspend. And at other times,
it just worked. I tried many things, but it seems if I go back to use
an earlier version (2.6.37 works perfectly for suspend-to-disk/mem),
then switched back to the newer broken version, it'll work for a few
cycles before hanging at suspend-to-disk.

I guess it could be CPU hotplug, and may be that thinkpad-acpi is
triggering the broken cpu hotplug to fail.

Jeff

  reply	other threads:[~2011-01-24  1:13 UTC|newest]

Thread overview: 99+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-20 11:26 [PATCH 0/11] ACPI: Fixes and cleanups related to iomaps management Rafael J. Wysocki
2011-01-20 11:27 ` [PATCH 1/11] ACPI: Introduce acpi_os_ioremap() Rafael J. Wysocki
2011-01-20 11:27 ` Rafael J. Wysocki
2011-01-20 11:28 ` [PATCH 2/11] ACPI / PM: Call suspend_nvs_free() earlier during resume Rafael J. Wysocki
2011-01-20 11:28 ` Rafael J. Wysocki
2011-01-20 11:30 ` [PATCH 3/11] ACPI: Fix acpi_os_read_memory() and acpi_os_write_memory() Rafael J. Wysocki
2011-01-20 11:30 ` Rafael J. Wysocki
2011-01-20 11:30 ` [PATCH 4/11] ACPI: Do not export functions that are only used in osl.c Rafael J. Wysocki
2011-01-20 11:30 ` Rafael J. Wysocki
2011-01-20 11:31 ` [PATCH 5/11] ACPI: Change acpi_ioremap_lock into a mutex Rafael J. Wysocki
2011-01-20 11:31 ` Rafael J. Wysocki
2011-01-20 11:32 ` [PATCH 6/11] ACPI: Avoid walking the list of iomaps in osl.c twice in a row Rafael J. Wysocki
2011-01-20 11:32 ` Rafael J. Wysocki
2011-01-20 11:33 ` [PATCH 7/11] ACPI: Make acpi_os_map_memory() avoid creating unnecessary mappings Rafael J. Wysocki
2011-01-20 11:33 ` Rafael J. Wysocki
2011-01-20 11:34 ` [PATCH 8/11] ACPI: Do not use krefs under a mutex in osl.c Rafael J. Wysocki
2011-01-20 11:34 ` Rafael J. Wysocki
2011-01-20 11:35 ` [PATCH 9/11] ACPI: Introduce acpi_os_get_iomem() Rafael J. Wysocki
2011-01-20 11:35 ` Rafael J. Wysocki
2011-01-20 11:36 ` [PATCH 10/11] ACPI / PM: Use existing ACPI iomaps for NVS save/restore Rafael J. Wysocki
2011-01-20 11:36 ` Rafael J. Wysocki
2011-01-20 11:37 ` [PATCH 11/11] ACPI / PM: Make NVS save/restore code use slightly less memory Rafael J. Wysocki
2011-01-20 11:37 ` Rafael J. Wysocki
2011-01-20 16:06 ` [PATCH 0/11] ACPI: Fixes and cleanups related to iomaps management Jeff Chua
2011-01-20 16:06 ` Jeff Chua
2011-01-20 16:06   ` Jeff Chua
2011-01-20 20:57   ` Rafael J. Wysocki
2011-01-20 21:46     ` Jeff Chua
2011-01-21  0:04       ` Rafael J. Wysocki
2011-01-21  2:51         ` Jeff Chua
2011-01-21  2:51         ` Jeff Chua
2011-01-21 12:01           ` Rafael J. Wysocki
2011-01-21 12:01           ` Rafael J. Wysocki
2011-01-21 21:06           ` Rafael J. Wysocki
2011-01-22  5:54             ` Jeff Chua
2011-01-22  5:54             ` Jeff Chua
2011-01-22  5:54               ` Jeff Chua
2011-01-22  5:58               ` Jeff Chua
2011-01-22  5:58               ` Jeff Chua
2011-01-22  9:25                 ` Rafael J. Wysocki
2011-01-22 17:24                   ` Jeff Chua
2011-01-22 17:24                   ` Jeff Chua
2011-01-22 17:24                     ` Jeff Chua
2011-01-22 19:12                     ` Rafael J. Wysocki
2011-01-23  0:14                       ` Jeff Chua
2011-01-23  0:14                         ` Jeff Chua
2011-01-23 20:28                         ` Rafael J. Wysocki
2011-01-23 20:28                         ` Rafael J. Wysocki
2011-01-24  1:13                           ` Jeff Chua [this message]
2011-01-24  1:13                             ` Jeff Chua
2011-01-24 21:36                             ` Rafael J. Wysocki
2011-01-24 21:36                             ` Rafael J. Wysocki
2011-01-24  1:13                           ` Jeff Chua
2011-01-23  0:14                       ` Jeff Chua
2011-01-22 19:12                     ` Rafael J. Wysocki
2011-01-22  9:25                 ` Rafael J. Wysocki
2011-01-22  9:13               ` Rafael J. Wysocki
2011-01-23 18:20                 ` Henrique de Moraes Holschuh
2011-01-23 18:20                 ` Henrique de Moraes Holschuh
2011-01-23 20:35                   ` Rafael J. Wysocki
2011-01-23 20:35                   ` Rafael J. Wysocki
2011-01-23 23:15                     ` Henrique de Moraes Holschuh
2011-01-24 21:37                       ` Rafael J. Wysocki
2011-01-24 21:37                       ` Rafael J. Wysocki
2011-01-23 23:15                     ` Henrique de Moraes Holschuh
2011-01-22  9:13               ` Rafael J. Wysocki
2011-01-21 21:06           ` Rafael J. Wysocki
2011-01-21  0:04       ` Rafael J. Wysocki
2011-01-20 21:46     ` Jeff Chua
2011-01-20 20:57   ` Rafael J. Wysocki
2011-01-24 23:25 ` [PATCH 0/8] ACPI: Fixes and cleanups related to iomaps management (v2) Rafael J. Wysocki
2011-01-24 23:25 ` Rafael J. Wysocki
2011-01-24 23:26   ` [PATCH 1/8] ACPI: Fix acpi_os_read_memory() and acpi_os_write_memory() (v2) Rafael J. Wysocki
2011-01-24 23:26   ` Rafael J. Wysocki
2011-01-24 23:27   ` [PATCH 2/8] ACPI: Do not export functions that are only used in osl.c Rafael J. Wysocki
2011-01-24 23:27   ` Rafael J. Wysocki
2011-01-24 23:28   ` [PATCH 3/8] ACPI: Change acpi_ioremap_lock into a mutex Rafael J. Wysocki
2011-01-24 23:28   ` Rafael J. Wysocki
2011-01-24 23:28   ` [PATCH 4/8] ACPI: Avoid walking the list of memory mappings in osl.c twice in a row Rafael J. Wysocki
2011-01-24 23:28   ` Rafael J. Wysocki
2011-01-24 23:29   ` [PATCH 5/8] ACPI: Make acpi_os_map_memory() avoid creating unnecessary mappings Rafael J. Wysocki
2011-01-24 23:29   ` Rafael J. Wysocki
2011-01-24 23:30   ` [PATCH 6/8] ACPI: Do not use krefs under a mutex in osl.c Rafael J. Wysocki
2011-01-24 23:30   ` Rafael J. Wysocki
2011-01-24 23:30   ` [PATCH 7/8] ACPI: Introduce acpi_os_get_iomem() Rafael J. Wysocki
2011-01-24 23:30   ` Rafael J. Wysocki
2011-01-24 23:32   ` [PATCH 8/8] ACPI / PM: Use existing ACPI iomaps for NVS save/restore (v2) Rafael J. Wysocki
2011-02-05 15:31     ` Jeff Chua
2011-02-05 15:31     ` Jeff Chua
2011-02-05 18:20       ` Rafael J. Wysocki
2011-02-05 18:20       ` Rafael J. Wysocki
2011-02-05 18:24       ` Rafael J. Wysocki
2011-02-05 18:24       ` Rafael J. Wysocki
2011-02-05 18:51       ` Linus Torvalds
2011-02-05 18:51       ` Linus Torvalds
2011-01-24 23:32   ` Rafael J. Wysocki
  -- strict thread matches above, loose matches on Subject: below --
2011-01-25  9:43 [PATCH 0/11] ACPI: Fixes and cleanups related to iomaps management Jeff Chua
2011-01-25  9:43 Jeff Chua
2011-01-20 11:26 Rafael J. Wysocki

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='AANLkTi=YwD7kBZ6XsnUpCfqCmTuwwhkcAYvjXJL3Fj8O@mail.gmail.com' \
    --to=jeff.chua.linux@gmail.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=mjg59@srcf.ucam.org \
    --cc=rjw@sisk.pl \
    /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.