[PATCH 097/109] drm/i915: Track HAS_RPS alongside HAS_RC6 in the device info
Chris Wilson
chris at chris-wilson.co.uk
Wed Jan 2 16:47:54 UTC 2019
For consistency (and elegance!), add intel_device_info.has_rps.
The immediate boon is that RPS support is now emitted along the other
capabilities in the debug log and after errors.
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Reviewed-by: Sagar Arun Kamble <sagar.a.kamble at intel.com>
---
drivers/gpu/drm/i915/i915_drv.h | 2 ++
drivers/gpu/drm/i915/i915_pci.c | 22 ++++++++++++++--------
drivers/gpu/drm/i915/intel_device_info.h | 1 +
drivers/gpu/drm/i915/intel_gt_pm.c | 20 ++++++++++++++++----
4 files changed, 33 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index a2f5f234cca8..9ce5d3c182e2 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -2397,6 +2397,8 @@ static inline unsigned int i915_sg_segment_size(void)
#define HAS_RC6p(dev_priv) (INTEL_INFO(dev_priv)->has_rc6p)
#define HAS_RC6pp(dev_priv) (false) /* HW was never validated */
+#define HAS_RPS(dev_priv) (INTEL_INFO(dev_priv)->has_rps)
+
#define HAS_CSR(dev_priv) (INTEL_INFO(dev_priv)->display.has_csr)
#define HAS_RUNTIME_PM(dev_priv) (INTEL_INFO(dev_priv)->has_runtime_pm)
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
index 5db464795b07..c3916cdc1d38 100644
--- a/drivers/gpu/drm/i915/i915_pci.c
+++ b/drivers/gpu/drm/i915/i915_pci.c
@@ -269,6 +269,7 @@ static const struct intel_device_info intel_ironlake_m_info = {
PLATFORM(INTEL_IRONLAKE),
.is_mobile = 1,
.display.has_fbc = 1,
+ .has_rps = true,
};
#define GEN6_FEATURES \
@@ -281,6 +282,7 @@ static const struct intel_device_info intel_ironlake_m_info = {
.has_llc = 1, \
.has_rc6 = 1, \
.has_rc6p = 1, \
+ .has_rps = true, \
.ppgtt = INTEL_PPGTT_ALIASING, \
GEN_DEFAULT_PIPEOFFSETS, \
GEN_DEFAULT_PAGE_SIZES, \
@@ -326,6 +328,7 @@ static const struct intel_device_info intel_sandybridge_m_gt2_info = {
.has_llc = 1, \
.has_rc6 = 1, \
.has_rc6p = 1, \
+ .has_rps = true, \
.ppgtt = INTEL_PPGTT_FULL, \
GEN_DEFAULT_PIPEOFFSETS, \
GEN_DEFAULT_PAGE_SIZES, \
@@ -377,12 +380,13 @@ static const struct intel_device_info intel_valleyview_info = {
.num_pipes = 2,
.has_runtime_pm = 1,
.has_rc6 = 1,
- .display.has_gmch_display = 1,
- .display.has_hotplug = 1,
+ .has_rps = true,
.ppgtt = INTEL_PPGTT_FULL,
.has_snoop = true,
.has_coherent_ggtt = false,
.ring_mask = RENDER_RING | BSD_RING | BLT_RING,
+ .display.has_gmch_display = 1,
+ .display.has_hotplug = 1,
.display_mmio_offset = VLV_DISPLAY_BASE,
GEN_DEFAULT_PAGE_SIZES,
GEN_DEFAULT_PIPEOFFSETS,
@@ -468,6 +472,7 @@ static const struct intel_device_info intel_cherryview_info = {
.has_64bit_reloc = 1,
.has_runtime_pm = 1,
.has_rc6 = 1,
+ .has_rps = true,
.has_logical_ring_contexts = 1,
.display.has_gmch_display = 1,
.ppgtt = INTEL_PPGTT_FULL,
@@ -530,18 +535,13 @@ static const struct intel_device_info intel_skylake_gt4_info = {
#define GEN9_LP_FEATURES \
GEN(9), \
.is_lp = 1, \
- .display.has_hotplug = 1, \
.ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, \
.num_pipes = 3, \
.has_64bit_reloc = 1, \
- .display.has_ddi = 1, \
.has_fpga_dbg = 1, \
- .display.has_fbc = 1, \
- .display.has_psr = 1, \
.has_runtime_pm = 1, \
- .display.has_csr = 1, \
.has_rc6 = 1, \
- .display.has_dp_mst = 1, \
+ .has_rps = true, \
.has_logical_ring_contexts = 1, \
.has_logical_ring_preemption = 1, \
.has_guc = 1, \
@@ -549,7 +549,13 @@ static const struct intel_device_info intel_skylake_gt4_info = {
.has_reset_engine = 1, \
.has_snoop = true, \
.has_coherent_ggtt = false, \
+ .display.has_csr = 1, \
+ .display.has_ddi = 1, \
+ .display.has_dp_mst = 1, \
+ .display.has_fbc = 1, \
+ .display.has_hotplug = 1, \
.display.has_ipc = 1, \
+ .display.has_psr = 1, \
GEN9_DEFAULT_PAGE_SIZES, \
GEN_DEFAULT_PIPEOFFSETS, \
IVB_CURSOR_OFFSETS, \
diff --git a/drivers/gpu/drm/i915/intel_device_info.h b/drivers/gpu/drm/i915/intel_device_info.h
index 957c6527f76b..1d5e79d4af51 100644
--- a/drivers/gpu/drm/i915/intel_device_info.h
+++ b/drivers/gpu/drm/i915/intel_device_info.h
@@ -102,6 +102,7 @@ enum intel_ppgtt {
func(has_pooled_eu); \
func(has_rc6); \
func(has_rc6p); \
+ func(has_rps); \
func(has_runtime_pm); \
func(has_snoop); \
func(has_coherent_ggtt); \
diff --git a/drivers/gpu/drm/i915/intel_gt_pm.c b/drivers/gpu/drm/i915/intel_gt_pm.c
index 97fb06e51a7c..df55ef8d99cc 100644
--- a/drivers/gpu/drm/i915/intel_gt_pm.c
+++ b/drivers/gpu/drm/i915/intel_gt_pm.c
@@ -848,6 +848,9 @@ void gen6_rps_busy(struct drm_i915_private *dev_priv)
{
struct intel_rps *rps = &dev_priv->gt_pm.rps;
+ if (!HAS_RPS(dev_priv))
+ return;
+
mutex_lock(&rps->lock);
if (rps->enabled) {
u8 freq;
@@ -877,6 +880,9 @@ void gen6_rps_idle(struct drm_i915_private *dev_priv)
{
struct intel_rps *rps = &dev_priv->gt_pm.rps;
+ if (!HAS_RPS(dev_priv))
+ return;
+
/*
* Flush our bottom-half so that it does not race with us
* setting the idle frequency and so that it is bounded by
@@ -904,6 +910,9 @@ void gen6_rps_boost(struct i915_request *rq, struct intel_rps_client *client)
unsigned long flags;
bool boost;
+ if (!HAS_RPS(rq->i915))
+ return;
+
/*
* This is intentionally racy! We peek at the state here, then
* validate inside the RPS worker.
@@ -1046,8 +1055,10 @@ static bool sanitize_rc6(struct drm_i915_private *i915)
struct intel_device_info *info = mkwrite_device_info(i915);
/* Powersaving is controlled by the host when inside a VM */
- if (intel_vgpu_active(i915))
+ if (intel_vgpu_active(i915)) {
info->has_rc6 = 0;
+ info->has_rps = 0;
+ }
if (info->has_rc6 &&
IS_GEN9_LP(i915) && !bxt_check_bios_rc6_setup(i915)) {
@@ -2680,7 +2691,7 @@ static void intel_disable_rps(struct drm_i915_private *i915)
valleyview_disable_rps(i915);
else if (INTEL_GEN(i915) >= 6)
gen6_disable_rps(i915);
- else if (IS_IRONLAKE_M(i915))
+ else if (INTEL_GEN(i915) >= 5)
ironlake_disable_drps(i915);
i915->gt_pm.rps.enabled = false;
@@ -2750,7 +2761,7 @@ static void intel_enable_rps(struct drm_i915_private *i915)
gen8_enable_rps(i915);
} else if (INTEL_GEN(i915) >= 6) {
gen6_enable_rps(i915);
- } else if (IS_IRONLAKE_M(i915)) {
+ } else if (INTEL_GEN(i915) >= 5) {
ironlake_enable_drps(i915);
intel_init_emon(i915);
}
@@ -2774,7 +2785,8 @@ void intel_enable_gt_powersave(struct drm_i915_private *i915)
if (HAS_RC6(i915))
intel_enable_rc6(i915);
- intel_enable_rps(i915);
+ if (HAS_RPS(i915))
+ intel_enable_rps(i915);
if (HAS_LLC(i915))
intel_enable_llc_pstate(i915);
--
2.20.1
More information about the Intel-gfx-trybot
mailing list