Re: [Intel-gfx] [PATCH 1/2] drm/i915: Extract GT powermanagement interrupt handling

2019-08-12 Thread Chris Wilson
Quoting Chris Wilson (2019-08-11 15:28:00)
> From: Andi Shyti 
> 
> i915_irq.c is large. It serves as the central dispatch and handler for
> all of our device interrupts. Pull out the GT pm interrupt handling
> (leaving the central dispatch) so that we can encapsulate the logic a
> little better.
> 
> Based on a patch by Chris Wilson.
> 
> Signed-off-by: Andi Shyti 
Reviewed-by: Chris Wilson 
-Chris
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH 1/2] drm/i915: Extract GT powermanagement interrupt handling

2019-08-11 Thread Chris Wilson
From: Andi Shyti 

i915_irq.c is large. It serves as the central dispatch and handler for
all of our device interrupts. Pull out the GT pm interrupt handling
(leaving the central dispatch) so that we can encapsulate the logic a
little better.

Based on a patch by Chris Wilson.

Signed-off-by: Andi Shyti 
---
 drivers/gpu/drm/i915/Makefile  |   1 +
 drivers/gpu/drm/i915/gt/intel_gt.c |   4 +-
 drivers/gpu/drm/i915/gt/intel_gt_pm_irq.c  | 108 ++
 drivers/gpu/drm/i915/gt/intel_gt_pm_irq.h  |  22 +++
 drivers/gpu/drm/i915/gt/intel_gt_types.h   |   3 +-
 drivers/gpu/drm/i915/gt/intel_ringbuffer.c |   5 +-
 drivers/gpu/drm/i915/i915_irq.c| 220 +
 drivers/gpu/drm/i915/i915_irq.h|   2 -
 8 files changed, 191 insertions(+), 174 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/gt/intel_gt_pm_irq.c
 create mode 100644 drivers/gpu/drm/i915/gt/intel_gt_pm_irq.h

diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
index 3962d9728dd7..d226659fdbb3 100644
--- a/drivers/gpu/drm/i915/Makefile
+++ b/drivers/gpu/drm/i915/Makefile
@@ -80,6 +80,7 @@ gt-y += \
gt/intel_engine_user.o \
gt/intel_gt.o \
gt/intel_gt_pm.o \
+   gt/intel_gt_pm_irq.o \
gt/intel_hangcheck.o \
gt/intel_lrc.o \
gt/intel_renderstate.o \
diff --git a/drivers/gpu/drm/i915/gt/intel_gt.c 
b/drivers/gpu/drm/i915/gt/intel_gt.c
index c543467a8a1c..914bd2db3bc7 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt.c
@@ -13,9 +13,11 @@ void intel_gt_init_early(struct intel_gt *gt, struct 
drm_i915_private *i915)
gt->i915 = i915;
gt->uncore = &i915->uncore;
 
+   spin_lock_init(>->irq_lock);
+
INIT_LIST_HEAD(>->active_rings);
-   INIT_LIST_HEAD(>->closed_vma);
 
+   INIT_LIST_HEAD(>->closed_vma);
spin_lock_init(>->closed_lock);
 
intel_gt_init_hangcheck(gt);
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm_irq.c 
b/drivers/gpu/drm/i915/gt/intel_gt_pm_irq.c
new file mode 100644
index ..83e11c2cc5ab
--- /dev/null
+++ b/drivers/gpu/drm/i915/gt/intel_gt_pm_irq.c
@@ -0,0 +1,108 @@
+/*
+ * SPDX-License-Identifier: MIT
+ *
+ * Copyright © 2019 Intel Corporation
+ */
+
+#include "i915_drv.h"
+#include "i915_reg.h"
+#include "intel_gt.h"
+#include "intel_gt_pm_irq.h"
+
+static void write_pm_imr(struct intel_gt *gt)
+{
+   struct drm_i915_private *i915 = gt->i915;
+   struct intel_uncore *uncore = gt->uncore;
+   u32 mask = gt->pm_imr;
+   i915_reg_t reg;
+
+   if (INTEL_GEN(i915) >= 11) {
+   reg = GEN11_GPM_WGBOXPERF_INTR_MASK;
+   mask <<= 16; /* pm is in upper half */
+   } else if (INTEL_GEN(i915) >= 8) {
+   reg = GEN8_GT_IMR(2);
+   } else {
+   reg = GEN6_PMIMR;
+   }
+
+   intel_uncore_write(uncore, reg, mask);
+}
+
+static void gen6_gt_pm_update_irq(struct intel_gt *gt,
+ u32 interrupt_mask,
+ u32 enabled_irq_mask)
+{
+   u32 new_val;
+
+   WARN_ON(enabled_irq_mask & ~interrupt_mask);
+
+   lockdep_assert_held(>->irq_lock);
+
+   new_val = gt->pm_imr;
+   new_val &= ~interrupt_mask;
+   new_val |= ~enabled_irq_mask & interrupt_mask;
+
+   if (new_val != gt->pm_imr) {
+   gt->pm_imr = new_val;
+   write_pm_imr(gt);
+   }
+}
+
+void gen6_gt_pm_unmask_irq(struct intel_gt *gt, u32 mask)
+{
+   gen6_gt_pm_update_irq(gt, mask, mask);
+}
+
+void gen6_gt_pm_mask_irq(struct intel_gt *gt, u32 mask)
+{
+   gen6_gt_pm_update_irq(gt, mask, 0);
+}
+
+void gen6_gt_pm_reset_iir(struct intel_gt *gt, u32 reset_mask)
+{
+   struct intel_uncore *uncore = gt->uncore;
+   i915_reg_t reg = INTEL_GEN(gt->i915) >= 8 ? GEN8_GT_IIR(2) : GEN6_PMIIR;
+
+   lockdep_assert_held(>->irq_lock);
+
+   intel_uncore_write(uncore, reg, reset_mask);
+   intel_uncore_write(uncore, reg, reset_mask);
+   intel_uncore_posting_read(uncore, reg);
+}
+
+static void write_pm_ier(struct intel_gt *gt)
+{
+   struct drm_i915_private *i915 = gt->i915;
+   struct intel_uncore *uncore = gt->uncore;
+   u32 mask = gt->pm_ier;
+   i915_reg_t reg;
+
+   if (INTEL_GEN(i915) >= 11) {
+   reg = GEN11_GPM_WGBOXPERF_INTR_ENABLE;
+   mask <<= 16; /* pm is in upper half */
+   } else if (INTEL_GEN(i915) >= 8) {
+   reg = GEN8_GT_IER(2);
+   } else {
+   reg = GEN6_PMIER;
+   }
+
+   intel_uncore_write(uncore, reg, mask);
+}
+
+void gen6_gt_pm_enable_irq(struct intel_gt *gt, u32 enable_mask)
+{
+   lockdep_assert_held(>->irq_lock);
+
+   gt->pm_ier |= enable_mask;
+   write_pm_ier(gt);
+   gen6_gt_pm_unmask_irq(gt, enable_mask);
+}
+
+void gen6_gt_pm_disable_irq(struct intel_gt *gt, u32 disable_mask)
+{
+   lockdep_assert_held(>->irq_lock);
+
+