Commit 915490d5 authored by Imre Deak's avatar Imre Deak

drm/i915: sseu: Move sseu_dev_status to i915_drv.h

The data in this struct is provided both by getting the
slice/subslice/eu features available on a given platform and the actual
runtime state of these same features which depends on the HW's current
power saving state.

Atm members of this struct are duplicated in sseu_dev_status and
intel_device_info. For clarity and code reuse we can share one struct
for both of the above purposes. This patch only moves the struct to the
header file, the next patch will convert users of intel_device_info to
use this struct too.

Instead of unsigned int u8 is used now, which is big enough and is used
anyway in intel_device_info.

No functional change.

v2:
- s/stat/sseu/ based on the new struct name (Ben)

Reviewed-by: Robert Bragg <robert@sixbynine.org> (v1)
Reviewed-by: Ben Widawsky <benjamin.widawsky@intel.com> (v1)
Tested-by: Ben Widawsky <benjamin.widawsky@intel.com> (v1)
Signed-off-by: default avatarImre Deak <imre.deak@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1472659987-10417-2-git-send-email-imre.deak@intel.com
parent 16162470
...@@ -4988,16 +4988,8 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_cache_sharing_fops, ...@@ -4988,16 +4988,8 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_cache_sharing_fops,
i915_cache_sharing_get, i915_cache_sharing_set, i915_cache_sharing_get, i915_cache_sharing_set,
"%llu\n"); "%llu\n");
struct sseu_dev_status {
unsigned int slice_total;
unsigned int subslice_total;
unsigned int subslice_per_slice;
unsigned int eu_total;
unsigned int eu_per_subslice;
};
static void cherryview_sseu_device_status(struct drm_i915_private *dev_priv, static void cherryview_sseu_device_status(struct drm_i915_private *dev_priv,
struct sseu_dev_status *stat) struct sseu_dev_info *sseu)
{ {
int ss_max = 2; int ss_max = 2;
int ss; int ss;
...@@ -5015,20 +5007,21 @@ static void cherryview_sseu_device_status(struct drm_i915_private *dev_priv, ...@@ -5015,20 +5007,21 @@ static void cherryview_sseu_device_status(struct drm_i915_private *dev_priv,
/* skip disabled subslice */ /* skip disabled subslice */
continue; continue;
stat->slice_total = 1; sseu->slice_total = 1;
stat->subslice_per_slice++; sseu->subslice_per_slice++;
eu_cnt = ((sig1[ss] & CHV_EU08_PG_ENABLE) ? 0 : 2) + eu_cnt = ((sig1[ss] & CHV_EU08_PG_ENABLE) ? 0 : 2) +
((sig1[ss] & CHV_EU19_PG_ENABLE) ? 0 : 2) + ((sig1[ss] & CHV_EU19_PG_ENABLE) ? 0 : 2) +
((sig1[ss] & CHV_EU210_PG_ENABLE) ? 0 : 2) + ((sig1[ss] & CHV_EU210_PG_ENABLE) ? 0 : 2) +
((sig2[ss] & CHV_EU311_PG_ENABLE) ? 0 : 2); ((sig2[ss] & CHV_EU311_PG_ENABLE) ? 0 : 2);
stat->eu_total += eu_cnt; sseu->eu_total += eu_cnt;
stat->eu_per_subslice = max(stat->eu_per_subslice, eu_cnt); sseu->eu_per_subslice = max_t(unsigned int,
sseu->eu_per_subslice, eu_cnt);
} }
stat->subslice_total = stat->subslice_per_slice; sseu->subslice_total = sseu->subslice_per_slice;
} }
static void gen9_sseu_device_status(struct drm_i915_private *dev_priv, static void gen9_sseu_device_status(struct drm_i915_private *dev_priv,
struct sseu_dev_status *stat) struct sseu_dev_info *sseu)
{ {
int s_max = 3, ss_max = 4; int s_max = 3, ss_max = 4;
int s, ss; int s, ss;
...@@ -5062,7 +5055,7 @@ static void gen9_sseu_device_status(struct drm_i915_private *dev_priv, ...@@ -5062,7 +5055,7 @@ static void gen9_sseu_device_status(struct drm_i915_private *dev_priv,
/* skip disabled slice */ /* skip disabled slice */
continue; continue;
stat->slice_total++; sseu->slice_total++;
if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv)) if (IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
ss_cnt = INTEL_INFO(dev_priv)->subslice_per_slice; ss_cnt = INTEL_INFO(dev_priv)->subslice_per_slice;
...@@ -5080,37 +5073,39 @@ static void gen9_sseu_device_status(struct drm_i915_private *dev_priv, ...@@ -5080,37 +5073,39 @@ static void gen9_sseu_device_status(struct drm_i915_private *dev_priv,
eu_cnt = 2 * hweight32(eu_reg[2*s + ss/2] & eu_cnt = 2 * hweight32(eu_reg[2*s + ss/2] &
eu_mask[ss%2]); eu_mask[ss%2]);
stat->eu_total += eu_cnt; sseu->eu_total += eu_cnt;
stat->eu_per_subslice = max(stat->eu_per_subslice, sseu->eu_per_subslice = max_t(unsigned int,
sseu->eu_per_subslice,
eu_cnt); eu_cnt);
} }
stat->subslice_total += ss_cnt; sseu->subslice_total += ss_cnt;
stat->subslice_per_slice = max(stat->subslice_per_slice, sseu->subslice_per_slice = max_t(unsigned int,
sseu->subslice_per_slice,
ss_cnt); ss_cnt);
} }
} }
static void broadwell_sseu_device_status(struct drm_i915_private *dev_priv, static void broadwell_sseu_device_status(struct drm_i915_private *dev_priv,
struct sseu_dev_status *stat) struct sseu_dev_info *sseu)
{ {
u32 slice_info = I915_READ(GEN8_GT_SLICE_INFO); u32 slice_info = I915_READ(GEN8_GT_SLICE_INFO);
int s; int s;
stat->slice_total = hweight32(slice_info & GEN8_LSLICESTAT_MASK); sseu->slice_total = hweight32(slice_info & GEN8_LSLICESTAT_MASK);
if (stat->slice_total) { if (sseu->slice_total) {
stat->subslice_per_slice = INTEL_INFO(dev_priv)->subslice_per_slice; sseu->subslice_per_slice = INTEL_INFO(dev_priv)->subslice_per_slice;
stat->subslice_total = stat->slice_total * sseu->subslice_total = sseu->slice_total *
stat->subslice_per_slice; sseu->subslice_per_slice;
stat->eu_per_subslice = INTEL_INFO(dev_priv)->eu_per_subslice; sseu->eu_per_subslice = INTEL_INFO(dev_priv)->eu_per_subslice;
stat->eu_total = stat->eu_per_subslice * stat->subslice_total; sseu->eu_total = sseu->eu_per_subslice * sseu->subslice_total;
/* subtract fused off EU(s) from enabled slice(s) */ /* subtract fused off EU(s) from enabled slice(s) */
for (s = 0; s < stat->slice_total; s++) { for (s = 0; s < sseu->slice_total; s++) {
u8 subslice_7eu = INTEL_INFO(dev_priv)->subslice_7eu[s]; u8 subslice_7eu = INTEL_INFO(dev_priv)->subslice_7eu[s];
stat->eu_total -= hweight8(subslice_7eu); sseu->eu_total -= hweight8(subslice_7eu);
} }
} }
} }
...@@ -5118,7 +5113,7 @@ static void broadwell_sseu_device_status(struct drm_i915_private *dev_priv, ...@@ -5118,7 +5113,7 @@ static void broadwell_sseu_device_status(struct drm_i915_private *dev_priv,
static int i915_sseu_status(struct seq_file *m, void *unused) static int i915_sseu_status(struct seq_file *m, void *unused)
{ {
struct drm_i915_private *dev_priv = node_to_i915(m->private); struct drm_i915_private *dev_priv = node_to_i915(m->private);
struct sseu_dev_status stat; struct sseu_dev_info sseu;
if (INTEL_GEN(dev_priv) < 8) if (INTEL_GEN(dev_priv) < 8)
return -ENODEV; return -ENODEV;
...@@ -5146,30 +5141,30 @@ static int i915_sseu_status(struct seq_file *m, void *unused) ...@@ -5146,30 +5141,30 @@ static int i915_sseu_status(struct seq_file *m, void *unused)
yesno(INTEL_INFO(dev_priv)->has_eu_pg)); yesno(INTEL_INFO(dev_priv)->has_eu_pg));
seq_puts(m, "SSEU Device Status\n"); seq_puts(m, "SSEU Device Status\n");
memset(&stat, 0, sizeof(stat)); memset(&sseu, 0, sizeof(sseu));
intel_runtime_pm_get(dev_priv); intel_runtime_pm_get(dev_priv);
if (IS_CHERRYVIEW(dev_priv)) { if (IS_CHERRYVIEW(dev_priv)) {
cherryview_sseu_device_status(dev_priv, &stat); cherryview_sseu_device_status(dev_priv, &sseu);
} else if (IS_BROADWELL(dev_priv)) { } else if (IS_BROADWELL(dev_priv)) {
broadwell_sseu_device_status(dev_priv, &stat); broadwell_sseu_device_status(dev_priv, &sseu);
} else if (INTEL_GEN(dev_priv) >= 9) { } else if (INTEL_GEN(dev_priv) >= 9) {
gen9_sseu_device_status(dev_priv, &stat); gen9_sseu_device_status(dev_priv, &sseu);
} }
intel_runtime_pm_put(dev_priv); intel_runtime_pm_put(dev_priv);
seq_printf(m, " Enabled Slice Total: %u\n", seq_printf(m, " Enabled Slice Total: %u\n",
stat.slice_total); sseu.slice_total);
seq_printf(m, " Enabled Subslice Total: %u\n", seq_printf(m, " Enabled Subslice Total: %u\n",
stat.subslice_total); sseu.subslice_total);
seq_printf(m, " Enabled Subslice Per Slice: %u\n", seq_printf(m, " Enabled Subslice Per Slice: %u\n",
stat.subslice_per_slice); sseu.subslice_per_slice);
seq_printf(m, " Enabled EU Total: %u\n", seq_printf(m, " Enabled EU Total: %u\n",
stat.eu_total); sseu.eu_total);
seq_printf(m, " Enabled EU Per Subslice: %u\n", seq_printf(m, " Enabled EU Per Subslice: %u\n",
stat.eu_per_subslice); sseu.eu_per_subslice);
return 0; return 0;
} }
......
...@@ -671,6 +671,14 @@ struct intel_csr { ...@@ -671,6 +671,14 @@ struct intel_csr {
#define DEFINE_FLAG(name) u8 name:1 #define DEFINE_FLAG(name) u8 name:1
#define SEP_SEMICOLON ; #define SEP_SEMICOLON ;
struct sseu_dev_info {
u8 slice_total;
u8 subslice_total;
u8 subslice_per_slice;
u8 eu_total;
u8 eu_per_subslice;
};
struct intel_device_info { struct intel_device_info {
u32 display_mmio_offset; u32 display_mmio_offset;
u16 device_id; u16 device_id;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment