* [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX @ 2020-02-13 0:14 Brian Welty 2020-02-13 0:34 ` Chris Wilson ` (7 more replies) 0 siblings, 8 replies; 14+ messages in thread From: Brian Welty @ 2020-02-13 0:14 UTC (permalink / raw) To: intel-gfx For DGFX devices, the MOCS control value is not initialized or used. Update the selftest to skip reading and checking control values for these devices. References: e6e2ac07118b ("drm/i915: do not set MOCS control values on dgfx") Fixes: 3fb33cd32ffd ("drm/i915/selftests: Add coverage of mocs registers") Signed-off-by: Brian Welty <brian.welty@intel.com> --- drivers/gpu/drm/i915/gt/selftest_mocs.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c index de1f83100fb6..8a94a546d580 100644 --- a/drivers/gpu/drm/i915/gt/selftest_mocs.c +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c @@ -199,7 +199,7 @@ static int check_l3cc_table(struct intel_engine_cs *engine, return 0; } -static int check_mocs_engine(struct live_mocs *arg, +static int check_mocs_engine(struct intel_gt *gt, struct live_mocs *arg, struct intel_context *ce) { struct i915_vma *vma = arg->scratch; @@ -222,7 +222,7 @@ static int check_mocs_engine(struct live_mocs *arg, /* Read the mocs tables back using SRM */ offset = i915_ggtt_offset(vma); - if (!err) + if (!err && !IS_DGFX(gt->i915)) err = read_mocs_table(rq, &arg->table, &offset); if (!err && ce->engine->class == RENDER_CLASS) err = read_l3cc_table(rq, &arg->table, &offset); @@ -235,7 +235,7 @@ static int check_mocs_engine(struct live_mocs *arg, /* Compare the results against the expected tables */ vaddr = arg->vaddr; - if (!err) + if (!err && !IS_DGFX(gt->i915)) err = check_mocs_table(ce->engine, &arg->table, &vaddr); if (!err && ce->engine->class == RENDER_CLASS) err = check_l3cc_table(ce->engine, &arg->table, &vaddr); @@ -262,7 +262,7 @@ static int live_mocs_kernel(void *arg) for_each_engine(engine, gt, id) { intel_engine_pm_get(engine); - err = check_mocs_engine(&mocs, engine->kernel_context); + err = check_mocs_engine(gt, &mocs, engine->kernel_context); intel_engine_pm_put(engine); if (err) break; @@ -295,7 +295,7 @@ static int live_mocs_clean(void *arg) break; } - err = check_mocs_engine(&mocs, ce); + err = check_mocs_engine(gt, &mocs, ce); intel_context_put(ce); if (err) break; @@ -332,7 +332,7 @@ static int active_engine_reset(struct intel_context *ce, return err; } -static int __live_mocs_reset(struct live_mocs *mocs, +static int __live_mocs_reset(struct intel_gt *gt, struct live_mocs *mocs, struct intel_context *ce) { int err; @@ -341,7 +341,7 @@ static int __live_mocs_reset(struct live_mocs *mocs, if (err) return err; - err = check_mocs_engine(mocs, ce); + err = check_mocs_engine(gt, mocs, ce); if (err) return err; @@ -349,13 +349,13 @@ static int __live_mocs_reset(struct live_mocs *mocs, if (err) return err; - err = check_mocs_engine(mocs, ce); + err = check_mocs_engine(gt, mocs, ce); if (err) return err; - intel_gt_reset(ce->engine->gt, ce->engine->mask, "mocs"); + intel_gt_reset(gt, ce->engine->mask, "mocs"); - err = check_mocs_engine(mocs, ce); + err = check_mocs_engine(gt, mocs, ce); if (err) return err; @@ -390,7 +390,7 @@ static int live_mocs_reset(void *arg) } intel_engine_pm_get(engine); - err = __live_mocs_reset(&mocs, ce); + err = __live_mocs_reset(gt, &mocs, ce); intel_engine_pm_put(engine); intel_context_put(ce); -- 2.20.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty @ 2020-02-13 0:34 ` Chris Wilson 2020-02-13 0:49 ` Brian Welty 2020-02-13 7:33 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork ` (6 subsequent siblings) 7 siblings, 1 reply; 14+ messages in thread From: Chris Wilson @ 2020-02-13 0:34 UTC (permalink / raw) To: Brian Welty, intel-gfx Quoting Brian Welty (2020-02-13 00:14:18) > For DGFX devices, the MOCS control value is not initialized or used. Then why is the table populated? -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-13 0:34 ` Chris Wilson @ 2020-02-13 0:49 ` Brian Welty 2020-02-14 17:56 ` Daniele Ceraolo Spurio 0 siblings, 1 reply; 14+ messages in thread From: Brian Welty @ 2020-02-13 0:49 UTC (permalink / raw) To: Chris Wilson, intel-gfx On 2/12/2020 4:34 PM, Chris Wilson wrote: > Quoting Brian Welty (2020-02-13 00:14:18) >> For DGFX devices, the MOCS control value is not initialized or used. > > Then why is the table populated? > -Chris > The format has changed (been reduced?) for DGFX. drm_i915_mocs_entry.l3cc_value is what is still initialized/used. Probably first needed is the patch that defines the table entries for DGFX. Ugh, I didn't notice this wasn't applied yet. Let me ask about this. -Brian _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-13 0:49 ` Brian Welty @ 2020-02-14 17:56 ` Daniele Ceraolo Spurio 2020-02-14 18:29 ` Chris Wilson 0 siblings, 1 reply; 14+ messages in thread From: Daniele Ceraolo Spurio @ 2020-02-14 17:56 UTC (permalink / raw) To: Brian Welty, Chris Wilson, intel-gfx On 2/12/20 4:49 PM, Brian Welty wrote: > > On 2/12/2020 4:34 PM, Chris Wilson wrote: >> Quoting Brian Welty (2020-02-13 00:14:18) >>> For DGFX devices, the MOCS control value is not initialized or used. >> >> Then why is the table populated? >> -Chris >> > > The format has changed (been reduced?) for DGFX. drm_i915_mocs_entry.l3cc_value is what is still initialized/used. > Probably first needed is the patch that defines the table entries for DGFX. > Ugh, I didn't notice this wasn't applied yet. Let me ask about this. > We do have: commit e6e2ac07118b15f25683fcbd59ea1be73ec9465d Author: Lucas De Marchi <lucas.demarchi@intel.com> Date: Thu Oct 24 12:51:21 2019 -0700 drm/i915: do not set MOCS control values on dgfx So I see no reason not to add this change to the test side to match that. Maybe we can add an additional check in the test to validate that all the control_entries are set to 0 in the table on DGFX? Daniele > -Brian > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-14 17:56 ` Daniele Ceraolo Spurio @ 2020-02-14 18:29 ` Chris Wilson 2020-02-14 18:37 ` Chris Wilson 2020-02-14 19:36 ` Daniele Ceraolo Spurio 0 siblings, 2 replies; 14+ messages in thread From: Chris Wilson @ 2020-02-14 18:29 UTC (permalink / raw) To: Brian Welty, Daniele Ceraolo Spurio, intel-gfx Quoting Daniele Ceraolo Spurio (2020-02-14 17:56:58) > > > On 2/12/20 4:49 PM, Brian Welty wrote: > > > > On 2/12/2020 4:34 PM, Chris Wilson wrote: > >> Quoting Brian Welty (2020-02-13 00:14:18) > >>> For DGFX devices, the MOCS control value is not initialized or used. > >> > >> Then why is the table populated? > >> -Chris > >> > > > > The format has changed (been reduced?) for DGFX. drm_i915_mocs_entry.l3cc_value is what is still initialized/used. > > Probably first needed is the patch that defines the table entries for DGFX. > > Ugh, I didn't notice this wasn't applied yet. Let me ask about this. > > > > We do have: > > commit e6e2ac07118b15f25683fcbd59ea1be73ec9465d > Author: Lucas De Marchi <lucas.demarchi@intel.com> > Date: Thu Oct 24 12:51:21 2019 -0700 > > drm/i915: do not set MOCS control values on dgfx > > So I see no reason not to add this change to the test side to match > that. Maybe we can add an additional check in the test to validate that > all the control_entries are set to 0 in the table on DGFX? My expectation was that as we were not setting mocs values, we would not have defined a table for it. However, the table is combined for mocs and l3cc. l3cc is still used, right? My ideal would be that our tables did remain the truth value we could use directly -- that would require splitting the tables though. If we did something like diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c index de1f83100fb6..2c636257f12c 100644 --- a/drivers/gpu/drm/i915/gt/selftest_mocs.c +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c @@ -12,7 +12,8 @@ #include "selftests/igt_spinner.h" struct live_mocs { - struct drm_i915_mocs_table table; + struct drm_i915_mocs_table mocs; + struct drm_i915_mocs_table l3cc; struct i915_vma *scratch; void *vaddr; }; @@ -68,13 +69,32 @@ static struct i915_vma *create_scratch(struct intel_gt *gt) return vma; } +static bool has_l3cc(struct drm_i915_private *i915) +{ + return true; +} + +static bool has_mocs(struct drm_i915_private *i915) +{ + return !IS_DGFX(i915); +} + static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) { + struct drm_i915_mocs_table table; int err; - if (!get_mocs_settings(gt->i915, &arg->table)) + memset(arg, 0, sizeof(*arg)); + + if (!get_mocs_settings(gt->i915, &table)) return -EINVAL; + if (has_l3cc(gt->i915)) + arg->l3cc = table; + + if (has_mocs(gt->i915)) + arg->mocs = table; + arg->scratch = create_scratch(gt); if (IS_ERR(arg->scratch)) return PTR_ERR(arg->scratch); @@ -223,9 +243,9 @@ static int check_mocs_engine(struct live_mocs *arg, /* Read the mocs tables back using SRM */ offset = i915_ggtt_offset(vma); if (!err) - err = read_mocs_table(rq, &arg->table, &offset); + err = read_mocs_table(rq, &arg->mocs, &offset); if (!err && ce->engine->class == RENDER_CLASS) - err = read_l3cc_table(rq, &arg->table, &offset); + err = read_l3cc_table(rq, &arg->l3cc, &offset); offset -= i915_ggtt_offset(vma); GEM_BUG_ON(offset > PAGE_SIZE); @@ -236,9 +256,9 @@ static int check_mocs_engine(struct live_mocs *arg, /* Compare the results against the expected tables */ vaddr = arg->vaddr; if (!err) - err = check_mocs_table(ce->engine, &arg->table, &vaddr); + err = check_mocs_table(ce->engine, &arg->mocs, &vaddr); if (!err && ce->engine->class == RENDER_CLASS) - err = check_l3cc_table(ce->engine, &arg->table, &vaddr); + err = check_l3cc_table(ce->engine, &arg->l3cc, &vaddr); if (err) return err; we could retain the table driven approach? -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-14 18:29 ` Chris Wilson @ 2020-02-14 18:37 ` Chris Wilson 2020-02-14 19:36 ` Daniele Ceraolo Spurio 1 sibling, 0 replies; 14+ messages in thread From: Chris Wilson @ 2020-02-14 18:37 UTC (permalink / raw) To: Brian Welty, Daniele Ceraolo Spurio, intel-gfx Quoting Chris Wilson (2020-02-14 18:29:31) > +static bool has_l3cc(struct drm_i915_private *i915) > +{ > + return true; > +} > + > +static bool has_mocs(struct drm_i915_private *i915) > +{ > + return !IS_DGFX(i915); > +} > + > static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) > { > + struct drm_i915_mocs_table table; > int err; > > - if (!get_mocs_settings(gt->i915, &arg->table)) > + memset(arg, 0, sizeof(*arg)); > + > + if (!get_mocs_settings(gt->i915, &table)) > return -EINVAL; On top of that, if we did something like return a mask from get_mocs_settings() for HAS_L3CC | HAS_MOCS (or put it into table) we can share the init with the main code and reduce potential drift. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-14 18:29 ` Chris Wilson 2020-02-14 18:37 ` Chris Wilson @ 2020-02-14 19:36 ` Daniele Ceraolo Spurio 1 sibling, 0 replies; 14+ messages in thread From: Daniele Ceraolo Spurio @ 2020-02-14 19:36 UTC (permalink / raw) To: Chris Wilson, Brian Welty, intel-gfx On 2/14/20 10:29 AM, Chris Wilson wrote: > Quoting Daniele Ceraolo Spurio (2020-02-14 17:56:58) >> >> >> On 2/12/20 4:49 PM, Brian Welty wrote: >>> >>> On 2/12/2020 4:34 PM, Chris Wilson wrote: >>>> Quoting Brian Welty (2020-02-13 00:14:18) >>>>> For DGFX devices, the MOCS control value is not initialized or used. >>>> >>>> Then why is the table populated? >>>> -Chris >>>> >>> >>> The format has changed (been reduced?) for DGFX. drm_i915_mocs_entry.l3cc_value is what is still initialized/used. >>> Probably first needed is the patch that defines the table entries for DGFX. >>> Ugh, I didn't notice this wasn't applied yet. Let me ask about this. >>> >> >> We do have: >> >> commit e6e2ac07118b15f25683fcbd59ea1be73ec9465d >> Author: Lucas De Marchi <lucas.demarchi@intel.com> >> Date: Thu Oct 24 12:51:21 2019 -0700 >> >> drm/i915: do not set MOCS control values on dgfx >> >> So I see no reason not to add this change to the test side to match >> that. Maybe we can add an additional check in the test to validate that >> all the control_entries are set to 0 in the table on DGFX? > > My expectation was that as we were not setting mocs values, we would not > have defined a table for it. However, the table is combined for mocs and > l3cc. l3cc is still used, right? > yes, l3cc is still used. The diff below looks ok to me to keep the table-driven approach. Daniele > My ideal would be that our tables did remain the truth value we could > use directly -- that would require splitting the tables though. > > If we did something like > > diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c > index de1f83100fb6..2c636257f12c 100644 > --- a/drivers/gpu/drm/i915/gt/selftest_mocs.c > +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c > @@ -12,7 +12,8 @@ > #include "selftests/igt_spinner.h" > > struct live_mocs { > - struct drm_i915_mocs_table table; > + struct drm_i915_mocs_table mocs; > + struct drm_i915_mocs_table l3cc; > struct i915_vma *scratch; > void *vaddr; > }; > @@ -68,13 +69,32 @@ static struct i915_vma *create_scratch(struct intel_gt *gt) > return vma; > } > > +static bool has_l3cc(struct drm_i915_private *i915) > +{ > + return true; > +} > + > +static bool has_mocs(struct drm_i915_private *i915) > +{ > + return !IS_DGFX(i915); > +} > + > static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) > { > + struct drm_i915_mocs_table table; > int err; > > - if (!get_mocs_settings(gt->i915, &arg->table)) > + memset(arg, 0, sizeof(*arg)); > + > + if (!get_mocs_settings(gt->i915, &table)) > return -EINVAL; > > + if (has_l3cc(gt->i915)) > + arg->l3cc = table; > + > + if (has_mocs(gt->i915)) > + arg->mocs = table; > + > arg->scratch = create_scratch(gt); > if (IS_ERR(arg->scratch)) > return PTR_ERR(arg->scratch); > @@ -223,9 +243,9 @@ static int check_mocs_engine(struct live_mocs *arg, > /* Read the mocs tables back using SRM */ > offset = i915_ggtt_offset(vma); > if (!err) > - err = read_mocs_table(rq, &arg->table, &offset); > + err = read_mocs_table(rq, &arg->mocs, &offset); > if (!err && ce->engine->class == RENDER_CLASS) > - err = read_l3cc_table(rq, &arg->table, &offset); > + err = read_l3cc_table(rq, &arg->l3cc, &offset); > offset -= i915_ggtt_offset(vma); > GEM_BUG_ON(offset > PAGE_SIZE); > > @@ -236,9 +256,9 @@ static int check_mocs_engine(struct live_mocs *arg, > /* Compare the results against the expected tables */ > vaddr = arg->vaddr; > if (!err) > - err = check_mocs_table(ce->engine, &arg->table, &vaddr); > + err = check_mocs_table(ce->engine, &arg->mocs, &vaddr); > if (!err && ce->engine->class == RENDER_CLASS) > - err = check_l3cc_table(ce->engine, &arg->table, &vaddr); > + err = check_l3cc_table(ce->engine, &arg->l3cc, &vaddr); > if (err) > return err; > > > we could retain the table driven approach? > -Chris > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty 2020-02-13 0:34 ` Chris Wilson @ 2020-02-13 7:33 ` Patchwork 2020-02-13 7:54 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork ` (5 subsequent siblings) 7 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-02-13 7:33 UTC (permalink / raw) To: Brian Welty; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Fix selftest_mocs for DGFX URL : https://patchwork.freedesktop.org/series/73387/ State : warning == Summary == $ dim checkpatch origin/drm-tip 682894ca6774 drm/i915/selftests: Fix selftest_mocs for DGFX -:10: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #10: References: e6e2ac07118b ("drm/i915: do not set MOCS control values on dgfx") -:10: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit e6e2ac07118b ("drm/i915: do not set MOCS control values on dgfx")' #10: References: e6e2ac07118b ("drm/i915: do not set MOCS control values on dgfx") total: 1 errors, 1 warnings, 0 checks, 80 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Fix selftest_mocs for DGFX 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty 2020-02-13 0:34 ` Chris Wilson 2020-02-13 7:33 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork @ 2020-02-13 7:54 ` Patchwork 2020-02-14 21:28 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) Patchwork ` (4 subsequent siblings) 7 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-02-13 7:54 UTC (permalink / raw) To: Brian Welty; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Fix selftest_mocs for DGFX URL : https://patchwork.freedesktop.org/series/73387/ State : success == Summary == CI Bug Log - changes from CI_DRM_7926 -> Patchwork_16551 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16551/index.html Known issues ------------ Here are the changes found in Patchwork_16551 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@i915_selftest@live_hangcheck: - fi-skl-guc: [PASS][1] -> [INCOMPLETE][2] ([fdo#108744]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7926/fi-skl-guc/igt@i915_selftest@live_hangcheck.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16551/fi-skl-guc/igt@i915_selftest@live_hangcheck.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-7500u: [PASS][3] -> [FAIL][4] ([fdo#111096] / [i915#323]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7926/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16551/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html #### Possible fixes #### * igt@gem_exec_parallel@fds: - fi-byt-n2820: [FAIL][5] ([i915#694]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7926/fi-byt-n2820/igt@gem_exec_parallel@fds.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16551/fi-byt-n2820/igt@gem_exec_parallel@fds.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744 [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096 [i915#323]: https://gitlab.freedesktop.org/drm/intel/issues/323 [i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694 [i915#937]: https://gitlab.freedesktop.org/drm/intel/issues/937 Participating hosts (45 -> 39) ------------------------------ Additional (6): fi-hsw-peppy fi-skl-6770hq fi-bdw-gvtdvm fi-glk-dsi fi-gdg-551 fi-bsw-kefka Missing (12): fi-ilk-m540 fi-bdw-5557u fi-tgl-dsi fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-hsw-4770 fi-ivb-3770 fi-icl-u3 fi-byt-clapper fi-bsw-nick fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7926 -> Patchwork_16551 CI-20190529: 20190529 CI_DRM_7926: 6b2fe829d300abf285e9db8b252ffacd216df3ed @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5437: ae42fedfd0c536c560e8e17b06d9c7b94a4e8f0c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16551: 682894ca677494b97822c78d9291f49f2e4b51e7 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 682894ca6774 drm/i915/selftests: Fix selftest_mocs for DGFX == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16551/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty ` (2 preceding siblings ...) 2020-02-13 7:54 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2020-02-14 21:28 ` Patchwork 2020-02-14 21:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork ` (3 subsequent siblings) 7 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-02-14 21:28 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) URL : https://patchwork.freedesktop.org/series/73387/ State : warning == Summary == $ dim checkpatch origin/drm-tip 9f9e214b4a7d drm/i915/selftests: Fix selftest_mocs for DGFX -:19: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #19: > > The format has changed (been reduced?) for DGFX. drm_i915_mocs_entry.l3cc_value is what is still initialized/used. -:26: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit e6e2ac07118b ("drm/i915: do not set MOCS control values on dgfx")' #26: > commit e6e2ac07118b15f25683fcbd59ea1be73ec9465d -:119: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s) total: 2 errors, 1 warnings, 0 checks, 64 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty ` (3 preceding siblings ...) 2020-02-14 21:28 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) Patchwork @ 2020-02-14 21:52 ` Patchwork 2020-02-16 15:56 ` [Intel-gfx] [PATCH] drm/i915/gt: Refactor l3cc/mocs availability Chris Wilson ` (2 subsequent siblings) 7 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-02-14 21:52 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) URL : https://patchwork.freedesktop.org/series/73387/ State : success == Summary == CI Bug Log - changes from CI_DRM_7942 -> Patchwork_16577 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16577/index.html Known issues ------------ Here are the changes found in Patchwork_16577 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_close_race@basic-threads: - fi-hsw-peppy: [PASS][1] -> [TIMEOUT][2] ([fdo#112271] / [i915#1084]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7942/fi-hsw-peppy/igt@gem_close_race@basic-threads.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16577/fi-hsw-peppy/igt@gem_close_race@basic-threads.html - fi-byt-n2820: [PASS][3] -> [INCOMPLETE][4] ([i915#45]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7942/fi-byt-n2820/igt@gem_close_race@basic-threads.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16577/fi-byt-n2820/igt@gem_close_race@basic-threads.html #### Possible fixes #### * igt@i915_selftest@live_gem_contexts: - fi-cfl-8700k: [INCOMPLETE][5] ([i915#424]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7942/fi-cfl-8700k/igt@i915_selftest@live_gem_contexts.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16577/fi-cfl-8700k/igt@i915_selftest@live_gem_contexts.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#1084]: https://gitlab.freedesktop.org/drm/intel/issues/1084 [i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424 [i915#45]: https://gitlab.freedesktop.org/drm/intel/issues/45 [i915#937]: https://gitlab.freedesktop.org/drm/intel/issues/937 Participating hosts (47 -> 45) ------------------------------ Additional (4): fi-bsw-kefka fi-blb-e6850 fi-cfl-8109u fi-ilk-650 Missing (6): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7942 -> Patchwork_16577 CI-20190529: 20190529 CI_DRM_7942: f4805f5a516d0a107438ff0f236c9f4187434819 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5442: 3f6080996885b997685f08ecb8b416b2dc485290 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16577: 9f9e214b4a7dc0d5e5e49a9f0b1224a21120960d @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 9f9e214b4a7d drm/i915/selftests: Fix selftest_mocs for DGFX == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16577/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] [PATCH] drm/i915/gt: Refactor l3cc/mocs availability 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty ` (4 preceding siblings ...) 2020-02-14 21:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2020-02-16 15:56 ` Chris Wilson 2020-02-16 16:50 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) Patchwork 2020-02-18 20:46 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 7 siblings, 0 replies; 14+ messages in thread From: Chris Wilson @ 2020-02-16 15:56 UTC (permalink / raw) To: intel-gfx On dgfx, we only use l3cc and not mocs, but we share the table of register definitions with Tigerlake (which includes the mocs). This confuses our selftest that verifies that the registers do contain the values in our tables after various events (idling, reset, activity etc). When constructing the table of register definitions, also include the flags for which registers are valid so that information is computed centrally and available to all callers. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Brian Welty <brian.welty@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> --- drivers/gpu/drm/i915/gt/intel_mocs.c | 72 +++++++++++++++++-------- drivers/gpu/drm/i915/gt/selftest_mocs.c | 24 ++++++--- 2 files changed, 67 insertions(+), 29 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c index 0afc1eb3c20f..632e08a4592b 100644 --- a/drivers/gpu/drm/i915/gt/intel_mocs.c +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c @@ -280,9 +280,32 @@ static const struct drm_i915_mocs_entry icl_mocs_table[] = { GEN11_MOCS_ENTRIES }; -static bool get_mocs_settings(const struct drm_i915_private *i915, - struct drm_i915_mocs_table *table) +enum { + HAS_GLOBAL_MOCS = BIT(0), + HAS_ENGINE_MOCS = BIT(1), + HAS_RENDER_L3CC = BIT(2), +}; + +static bool has_l3cc(const struct drm_i915_private *i915) { + return true; +} + +static bool has_global_mocs(const struct drm_i915_private *i915) +{ + return HAS_GLOBAL_MOCS_REGISTERS(i915); +} + +static bool has_mocs(const struct drm_i915_private *i915) +{ + return !IS_DGFX(i915); +} + +static unsigned int get_mocs_settings(const struct drm_i915_private *i915, + struct drm_i915_mocs_table *table) +{ + unsigned int flags; + if (INTEL_GEN(i915) >= 12) { table->size = ARRAY_SIZE(tgl_mocs_table); table->table = tgl_mocs_table; @@ -302,11 +325,11 @@ static bool get_mocs_settings(const struct drm_i915_private *i915, } else { drm_WARN_ONCE(&i915->drm, INTEL_GEN(i915) >= 9, "Platform that should have a MOCS table does not.\n"); - return false; + return 0; } if (GEM_DEBUG_WARN_ON(table->size > table->n_entries)) - return false; + return 0; /* WaDisableSkipCaching:skl,bxt,kbl,glk */ if (IS_GEN(i915, 9)) { @@ -315,10 +338,20 @@ static bool get_mocs_settings(const struct drm_i915_private *i915, for (i = 0; i < table->size; i++) if (GEM_DEBUG_WARN_ON(table->table[i].l3cc_value & (L3_ESC(1) | L3_SCC(0x7)))) - return false; + return 0; } - return true; + flags = 0; + if (has_mocs(i915)) { + if (has_global_mocs(i915)) + flags |= HAS_GLOBAL_MOCS; + else + flags |= HAS_ENGINE_MOCS; + } + if (has_l3cc(i915)) + flags |= HAS_RENDER_L3CC; + + return flags; } /* @@ -411,18 +444,20 @@ static void init_l3cc_table(struct intel_engine_cs *engine, void intel_mocs_init_engine(struct intel_engine_cs *engine) { struct drm_i915_mocs_table table; + unsigned int flags; /* Called under a blanket forcewake */ assert_forcewakes_active(engine->uncore, FORCEWAKE_ALL); - if (!get_mocs_settings(engine->i915, &table)) + flags = get_mocs_settings(engine->i915, &table); + if (!flags) return; /* Platforms with global MOCS do not need per-engine initialization. */ - if (!HAS_GLOBAL_MOCS_REGISTERS(engine->i915)) + if (flags & HAS_ENGINE_MOCS) init_mocs_table(engine, &table); - if (engine->class == RENDER_CLASS) + if (flags & HAS_RENDER_L3CC && engine->class == RENDER_CLASS) init_l3cc_table(engine, &table); } @@ -431,26 +466,17 @@ static u32 global_mocs_offset(void) return i915_mmio_reg_offset(GEN12_GLOBAL_MOCS(0)); } -static void init_global_mocs(struct intel_gt *gt) +void intel_mocs_init(struct intel_gt *gt) { struct drm_i915_mocs_table table; + unsigned int flags; /* * LLC and eDRAM control values are not applicable to dgfx */ - if (IS_DGFX(gt->i915)) - return; - - if (!get_mocs_settings(gt->i915, &table)) - return; - - __init_mocs_table(gt->uncore, &table, global_mocs_offset()); -} - -void intel_mocs_init(struct intel_gt *gt) -{ - if (HAS_GLOBAL_MOCS_REGISTERS(gt->i915)) - init_global_mocs(gt); + flags = get_mocs_settings(gt->i915, &table); + if (flags & HAS_GLOBAL_MOCS) + __init_mocs_table(gt->uncore, &table, global_mocs_offset()); } #if IS_ENABLED(CONFIG_DRM_I915_SELFTEST) diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c index de1f83100fb6..8831ffee2061 100644 --- a/drivers/gpu/drm/i915/gt/selftest_mocs.c +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c @@ -12,7 +12,8 @@ #include "selftests/igt_spinner.h" struct live_mocs { - struct drm_i915_mocs_table table; + struct drm_i915_mocs_table mocs; + struct drm_i915_mocs_table l3cc; struct i915_vma *scratch; void *vaddr; }; @@ -70,11 +71,22 @@ static struct i915_vma *create_scratch(struct intel_gt *gt) static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) { + struct drm_i915_mocs_table table; + unsigned int flags; int err; - if (!get_mocs_settings(gt->i915, &arg->table)) + memset(arg, 0, sizeof(*arg)); + + flags = get_mocs_settings(gt->i915, &table); + if (!flags) return -EINVAL; + if (flags & HAS_RENDER_L3CC) + arg->l3cc = table; + + if (flags & (HAS_GLOBAL_MOCS | HAS_ENGINE_MOCS)) + arg->mocs = table; + arg->scratch = create_scratch(gt); if (IS_ERR(arg->scratch)) return PTR_ERR(arg->scratch); @@ -223,9 +235,9 @@ static int check_mocs_engine(struct live_mocs *arg, /* Read the mocs tables back using SRM */ offset = i915_ggtt_offset(vma); if (!err) - err = read_mocs_table(rq, &arg->table, &offset); + err = read_mocs_table(rq, &arg->mocs, &offset); if (!err && ce->engine->class == RENDER_CLASS) - err = read_l3cc_table(rq, &arg->table, &offset); + err = read_l3cc_table(rq, &arg->l3cc, &offset); offset -= i915_ggtt_offset(vma); GEM_BUG_ON(offset > PAGE_SIZE); @@ -236,9 +248,9 @@ static int check_mocs_engine(struct live_mocs *arg, /* Compare the results against the expected tables */ vaddr = arg->vaddr; if (!err) - err = check_mocs_table(ce->engine, &arg->table, &vaddr); + err = check_mocs_table(ce->engine, &arg->mocs, &vaddr); if (!err && ce->engine->class == RENDER_CLASS) - err = check_l3cc_table(ce->engine, &arg->table, &vaddr); + err = check_l3cc_table(ce->engine, &arg->l3cc, &vaddr); if (err) return err; -- 2.25.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty ` (5 preceding siblings ...) 2020-02-16 15:56 ` [Intel-gfx] [PATCH] drm/i915/gt: Refactor l3cc/mocs availability Chris Wilson @ 2020-02-16 16:50 ` Patchwork 2020-02-18 20:46 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 7 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-02-16 16:50 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) URL : https://patchwork.freedesktop.org/series/73387/ State : success == Summary == CI Bug Log - changes from CI_DRM_7950 -> Patchwork_16589 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_16589: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@i915_selftest@live_gt_lrc: - {fi-tgl-dsi}: NOTRUN -> [INCOMPLETE][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/fi-tgl-dsi/igt@i915_selftest@live_gt_lrc.html Known issues ------------ Here are the changes found in Patchwork_16589 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@i915_selftest@live_gtt: - fi-kbl-7500u: [PASS][2] -> [TIMEOUT][3] ([fdo#112271]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/fi-kbl-7500u/igt@i915_selftest@live_gtt.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/fi-kbl-7500u/igt@i915_selftest@live_gtt.html * igt@kms_chamelium@dp-edid-read: - fi-cml-u2: [PASS][4] -> [FAIL][5] ([i915#217] / [i915#976]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/fi-cml-u2/igt@kms_chamelium@dp-edid-read.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/fi-cml-u2/igt@kms_chamelium@dp-edid-read.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#217]: https://gitlab.freedesktop.org/drm/intel/issues/217 [i915#976]: https://gitlab.freedesktop.org/drm/intel/issues/976 Participating hosts (40 -> 36) ------------------------------ Additional (7): fi-hsw-peppy fi-ilk-650 fi-snb-2520m fi-bsw-kefka fi-byt-n2820 fi-skl-6700k2 fi-kbl-r Missing (11): fi-ilk-m540 fi-bsw-n3050 fi-hsw-4200u fi-icl-u2 fi-bwr-2160 fi-ctg-p8600 fi-gdg-551 fi-elk-e7500 fi-pnv-d510 fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7950 -> Patchwork_16589 CI-20190529: 20190529 CI_DRM_7950: eebe6906446d83547d0a3f29ffa3f124a8971b40 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5444: c46bae259d427f53fcfcd5f05de0181a9e82d6fe @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16589: 57a0abd044c141876bb459321a6938611ee1b428 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 57a0abd044c1 drm/i915/gt: Refactor l3cc/mocs availability == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty ` (6 preceding siblings ...) 2020-02-16 16:50 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) Patchwork @ 2020-02-18 20:46 ` Patchwork 7 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-02-18 20:46 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) URL : https://patchwork.freedesktop.org/series/73387/ State : success == Summary == CI Bug Log - changes from CI_DRM_7950_full -> Patchwork_16589_full ==================================================== Summary ------- **SUCCESS** No regressions found. Known issues ------------ Here are the changes found in Patchwork_16589_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_shared@exec-single-timeline-bsd: - shard-iclb: [PASS][1] -> [SKIP][2] ([fdo#110841]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb5/igt@gem_ctx_shared@exec-single-timeline-bsd.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb1/igt@gem_ctx_shared@exec-single-timeline-bsd.html * igt@gem_exec_schedule@in-order-bsd: - shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#112146]) +6 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb5/igt@gem_exec_schedule@in-order-bsd.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb1/igt@gem_exec_schedule@in-order-bsd.html * igt@gem_exec_schedule@pi-common-bsd: - shard-iclb: [PASS][5] -> [SKIP][6] ([i915#677]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb6/igt@gem_exec_schedule@pi-common-bsd.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb1/igt@gem_exec_schedule@pi-common-bsd.html * igt@gem_exec_schedule@preempt-queue-bsd1: - shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#109276]) +20 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb1/igt@gem_exec_schedule@preempt-queue-bsd1.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb8/igt@gem_exec_schedule@preempt-queue-bsd1.html * igt@gem_partial_pwrite_pread@writes-after-reads-display: - shard-hsw: [PASS][9] -> [FAIL][10] ([i915#694]) +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-hsw6/igt@gem_partial_pwrite_pread@writes-after-reads-display.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-hsw5/igt@gem_partial_pwrite_pread@writes-after-reads-display.html * igt@gem_ppgtt@flink-and-close-vma-leak: - shard-glk: [PASS][11] -> [FAIL][12] ([i915#644]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-glk6/igt@gem_ppgtt@flink-and-close-vma-leak.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-glk9/igt@gem_ppgtt@flink-and-close-vma-leak.html * igt@gem_userptr_blits@sync-unmap-after-close: - shard-apl: [PASS][13] -> [DMESG-WARN][14] ([fdo#111870] / [i915#211] / [i915#836]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-apl3/igt@gem_userptr_blits@sync-unmap-after-close.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl3/igt@gem_userptr_blits@sync-unmap-after-close.html * igt@i915_pm_dc@dc6-dpms: - shard-iclb: [PASS][15] -> [FAIL][16] ([i915#454]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb5/igt@i915_pm_dc@dc6-dpms.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb1/igt@i915_pm_dc@dc6-dpms.html * igt@kms_cursor_crc@pipe-a-cursor-suspend: - shard-kbl: [PASS][17] -> [DMESG-WARN][18] ([i915#180]) +3 similar issues [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-kbl2/igt@kms_cursor_crc@pipe-a-cursor-suspend.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-kbl4/igt@kms_cursor_crc@pipe-a-cursor-suspend.html * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic: - shard-glk: [PASS][19] -> [FAIL][20] ([i915#72]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-glk6/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-glk9/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html * igt@kms_cursor_legacy@cursor-vs-flip-toggle: - shard-hsw: [PASS][21] -> [FAIL][22] ([i915#57]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-hsw2/igt@kms_cursor_legacy@cursor-vs-flip-toggle.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-hsw4/igt@kms_cursor_legacy@cursor-vs-flip-toggle.html * igt@kms_flip@flip-vs-expired-vblank-interruptible: - shard-skl: [PASS][23] -> [FAIL][24] ([i915#79]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-skl9/igt@kms_flip@flip-vs-expired-vblank-interruptible.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-skl7/igt@kms_flip@flip-vs-expired-vblank-interruptible.html * igt@kms_frontbuffer_tracking@fbcpsr-rgb101010-draw-render: - shard-tglb: [PASS][25] -> [SKIP][26] ([i915#668]) +5 similar issues [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-tglb6/igt@kms_frontbuffer_tracking@fbcpsr-rgb101010-draw-render.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-tglb1/igt@kms_frontbuffer_tracking@fbcpsr-rgb101010-draw-render.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes: - shard-apl: [PASS][27] -> [DMESG-WARN][28] ([i915#180]) +1 similar issue [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-apl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl8/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html * igt@kms_psr@psr2_primary_page_flip: - shard-iclb: [PASS][29] -> [SKIP][30] ([fdo#109441]) +1 similar issue [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb5/igt@kms_psr@psr2_primary_page_flip.html * igt@perf_pmu@busy-no-semaphores-vcs1: - shard-iclb: [PASS][31] -> [SKIP][32] ([fdo#112080]) +11 similar issues [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb4/igt@perf_pmu@busy-no-semaphores-vcs1.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb7/igt@perf_pmu@busy-no-semaphores-vcs1.html #### Possible fixes #### * {igt@gem_ctx_persistence@close-replace-race}: - shard-tglb: [FAIL][33] ([i915#1241]) -> [PASS][34] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-tglb7/igt@gem_ctx_persistence@close-replace-race.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-tglb2/igt@gem_ctx_persistence@close-replace-race.html - shard-skl: [FAIL][35] ([i915#1241]) -> [PASS][36] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-skl3/igt@gem_ctx_persistence@close-replace-race.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-skl9/igt@gem_ctx_persistence@close-replace-race.html * igt@gem_exec_balancer@hang: - shard-iclb: [FAIL][37] -> [PASS][38] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb4/igt@gem_exec_balancer@hang.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb2/igt@gem_exec_balancer@hang.html * igt@gem_exec_parallel@vcs1-fds: - shard-iclb: [SKIP][39] ([fdo#112080]) -> [PASS][40] +16 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb6/igt@gem_exec_parallel@vcs1-fds.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb4/igt@gem_exec_parallel@vcs1-fds.html * igt@gem_exec_schedule@pi-shared-iova-bsd: - shard-iclb: [SKIP][41] ([i915#677]) -> [PASS][42] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb2/igt@gem_exec_schedule@pi-shared-iova-bsd.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb6/igt@gem_exec_schedule@pi-shared-iova-bsd.html * igt@gem_exec_schedule@preempt-other-chain-bsd: - shard-iclb: [SKIP][43] ([fdo#112146]) -> [PASS][44] +8 similar issues [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb2/igt@gem_exec_schedule@preempt-other-chain-bsd.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb5/igt@gem_exec_schedule@preempt-other-chain-bsd.html * igt@gem_linear_blits@normal: - shard-hsw: [FAIL][45] ([i915#694]) -> [PASS][46] [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-hsw8/igt@gem_linear_blits@normal.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-hsw2/igt@gem_linear_blits@normal.html * igt@i915_pm_dc@dc5-dpms: - shard-iclb: [FAIL][47] ([i915#447]) -> [PASS][48] [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb3/igt@i915_pm_dc@dc5-dpms.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb6/igt@i915_pm_dc@dc5-dpms.html * igt@kms_cursor_legacy@nonblocking-modeset-vs-cursor-atomic: - shard-glk: [FAIL][49] ([i915#67]) -> [PASS][50] [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-glk7/igt@kms_cursor_legacy@nonblocking-modeset-vs-cursor-atomic.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-glk1/igt@kms_cursor_legacy@nonblocking-modeset-vs-cursor-atomic.html * igt@kms_flip@flip-vs-expired-vblank-interruptible: - shard-glk: [FAIL][51] ([i915#79]) -> [PASS][52] [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-glk7/igt@kms_flip@flip-vs-expired-vblank-interruptible.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-glk1/igt@kms_flip@flip-vs-expired-vblank-interruptible.html * igt@kms_flip@flip-vs-suspend-interruptible: - shard-apl: [DMESG-WARN][53] ([i915#180]) -> [PASS][54] +4 similar issues [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-apl6/igt@kms_flip@flip-vs-suspend-interruptible.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible.html - shard-hsw: [INCOMPLETE][55] ([i915#61]) -> [PASS][56] [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-hsw5/igt@kms_flip@flip-vs-suspend-interruptible.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-hsw1/igt@kms_flip@flip-vs-suspend-interruptible.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt: - shard-tglb: [SKIP][57] ([i915#668]) -> [PASS][58] +2 similar issues [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-tglb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-tglb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-plflip-blt.html * {igt@kms_hdr@bpc-switch-dpms}: - shard-skl: [FAIL][59] ([i915#1188]) -> [PASS][60] +1 similar issue [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-skl4/igt@kms_hdr@bpc-switch-dpms.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-skl9/igt@kms_hdr@bpc-switch-dpms.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes: - shard-kbl: [DMESG-WARN][61] ([i915#180]) -> [PASS][62] +1 similar issue [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-kbl4/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-kbl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes.html * igt@kms_psr2_su@frontbuffer: - shard-iclb: [SKIP][63] ([fdo#109642] / [fdo#111068]) -> [PASS][64] [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb5/igt@kms_psr2_su@frontbuffer.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb2/igt@kms_psr2_su@frontbuffer.html * igt@kms_psr@psr2_basic: - shard-iclb: [SKIP][65] ([fdo#109441]) -> [PASS][66] +1 similar issue [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb4/igt@kms_psr@psr2_basic.html [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb2/igt@kms_psr@psr2_basic.html * igt@kms_setmode@basic: - shard-apl: [FAIL][67] ([i915#31]) -> [PASS][68] [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-apl6/igt@kms_setmode@basic.html [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl7/igt@kms_setmode@basic.html * igt@perf@short-reads: - shard-apl: [TIMEOUT][69] ([fdo#112271] / [i915#51]) -> [PASS][70] [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-apl4/igt@perf@short-reads.html [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl1/igt@perf@short-reads.html * igt@prime_vgem@fence-wait-bsd2: - shard-iclb: [SKIP][71] ([fdo#109276]) -> [PASS][72] +23 similar issues [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-iclb5/igt@prime_vgem@fence-wait-bsd2.html [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-iclb1/igt@prime_vgem@fence-wait-bsd2.html #### Warnings #### * igt@gem_tiled_blits@interruptible: - shard-hsw: [FAIL][73] ([i915#818]) -> [DMESG-FAIL][74] ([i915#44]) [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-hsw1/igt@gem_tiled_blits@interruptible.html [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-hsw8/igt@gem_tiled_blits@interruptible.html * igt@gem_tiled_blits@normal: - shard-hsw: [FAIL][75] ([i915#818]) -> [FAIL][76] ([i915#694]) [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-hsw1/igt@gem_tiled_blits@normal.html [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-hsw2/igt@gem_tiled_blits@normal.html * igt@runner@aborted: - shard-apl: [FAIL][77] ([fdo#103927] / [i915#974]) -> ([FAIL][78], [FAIL][79]) ([fdo#103927] / [fdo#111870] / [i915#211] / [i915#771] / [i915#974]) [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7950/shard-apl2/igt@runner@aborted.html [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl8/igt@runner@aborted.html [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/shard-apl3/igt@runner@aborted.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642 [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841 [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068 [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870 [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080 [fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146 [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188 [i915#1241]: https://gitlab.freedesktop.org/drm/intel/issues/1241 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#211]: https://gitlab.freedesktop.org/drm/intel/issues/211 [i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31 [i915#44]: https://gitlab.freedesktop.org/drm/intel/issues/44 [i915#447]: https://gitlab.freedesktop.org/drm/intel/issues/447 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#51]: https://gitlab.freedesktop.org/drm/intel/issues/51 [i915#57]: https://gitlab.freedesktop.org/drm/intel/issues/57 [i915#61]: https://gitlab.freedesktop.org/drm/intel/issues/61 [i915#644]: https://gitlab.freedesktop.org/drm/intel/issues/644 [i915#668]: https://gitlab.freedesktop.org/drm/intel/issues/668 [i915#67]: https://gitlab.freedesktop.org/drm/intel/issues/67 [i915#677]: https://gitlab.freedesktop.org/drm/intel/issues/677 [i915#694]: https://gitlab.freedesktop.org/drm/intel/issues/694 [i915#72]: https://gitlab.freedesktop.org/drm/intel/issues/72 [i915#771]: https://gitlab.freedesktop.org/drm/intel/issues/771 [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79 [i915#818]: https://gitlab.freedesktop.org/drm/intel/issues/818 [i915#836]: https://gitlab.freedesktop.org/drm/intel/issues/836 [i915#974]: https://gitlab.freedesktop.org/drm/intel/issues/974 Participating hosts (10 -> 10) ------------------------------ No changes in participating hosts Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7950 -> Patchwork_16589 CI-20190529: 20190529 CI_DRM_7950: eebe6906446d83547d0a3f29ffa3f124a8971b40 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5444: c46bae259d427f53fcfcd5f05de0181a9e82d6fe @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16589: 57a0abd044c141876bb459321a6938611ee1b428 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16589/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2020-02-18 20:46 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-02-13 0:14 [Intel-gfx] [PATCH] drm/i915/selftests: Fix selftest_mocs for DGFX Brian Welty 2020-02-13 0:34 ` Chris Wilson 2020-02-13 0:49 ` Brian Welty 2020-02-14 17:56 ` Daniele Ceraolo Spurio 2020-02-14 18:29 ` Chris Wilson 2020-02-14 18:37 ` Chris Wilson 2020-02-14 19:36 ` Daniele Ceraolo Spurio 2020-02-13 7:33 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for " Patchwork 2020-02-13 7:54 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2020-02-14 21:28 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/selftests: Fix selftest_mocs for DGFX (rev2) Patchwork 2020-02-14 21:52 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2020-02-16 15:56 ` [Intel-gfx] [PATCH] drm/i915/gt: Refactor l3cc/mocs availability Chris Wilson 2020-02-16 16:50 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Fix selftest_mocs for DGFX (rev3) Patchwork 2020-02-18 20:46 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).