User: Password:
|
|
Subscribe / Log in / New account

Re: Async suspend-resume patch w/ completions (was: Re: Async suspend-resume patch w/ rwsems)

From:  Linus Torvalds <torvalds-AT-linux-foundation.org>
To:  "Rafael J. Wysocki" <rjw-AT-sisk.pl>
Subject:  Re: Async suspend-resume patch w/ completions (was: Re: Async suspend-resume patch w/ rwsems)
Date:  Fri, 11 Dec 2009 15:53:14 -0800 (PST)
Cc:  Alan Stern <stern-AT-rowland.harvard.edu>, Zhang Rui <rui.zhang-AT-intel.com>, LKML <linux-kernel-AT-vger.kernel.org>, ACPI Devel Maling List <linux-acpi-AT-vger.kernel.org>, pm list <linux-pm-AT-lists.linux-foundation.org>
Archive-link:  Article, Thread



On Sat, 12 Dec 2009, Rafael J. Wysocki wrote:
> 
> Below is a patch I've just tested, but there's a lockdep problem in it I don't
> know how to solve.  Namely, lockdep is apparently unhappy with us not releasing
> the lock taken in device_suspend() and it complains we take it twice in a row
> (which we do, but for another device).  I need to use down_read_non_owner()
> to make it shut up and then I also need to use up_read_non_owner() in
> __device_suspend(),

Ok, that I admit is actually a problem.

Ok, ok, I'll accept that completion() version, even though I think it's 
inferior.

		Linus
--
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



(Log in to post comments)


Copyright © 2009, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds