[PATCH 07/10] drm/i915: Introduce struct intel_wakeref
Chris Wilson
chris at chris-wilson.co.uk
Thu Apr 4 23:27:37 UTC 2019
For controlling runtime pm of the GT and engines, we would like to have
a callback to do extra work the first time we wake up and the last time
we drop the wakeref. This first/last access needs serialisation and so
we encompass a mutex with the regular intel_wakeref_t tracker.
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
drivers/gpu/drm/i915/Makefile | 1 +
drivers/gpu/drm/i915/Makefile.header-test | 3 +-
drivers/gpu/drm/i915/i915_drv.h | 3 +-
drivers/gpu/drm/i915/intel_wakeref.c | 47 ++++++++++++++++++
drivers/gpu/drm/i915/intel_wakeref.h | 59 +++++++++++++++++++++++
5 files changed, 110 insertions(+), 3 deletions(-)
create mode 100644 drivers/gpu/drm/i915/intel_wakeref.c
create mode 100644 drivers/gpu/drm/i915/intel_wakeref.h
diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
index 536ec3de47a6..9f6150b55168 100644
--- a/drivers/gpu/drm/i915/Makefile
+++ b/drivers/gpu/drm/i915/Makefile
@@ -50,6 +50,7 @@ i915-y += i915_drv.o \
intel_device_info.o \
intel_pm.o \
intel_runtime_pm.o \
+ intel_wakeref.o \
intel_uncore.o
# core library code
diff --git a/drivers/gpu/drm/i915/Makefile.header-test b/drivers/gpu/drm/i915/Makefile.header-test
index 4354db53fa19..14dbcfb5b5ea 100644
--- a/drivers/gpu/drm/i915/Makefile.header-test
+++ b/drivers/gpu/drm/i915/Makefile.header-test
@@ -7,7 +7,8 @@ header_test := \
i915_gem_context_types.h \
i915_priolist_types.h \
i915_scheduler_types.h \
- i915_timeline_types.h
+ i915_timeline_types.h \
+ intel_wakeref.h
quiet_cmd_header_test = HDRTEST $@
cmd_header_test = echo "\#include \"$(<F)\"" > $@
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index d45affca64b3..9636a388a5c1 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -72,6 +72,7 @@
#include "intel_dpll_mgr.h"
#include "intel_opregion.h"
#include "intel_uncore.h"
+#include "intel_wakeref.h"
#include "intel_wopcm.h"
#include "intel_uc.h"
@@ -133,8 +134,6 @@ bool i915_error_injected(void);
__i915_printk(i915, i915_error_injected() ? KERN_DEBUG : KERN_ERR, \
fmt, ##__VA_ARGS__)
-typedef depot_stack_handle_t intel_wakeref_t;
-
enum hpd_pin {
HPD_NONE = 0,
HPD_TV = HPD_NONE, /* TV is known to be unreliable */
diff --git a/drivers/gpu/drm/i915/intel_wakeref.c b/drivers/gpu/drm/i915/intel_wakeref.c
new file mode 100644
index 000000000000..34c184fb57e5
--- /dev/null
+++ b/drivers/gpu/drm/i915/intel_wakeref.c
@@ -0,0 +1,47 @@
+/*
+ * SPDX-License-Identifier: MIT
+ *
+ * Copyright © 2019 Intel Corporation
+ */
+
+#include "intel_drv.h"
+#include "intel_wakeref.h"
+
+void __intel_wakeref_get_once(struct drm_i915_private *i915,
+ struct intel_wakeref_count *wc,
+ bool (*fn)(struct intel_wakeref_count *wc))
+{
+ mutex_lock(&wc->mutex);
+ if (!atomic_read(&wc->count)) {
+ wc->wakeref = intel_runtime_pm_get(i915);
+ if (unlikely(!fn(wc))) {
+ intel_runtime_pm_put(i915, wc->wakeref);
+ mutex_unlock(&wc->mutex);
+ return;
+ }
+
+ smp_mb__before_atomic(); /* release wc->count */
+ }
+ atomic_inc(&wc->count);
+ mutex_unlock(&wc->mutex);
+}
+
+void __intel_wakeref_put_once(struct drm_i915_private *i915,
+ struct intel_wakeref_count *wc,
+ bool (*fn)(struct intel_wakeref_count *wc))
+{
+ if (likely(fn(wc)))
+ intel_runtime_pm_put(i915, wc->wakeref);
+ mutex_unlock(&wc->mutex);
+}
+
+void intel_wakeref_init(struct intel_wakeref_count *wc,
+ enum intel_wakeref_class class)
+{
+ BUILD_BUG_ON(__INTEL_WAKEREF_NUM_CLASSES >= MAX_LOCKDEP_SUBCLASSES);
+
+ mutex_init(&wc->mutex);
+ lockdep_set_subclass(&wc->mutex, class);
+
+ atomic_set(&wc->count, 0);
+}
diff --git a/drivers/gpu/drm/i915/intel_wakeref.h b/drivers/gpu/drm/i915/intel_wakeref.h
new file mode 100644
index 000000000000..2c414453489c
--- /dev/null
+++ b/drivers/gpu/drm/i915/intel_wakeref.h
@@ -0,0 +1,59 @@
+/*
+ * SPDX-License-Identifier: MIT
+ *
+ * Copyright © 2019 Intel Corporation
+ */
+
+#ifndef INTEL_WAKEREF_H
+#define INTEL_WAKEREF_H
+
+#include <linux/atomic.h>
+#include <linux/mutex.h>
+#include <linux/stackdepot.h>
+
+struct drm_i915_private;
+
+typedef depot_stack_handle_t intel_wakeref_t;
+
+struct intel_wakeref_count {
+ atomic_t count;
+ struct mutex mutex;
+ intel_wakeref_t wakeref;
+};
+
+enum intel_wakeref_class {
+ INTEL_WAKEREF_NONE = 0,
+ __INTEL_WAKEREF_NUM_CLASSES
+};
+
+void intel_wakeref_init(struct intel_wakeref_count *wc,
+ enum intel_wakeref_class class);
+
+void __intel_wakeref_get_once(struct drm_i915_private *i915,
+ struct intel_wakeref_count *wc,
+ bool (*fn)(struct intel_wakeref_count *wc));
+void __intel_wakeref_put_once(struct drm_i915_private *i915,
+ struct intel_wakeref_count *wc,
+ bool (*fn)(struct intel_wakeref_count *wc));
+
+static inline void
+intel_wakeref_get_once(struct drm_i915_private *i915,
+ struct intel_wakeref_count *wc,
+ bool (*fn)(struct intel_wakeref_count *wc))
+{
+ might_lock(&wc->mutex);
+ if (unlikely(!atomic_inc_not_zero(&wc->count)))
+ __intel_wakeref_get_once(i915, wc, fn);
+}
+
+static inline void
+intel_wakeref_put_once(struct drm_i915_private *i915,
+ struct intel_wakeref_count *wc,
+ bool (*fn)(struct intel_wakeref_count *wc))
+{
+ might_lock(&wc->mutex);
+ if (atomic_dec_and_mutex_lock(&wc->count, &wc->mutex))
+ __intel_wakeref_put_once(i915, wc, fn);
+}
+
+#endif /* INTEL_WAKEREF_H */
--
2.20.1
More information about the Intel-gfx-trybot
mailing list