Re: [linux-pm] Re: [PATCH] Remove process freezer from suspend to RAM pathway

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Alan Stern
Date: Thursday, July 5, 2007 - 7:42 am

On Thu, 5 Jul 2007, Paul Mackerras wrote:


The example I quoted earlier about binding during a suspend will do.  I 
agree that we can and should try to prevent it from ever occurring.

Read and write are a problem only in that fixing them would potentially
involve changing lots of drivers; I don't think they pose a serious
theoretical obstacle.  (Lord knows what will happen with async I/O!)

Any other entry points to drivers are also potential problems, but it's 
hard to say anything definite about them since they are so varied.


Yes.


Getting the PM core to resume a device before probing could be 
difficult; in general it doesn't know enough about specific device 
behaviors to do something like that.  But the subsystem certainly ought 
to take care of it.  USB does.

Yes, bind/unbind/etc. should be deferred during a system suspend.  But
it has to be done carefully, because these operations generally involve
locks that can't be released.  They need to be prevented at their
source, not in the driver core.  That's one reason why khubd needs to
be frozen (being part of the USB hub driver, it is the task responsible
for binding and unbinding drivers to USB devices).

Another thing to look out for is registration and unregistration of 
drivers.  These activities also cause bind/unbind operations.  Note 
that if userspace is frozen then neither insmod nor rmmod can run.  :-)


All I/O to sysfs attributes passes through the routines in fs/sysfs/*.  
It could be blocked there.  (But if userspace is frozen it won't need 
to be.)


That's certainly true of block devices, whose drivers use the block 
subsystem.  It's not true for lots of other devices, though.


In my experience, most non-block drivers do not have any queue of 
pending I/O operations.  They simply carry out requests as they arrive.

Alan Stern

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Remove process freezer from suspend to RAM pathway, Matthew Garrett, (Mon Jul 2, 9:29 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 2, 10:48 pm)
Re: [linux-pm] [PATCH] Remove process freezer from suspend ..., Benjamin Herrenschmidt, (Mon Jul 2, 10:49 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 2, 10:51 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 12:19 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Tue Jul 3, 4:40 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Tue Jul 3, 4:44 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Tue Jul 3, 4:45 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:14 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:16 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:20 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:35 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:35 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 2:56 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Tue Jul 3, 4:08 pm)
The big suspend mess, Adrian Bunk, (Wed Jul 4, 3:19 pm)
Re: The big suspend mess, Pavel Machek, (Wed Jul 4, 5:27 pm)
Re: The big suspend mess, Paul Mackerras, (Wed Jul 4, 5:53 pm)
Re: The big suspend mess, Adrian Bunk, (Wed Jul 4, 6:22 pm)
Re: The big suspend mess, Pavel Machek, (Thu Jul 5, 2:32 am)
Re: The big suspend mess, Gabriel C, (Thu Jul 5, 3:29 am)
Re: The big suspend mess, Rafael J. Wysocki, (Thu Jul 5, 5:18 am)
Re: [linux-pm] The big suspend mess, Alan Stern, (Thu Jul 5, 7:14 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Alan Stern, (Thu Jul 5, 7:42 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Jeremy Maitin-Shepard, (Thu Jul 5, 9:06 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Thu Jul 5, 3:38 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 3:46 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Thu Jul 5, 3:59 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Thu Jul 5, 4:05 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 4:20 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 8:54 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Jeremy Maitin-Shepard, (Thu Jul 5, 8:59 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 8:59 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Thu Jul 5, 9:41 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:04 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:13 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:16 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Rafael J. Wysocki, (Fri Jul 6, 12:20 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Fri Jul 6, 12:35 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Fri Jul 6, 12:40 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Fri Jul 6, 1:59 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Fri Jul 6, 2:03 am)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Fri Jul 6, 7:44 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Fri Jul 6, 7:46 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Fri Jul 6, 8:44 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Fri Jul 6, 9:06 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 5:40 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 5:42 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 5:48 pm)
Re: [linux-pm] Re: [PATCH] Remove process freezer from sus ..., Benjamin Herrenschmidt, (Sat Jul 7, 5:50 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 9:39 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 10:14 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sat Jul 7, 10:19 pm)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Rafael J. Wysocki, (Sun Jul 8, 7:06 am)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Rafael J. Wysocki, (Sun Jul 8, 11:08 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 11:20 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 11:26 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 11:46 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Rafael J. Wysocki, (Sun Jul 8, 12:15 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:03 pm)
Re: malicious filesystems (was Re: [linux-pm] Re: [PATCH] ..., Rafael J. Wysocki, (Sun Jul 8, 2:07 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:20 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:21 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 2:54 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 5:33 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Sun Jul 8, 6:32 pm)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 2:13 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 2:14 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 2:33 am)
Re: [PATCH] Remove process freezer from suspend to RAM pathway, Benjamin Herrenschmidt, (Mon Jul 9, 3:05 am)
Re: hibernation/snapshot design, Jeremy Maitin-Shepard, (Mon Jul 9, 8:23 am)
Re: sysrq-t dumps of s2ram/fuse deadlock, Jeremy Maitin-Shepard, (Wed Jul 11, 6:45 am)