[Intel-gfx] [PATCH v2] PM / Runtime: Introduce pm_runtime_get_noidle
imre.deak at intel.com
Thu Dec 10 01:43:50 PST 2015
On Thu, 2015-12-10 at 01:58 +0100, Rafael J. Wysocki wrote:
> On Wednesday, December 09, 2015 06:22:19 PM Joonas Lahtinen wrote:
> > Introduce pm_runtime_get_noidle to for situations where it is not
> > desireable to touch an idling device. One use scenario is periodic
> > hangchecks performed by the drm/i915 driver which can be omitted
> > on a device in a runtime idle state.
> > v2:
> > - Fix inconsistent return value when !CONFIG_PM.
> > - Update documentation for bool return value
> > Signed-off-by: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> > Reported-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: "Rafael J. Wysocki" <rjw at rjwysocki.net>
> > Cc: linux-pm at vger.kernel.org
> Well, I don't quite see how this can be used in a non-racy way
> without doing an additional pm_runtime_resume() or something like
> that in the same code path.
We don't want to resume, that would be the whole point. We'd like to
ensure that we hold a reference _and_ the device is already active. So
AFAICS we'd need to check runtime_status == RPM_ACTIVE in addition
after taking the reference.
More information about the Intel-gfx