Re: [PATCH 0/5] PM: Enable option of re-use runtime PM callbacks atsystem suspend

From: Alan Stern
Date: Fri Nov 29 2013 - 10:31:10 EST


On Fri, 29 Nov 2013, Rafael J. Wysocki wrote:

> That should have been
>
> driver->runtime_suspend(dev)
> do_X(dev)
>
> because do_Y(dev) is for runtime suspend. Sorry.
>
> And of course, the subsystem-level code you're developing the driver for may not
> do the do_X(dev) thing at all, in which case all will work. But what if someone
> tries to use the driver with a different subsystem-level code (like a new PM
> domain)?

It sounds like the problem is one of division of responsibilities.
What part of the work is done by the subsystem, and what part is done
by the driver? Obviously the answer will vary from one subsystem to
another.

As I understand it, in Ulf's situation the subsystem knows that the
operations needed for the suspend_late phase of system suspend are the
same as those carried out by the driver's runtime_suspend callback (if
the device isn't already in a runtime-PM low-power state). Or perhaps
it knows they are the same as those carried out by the subsystem's
runtime_suspend callback. Or perhaps the _driver_ knows that the
operations it needs for suspend_late are the same as those carried out
by its own runtime_suspend callback (if the device isn't already in a
low-power state).

Regardless, under those conditions the suspend_late routine merely has
to invoke the appropriate runtime_suspend routine, after checking the
device's runtime PM status. We can't do this by calling
pm_runtime_suspend, because runtime PM is disabled during suspend_late.
Instead, we have to call the proper runtime-suspend routine directly.

The amount of code needed is quite small; basically nothing more than:

if (!pm_runtime_status_suspended(dev))
dev->driver->pm->runtime_suspend(dev);

The problems are:

1. Which callback does this code belong in?

2. Which runtime_suspend callback should be invoked? The example
above uses dev->driver->pm->runtime_suspend, but this may not
always be the right one.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/