Re: [RFC][PATCH -mm] PM: Do not sync from within the freezer during suspend to RAM

Hi.

On Thursday 05 July 2007 08:49:42 Pavel Machek wrote:

On Thu 2007-07-05 08:48:15, Nigel Cunningham wrote:

quoted

Hi.

On Thursday 05 July 2007 00:58:58 Rafael J. Wysocki wrote:

quoted

From: Rafael J. Wysocki <redacted>

The syncing of filesystems from within the freezer in not needed for 

suspend 

quoted

to

quoted

RAM.  Change freeze_processes() so that it doesn't execute sys_sync() 

and

quoted

quoted

introduce the "syncing" version of it to be called from the hibernation 

code

quoted

quoted

paths.

Signed-off-by: Rafael J. Wysocki <redacted>
---
 include/linux/freezer.h |   14 ++++++++++++--
 kernel/power/disk.c     |    2 +-
 kernel/power/main.c     |    6 ++++++
 kernel/power/process.c  |    8 +++++---
 kernel/power/user.c     |    2 +-
 5 files changed, 25 insertions(+), 7 deletions(-)

Looks ok, except that I wonder if you want the following fragment. It 

looks to 

quoted

me (looking at rc6) like with this code, you'll currently call sys_sync 

twice 

quoted

when suspending to ram. Maybe I'm misreading it. Also, shouldn't it be 

done 

quoted

after taking the mutex?

sys_sync() should be okay to call, mutex or not.

Yeah. That wasn't my point, sorry. Calling sys_sync is pointless if you're 
going to fail to take the mutex. It makes more sense to know you've got it 
before you start doing things related to actually suspending.

Regards,

Nigel
-- 
See http://www.tuxonice.net for Howtos, FAQs, mailing
lists, wiki and bugzilla info.