From: Michael Buesch <mb@bu3sch.de>
To: Pavel Machek <pavel@ucw.cz>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>,
Andrew Morton <akpm@linux-foundation.org>,
pm list <linux-pm@lists.linux-foundation.org>,
Alan Stern <stern@rowland.harvard.edu>,
Len Brown <lenb@kernel.org>, LKML <linux-kernel@vger.kernel.org>,
bcm43xx-dev@lists.berlios.de,
Alessandro Rubini <rubini@vision.unipv.it>,
Richard Purdie <rpurdie@rpsys.net>
Subject: Re: [PATCH -mm 5/5] b43: Avoid unregistering device objects during suspend
Date: Fri, 25 Jan 2008 11:13:30 +0100 [thread overview]
Message-ID: <200801251113.31325.mb@bu3sch.de> (raw)
In-Reply-To: <20080125074746.GC4134@ucw.cz>
On Friday 25 January 2008 08:47:46 Pavel Machek wrote:
> On Fri 2008-01-25 01:37:33, Rafael J. Wysocki wrote:
> > From: Rafael J. Wysocki <rjw@sisk.pl>
> >
> > Modify the b43 driver to avoid deadlocking suspend and resume,
> > which happens as a result of attempting to unregister device objects
> > locked by the PM core during suspend/resume cycles. Also, make it
> > use a suspend-safe method of unregistering device object in the
> > resume error path.
> >
> > Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
> > Acked-by: Michael Buesch <mb@bu3sch.de>
>
> Maybe we should have global suspend_in_progress (or maybe system_state
> == suspending?) and automatically switch to schedule_removal() while
> it is set?
>
That would be great, from my perspective :)
--
Greetings Michael.
next prev parent reply other threads:[~2008-01-25 10:15 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-25 0:27 [PATCH -mm 0/5] b43: Fix suspend/resume deadlock Rafael J. Wysocki
2008-01-25 0:27 ` Rafael J. Wysocki
2008-01-25 0:30 ` [PATCH -mm 1/5] PM: Export device_pm_schedule_removal Rafael J. Wysocki
2008-01-25 0:30 ` Rafael J. Wysocki
2008-01-25 7:44 ` Pavel Machek
2008-01-25 7:44 ` Pavel Machek
2008-01-28 17:59 ` patch pm-export-device_pm_schedule_removal.patch added to gregkh-2.6 tree gregkh
2008-01-28 17:59 ` gregkh
2008-01-25 0:31 ` [PATCH -mm 2/5] Misc: Add possibility to remove misc devices during suspend/resume Rafael J. Wysocki
2008-01-25 0:31 ` Rafael J. Wysocki
2008-01-25 0:35 ` [PATCH -mm 3/5] HWRNG: Add possibility to remove hwrng " Rafael J. Wysocki
2008-01-25 0:35 ` Rafael J. Wysocki
2008-01-25 0:36 ` [PATCH -mm 4/5] Leds: Add possibility to remove leds classdevs " Rafael J. Wysocki
2008-01-25 0:36 ` Rafael J. Wysocki
2008-01-25 0:37 ` [PATCH -mm 5/5] b43: Avoid unregistering device objects during suspend Rafael J. Wysocki
2008-01-25 0:37 ` Rafael J. Wysocki
2008-01-25 7:47 ` Pavel Machek
2008-01-25 7:47 ` Pavel Machek
2008-01-25 10:13 ` Michael Buesch [this message]
2008-01-25 11:45 ` Rafael J. Wysocki
2008-01-25 11:45 ` Rafael J. Wysocki
2008-01-25 10:13 ` Michael Buesch
2008-01-25 14:58 ` Alan Stern
2008-01-25 14:58 ` Alan Stern
2008-01-25 21:16 ` Pavel Machek
2008-01-25 21:16 ` Pavel Machek
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=200801251113.31325.mb@bu3sch.de \
--to=mb@bu3sch.de \
--cc=akpm@linux-foundation.org \
--cc=bcm43xx-dev@lists.berlios.de \
--cc=lenb@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
--cc=rpurdie@rpsys.net \
--cc=rubini@vision.unipv.it \
--cc=stern@rowland.harvard.edu \
/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.