On Tue, 8 Dec 2009, Rafael J. Wysocki wrote: > Suppose we use rwsem and during suspend each child uses a down_read() on a > parent and then the parent uses down_write() on itself. What if, whatever the > reason, the parent is a bit early and does the down_write() before one of the > children has a chance to do the down_read()? Aren't we toast? > > Do we need any direct protection against that or does it just work itself out > in a way I just don't see right now? That's not the way it should be done. Linus had children taking their parents' locks during suspend, which is simple but leads to difficulties. Instead, the PM core should do a down_write() on each device before starting the device's async suspend routine, and an up_write() when the routine finishes. Parents should, at the start of their async routine, do down_read() on each of their children plus whatever other devices they need to wait for. The core can do the waiting for children part and the driver's suspend routine can handle any other waiting. This is a little more awkward because it requires the parent to iterate through its children. But it does solve the off-tree dependency problem for suspends. Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
