Commit 905da43c authored by Chris Wilson's avatar Chris Wilson

drm/i915/selftests: Move uncore fw selftests to operate on intel_gt

Forcewake is the speciality of the GT, so it is natural to run the
intel_uncore_forcewake tests over the GT. So pass intel_gt as the
parameter to our selftests.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: default avatarDaniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191022131016.9065-1-chris@chris-wilson.co.uk
parent c31c9e82
...@@ -140,19 +140,19 @@ static int live_forcewake_ops(void *arg) ...@@ -140,19 +140,19 @@ static int live_forcewake_ops(void *arg)
} }
}; };
const struct reg *r; const struct reg *r;
struct drm_i915_private *i915 = arg; struct intel_gt *gt = arg;
struct intel_uncore_forcewake_domain *domain; struct intel_uncore_forcewake_domain *domain;
struct intel_uncore *uncore = &i915->uncore; struct intel_uncore *uncore = gt->uncore;
struct intel_engine_cs *engine; struct intel_engine_cs *engine;
enum intel_engine_id id; enum intel_engine_id id;
intel_wakeref_t wakeref; intel_wakeref_t wakeref;
unsigned int tmp; unsigned int tmp;
int err = 0; int err = 0;
GEM_BUG_ON(i915->gt.awake); GEM_BUG_ON(gt->awake);
/* vlv/chv with their pcu behave differently wrt reads */ /* vlv/chv with their pcu behave differently wrt reads */
if (IS_VALLEYVIEW(i915) || IS_CHERRYVIEW(i915)) { if (IS_VALLEYVIEW(gt->i915) || IS_CHERRYVIEW(gt->i915)) {
pr_debug("PCU fakes forcewake badly; skipping\n"); pr_debug("PCU fakes forcewake badly; skipping\n");
return 0; return 0;
} }
...@@ -170,15 +170,15 @@ static int live_forcewake_ops(void *arg) ...@@ -170,15 +170,15 @@ static int live_forcewake_ops(void *arg)
/* We have to pick carefully to get the exact behaviour we need */ /* We have to pick carefully to get the exact behaviour we need */
for (r = registers; r->name; r++) for (r = registers; r->name; r++)
if (r->platforms & INTEL_INFO(i915)->gen_mask) if (r->platforms & INTEL_INFO(gt->i915)->gen_mask)
break; break;
if (!r->name) { if (!r->name) {
pr_debug("Forcewaked register not known for %s; skipping\n", pr_debug("Forcewaked register not known for %s; skipping\n",
intel_platform_name(INTEL_INFO(i915)->platform)); intel_platform_name(INTEL_INFO(gt->i915)->platform));
return 0; return 0;
} }
wakeref = intel_runtime_pm_get(&i915->runtime_pm); wakeref = intel_runtime_pm_get(uncore->rpm);
for_each_fw_domain(domain, uncore, tmp) { for_each_fw_domain(domain, uncore, tmp) {
smp_store_mb(domain->active, false); smp_store_mb(domain->active, false);
...@@ -188,7 +188,7 @@ static int live_forcewake_ops(void *arg) ...@@ -188,7 +188,7 @@ static int live_forcewake_ops(void *arg)
intel_uncore_fw_release_timer(&domain->timer); intel_uncore_fw_release_timer(&domain->timer);
} }
for_each_engine(engine, i915, id) { for_each_engine(engine, gt, id) {
i915_reg_t mmio = _MMIO(engine->mmio_base + r->offset); i915_reg_t mmio = _MMIO(engine->mmio_base + r->offset);
u32 __iomem *reg = uncore->regs + engine->mmio_base + r->offset; u32 __iomem *reg = uncore->regs + engine->mmio_base + r->offset;
enum forcewake_domains fw_domains; enum forcewake_domains fw_domains;
...@@ -249,22 +249,22 @@ static int live_forcewake_ops(void *arg) ...@@ -249,22 +249,22 @@ static int live_forcewake_ops(void *arg)
} }
out_rpm: out_rpm:
intel_runtime_pm_put(&i915->runtime_pm, wakeref); intel_runtime_pm_put(uncore->rpm, wakeref);
return err; return err;
} }
static int live_forcewake_domains(void *arg) static int live_forcewake_domains(void *arg)
{ {
#define FW_RANGE 0x40000 #define FW_RANGE 0x40000
struct drm_i915_private *dev_priv = arg; struct intel_gt *gt = arg;
struct intel_uncore *uncore = &dev_priv->uncore; struct intel_uncore *uncore = gt->uncore;
unsigned long *valid; unsigned long *valid;
u32 offset; u32 offset;
int err; int err;
if (!HAS_FPGA_DBG_UNCLAIMED(dev_priv) && if (!HAS_FPGA_DBG_UNCLAIMED(gt->i915) &&
!IS_VALLEYVIEW(dev_priv) && !IS_VALLEYVIEW(gt->i915) &&
!IS_CHERRYVIEW(dev_priv)) !IS_CHERRYVIEW(gt->i915))
return 0; return 0;
/* /*
...@@ -283,7 +283,7 @@ static int live_forcewake_domains(void *arg) ...@@ -283,7 +283,7 @@ static int live_forcewake_domains(void *arg)
for (offset = 0; offset < FW_RANGE; offset += 4) { for (offset = 0; offset < FW_RANGE; offset += 4) {
i915_reg_t reg = { offset }; i915_reg_t reg = { offset };
(void)I915_READ_FW(reg); intel_uncore_posting_read_fw(uncore, reg);
if (!check_for_unclaimed_mmio(uncore)) if (!check_for_unclaimed_mmio(uncore))
set_bit(offset, valid); set_bit(offset, valid);
} }
...@@ -300,7 +300,7 @@ static int live_forcewake_domains(void *arg) ...@@ -300,7 +300,7 @@ static int live_forcewake_domains(void *arg)
check_for_unclaimed_mmio(uncore); check_for_unclaimed_mmio(uncore);
(void)I915_READ(reg); intel_uncore_posting_read_fw(uncore, reg);
if (check_for_unclaimed_mmio(uncore)) { if (check_for_unclaimed_mmio(uncore)) {
pr_err("Unclaimed mmio read to register 0x%04x\n", pr_err("Unclaimed mmio read to register 0x%04x\n",
offset); offset);
...@@ -312,21 +312,23 @@ static int live_forcewake_domains(void *arg) ...@@ -312,21 +312,23 @@ static int live_forcewake_domains(void *arg)
return err; return err;
} }
static int live_fw_table(void *arg)
{
struct intel_gt *gt = arg;
/* Confirm the table we load is still valid */
return intel_fw_table_check(gt->uncore->fw_domains_table,
gt->uncore->fw_domains_table_entries,
INTEL_GEN(gt->i915) >= 9);
}
int intel_uncore_live_selftests(struct drm_i915_private *i915) int intel_uncore_live_selftests(struct drm_i915_private *i915)
{ {
static const struct i915_subtest tests[] = { static const struct i915_subtest tests[] = {
SUBTEST(live_fw_table),
SUBTEST(live_forcewake_ops), SUBTEST(live_forcewake_ops),
SUBTEST(live_forcewake_domains), SUBTEST(live_forcewake_domains),
}; };
int err; return intel_gt_live_subtests(tests, &i915->gt);
/* Confirm the table we load is still valid */
err = intel_fw_table_check(i915->uncore.fw_domains_table,
i915->uncore.fw_domains_table_entries,
INTEL_GEN(i915) >= 9);
if (err)
return err;
return i915_subtests(tests, i915);
} }
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