Patrik Jakobsson b8e0c7
From 638d40bb4f3d14ba693af7cdb74b6649a45f88a6 Mon Sep 17 00:00:00 2001
Patrik Jakobsson b8e0c7
From: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
Patrik Jakobsson b8e0c7
Date: Fri, 30 Jul 2021 13:21:06 -0700
Patrik Jakobsson b8e0c7
Subject: drm/i915/guc/slpc: Initial definitions for SLPC
Patrik Jakobsson b8e0c7
Git-commit: dff0fc4990929858eccab824bd310e7fb4bb20ee
Patrik Jakobsson b8e0c7
Patch-mainline: v5.15-rc1
Patrik Jakobsson b8e0c7
References: jsc#SLE-22601
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
Add macros to check for SLPC support. This feature is currently supported
Patrik Jakobsson b8e0c7
for Gen12+ and enabled whenever GuC submission is enabled/selected.
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
Include templates for SLPC init/fini and enable.
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
v2: Move SLPC helper functions to intel_guc_slpc.c/.h. Define
Patrik Jakobsson b8e0c7
basic template for SLPC structure in intel_guc_slpc_types.h.
Patrik Jakobsson b8e0c7
Fix copyright (Michal W)
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
v3: Review comments (Michal W)
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
v4: Include supported/selected inside slpc struct (Michal W)
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Patrik Jakobsson b8e0c7
Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
Patrik Jakobsson b8e0c7
Signed-off-by: Sundaresan Sujaritha <sujaritha.sundaresan@intel.com>
Patrik Jakobsson b8e0c7
Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Patrik Jakobsson b8e0c7
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Patrik Jakobsson b8e0c7
Link: https://patchwork.freedesktop.org/patch/msgid/20210730202119.23810-2-vinay.belgaumkar@intel.com
Patrik Jakobsson b8e0c7
Signed-off-by: Patrik Jakobsson <pjakobsson@suse.de>
Patrik Jakobsson b8e0c7
---
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/Makefile                 |  1 +
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/gt/uc/intel_guc.c        |  2 +
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/gt/uc/intel_guc.h        |  2 +
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c   | 45 +++++++++++++++++++
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h   | 33 ++++++++++++++
Patrik Jakobsson b8e0c7
 .../gpu/drm/i915/gt/uc/intel_guc_slpc_types.h | 16 +++++++
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/gt/uc/intel_uc.c         |  6 ++-
Patrik Jakobsson b8e0c7
 drivers/gpu/drm/i915/gt/uc/intel_uc.h         |  2 +
Patrik Jakobsson b8e0c7
 8 files changed, 105 insertions(+), 2 deletions(-)
Patrik Jakobsson b8e0c7
 create mode 100644 drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
Patrik Jakobsson b8e0c7
 create mode 100644 drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h
Patrik Jakobsson b8e0c7
 create mode 100644 drivers/gpu/drm/i915/gt/uc/intel_guc_slpc_types.h
Patrik Jakobsson b8e0c7
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
Patrik Jakobsson b8e0c7
index a985c92c8c19..134b0edf4a2e 100644
Patrik Jakobsson b8e0c7
--- a/drivers/gpu/drm/i915/Makefile
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/Makefile
Patrik Jakobsson b8e0c7
@@ -186,6 +186,7 @@ i915-y += gt/uc/intel_uc.o \
Patrik Jakobsson b8e0c7
 	  gt/uc/intel_guc_fw.o \
Patrik Jakobsson b8e0c7
 	  gt/uc/intel_guc_log.o \
Patrik Jakobsson b8e0c7
 	  gt/uc/intel_guc_log_debugfs.o \
Patrik Jakobsson b8e0c7
+	  gt/uc/intel_guc_slpc.o \
Patrik Jakobsson b8e0c7
 	  gt/uc/intel_guc_submission.o \
Patrik Jakobsson b8e0c7
 	  gt/uc/intel_huc.o \
Patrik Jakobsson b8e0c7
 	  gt/uc/intel_huc_debugfs.o \
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
Patrik Jakobsson b8e0c7
index 979128e28372..39bc3c16057b 100644
Patrik Jakobsson b8e0c7
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.c
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.c
Patrik Jakobsson b8e0c7
@@ -7,6 +7,7 @@
Patrik Jakobsson b8e0c7
 #include "gt/intel_gt_irq.h"
Patrik Jakobsson b8e0c7
 #include "gt/intel_gt_pm_irq.h"
Patrik Jakobsson b8e0c7
 #include "intel_guc.h"
Patrik Jakobsson b8e0c7
+#include "intel_guc_slpc.h"
Patrik Jakobsson b8e0c7
 #include "intel_guc_ads.h"
Patrik Jakobsson b8e0c7
 #include "intel_guc_submission.h"
Patrik Jakobsson b8e0c7
 #include "i915_drv.h"
Patrik Jakobsson b8e0c7
@@ -157,6 +158,7 @@ void intel_guc_init_early(struct intel_guc *guc)
Patrik Jakobsson b8e0c7
 	intel_guc_ct_init_early(&guc->ct);
Patrik Jakobsson b8e0c7
 	intel_guc_log_init_early(&guc->log);
Patrik Jakobsson b8e0c7
 	intel_guc_submission_init_early(guc);
Patrik Jakobsson b8e0c7
+	intel_guc_slpc_init_early(&guc->slpc);
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
 	mutex_init(&guc->send_mutex);
Patrik Jakobsson b8e0c7
 	spin_lock_init(&guc->irq_lock);
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
Patrik Jakobsson b8e0c7
index a9547069ee7e..7da11a0b6059 100644
Patrik Jakobsson b8e0c7
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc.h
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc.h
Patrik Jakobsson b8e0c7
@@ -15,6 +15,7 @@
Patrik Jakobsson b8e0c7
 #include "intel_guc_ct.h"
Patrik Jakobsson b8e0c7
 #include "intel_guc_log.h"
Patrik Jakobsson b8e0c7
 #include "intel_guc_reg.h"
Patrik Jakobsson b8e0c7
+#include "intel_guc_slpc_types.h"
Patrik Jakobsson b8e0c7
 #include "intel_uc_fw.h"
Patrik Jakobsson b8e0c7
 #include "i915_utils.h"
Patrik Jakobsson b8e0c7
 #include "i915_vma.h"
Patrik Jakobsson b8e0c7
@@ -30,6 +31,7 @@ struct intel_guc {
Patrik Jakobsson b8e0c7
 	struct intel_uc_fw fw;
Patrik Jakobsson b8e0c7
 	struct intel_guc_log log;
Patrik Jakobsson b8e0c7
 	struct intel_guc_ct ct;
Patrik Jakobsson b8e0c7
+	struct intel_guc_slpc slpc;
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
 	/* Global engine used to submit requests to GuC */
Patrik Jakobsson b8e0c7
 	struct i915_sched_engine *sched_engine;
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
Patrik Jakobsson b8e0c7
new file mode 100644
Patrik Jakobsson b8e0c7
index 000000000000..40950f1bf05c
Patrik Jakobsson b8e0c7
--- /dev/null
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
Patrik Jakobsson b8e0c7
@@ -0,0 +1,45 @@
Patrik Jakobsson b8e0c7
+// SPDX-License-Identifier: MIT
Patrik Jakobsson b8e0c7
+/*
Patrik Jakobsson b8e0c7
+ * Copyright © 2021 Intel Corporation
Patrik Jakobsson b8e0c7
+ */
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#include "i915_drv.h"
Patrik Jakobsson b8e0c7
+#include "intel_guc_slpc.h"
Patrik Jakobsson b8e0c7
+#include "gt/intel_gt.h"
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+static inline struct intel_guc *slpc_to_guc(struct intel_guc_slpc *slpc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	return container_of(slpc, struct intel_guc, slpc);
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+static bool __detect_slpc_supported(struct intel_guc *guc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	/* GuC SLPC is unavailable for pre-Gen12 */
Patrik Jakobsson b8e0c7
+	return guc->submission_supported &&
Patrik Jakobsson b8e0c7
+		GRAPHICS_VER(guc_to_gt(guc)->i915) >= 12;
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+static bool __guc_slpc_selected(struct intel_guc *guc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	if (!intel_guc_slpc_is_supported(guc))
Patrik Jakobsson b8e0c7
+		return false;
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+	return guc->submission_selected;
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+void intel_guc_slpc_init_early(struct intel_guc_slpc *slpc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	struct intel_guc *guc = slpc_to_guc(slpc);
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+	slpc->supported = __detect_slpc_supported(guc);
Patrik Jakobsson b8e0c7
+	slpc->selected = __guc_slpc_selected(guc);
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+int intel_guc_slpc_init(struct intel_guc_slpc *slpc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	return 0;
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+void intel_guc_slpc_fini(struct intel_guc_slpc *slpc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h
Patrik Jakobsson b8e0c7
new file mode 100644
Patrik Jakobsson b8e0c7
index 000000000000..bc139682ad0f
Patrik Jakobsson b8e0c7
--- /dev/null
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.h
Patrik Jakobsson b8e0c7
@@ -0,0 +1,33 @@
Patrik Jakobsson b8e0c7
+/* SPDX-License-Identifier: MIT */
Patrik Jakobsson b8e0c7
+/*
Patrik Jakobsson b8e0c7
+ * Copyright © 2021 Intel Corporation
Patrik Jakobsson b8e0c7
+ */
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#ifndef _INTEL_GUC_SLPC_H_
Patrik Jakobsson b8e0c7
+#define _INTEL_GUC_SLPC_H_
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#include "intel_guc_submission.h"
Patrik Jakobsson b8e0c7
+#include "intel_guc_slpc_types.h"
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+static inline bool intel_guc_slpc_is_supported(struct intel_guc *guc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	return guc->slpc.supported;
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+static inline bool intel_guc_slpc_is_wanted(struct intel_guc *guc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	return guc->slpc.selected;
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+static inline bool intel_guc_slpc_is_used(struct intel_guc *guc)
Patrik Jakobsson b8e0c7
+{
Patrik Jakobsson b8e0c7
+	return intel_guc_submission_is_used(guc) && intel_guc_slpc_is_wanted(guc);
Patrik Jakobsson b8e0c7
+}
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+void intel_guc_slpc_init_early(struct intel_guc_slpc *slpc);
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+int intel_guc_slpc_init(struct intel_guc_slpc *slpc);
Patrik Jakobsson b8e0c7
+int intel_guc_slpc_enable(struct intel_guc_slpc *slpc);
Patrik Jakobsson b8e0c7
+void intel_guc_slpc_fini(struct intel_guc_slpc *slpc);
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#endif
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc_types.h b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc_types.h
Patrik Jakobsson b8e0c7
new file mode 100644
Patrik Jakobsson b8e0c7
index 000000000000..769c162305a0
Patrik Jakobsson b8e0c7
--- /dev/null
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc_types.h
Patrik Jakobsson b8e0c7
@@ -0,0 +1,16 @@
Patrik Jakobsson b8e0c7
+/* SPDX-License-Identifier: MIT */
Patrik Jakobsson b8e0c7
+/*
Patrik Jakobsson b8e0c7
+ * Copyright © 2021 Intel Corporation
Patrik Jakobsson b8e0c7
+ */
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#ifndef _INTEL_GUC_SLPC_TYPES_H_
Patrik Jakobsson b8e0c7
+#define _INTEL_GUC_SLPC_TYPES_H_
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#include <linux/types.h>
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+struct intel_guc_slpc {
Patrik Jakobsson b8e0c7
+	bool supported;
Patrik Jakobsson b8e0c7
+	bool selected;
Patrik Jakobsson b8e0c7
+};
Patrik Jakobsson b8e0c7
+
Patrik Jakobsson b8e0c7
+#endif
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc.c b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
Patrik Jakobsson b8e0c7
index da57d18d9f6b..e6bd9406c7b2 100644
Patrik Jakobsson b8e0c7
--- a/drivers/gpu/drm/i915/gt/uc/intel_uc.c
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
Patrik Jakobsson b8e0c7
@@ -75,16 +75,18 @@ static void __confirm_options(struct intel_uc *uc)
Patrik Jakobsson b8e0c7
 	struct drm_i915_private *i915 = uc_to_gt(uc)->i915;
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
 	drm_dbg(&i915->drm,
Patrik Jakobsson b8e0c7
-		"enable_guc=%d (guc:%s submission:%s huc:%s)\n",
Patrik Jakobsson b8e0c7
+		"enable_guc=%d (guc:%s submission:%s huc:%s slpc:%s)\n",
Patrik Jakobsson b8e0c7
 		i915->params.enable_guc,
Patrik Jakobsson b8e0c7
 		yesno(intel_uc_wants_guc(uc)),
Patrik Jakobsson b8e0c7
 		yesno(intel_uc_wants_guc_submission(uc)),
Patrik Jakobsson b8e0c7
-		yesno(intel_uc_wants_huc(uc)));
Patrik Jakobsson b8e0c7
+		yesno(intel_uc_wants_huc(uc)),
Patrik Jakobsson b8e0c7
+		yesno(intel_uc_wants_guc_slpc(uc)));
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
 	if (i915->params.enable_guc == 0) {
Patrik Jakobsson b8e0c7
 		GEM_BUG_ON(intel_uc_wants_guc(uc));
Patrik Jakobsson b8e0c7
 		GEM_BUG_ON(intel_uc_wants_guc_submission(uc));
Patrik Jakobsson b8e0c7
 		GEM_BUG_ON(intel_uc_wants_huc(uc));
Patrik Jakobsson b8e0c7
+		GEM_BUG_ON(intel_uc_wants_guc_slpc(uc));
Patrik Jakobsson b8e0c7
 		return;
Patrik Jakobsson b8e0c7
 	}
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc.h b/drivers/gpu/drm/i915/gt/uc/intel_uc.h
Patrik Jakobsson b8e0c7
index e2da2b6e76e1..925a58ca6b94 100644
Patrik Jakobsson b8e0c7
--- a/drivers/gpu/drm/i915/gt/uc/intel_uc.h
Patrik Jakobsson b8e0c7
+++ b/drivers/gpu/drm/i915/gt/uc/intel_uc.h
Patrik Jakobsson b8e0c7
@@ -8,6 +8,7 @@
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
 #include "intel_guc.h"
Patrik Jakobsson b8e0c7
 #include "intel_guc_submission.h"
Patrik Jakobsson b8e0c7
+#include "intel_guc_slpc.h"
Patrik Jakobsson b8e0c7
 #include "intel_huc.h"
Patrik Jakobsson b8e0c7
 #include "i915_params.h"
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
@@ -83,6 +84,7 @@ __uc_state_checker(x, func, uses, used)
Patrik Jakobsson b8e0c7
 uc_state_checkers(guc, guc);
Patrik Jakobsson b8e0c7
 uc_state_checkers(huc, huc);
Patrik Jakobsson b8e0c7
 uc_state_checkers(guc, guc_submission);
Patrik Jakobsson b8e0c7
+uc_state_checkers(guc, guc_slpc);
Patrik Jakobsson b8e0c7
 
Patrik Jakobsson b8e0c7
 #undef uc_state_checkers
Patrik Jakobsson b8e0c7
 #undef __uc_state_checker
Patrik Jakobsson b8e0c7
-- 
Patrik Jakobsson b8e0c7
2.33.1
Patrik Jakobsson b8e0c7