From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: Re: [PATCH] swsusp: do not use pm_ops (was: Re: suspend2 merge (was: Re: CFS and suspend2: hang in atomic copy)) Date: Fri, 4 May 2007 23:55:18 +0200 Message-ID: <200705042355.19145.rjw@sisk.pl> References: <200705042311.23557.rjw@sisk.pl> <1178313830.7408.43.camel@johannes.berg> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1178313830.7408.43.camel@johannes.berg> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Johannes Berg Cc: linux-pm@lists.linux-foundation.org, Pekka Enberg , Nigel Cunningham , Pavel Machek List-Id: linux-pm@vger.kernel.org On Friday, 4 May 2007 23:23, Johannes Berg wrote: > On Fri, 2007-05-04 at 23:11 +0200, Rafael J. Wysocki wrote: > > > Actually, prethaw is to prevent drivers loaded before the image is restored > > from doing unreasonable things. It doesn't have any effect on the drivers' > > .resume() routines. > > Oh, but it can, you could have a flag in your driver saying "the next > resume is after restore" and you set that flag in prethaw. No, you should have set that flag in .suspend(), really. :-) Greetings, Rafael