|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>|
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 firstname.lastname@example.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Copyright © 2009, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds