From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Jeff Chua <jeff.chua.linux@gmail.com>
Cc: ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
Linux-pm mailing list <linux-pm@lists.linux-foundation.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/11] ACPI: Fixes and cleanups related to iomaps management
Date: Sat, 22 Jan 2011 10:25:49 +0100 [thread overview]
Message-ID: <201101221025.49327.rjw__47585.3094518699$1295688425$gmane$org@sisk.pl> (raw)
In-Reply-To: <AANLkTin3d4LdnqDYtrTK=KdmjADA2MsaWYoLG19FxOzG@mail.gmail.com>
On Saturday, January 22, 2011, Jeff Chua wrote:
> On Sat, Jan 22, 2011 at 1:54 PM, Jeff Chua <jeff.chua.linux@gmail.com> wrote:
> > 2011/1/22 Rafael J. Wysocki <rjw@sisk.pl>:
> >> On Friday, January 21, 2011, Jeff Chua wrote:
> >>> 2011/1/21 Rafael J. Wysocki <rjw@sisk.pl>:
> >>> > Thanks, but unfortunately this wasn't conclusive. Please apply the patch below
> >>> > instead of the previous one (on top of [1/11] - [11/11]) and collect dmesg
> >> So, below is a replacement for [11/11]. Please test it on top of
> >> [1/11] - [10/11] (the current Linus' tree already contains [1/11] and [2/11])
> >> and let me know if it works for you (in either case, please attach dmesg
> >> output containing a suspend attempt).
> >>
> >> If it works, I'll remove the diagnostic messages and submit along with the
> >> rest of the patchset.
> >
> > Rafael,
> >
> > dmesg attached. This time, it worked!
> >
> > I discovered CONFIG_THINKPAD_ACPI caused suspend-to-disk to hang. I
> > need the Thinkpad ACPI to control the fan and bluetooth. It looks like
> > the thinkpad acpi is trying acquire locks while suspending. Disabling
> > cmos, light, led and hotkeys makes suspend-to-disk works again.
>
> Here's the dmesg.
Thanks!
Well, and this:
Brought up 4 CPUs
2011-01-22T11:32:56.424328+08:00 boston kernel: Total of 4 processors activated (17023.83 BogoMIPS).
2011-01-22T11:32:56.424329+08:00 boston kernel: PM: Registering ACPI NVS region at bb371000 (528384 bytes)
2011-01-22T11:32:56.424332+08:00 boston kernel: PM: Registering ACPI NVS region at bb668000 (524288 bytes)
2011-01-22T11:32:56.424334+08:00 boston kernel: PM: Registering ACPI NVS region at bb76b000 (49152 bytes)
2011-01-22T11:32:56.424335+08:00 boston kernel: PM: Registering ACPI NVS region at bb77a000 (28672 bytes)
2011-01-22T11:32:56.424337+08:00 boston kernel: PM: Registering ACPI NVS region at bb782000 (36864 bytes)
2011-01-22T11:32:56.424338+08:00 boston kernel: PM: Registering ACPI NVS region at bb78c000 (77824 bytes)
2011-01-22T11:32:56.424339+08:00 boston kernel: NET: Registered protocol family 16
means that my patch [11/11] _and_ the replacement are totally wrong, because you have
_multiple_ NVS regions. That really helped, thanks a lot.
Rafael
next prev parent reply other threads:[~2011-01-22 9:25 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
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 [this message]
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='201101221025.49327.rjw__47585.3094518699$1295688425$gmane$org@sisk.pl' \
--to=rjw@sisk.pl \
--cc=jeff.chua.linux@gmail.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
/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.