* [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others @ 2019-10-22 11:51 Chris Wilson 2019-10-22 11:51 ` [PATCH 2/4] drm/i915/gt: Tidy up debug-warns for the mocs control table Chris Wilson ` (5 more replies) 0 siblings, 6 replies; 15+ messages in thread From: Chris Wilson @ 2019-10-22 11:51 UTC (permalink / raw) To: intel-gfx Be consistent in our mocs setup on Tigerlake and set the unused control value to follow the PTE entry as we previously have done. The unused values are beyond the defines of the ABI, the consistency simplifies our checking. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> --- drivers/gpu/drm/i915/gt/intel_mocs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c index 5bac3966906b..aea3896cc139 100644 --- a/drivers/gpu/drm/i915/gt/intel_mocs.c +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c @@ -482,7 +482,7 @@ static void intel_mocs_init_global(struct intel_gt *gt) for (; index < table.n_entries; index++) intel_uncore_write(uncore, GEN12_GLOBAL_MOCS(index), - table.table[0].control_value); + table.table[I915_MOCS_PTE].control_value); } void intel_mocs_init(struct intel_gt *gt) -- 2.24.0.rc0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/4] drm/i915/gt: Tidy up debug-warns for the mocs control table 2019-10-22 11:51 [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others Chris Wilson @ 2019-10-22 11:51 ` Chris Wilson 2019-10-22 11:51 ` [PATCH 3/4] drm/i915/gt: Refactor mocs loops into single control macro Chris Wilson ` (4 subsequent siblings) 5 siblings, 0 replies; 15+ messages in thread From: Chris Wilson @ 2019-10-22 11:51 UTC (permalink / raw) To: intel-gfx As we always run new platforms through CI, we only need the debug code compiled in during CI runs. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> --- drivers/gpu/drm/i915/gt/intel_mocs.c | 30 ++++++++++------------------ 1 file changed, 11 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c index aea3896cc139..749dc73ea938 100644 --- a/drivers/gpu/drm/i915/gt/intel_mocs.c +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c @@ -290,44 +290,42 @@ static const struct drm_i915_mocs_entry icelake_mocs_table[] = { static bool get_mocs_settings(const struct drm_i915_private *i915, struct drm_i915_mocs_table *table) { - bool result = false; - if (INTEL_GEN(i915) >= 12) { table->size = ARRAY_SIZE(tigerlake_mocs_table); table->table = tigerlake_mocs_table; table->n_entries = GEN11_NUM_MOCS_ENTRIES; - result = true; } else if (IS_GEN(i915, 11)) { table->size = ARRAY_SIZE(icelake_mocs_table); table->table = icelake_mocs_table; table->n_entries = GEN11_NUM_MOCS_ENTRIES; - result = true; } else if (IS_GEN9_BC(i915) || IS_CANNONLAKE(i915)) { table->size = ARRAY_SIZE(skylake_mocs_table); table->n_entries = GEN9_NUM_MOCS_ENTRIES; table->table = skylake_mocs_table; - result = true; } else if (IS_GEN9_LP(i915)) { table->size = ARRAY_SIZE(broxton_mocs_table); table->n_entries = GEN9_NUM_MOCS_ENTRIES; table->table = broxton_mocs_table; - result = true; } else { WARN_ONCE(INTEL_GEN(i915) >= 9, "Platform that should have a MOCS table does not.\n"); + return false; } + if (GEM_DEBUG_WARN_ON(table->size > table->n_entries)) + return false; + /* WaDisableSkipCaching:skl,bxt,kbl,glk */ if (IS_GEN(i915, 9)) { int i; for (i = 0; i < table->size; i++) - if (WARN_ON(table->table[i].l3cc_value & - (L3_ESC(1) | L3_SCC(0x7)))) + if (GEM_DEBUG_WARN_ON(table->table[i].l3cc_value & + (L3_ESC(1) | L3_SCC(0x7)))) return false; } - return result; + return true; } static i915_reg_t mocs_register(const struct intel_engine_cs *engine, int index) @@ -396,9 +394,7 @@ static u16 get_entry_l3cc(const struct drm_i915_mocs_table *table, return table->table[I915_MOCS_PTE].l3cc_value; } -static inline u32 l3cc_combine(const struct drm_i915_mocs_table *table, - u16 low, - u16 high) +static inline u32 l3cc_combine(u16 low, u16 high) { return low | (u32)high << 16; } @@ -416,7 +412,7 @@ static void init_l3cc_table(struct intel_engine_cs *engine, intel_uncore_write(uncore, GEN9_LNCFCMOCS(i), - l3cc_combine(table, low, high)); + l3cc_combine(low, high)); } /* Odd table size - 1 left over */ @@ -425,7 +421,7 @@ static void init_l3cc_table(struct intel_engine_cs *engine, intel_uncore_write(uncore, GEN9_LNCFCMOCS(i), - l3cc_combine(table, low, unused_value)); + l3cc_combine(low, unused_value)); i++; } @@ -433,8 +429,7 @@ static void init_l3cc_table(struct intel_engine_cs *engine, for (; i < table->n_entries / 2; i++) intel_uncore_write(uncore, GEN9_LNCFCMOCS(i), - l3cc_combine(table, unused_value, - unused_value)); + l3cc_combine(unused_value, unused_value)); } void intel_mocs_init_engine(struct intel_engine_cs *engine) @@ -466,9 +461,6 @@ static void intel_mocs_init_global(struct intel_gt *gt) if (!get_mocs_settings(gt->i915, &table)) return; - if (GEM_DEBUG_WARN_ON(table.size > table.n_entries)) - return; - for (index = 0; index < table.size; index++) intel_uncore_write(uncore, GEN12_GLOBAL_MOCS(index), -- 2.24.0.rc0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/4] drm/i915/gt: Refactor mocs loops into single control macro 2019-10-22 11:51 [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others Chris Wilson 2019-10-22 11:51 ` [PATCH 2/4] drm/i915/gt: Tidy up debug-warns for the mocs control table Chris Wilson @ 2019-10-22 11:51 ` Chris Wilson 2019-10-22 11:51 ` [PATCH 4/4] drm/i915/selftests: Add coverage of mocs registers Chris Wilson ` (3 subsequent siblings) 5 siblings, 0 replies; 15+ messages in thread From: Chris Wilson @ 2019-10-22 11:51 UTC (permalink / raw) To: intel-gfx We repeatedly (and more so in future) use the same looping construct over the mocs definition table to setup the register state. Refactor the loop construct into a reusable macro. add/remove: 2/1 grow/shrink: 1/2 up/down: 113/-330 (-217) Function old new delta intel_mocs_init_engine.cold - 71 +71 offset - 28 +28 __func__ 17273 17287 +14 intel_mocs_init 143 113 -30 mocs_register.isra 91 - -91 intel_mocs_init_engine 503 294 -209 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> --- drivers/gpu/drm/i915/gt/intel_mocs.c | 128 ++++++++++----------------- 1 file changed, 47 insertions(+), 81 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c index 749dc73ea938..445ec025bda0 100644 --- a/drivers/gpu/drm/i915/gt/intel_mocs.c +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c @@ -328,27 +328,6 @@ static bool get_mocs_settings(const struct drm_i915_private *i915, return true; } -static i915_reg_t mocs_register(const struct intel_engine_cs *engine, int index) -{ - switch (engine->id) { - case RCS0: - return GEN9_GFX_MOCS(index); - case VCS0: - return GEN9_MFX0_MOCS(index); - case BCS0: - return GEN9_BLT_MOCS(index); - case VECS0: - return GEN9_VEBOX_MOCS(index); - case VCS1: - return GEN9_MFX1_MOCS(index); - case VCS2: - return GEN11_MFX2_MOCS(index); - default: - MISSING_CASE(engine->id); - return INVALID_MMIO_REG; - } -} - /* * Get control_value from MOCS entry taking into account when it's not used: * I915_MOCS_PTE's value is returned in this case. @@ -356,29 +335,47 @@ static i915_reg_t mocs_register(const struct intel_engine_cs *engine, int index) static u32 get_entry_control(const struct drm_i915_mocs_table *table, unsigned int index) { - if (table->table[index].used) + if (index < table->size && table->table[index].used) return table->table[index].control_value; return table->table[I915_MOCS_PTE].control_value; } -static void init_mocs_table(struct intel_engine_cs *engine, - const struct drm_i915_mocs_table *table) +#define for_each_mocs(mocs, t, i) \ + for (i = 0; \ + i < t->n_entries ? (mocs = get_entry_control(t, i)), 1 : 0;\ + i++) + +static void __init_mocs_table(struct intel_uncore *uncore, + const struct drm_i915_mocs_table *table, + u32 addr) { - struct intel_uncore *uncore = engine->uncore; - u32 unused_value = table->table[I915_MOCS_PTE].control_value; unsigned int i; + u32 mocs; - for (i = 0; i < table->size; i++) - intel_uncore_write_fw(uncore, - mocs_register(engine, i), - get_entry_control(table, i)); + for_each_mocs(mocs, table, i) + intel_uncore_write_fw(uncore, _MMIO(addr + i * 4), mocs); +} - /* All remaining entries are unused */ - for (; i < table->n_entries; i++) - intel_uncore_write_fw(uncore, - mocs_register(engine, i), - unused_value); +static u32 mocs_register(const struct intel_engine_cs *engine) +{ + static const u32 offset[] = { + [RCS0] = 0x0c800, + [VCS0] = 0x0c900, + [VCS1] = 0x0ca00, + [VECS0] = 0x0cb00, + [BCS0] = 0x0cc00, + [VCS2] = 0x10000, + }; + + GEM_BUG_ON(engine->id > ARRAY_SIZE(offset)); + return offset[engine->id]; +} + +static void init_mocs_table(struct intel_engine_cs *engine, + const struct drm_i915_mocs_table *table) +{ + __init_mocs_table(engine->uncore, table, mocs_register(engine)); } /* @@ -388,7 +385,7 @@ static void init_mocs_table(struct intel_engine_cs *engine, static u16 get_entry_l3cc(const struct drm_i915_mocs_table *table, unsigned int index) { - if (table->table[index].used) + if (index < table->size && table->table[index].used) return table->table[index].l3cc_value; return table->table[I915_MOCS_PTE].l3cc_value; @@ -399,37 +396,23 @@ static inline u32 l3cc_combine(u16 low, u16 high) return low | (u32)high << 16; } +#define for_each_l3cc(l3cc, t, i) \ + for (i = 0; \ + i < (t->n_entries + 1) / 2 ? \ + (l3cc = l3cc_combine(get_entry_l3cc(t, 2 * i), \ + get_entry_l3cc(t, 2 * i + 1))), 1 : \ + 0; \ + i++) + static void init_l3cc_table(struct intel_engine_cs *engine, const struct drm_i915_mocs_table *table) { struct intel_uncore *uncore = engine->uncore; - u16 unused_value = table->table[I915_MOCS_PTE].l3cc_value; unsigned int i; + u32 l3cc; - for (i = 0; i < table->size / 2; i++) { - u16 low = get_entry_l3cc(table, 2 * i); - u16 high = get_entry_l3cc(table, 2 * i + 1); - - intel_uncore_write(uncore, - GEN9_LNCFCMOCS(i), - l3cc_combine(low, high)); - } - - /* Odd table size - 1 left over */ - if (table->size & 1) { - u16 low = get_entry_l3cc(table, 2 * i); - - intel_uncore_write(uncore, - GEN9_LNCFCMOCS(i), - l3cc_combine(low, unused_value)); - i++; - } - - /* All remaining entries are also unused */ - for (; i < table->n_entries / 2; i++) - intel_uncore_write(uncore, - GEN9_LNCFCMOCS(i), - l3cc_combine(unused_value, unused_value)); + for_each_l3cc(l3cc, table, i) + intel_uncore_write_fw(uncore, GEN9_LNCFCMOCS(i), l3cc); } void intel_mocs_init_engine(struct intel_engine_cs *engine) @@ -450,35 +433,18 @@ void intel_mocs_init_engine(struct intel_engine_cs *engine) init_l3cc_table(engine, &table); } -static void intel_mocs_init_global(struct intel_gt *gt) +static void init_global_mocs(struct intel_gt *gt) { - struct intel_uncore *uncore = gt->uncore; struct drm_i915_mocs_table table; - unsigned int index; - - GEM_BUG_ON(!HAS_GLOBAL_MOCS_REGISTERS(gt->i915)); if (!get_mocs_settings(gt->i915, &table)) return; - for (index = 0; index < table.size; index++) - intel_uncore_write(uncore, - GEN12_GLOBAL_MOCS(index), - table.table[index].control_value); - - /* - * Ok, now set the unused entries to the invalid entry (index 0). These - * entries are officially undefined and no contract for the contents and - * settings is given for these entries. - */ - for (; index < table.n_entries; index++) - intel_uncore_write(uncore, - GEN12_GLOBAL_MOCS(index), - table.table[I915_MOCS_PTE].control_value); + __init_mocs_table(gt->uncore, &table, 0x4000); } void intel_mocs_init(struct intel_gt *gt) { if (HAS_GLOBAL_MOCS_REGISTERS(gt->i915)) - intel_mocs_init_global(gt); + init_global_mocs(gt); } -- 2.24.0.rc0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 4/4] drm/i915/selftests: Add coverage of mocs registers 2019-10-22 11:51 [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others Chris Wilson 2019-10-22 11:51 ` [PATCH 2/4] drm/i915/gt: Tidy up debug-warns for the mocs control table Chris Wilson 2019-10-22 11:51 ` [PATCH 3/4] drm/i915/gt: Refactor mocs loops into single control macro Chris Wilson @ 2019-10-22 11:51 ` Chris Wilson 2019-10-22 11:57 ` [PATCH] " Chris Wilson 2019-10-22 19:28 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) Patchwork ` (2 subsequent siblings) 5 siblings, 1 reply; 15+ messages in thread From: Chris Wilson @ 2019-10-22 11:51 UTC (permalink / raw) To: intel-gfx Probe the mocs registers for new contexts and across GPU resets. Similar to intel_workarounds, we have tables of what register values we expect to see, so verify that user contexts are affected by them. In the future, we should add tests similar to intel_sseu to cover dynamic reconfigurations. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> --- drivers/gpu/drm/i915/gt/intel_mocs.c | 4 + drivers/gpu/drm/i915/gt/selftest_mocs.c | 405 ++++++++++++++++++ .../drm/i915/selftests/i915_live_selftests.h | 1 + 3 files changed, 410 insertions(+) create mode 100644 drivers/gpu/drm/i915/gt/selftest_mocs.c diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c index 445ec025bda0..06dba7ff294e 100644 --- a/drivers/gpu/drm/i915/gt/intel_mocs.c +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c @@ -448,3 +448,7 @@ void intel_mocs_init(struct intel_gt *gt) if (HAS_GLOBAL_MOCS_REGISTERS(gt->i915)) init_global_mocs(gt); } + +#if IS_ENABLED(CONFIG_DRM_I915_SELFTEST) +#include "selftest_mocs.c" +#endif diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c new file mode 100644 index 000000000000..f4dfb36f450d --- /dev/null +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c @@ -0,0 +1,405 @@ +/* + * SPDX-License-Identifier: MIT + * + * Copyright © 2019 Intel Corporation + */ + +#include "gt/intel_engine_pm.h" +#include "i915_selftest.h" + +#include "gem/selftests/mock_context.h" +#include "selftests/igt_reset.h" +#include "selftests/igt_spinner.h" + +struct live_mocs { + struct drm_i915_mocs_table table; + struct i915_vma *scratch; + void *vaddr; +}; + +static int request_add_sync(struct i915_request *rq, int err) +{ + i915_request_get(rq); + i915_request_add(rq); + if (i915_request_wait(rq, 0, HZ / 5) < 0) + err = -ETIME; + i915_request_put(rq); + + return err; +} + +static int request_add_spin(struct i915_request *rq, struct igt_spinner *spin) +{ + int err = 0; + + i915_request_get(rq); + i915_request_add(rq); + if (spin && !igt_wait_for_spinner(spin, rq)) + err = -ETIME; + i915_request_put(rq); + + return err; +} + +static struct i915_vma *create_scratch(struct intel_gt *gt) +{ + struct drm_i915_gem_object *obj; + struct i915_vma *vma; + int err; + + obj = i915_gem_object_create_internal(gt->i915, PAGE_SIZE); + if (IS_ERR(obj)) + return ERR_CAST(obj); + + i915_gem_object_set_cache_coherency(obj, I915_CACHING_CACHED); + + vma = i915_vma_instance(obj, >->ggtt->vm, NULL); + if (IS_ERR(vma)) { + i915_gem_object_put(obj); + return vma; + } + + err = i915_vma_pin(vma, 0, 0, PIN_GLOBAL); + if (err) { + i915_gem_object_put(obj); + return ERR_PTR(err); + } + + return vma; +} + +static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) +{ + int err; + + if (!get_mocs_settings(gt->i915, &arg->table)) + return -EINVAL; + + arg->scratch = create_scratch(gt); + if (IS_ERR(arg->scratch)) + return PTR_ERR(arg->scratch); + + arg->vaddr = i915_gem_object_pin_map(arg->scratch->obj, I915_MAP_WB); + if (IS_ERR(arg->vaddr)) { + err = PTR_ERR(arg->vaddr); + goto err_scratch; + } + + return 0; + +err_scratch: + i915_vma_unpin_and_release(&arg->scratch, 0); + return err; +} + +static void live_mocs_fini(struct live_mocs *arg) +{ + i915_vma_unpin_and_release(&arg->scratch, I915_VMA_RELEASE_MAP); +} + +static int read_mocs_table(struct i915_request *rq, + const struct drm_i915_mocs_table *table, + uint32_t *offset) +{ + unsigned int i; + u32 addr; + u32 *cs; + + GEM_BUG_ON(!IS_ALIGNED(*offset, sizeof(u32))); + + if (HAS_GLOBAL_MOCS_REGISTERS(rq->i915)) + addr = 0x4000; + else + addr = mocs_register(rq->engine); + + cs = intel_ring_begin(rq, 4 * table->n_entries); + if (IS_ERR(cs)) + return PTR_ERR(cs); + + for (i = 0; i < table->n_entries; i++) { + *cs++ = MI_STORE_REGISTER_MEM_GEN8 | MI_USE_GGTT; + *cs++ = addr; + *cs++ = *offset; + *cs++ = 0; + + addr += sizeof(u32); + *offset += sizeof(u32); + } + + intel_ring_advance(rq, cs); + + return 0; +} + +static int read_l3cc_table(struct i915_request *rq, + const struct drm_i915_mocs_table *table, + uint32_t *offset) +{ + u32 addr = i915_mmio_reg_offset(GEN9_LNCFCMOCS(0)); + unsigned int i; + u32 *cs; + + GEM_BUG_ON(!IS_ALIGNED(*offset, sizeof(u32))); + + cs = intel_ring_begin(rq, (table->n_entries + 1) / 2 * 4); + if (IS_ERR(cs)) + return PTR_ERR(cs); + + /* Can we read the MCR range 0xb00 directly? See intel_workarounds! */ + for (i = 0; i < (table->n_entries + 1) / 2; i++) { + *cs++ = MI_STORE_REGISTER_MEM_GEN8 | MI_USE_GGTT; + *cs++ = addr; + *cs++ = *offset; + *cs++ = 0; + + addr += sizeof(u32); + *offset += sizeof(u32); + } + + intel_ring_advance(rq, cs); + + return 0; +} + +static int check_mocs_table(struct intel_engine_cs *engine, + const struct drm_i915_mocs_table *table, + uint32_t **vaddr) +{ + unsigned int i; + u32 expect; + + for_each_mocs(expect, table, i) { + if (**vaddr != expect) { + pr_err("%s: Invalid MOCS[%d] entry, found %08x, expected %08x\n", + engine->name, i, **vaddr, expect); + return -EINVAL; + } + ++*vaddr; + } + + return 0; +} + +static int check_l3cc_table(struct intel_engine_cs *engine, + const struct drm_i915_mocs_table *table, + uint32_t **vaddr) +{ + unsigned int i; + u32 expect; + + for_each_l3cc(expect, table, i) { + if (**vaddr != expect) { + pr_err("%s: Invalid L3CC[%d] entry, found %08x, expected %08x\n", + engine->name, i, **vaddr, expect); + return -EINVAL; + } + ++*vaddr; + } + + return 0; +} + +static int check_mocs_engine(struct live_mocs *arg, + struct intel_context *ce) +{ + struct i915_vma *vma = arg->scratch; + struct i915_request *rq; + u32 offset; + u32 *vaddr; + int err; + + memset32(arg->vaddr, STACK_MAGIC, PAGE_SIZE / sizeof(u32)); + + rq = intel_context_create_request(ce); + if (IS_ERR(rq)) + return PTR_ERR(rq); + + i915_vma_lock(vma); + err = i915_request_await_object(rq, vma->obj, true); + if (!err) + err = i915_vma_move_to_active(vma, rq, EXEC_OBJECT_WRITE); + i915_vma_unlock(vma); + + offset = i915_ggtt_offset(vma); + if (!err) + err = read_mocs_table(rq, &arg->table, &offset); + if (!err && ce->engine->class == RENDER_CLASS) + err = read_l3cc_table(rq, &arg->table, &offset); + offset -= i915_ggtt_offset(vma); + GEM_BUG_ON(offset > PAGE_SIZE); + + err = request_add_sync(rq, err); + if (err) + return err; + + vaddr = arg->vaddr; + if (!err) + 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); + if (err) + return err; + + GEM_BUG_ON(arg->vaddr + offset != vaddr); + return 0; +} + +static int live_mocs_kernel(void *arg) +{ + struct intel_gt *gt = arg; + struct intel_engine_cs *engine; + enum intel_engine_id id; + struct live_mocs mocs; + int err; + + err = live_mocs_init(&mocs, gt); + if (err) + return err; + + for_each_engine(engine, gt, id) { + err = check_mocs_engine(&mocs, engine->kernel_context); + if (err) + break; + } + + live_mocs_fini(&mocs); + return err; +} + +static int live_mocs_clean(void *arg) +{ + struct intel_gt *gt = arg; + struct intel_engine_cs *engine; + enum intel_engine_id id; + struct live_mocs mocs; + int err; + + err = live_mocs_init(&mocs, gt); + if (err) + return err; + + for_each_engine(engine, gt, id) { + struct intel_context *ce; + + ce = intel_context_create(engine->kernel_context->gem_context, + engine); + if (IS_ERR(ce)) { + err = PTR_ERR(ce); + break; + } + + err = check_mocs_engine(&mocs, ce); + intel_context_put(ce); + if (err) + break; + } + + live_mocs_fini(&mocs); + return err; +} + +static int active_engine_reset(struct intel_context *ce, + const char *reason) +{ + struct igt_spinner spin; + struct i915_request *rq; + int err; + + err = igt_spinner_init(&spin, ce->engine->gt); + if (err) + return err; + + rq = igt_spinner_create_request(&spin, ce, MI_NOOP); + if (IS_ERR(rq)) { + igt_spinner_fini(&spin); + return PTR_ERR(rq); + } + + err = request_add_spin(rq, &spin); + if (err == 0) + err = intel_engine_reset(ce->engine, reason); + + igt_spinner_end(&spin); + igt_spinner_fini(&spin); + + return err; +} + +static int __live_mocs_reset(struct live_mocs *mocs, + struct intel_context *ce) +{ + int err; + + err = intel_engine_reset(ce->engine, "mocs"); + if (err) + return err; + + err = check_mocs_engine(mocs, ce); + if (err) + return err; + + err = active_engine_reset(ce, "mocs"); + if (err) + return err; + + err = check_mocs_engine(mocs, ce); + if (err) + return err; + + return 0; +} + +static int live_mocs_reset(void *arg) +{ + struct intel_gt *gt = arg; + struct intel_engine_cs *engine; + enum intel_engine_id id; + struct live_mocs mocs; + int err = 0; + + if (!intel_has_reset_engine(gt)) + return 0; + + err = live_mocs_init(&mocs, gt); + if (err) + return err; + + igt_global_reset_lock(gt); + for_each_engine(engine, gt, id) { + struct intel_context *ce; + + ce = intel_context_create(engine->kernel_context->gem_context, + engine); + if (IS_ERR(ce)) { + err = PTR_ERR(ce); + break; + } + + intel_engine_pm_get(engine); + err = __live_mocs_reset(&mocs, ce); + intel_engine_pm_put(engine); + + intel_context_put(ce); + if (err) + break; + } + igt_global_reset_unlock(gt); + + live_mocs_fini(&mocs); + return err; +} + +int intel_mocs_live_selftests(struct drm_i915_private *i915) +{ + static const struct i915_subtest tests[] = { + SUBTEST(live_mocs_kernel), + SUBTEST(live_mocs_clean), + SUBTEST(live_mocs_reset), + }; + struct drm_i915_mocs_table table; + + if (!get_mocs_settings(i915, &table)) + return 0; + + return intel_gt_live_subtests(tests, &i915->gt); +} diff --git a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h index 00a063730bc3..c23d06bca09e 100644 --- a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h +++ b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h @@ -16,6 +16,7 @@ selftest(gt_engines, intel_engine_live_selftests) selftest(gt_timelines, intel_timeline_live_selftests) selftest(gt_contexts, intel_context_live_selftests) selftest(gt_lrc, intel_lrc_live_selftests) +selftest(gt_mocs, intel_mocs_live_selftests) selftest(gt_pm, intel_gt_pm_live_selftests) selftest(gt_heartbeat, intel_heartbeat_live_selftests) selftest(requests, i915_request_live_selftests) -- 2.24.0.rc0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH] drm/i915/selftests: Add coverage of mocs registers 2019-10-22 11:51 ` [PATCH 4/4] drm/i915/selftests: Add coverage of mocs registers Chris Wilson @ 2019-10-22 11:57 ` Chris Wilson 2019-10-23 21:03 ` [Intel-gfx] " Kumar Valsan, Prathap 0 siblings, 1 reply; 15+ messages in thread From: Chris Wilson @ 2019-10-22 11:57 UTC (permalink / raw) To: intel-gfx Probe the mocs registers for new contexts and across GPU resets. Similar to intel_workarounds, we have tables of what register values we expect to see, so verify that user contexts are affected by them. In the future, we should add tests similar to intel_sseu to cover dynamic reconfigurations. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> --- drivers/gpu/drm/i915/gt/intel_mocs.c | 4 + drivers/gpu/drm/i915/gt/selftest_mocs.c | 393 ++++++++++++++++++ .../drm/i915/selftests/i915_live_selftests.h | 1 + 3 files changed, 398 insertions(+) create mode 100644 drivers/gpu/drm/i915/gt/selftest_mocs.c diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c index 445ec025bda0..06dba7ff294e 100644 --- a/drivers/gpu/drm/i915/gt/intel_mocs.c +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c @@ -448,3 +448,7 @@ void intel_mocs_init(struct intel_gt *gt) if (HAS_GLOBAL_MOCS_REGISTERS(gt->i915)) init_global_mocs(gt); } + +#if IS_ENABLED(CONFIG_DRM_I915_SELFTEST) +#include "selftest_mocs.c" +#endif diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c new file mode 100644 index 000000000000..ca9679c3ee68 --- /dev/null +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c @@ -0,0 +1,393 @@ +/* + * SPDX-License-Identifier: MIT + * + * Copyright © 2019 Intel Corporation + */ + +#include "gt/intel_engine_pm.h" +#include "i915_selftest.h" + +#include "gem/selftests/mock_context.h" +#include "selftests/igt_reset.h" +#include "selftests/igt_spinner.h" + +struct live_mocs { + struct drm_i915_mocs_table table; + struct i915_vma *scratch; + void *vaddr; +}; + +static int request_add_sync(struct i915_request *rq, int err) +{ + i915_request_get(rq); + i915_request_add(rq); + if (i915_request_wait(rq, 0, HZ / 5) < 0) + err = -ETIME; + i915_request_put(rq); + + return err; +} + +static int request_add_spin(struct i915_request *rq, struct igt_spinner *spin) +{ + int err = 0; + + i915_request_get(rq); + i915_request_add(rq); + if (spin && !igt_wait_for_spinner(spin, rq)) + err = -ETIME; + i915_request_put(rq); + + return err; +} + +static struct i915_vma *create_scratch(struct intel_gt *gt) +{ + struct drm_i915_gem_object *obj; + struct i915_vma *vma; + int err; + + obj = i915_gem_object_create_internal(gt->i915, PAGE_SIZE); + if (IS_ERR(obj)) + return ERR_CAST(obj); + + i915_gem_object_set_cache_coherency(obj, I915_CACHING_CACHED); + + vma = i915_vma_instance(obj, >->ggtt->vm, NULL); + if (IS_ERR(vma)) { + i915_gem_object_put(obj); + return vma; + } + + err = i915_vma_pin(vma, 0, 0, PIN_GLOBAL); + if (err) { + i915_gem_object_put(obj); + return ERR_PTR(err); + } + + return vma; +} + +static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) +{ + int err; + + if (!get_mocs_settings(gt->i915, &arg->table)) + return -EINVAL; + + arg->scratch = create_scratch(gt); + if (IS_ERR(arg->scratch)) + return PTR_ERR(arg->scratch); + + arg->vaddr = i915_gem_object_pin_map(arg->scratch->obj, I915_MAP_WB); + if (IS_ERR(arg->vaddr)) { + err = PTR_ERR(arg->vaddr); + goto err_scratch; + } + + return 0; + +err_scratch: + i915_vma_unpin_and_release(&arg->scratch, 0); + return err; +} + +static void live_mocs_fini(struct live_mocs *arg) +{ + i915_vma_unpin_and_release(&arg->scratch, I915_VMA_RELEASE_MAP); +} + +static int read_regs(struct i915_request *rq, + u32 addr, unsigned int count, + uint32_t *offset) +{ + unsigned int i; + u32 *cs; + + GEM_BUG_ON(!IS_ALIGNED(*offset, sizeof(u32))); + + cs = intel_ring_begin(rq, 4 * count); + if (IS_ERR(cs)) + return PTR_ERR(cs); + + for (i = 0; i < count; i++) { + *cs++ = MI_STORE_REGISTER_MEM_GEN8 | MI_USE_GGTT; + *cs++ = addr; + *cs++ = *offset; + *cs++ = 0; + + addr += sizeof(u32); + *offset += sizeof(u32); + } + + intel_ring_advance(rq, cs); + + return 0; +} + +static int read_mocs_table(struct i915_request *rq, + const struct drm_i915_mocs_table *table, + uint32_t *offset) +{ + u32 addr; + + if (HAS_GLOBAL_MOCS_REGISTERS(rq->i915)) + addr = 0x4000; + else + addr = mocs_register(rq->engine); + + return read_regs(rq, addr, table->n_entries, offset); +} + +static int read_l3cc_table(struct i915_request *rq, + const struct drm_i915_mocs_table *table, + uint32_t *offset) +{ + /* Can we read the MCR range 0xb00 directly? See intel_workarounds! */ + u32 addr = i915_mmio_reg_offset(GEN9_LNCFCMOCS(0)); + + return read_regs(rq, addr, (table->n_entries + 1) / 2, offset); +} + +static int check_mocs_table(struct intel_engine_cs *engine, + const struct drm_i915_mocs_table *table, + uint32_t **vaddr) +{ + unsigned int i; + u32 expect; + + for_each_mocs(expect, table, i) { + if (**vaddr != expect) { + pr_err("%s: Invalid MOCS[%d] entry, found %08x, expected %08x\n", + engine->name, i, **vaddr, expect); + return -EINVAL; + } + ++*vaddr; + } + + return 0; +} + +static int check_l3cc_table(struct intel_engine_cs *engine, + const struct drm_i915_mocs_table *table, + uint32_t **vaddr) +{ + unsigned int i; + u32 expect; + + for_each_l3cc(expect, table, i) { + if (**vaddr != expect) { + pr_err("%s: Invalid L3CC[%d] entry, found %08x, expected %08x\n", + engine->name, i, **vaddr, expect); + return -EINVAL; + } + ++*vaddr; + } + + return 0; +} + +static int check_mocs_engine(struct live_mocs *arg, + struct intel_context *ce) +{ + struct i915_vma *vma = arg->scratch; + struct i915_request *rq; + u32 offset; + u32 *vaddr; + int err; + + memset32(arg->vaddr, STACK_MAGIC, PAGE_SIZE / sizeof(u32)); + + rq = intel_context_create_request(ce); + if (IS_ERR(rq)) + return PTR_ERR(rq); + + i915_vma_lock(vma); + err = i915_request_await_object(rq, vma->obj, true); + if (!err) + err = i915_vma_move_to_active(vma, rq, EXEC_OBJECT_WRITE); + i915_vma_unlock(vma); + + offset = i915_ggtt_offset(vma); + if (!err) + err = read_mocs_table(rq, &arg->table, &offset); + if (!err && ce->engine->class == RENDER_CLASS) + err = read_l3cc_table(rq, &arg->table, &offset); + offset -= i915_ggtt_offset(vma); + GEM_BUG_ON(offset > PAGE_SIZE); + + err = request_add_sync(rq, err); + if (err) + return err; + + vaddr = arg->vaddr; + if (!err) + 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); + if (err) + return err; + + GEM_BUG_ON(arg->vaddr + offset != vaddr); + return 0; +} + +static int live_mocs_kernel(void *arg) +{ + struct intel_gt *gt = arg; + struct intel_engine_cs *engine; + enum intel_engine_id id; + struct live_mocs mocs; + int err; + + err = live_mocs_init(&mocs, gt); + if (err) + return err; + + for_each_engine(engine, gt, id) { + err = check_mocs_engine(&mocs, engine->kernel_context); + if (err) + break; + } + + live_mocs_fini(&mocs); + return err; +} + +static int live_mocs_clean(void *arg) +{ + struct intel_gt *gt = arg; + struct intel_engine_cs *engine; + enum intel_engine_id id; + struct live_mocs mocs; + int err; + + err = live_mocs_init(&mocs, gt); + if (err) + return err; + + for_each_engine(engine, gt, id) { + struct intel_context *ce; + + ce = intel_context_create(engine->kernel_context->gem_context, + engine); + if (IS_ERR(ce)) { + err = PTR_ERR(ce); + break; + } + + err = check_mocs_engine(&mocs, ce); + intel_context_put(ce); + if (err) + break; + } + + live_mocs_fini(&mocs); + return err; +} + +static int active_engine_reset(struct intel_context *ce, + const char *reason) +{ + struct igt_spinner spin; + struct i915_request *rq; + int err; + + err = igt_spinner_init(&spin, ce->engine->gt); + if (err) + return err; + + rq = igt_spinner_create_request(&spin, ce, MI_NOOP); + if (IS_ERR(rq)) { + igt_spinner_fini(&spin); + return PTR_ERR(rq); + } + + err = request_add_spin(rq, &spin); + if (err == 0) + err = intel_engine_reset(ce->engine, reason); + + igt_spinner_end(&spin); + igt_spinner_fini(&spin); + + return err; +} + +static int __live_mocs_reset(struct live_mocs *mocs, + struct intel_context *ce) +{ + int err; + + err = intel_engine_reset(ce->engine, "mocs"); + if (err) + return err; + + err = check_mocs_engine(mocs, ce); + if (err) + return err; + + err = active_engine_reset(ce, "mocs"); + if (err) + return err; + + err = check_mocs_engine(mocs, ce); + if (err) + return err; + + return 0; +} + +static int live_mocs_reset(void *arg) +{ + struct intel_gt *gt = arg; + struct intel_engine_cs *engine; + enum intel_engine_id id; + struct live_mocs mocs; + int err = 0; + + if (!intel_has_reset_engine(gt)) + return 0; + + err = live_mocs_init(&mocs, gt); + if (err) + return err; + + igt_global_reset_lock(gt); + for_each_engine(engine, gt, id) { + struct intel_context *ce; + + ce = intel_context_create(engine->kernel_context->gem_context, + engine); + if (IS_ERR(ce)) { + err = PTR_ERR(ce); + break; + } + + intel_engine_pm_get(engine); + err = __live_mocs_reset(&mocs, ce); + intel_engine_pm_put(engine); + + intel_context_put(ce); + if (err) + break; + } + igt_global_reset_unlock(gt); + + live_mocs_fini(&mocs); + return err; +} + +int intel_mocs_live_selftests(struct drm_i915_private *i915) +{ + static const struct i915_subtest tests[] = { + SUBTEST(live_mocs_kernel), + SUBTEST(live_mocs_clean), + SUBTEST(live_mocs_reset), + }; + struct drm_i915_mocs_table table; + + if (!get_mocs_settings(i915, &table)) + return 0; + + return intel_gt_live_subtests(tests, &i915->gt); +} diff --git a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h index 00a063730bc3..c23d06bca09e 100644 --- a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h +++ b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h @@ -16,6 +16,7 @@ selftest(gt_engines, intel_engine_live_selftests) selftest(gt_timelines, intel_timeline_live_selftests) selftest(gt_contexts, intel_context_live_selftests) selftest(gt_lrc, intel_lrc_live_selftests) +selftest(gt_mocs, intel_mocs_live_selftests) selftest(gt_pm, intel_gt_pm_live_selftests) selftest(gt_heartbeat, intel_heartbeat_live_selftests) selftest(requests, i915_request_live_selftests) -- 2.24.0.rc0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] drm/i915/selftests: Add coverage of mocs registers @ 2019-10-23 21:03 ` Kumar Valsan, Prathap 0 siblings, 0 replies; 15+ messages in thread From: Kumar Valsan, Prathap @ 2019-10-23 21:03 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx On Tue, Oct 22, 2019 at 12:57:05PM +0100, Chris Wilson wrote: > Probe the mocs registers for new contexts and across GPU resets. Similar > to intel_workarounds, we have tables of what register values we expect > to see, so verify that user contexts are affected by them. In the > future, we should add tests similar to intel_sseu to cover dynamic > reconfigurations. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> s/for_each_engine/for_each_uabi_engine ? Otherwise Reviewed-by: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > --- > drivers/gpu/drm/i915/gt/intel_mocs.c | 4 + > drivers/gpu/drm/i915/gt/selftest_mocs.c | 393 ++++++++++++++++++ > .../drm/i915/selftests/i915_live_selftests.h | 1 + > 3 files changed, 398 insertions(+) > create mode 100644 drivers/gpu/drm/i915/gt/selftest_mocs.c > > diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c > index 445ec025bda0..06dba7ff294e 100644 > --- a/drivers/gpu/drm/i915/gt/intel_mocs.c > +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c > @@ -448,3 +448,7 @@ void intel_mocs_init(struct intel_gt *gt) > if (HAS_GLOBAL_MOCS_REGISTERS(gt->i915)) > init_global_mocs(gt); > } > + > +#if IS_ENABLED(CONFIG_DRM_I915_SELFTEST) > +#include "selftest_mocs.c" > +#endif > diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c > new file mode 100644 > index 000000000000..ca9679c3ee68 > --- /dev/null > +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c > @@ -0,0 +1,393 @@ > +/* > + * SPDX-License-Identifier: MIT > + * > + * Copyright © 2019 Intel Corporation > + */ > + > +#include "gt/intel_engine_pm.h" > +#include "i915_selftest.h" > + > +#include "gem/selftests/mock_context.h" > +#include "selftests/igt_reset.h" > +#include "selftests/igt_spinner.h" > + > +struct live_mocs { > + struct drm_i915_mocs_table table; > + struct i915_vma *scratch; > + void *vaddr; > +}; > + > +static int request_add_sync(struct i915_request *rq, int err) > +{ > + i915_request_get(rq); > + i915_request_add(rq); > + if (i915_request_wait(rq, 0, HZ / 5) < 0) > + err = -ETIME; > + i915_request_put(rq); > + > + return err; > +} > + > +static int request_add_spin(struct i915_request *rq, struct igt_spinner *spin) > +{ > + int err = 0; > + > + i915_request_get(rq); > + i915_request_add(rq); > + if (spin && !igt_wait_for_spinner(spin, rq)) > + err = -ETIME; > + i915_request_put(rq); > + > + return err; > +} > + > +static struct i915_vma *create_scratch(struct intel_gt *gt) > +{ > + struct drm_i915_gem_object *obj; > + struct i915_vma *vma; > + int err; > + > + obj = i915_gem_object_create_internal(gt->i915, PAGE_SIZE); > + if (IS_ERR(obj)) > + return ERR_CAST(obj); > + > + i915_gem_object_set_cache_coherency(obj, I915_CACHING_CACHED); > + > + vma = i915_vma_instance(obj, >->ggtt->vm, NULL); > + if (IS_ERR(vma)) { > + i915_gem_object_put(obj); > + return vma; > + } > + > + err = i915_vma_pin(vma, 0, 0, PIN_GLOBAL); > + if (err) { > + i915_gem_object_put(obj); > + return ERR_PTR(err); > + } > + > + return vma; > +} > + > +static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) > +{ > + int err; > + > + if (!get_mocs_settings(gt->i915, &arg->table)) > + return -EINVAL; > + > + arg->scratch = create_scratch(gt); > + if (IS_ERR(arg->scratch)) > + return PTR_ERR(arg->scratch); > + > + arg->vaddr = i915_gem_object_pin_map(arg->scratch->obj, I915_MAP_WB); > + if (IS_ERR(arg->vaddr)) { > + err = PTR_ERR(arg->vaddr); > + goto err_scratch; > + } > + > + return 0; > + > +err_scratch: > + i915_vma_unpin_and_release(&arg->scratch, 0); > + return err; > +} > + > +static void live_mocs_fini(struct live_mocs *arg) > +{ > + i915_vma_unpin_and_release(&arg->scratch, I915_VMA_RELEASE_MAP); > +} > + > +static int read_regs(struct i915_request *rq, > + u32 addr, unsigned int count, > + uint32_t *offset) > +{ > + unsigned int i; > + u32 *cs; > + > + GEM_BUG_ON(!IS_ALIGNED(*offset, sizeof(u32))); > + > + cs = intel_ring_begin(rq, 4 * count); > + if (IS_ERR(cs)) > + return PTR_ERR(cs); > + > + for (i = 0; i < count; i++) { > + *cs++ = MI_STORE_REGISTER_MEM_GEN8 | MI_USE_GGTT; > + *cs++ = addr; > + *cs++ = *offset; > + *cs++ = 0; > + > + addr += sizeof(u32); > + *offset += sizeof(u32); > + } > + > + intel_ring_advance(rq, cs); > + > + return 0; > +} > + > +static int read_mocs_table(struct i915_request *rq, > + const struct drm_i915_mocs_table *table, > + uint32_t *offset) > +{ > + u32 addr; > + > + if (HAS_GLOBAL_MOCS_REGISTERS(rq->i915)) > + addr = 0x4000; > + else > + addr = mocs_register(rq->engine); > + > + return read_regs(rq, addr, table->n_entries, offset); > +} > + > +static int read_l3cc_table(struct i915_request *rq, > + const struct drm_i915_mocs_table *table, > + uint32_t *offset) > +{ > + /* Can we read the MCR range 0xb00 directly? See intel_workarounds! */ > + u32 addr = i915_mmio_reg_offset(GEN9_LNCFCMOCS(0)); > + > + return read_regs(rq, addr, (table->n_entries + 1) / 2, offset); > +} > + > +static int check_mocs_table(struct intel_engine_cs *engine, > + const struct drm_i915_mocs_table *table, > + uint32_t **vaddr) > +{ > + unsigned int i; > + u32 expect; > + > + for_each_mocs(expect, table, i) { > + if (**vaddr != expect) { > + pr_err("%s: Invalid MOCS[%d] entry, found %08x, expected %08x\n", > + engine->name, i, **vaddr, expect); > + return -EINVAL; > + } > + ++*vaddr; > + } > + > + return 0; > +} > + > +static int check_l3cc_table(struct intel_engine_cs *engine, > + const struct drm_i915_mocs_table *table, > + uint32_t **vaddr) > +{ > + unsigned int i; > + u32 expect; > + > + for_each_l3cc(expect, table, i) { > + if (**vaddr != expect) { > + pr_err("%s: Invalid L3CC[%d] entry, found %08x, expected %08x\n", > + engine->name, i, **vaddr, expect); > + return -EINVAL; > + } > + ++*vaddr; > + } > + > + return 0; > +} > + > +static int check_mocs_engine(struct live_mocs *arg, > + struct intel_context *ce) > +{ > + struct i915_vma *vma = arg->scratch; > + struct i915_request *rq; > + u32 offset; > + u32 *vaddr; > + int err; > + > + memset32(arg->vaddr, STACK_MAGIC, PAGE_SIZE / sizeof(u32)); > + > + rq = intel_context_create_request(ce); > + if (IS_ERR(rq)) > + return PTR_ERR(rq); > + > + i915_vma_lock(vma); > + err = i915_request_await_object(rq, vma->obj, true); > + if (!err) > + err = i915_vma_move_to_active(vma, rq, EXEC_OBJECT_WRITE); > + i915_vma_unlock(vma); > + > + offset = i915_ggtt_offset(vma); > + if (!err) > + err = read_mocs_table(rq, &arg->table, &offset); > + if (!err && ce->engine->class == RENDER_CLASS) > + err = read_l3cc_table(rq, &arg->table, &offset); > + offset -= i915_ggtt_offset(vma); > + GEM_BUG_ON(offset > PAGE_SIZE); > + > + err = request_add_sync(rq, err); > + if (err) > + return err; > + > + vaddr = arg->vaddr; > + if (!err) > + 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); > + if (err) > + return err; > + > + GEM_BUG_ON(arg->vaddr + offset != vaddr); > + return 0; > +} > + > +static int live_mocs_kernel(void *arg) > +{ > + struct intel_gt *gt = arg; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + struct live_mocs mocs; > + int err; > + > + err = live_mocs_init(&mocs, gt); > + if (err) > + return err; > + > + for_each_engine(engine, gt, id) { > + err = check_mocs_engine(&mocs, engine->kernel_context); > + if (err) > + break; > + } > + > + live_mocs_fini(&mocs); > + return err; > +} > + > +static int live_mocs_clean(void *arg) > +{ > + struct intel_gt *gt = arg; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + struct live_mocs mocs; > + int err; > + > + err = live_mocs_init(&mocs, gt); > + if (err) > + return err; > + > + for_each_engine(engine, gt, id) { > + struct intel_context *ce; > + > + ce = intel_context_create(engine->kernel_context->gem_context, > + engine); > + if (IS_ERR(ce)) { > + err = PTR_ERR(ce); > + break; > + } > + > + err = check_mocs_engine(&mocs, ce); > + intel_context_put(ce); > + if (err) > + break; > + } > + > + live_mocs_fini(&mocs); > + return err; > +} > + > +static int active_engine_reset(struct intel_context *ce, > + const char *reason) > +{ > + struct igt_spinner spin; > + struct i915_request *rq; > + int err; > + > + err = igt_spinner_init(&spin, ce->engine->gt); > + if (err) > + return err; > + > + rq = igt_spinner_create_request(&spin, ce, MI_NOOP); > + if (IS_ERR(rq)) { > + igt_spinner_fini(&spin); > + return PTR_ERR(rq); > + } > + > + err = request_add_spin(rq, &spin); > + if (err == 0) > + err = intel_engine_reset(ce->engine, reason); > + > + igt_spinner_end(&spin); > + igt_spinner_fini(&spin); > + > + return err; > +} > + > +static int __live_mocs_reset(struct live_mocs *mocs, > + struct intel_context *ce) > +{ > + int err; > + > + err = intel_engine_reset(ce->engine, "mocs"); > + if (err) > + return err; > + > + err = check_mocs_engine(mocs, ce); > + if (err) > + return err; > + > + err = active_engine_reset(ce, "mocs"); > + if (err) > + return err; > + > + err = check_mocs_engine(mocs, ce); > + if (err) > + return err; > + > + return 0; > +} > + > +static int live_mocs_reset(void *arg) > +{ > + struct intel_gt *gt = arg; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + struct live_mocs mocs; > + int err = 0; > + > + if (!intel_has_reset_engine(gt)) > + return 0; > + > + err = live_mocs_init(&mocs, gt); > + if (err) > + return err; > + > + igt_global_reset_lock(gt); > + for_each_engine(engine, gt, id) { > + struct intel_context *ce; > + > + ce = intel_context_create(engine->kernel_context->gem_context, > + engine); > + if (IS_ERR(ce)) { > + err = PTR_ERR(ce); > + break; > + } > + > + intel_engine_pm_get(engine); > + err = __live_mocs_reset(&mocs, ce); > + intel_engine_pm_put(engine); > + > + intel_context_put(ce); > + if (err) > + break; > + } > + igt_global_reset_unlock(gt); > + > + live_mocs_fini(&mocs); > + return err; > +} > + > +int intel_mocs_live_selftests(struct drm_i915_private *i915) > +{ > + static const struct i915_subtest tests[] = { > + SUBTEST(live_mocs_kernel), > + SUBTEST(live_mocs_clean), > + SUBTEST(live_mocs_reset), > + }; > + struct drm_i915_mocs_table table; > + > + if (!get_mocs_settings(i915, &table)) > + return 0; > + > + return intel_gt_live_subtests(tests, &i915->gt); > +} > diff --git a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h > index 00a063730bc3..c23d06bca09e 100644 > --- a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h > +++ b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h > @@ -16,6 +16,7 @@ selftest(gt_engines, intel_engine_live_selftests) > selftest(gt_timelines, intel_timeline_live_selftests) > selftest(gt_contexts, intel_context_live_selftests) > selftest(gt_lrc, intel_lrc_live_selftests) > +selftest(gt_mocs, intel_mocs_live_selftests) > selftest(gt_pm, intel_gt_pm_live_selftests) > selftest(gt_heartbeat, intel_heartbeat_live_selftests) > selftest(requests, i915_request_live_selftests) > -- > 2.24.0.rc0 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Add coverage of mocs registers @ 2019-10-23 21:03 ` Kumar Valsan, Prathap 0 siblings, 0 replies; 15+ messages in thread From: Kumar Valsan, Prathap @ 2019-10-23 21:03 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx On Tue, Oct 22, 2019 at 12:57:05PM +0100, Chris Wilson wrote: > Probe the mocs registers for new contexts and across GPU resets. Similar > to intel_workarounds, we have tables of what register values we expect > to see, so verify that user contexts are affected by them. In the > future, we should add tests similar to intel_sseu to cover dynamic > reconfigurations. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> s/for_each_engine/for_each_uabi_engine ? Otherwise Reviewed-by: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > --- > drivers/gpu/drm/i915/gt/intel_mocs.c | 4 + > drivers/gpu/drm/i915/gt/selftest_mocs.c | 393 ++++++++++++++++++ > .../drm/i915/selftests/i915_live_selftests.h | 1 + > 3 files changed, 398 insertions(+) > create mode 100644 drivers/gpu/drm/i915/gt/selftest_mocs.c > > diff --git a/drivers/gpu/drm/i915/gt/intel_mocs.c b/drivers/gpu/drm/i915/gt/intel_mocs.c > index 445ec025bda0..06dba7ff294e 100644 > --- a/drivers/gpu/drm/i915/gt/intel_mocs.c > +++ b/drivers/gpu/drm/i915/gt/intel_mocs.c > @@ -448,3 +448,7 @@ void intel_mocs_init(struct intel_gt *gt) > if (HAS_GLOBAL_MOCS_REGISTERS(gt->i915)) > init_global_mocs(gt); > } > + > +#if IS_ENABLED(CONFIG_DRM_I915_SELFTEST) > +#include "selftest_mocs.c" > +#endif > diff --git a/drivers/gpu/drm/i915/gt/selftest_mocs.c b/drivers/gpu/drm/i915/gt/selftest_mocs.c > new file mode 100644 > index 000000000000..ca9679c3ee68 > --- /dev/null > +++ b/drivers/gpu/drm/i915/gt/selftest_mocs.c > @@ -0,0 +1,393 @@ > +/* > + * SPDX-License-Identifier: MIT > + * > + * Copyright © 2019 Intel Corporation > + */ > + > +#include "gt/intel_engine_pm.h" > +#include "i915_selftest.h" > + > +#include "gem/selftests/mock_context.h" > +#include "selftests/igt_reset.h" > +#include "selftests/igt_spinner.h" > + > +struct live_mocs { > + struct drm_i915_mocs_table table; > + struct i915_vma *scratch; > + void *vaddr; > +}; > + > +static int request_add_sync(struct i915_request *rq, int err) > +{ > + i915_request_get(rq); > + i915_request_add(rq); > + if (i915_request_wait(rq, 0, HZ / 5) < 0) > + err = -ETIME; > + i915_request_put(rq); > + > + return err; > +} > + > +static int request_add_spin(struct i915_request *rq, struct igt_spinner *spin) > +{ > + int err = 0; > + > + i915_request_get(rq); > + i915_request_add(rq); > + if (spin && !igt_wait_for_spinner(spin, rq)) > + err = -ETIME; > + i915_request_put(rq); > + > + return err; > +} > + > +static struct i915_vma *create_scratch(struct intel_gt *gt) > +{ > + struct drm_i915_gem_object *obj; > + struct i915_vma *vma; > + int err; > + > + obj = i915_gem_object_create_internal(gt->i915, PAGE_SIZE); > + if (IS_ERR(obj)) > + return ERR_CAST(obj); > + > + i915_gem_object_set_cache_coherency(obj, I915_CACHING_CACHED); > + > + vma = i915_vma_instance(obj, >->ggtt->vm, NULL); > + if (IS_ERR(vma)) { > + i915_gem_object_put(obj); > + return vma; > + } > + > + err = i915_vma_pin(vma, 0, 0, PIN_GLOBAL); > + if (err) { > + i915_gem_object_put(obj); > + return ERR_PTR(err); > + } > + > + return vma; > +} > + > +static int live_mocs_init(struct live_mocs *arg, struct intel_gt *gt) > +{ > + int err; > + > + if (!get_mocs_settings(gt->i915, &arg->table)) > + return -EINVAL; > + > + arg->scratch = create_scratch(gt); > + if (IS_ERR(arg->scratch)) > + return PTR_ERR(arg->scratch); > + > + arg->vaddr = i915_gem_object_pin_map(arg->scratch->obj, I915_MAP_WB); > + if (IS_ERR(arg->vaddr)) { > + err = PTR_ERR(arg->vaddr); > + goto err_scratch; > + } > + > + return 0; > + > +err_scratch: > + i915_vma_unpin_and_release(&arg->scratch, 0); > + return err; > +} > + > +static void live_mocs_fini(struct live_mocs *arg) > +{ > + i915_vma_unpin_and_release(&arg->scratch, I915_VMA_RELEASE_MAP); > +} > + > +static int read_regs(struct i915_request *rq, > + u32 addr, unsigned int count, > + uint32_t *offset) > +{ > + unsigned int i; > + u32 *cs; > + > + GEM_BUG_ON(!IS_ALIGNED(*offset, sizeof(u32))); > + > + cs = intel_ring_begin(rq, 4 * count); > + if (IS_ERR(cs)) > + return PTR_ERR(cs); > + > + for (i = 0; i < count; i++) { > + *cs++ = MI_STORE_REGISTER_MEM_GEN8 | MI_USE_GGTT; > + *cs++ = addr; > + *cs++ = *offset; > + *cs++ = 0; > + > + addr += sizeof(u32); > + *offset += sizeof(u32); > + } > + > + intel_ring_advance(rq, cs); > + > + return 0; > +} > + > +static int read_mocs_table(struct i915_request *rq, > + const struct drm_i915_mocs_table *table, > + uint32_t *offset) > +{ > + u32 addr; > + > + if (HAS_GLOBAL_MOCS_REGISTERS(rq->i915)) > + addr = 0x4000; > + else > + addr = mocs_register(rq->engine); > + > + return read_regs(rq, addr, table->n_entries, offset); > +} > + > +static int read_l3cc_table(struct i915_request *rq, > + const struct drm_i915_mocs_table *table, > + uint32_t *offset) > +{ > + /* Can we read the MCR range 0xb00 directly? See intel_workarounds! */ > + u32 addr = i915_mmio_reg_offset(GEN9_LNCFCMOCS(0)); > + > + return read_regs(rq, addr, (table->n_entries + 1) / 2, offset); > +} > + > +static int check_mocs_table(struct intel_engine_cs *engine, > + const struct drm_i915_mocs_table *table, > + uint32_t **vaddr) > +{ > + unsigned int i; > + u32 expect; > + > + for_each_mocs(expect, table, i) { > + if (**vaddr != expect) { > + pr_err("%s: Invalid MOCS[%d] entry, found %08x, expected %08x\n", > + engine->name, i, **vaddr, expect); > + return -EINVAL; > + } > + ++*vaddr; > + } > + > + return 0; > +} > + > +static int check_l3cc_table(struct intel_engine_cs *engine, > + const struct drm_i915_mocs_table *table, > + uint32_t **vaddr) > +{ > + unsigned int i; > + u32 expect; > + > + for_each_l3cc(expect, table, i) { > + if (**vaddr != expect) { > + pr_err("%s: Invalid L3CC[%d] entry, found %08x, expected %08x\n", > + engine->name, i, **vaddr, expect); > + return -EINVAL; > + } > + ++*vaddr; > + } > + > + return 0; > +} > + > +static int check_mocs_engine(struct live_mocs *arg, > + struct intel_context *ce) > +{ > + struct i915_vma *vma = arg->scratch; > + struct i915_request *rq; > + u32 offset; > + u32 *vaddr; > + int err; > + > + memset32(arg->vaddr, STACK_MAGIC, PAGE_SIZE / sizeof(u32)); > + > + rq = intel_context_create_request(ce); > + if (IS_ERR(rq)) > + return PTR_ERR(rq); > + > + i915_vma_lock(vma); > + err = i915_request_await_object(rq, vma->obj, true); > + if (!err) > + err = i915_vma_move_to_active(vma, rq, EXEC_OBJECT_WRITE); > + i915_vma_unlock(vma); > + > + offset = i915_ggtt_offset(vma); > + if (!err) > + err = read_mocs_table(rq, &arg->table, &offset); > + if (!err && ce->engine->class == RENDER_CLASS) > + err = read_l3cc_table(rq, &arg->table, &offset); > + offset -= i915_ggtt_offset(vma); > + GEM_BUG_ON(offset > PAGE_SIZE); > + > + err = request_add_sync(rq, err); > + if (err) > + return err; > + > + vaddr = arg->vaddr; > + if (!err) > + 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); > + if (err) > + return err; > + > + GEM_BUG_ON(arg->vaddr + offset != vaddr); > + return 0; > +} > + > +static int live_mocs_kernel(void *arg) > +{ > + struct intel_gt *gt = arg; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + struct live_mocs mocs; > + int err; > + > + err = live_mocs_init(&mocs, gt); > + if (err) > + return err; > + > + for_each_engine(engine, gt, id) { > + err = check_mocs_engine(&mocs, engine->kernel_context); > + if (err) > + break; > + } > + > + live_mocs_fini(&mocs); > + return err; > +} > + > +static int live_mocs_clean(void *arg) > +{ > + struct intel_gt *gt = arg; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + struct live_mocs mocs; > + int err; > + > + err = live_mocs_init(&mocs, gt); > + if (err) > + return err; > + > + for_each_engine(engine, gt, id) { > + struct intel_context *ce; > + > + ce = intel_context_create(engine->kernel_context->gem_context, > + engine); > + if (IS_ERR(ce)) { > + err = PTR_ERR(ce); > + break; > + } > + > + err = check_mocs_engine(&mocs, ce); > + intel_context_put(ce); > + if (err) > + break; > + } > + > + live_mocs_fini(&mocs); > + return err; > +} > + > +static int active_engine_reset(struct intel_context *ce, > + const char *reason) > +{ > + struct igt_spinner spin; > + struct i915_request *rq; > + int err; > + > + err = igt_spinner_init(&spin, ce->engine->gt); > + if (err) > + return err; > + > + rq = igt_spinner_create_request(&spin, ce, MI_NOOP); > + if (IS_ERR(rq)) { > + igt_spinner_fini(&spin); > + return PTR_ERR(rq); > + } > + > + err = request_add_spin(rq, &spin); > + if (err == 0) > + err = intel_engine_reset(ce->engine, reason); > + > + igt_spinner_end(&spin); > + igt_spinner_fini(&spin); > + > + return err; > +} > + > +static int __live_mocs_reset(struct live_mocs *mocs, > + struct intel_context *ce) > +{ > + int err; > + > + err = intel_engine_reset(ce->engine, "mocs"); > + if (err) > + return err; > + > + err = check_mocs_engine(mocs, ce); > + if (err) > + return err; > + > + err = active_engine_reset(ce, "mocs"); > + if (err) > + return err; > + > + err = check_mocs_engine(mocs, ce); > + if (err) > + return err; > + > + return 0; > +} > + > +static int live_mocs_reset(void *arg) > +{ > + struct intel_gt *gt = arg; > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + struct live_mocs mocs; > + int err = 0; > + > + if (!intel_has_reset_engine(gt)) > + return 0; > + > + err = live_mocs_init(&mocs, gt); > + if (err) > + return err; > + > + igt_global_reset_lock(gt); > + for_each_engine(engine, gt, id) { > + struct intel_context *ce; > + > + ce = intel_context_create(engine->kernel_context->gem_context, > + engine); > + if (IS_ERR(ce)) { > + err = PTR_ERR(ce); > + break; > + } > + > + intel_engine_pm_get(engine); > + err = __live_mocs_reset(&mocs, ce); > + intel_engine_pm_put(engine); > + > + intel_context_put(ce); > + if (err) > + break; > + } > + igt_global_reset_unlock(gt); > + > + live_mocs_fini(&mocs); > + return err; > +} > + > +int intel_mocs_live_selftests(struct drm_i915_private *i915) > +{ > + static const struct i915_subtest tests[] = { > + SUBTEST(live_mocs_kernel), > + SUBTEST(live_mocs_clean), > + SUBTEST(live_mocs_reset), > + }; > + struct drm_i915_mocs_table table; > + > + if (!get_mocs_settings(i915, &table)) > + return 0; > + > + return intel_gt_live_subtests(tests, &i915->gt); > +} > diff --git a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h > index 00a063730bc3..c23d06bca09e 100644 > --- a/drivers/gpu/drm/i915/selftests/i915_live_selftests.h > +++ b/drivers/gpu/drm/i915/selftests/i915_live_selftests.h > @@ -16,6 +16,7 @@ selftest(gt_engines, intel_engine_live_selftests) > selftest(gt_timelines, intel_timeline_live_selftests) > selftest(gt_contexts, intel_context_live_selftests) > selftest(gt_lrc, intel_lrc_live_selftests) > +selftest(gt_mocs, intel_mocs_live_selftests) > selftest(gt_pm, intel_gt_pm_live_selftests) > selftest(gt_heartbeat, intel_heartbeat_live_selftests) > selftest(requests, i915_request_live_selftests) > -- > 2.24.0.rc0 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] drm/i915/selftests: Add coverage of mocs registers @ 2019-10-24 7:13 ` Chris Wilson 0 siblings, 0 replies; 15+ messages in thread From: Chris Wilson @ 2019-10-24 7:13 UTC (permalink / raw) To: Kumar Valsan, Prathap; +Cc: intel-gfx Quoting Kumar Valsan, Prathap (2019-10-23 22:03:40) > On Tue, Oct 22, 2019 at 12:57:05PM +0100, Chris Wilson wrote: > > Probe the mocs registers for new contexts and across GPU resets. Similar > > to intel_workarounds, we have tables of what register values we expect > > to see, so verify that user contexts are affected by them. In the > > future, we should add tests similar to intel_sseu to cover dynamic > > reconfigurations. > > > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > > Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> > > s/for_each_engine/for_each_uabi_engine ? No, we are inside the gt compartment, so we only operate within our little enclosure. Think parallelism... -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Add coverage of mocs registers @ 2019-10-24 7:13 ` Chris Wilson 0 siblings, 0 replies; 15+ messages in thread From: Chris Wilson @ 2019-10-24 7:13 UTC (permalink / raw) To: Kumar Valsan, Prathap; +Cc: intel-gfx Quoting Kumar Valsan, Prathap (2019-10-23 22:03:40) > On Tue, Oct 22, 2019 at 12:57:05PM +0100, Chris Wilson wrote: > > Probe the mocs registers for new contexts and across GPU resets. Similar > > to intel_workarounds, we have tables of what register values we expect > > to see, so verify that user contexts are affected by them. In the > > future, we should add tests similar to intel_sseu to cover dynamic > > reconfigurations. > > > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > > Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> > > s/for_each_engine/for_each_uabi_engine ? No, we are inside the gt compartment, so we only operate within our little enclosure. Think parallelism... -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] drm/i915/selftests: Add coverage of mocs registers @ 2019-10-24 17:01 ` Kumar Valsan, Prathap 0 siblings, 0 replies; 15+ messages in thread From: Kumar Valsan, Prathap @ 2019-10-24 17:01 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx On Thu, Oct 24, 2019 at 08:13:29AM +0100, Chris Wilson wrote: > Quoting Kumar Valsan, Prathap (2019-10-23 22:03:40) > > On Tue, Oct 22, 2019 at 12:57:05PM +0100, Chris Wilson wrote: > > > Probe the mocs registers for new contexts and across GPU resets. Similar > > > to intel_workarounds, we have tables of what register values we expect > > > to see, so verify that user contexts are affected by them. In the > > > future, we should add tests similar to intel_sseu to cover dynamic > > > reconfigurations. > > > > > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > > > Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > > > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> > > > > s/for_each_engine/for_each_uabi_engine ? > > No, we are inside the gt compartment, so we only operate within our > little enclosure. Think parallelism... Ok. Thanks. > -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/selftests: Add coverage of mocs registers @ 2019-10-24 17:01 ` Kumar Valsan, Prathap 0 siblings, 0 replies; 15+ messages in thread From: Kumar Valsan, Prathap @ 2019-10-24 17:01 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx On Thu, Oct 24, 2019 at 08:13:29AM +0100, Chris Wilson wrote: > Quoting Kumar Valsan, Prathap (2019-10-23 22:03:40) > > On Tue, Oct 22, 2019 at 12:57:05PM +0100, Chris Wilson wrote: > > > Probe the mocs registers for new contexts and across GPU resets. Similar > > > to intel_workarounds, we have tables of what register values we expect > > > to see, so verify that user contexts are affected by them. In the > > > future, we should add tests similar to intel_sseu to cover dynamic > > > reconfigurations. > > > > > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > > > Cc: Prathap Kumar Valsan <prathap.kumar.valsan@intel.com> > > > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> > > > > s/for_each_engine/for_each_uabi_engine ? > > No, we are inside the gt compartment, so we only operate within our > little enclosure. Think parallelism... Ok. Thanks. > -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) 2019-10-22 11:51 [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others Chris Wilson ` (2 preceding siblings ...) 2019-10-22 11:51 ` [PATCH 4/4] drm/i915/selftests: Add coverage of mocs registers Chris Wilson @ 2019-10-22 19:28 ` Patchwork 2019-10-22 19:55 ` ✓ Fi.CI.BAT: success " Patchwork 2019-10-23 11:00 ` [Intel-gfx] " Patchwork 5 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2019-10-22 19:28 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) URL : https://patchwork.freedesktop.org/series/68388/ State : warning == Summary == $ dim checkpatch origin/drm-tip 0331f204fce6 drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others 552119a6b590 drm/i915/gt: Tidy up debug-warns for the mocs control table 6ddfeb99f8cc drm/i915/gt: Refactor mocs loops into single control macro -:66: CHECK:MACRO_ARG_REUSE: Macro argument reuse 't' - possible side-effects? #66: FILE: drivers/gpu/drm/i915/gt/intel_mocs.c:344: +#define for_each_mocs(mocs, t, i) \ + for (i = 0; \ + i < t->n_entries ? (mocs = get_entry_control(t, i)), 1 : 0;\ + i++) -:66: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 't' may be better as '(t)' to avoid precedence issues #66: FILE: drivers/gpu/drm/i915/gt/intel_mocs.c:344: +#define for_each_mocs(mocs, t, i) \ + for (i = 0; \ + i < t->n_entries ? (mocs = get_entry_control(t, i)), 1 : 0;\ + i++) -:66: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects? #66: FILE: drivers/gpu/drm/i915/gt/intel_mocs.c:344: +#define for_each_mocs(mocs, t, i) \ + for (i = 0; \ + i < t->n_entries ? (mocs = get_entry_control(t, i)), 1 : 0;\ + i++) -:128: CHECK:MACRO_ARG_REUSE: Macro argument reuse 't' - possible side-effects? #128: FILE: drivers/gpu/drm/i915/gt/intel_mocs.c:399: +#define for_each_l3cc(l3cc, t, i) \ + for (i = 0; \ + i < (t->n_entries + 1) / 2 ? \ + (l3cc = l3cc_combine(get_entry_l3cc(t, 2 * i), \ + get_entry_l3cc(t, 2 * i + 1))), 1 : \ + 0; \ + i++) -:128: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 't' may be better as '(t)' to avoid precedence issues #128: FILE: drivers/gpu/drm/i915/gt/intel_mocs.c:399: +#define for_each_l3cc(l3cc, t, i) \ + for (i = 0; \ + i < (t->n_entries + 1) / 2 ? \ + (l3cc = l3cc_combine(get_entry_l3cc(t, 2 * i), \ + get_entry_l3cc(t, 2 * i + 1))), 1 : \ + 0; \ + i++) -:128: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i' - possible side-effects? #128: FILE: drivers/gpu/drm/i915/gt/intel_mocs.c:399: +#define for_each_l3cc(l3cc, t, i) \ + for (i = 0; \ + i < (t->n_entries + 1) / 2 ? \ + (l3cc = l3cc_combine(get_entry_l3cc(t, 2 * i), \ + get_entry_l3cc(t, 2 * i + 1))), 1 : \ + 0; \ + i++) total: 0 errors, 0 warnings, 6 checks, 182 lines checked 49c1502301e4 drm/i915/selftests: Add coverage of mocs registers -:29: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #29: new file mode 100644 -:34: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #34: FILE: drivers/gpu/drm/i915/gt/selftest_mocs.c:1: +/* -:35: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead #35: FILE: drivers/gpu/drm/i915/gt/selftest_mocs.c:2: + * SPDX-License-Identifier: MIT total: 0 errors, 3 warnings, 0 checks, 407 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* ✓ Fi.CI.BAT: success for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) 2019-10-22 11:51 [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others Chris Wilson ` (3 preceding siblings ...) 2019-10-22 19:28 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) Patchwork @ 2019-10-22 19:55 ` Patchwork 2019-10-23 11:00 ` [Intel-gfx] " Patchwork 5 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2019-10-22 19:55 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) URL : https://patchwork.freedesktop.org/series/68388/ State : success == Summary == CI Bug Log - changes from CI_DRM_7154 -> Patchwork_14923 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_14923: ### IGT changes ### #### Possible regressions #### * {igt@i915_selftest@live_gt_mocs} (NEW): - fi-skl-lmem: NOTRUN -> [DMESG-FAIL][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-skl-lmem/igt@i915_selftest@live_gt_mocs.html - fi-skl-6770hq: NOTRUN -> [DMESG-FAIL][2] [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-skl-6770hq/igt@i915_selftest@live_gt_mocs.html - fi-cfl-8109u: NOTRUN -> [DMESG-FAIL][3] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-cfl-8109u/igt@i915_selftest@live_gt_mocs.html New tests --------- New tests have been introduced between CI_DRM_7154 and Patchwork_14923: ### New IGT tests (1) ### * igt@i915_selftest@live_gt_mocs: - Statuses : 3 dmesg-fail(s) 42 pass(s) - Exec time: [0.40, 2.38] s Known issues ------------ Here are the changes found in Patchwork_14923 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_mmap@basic: - fi-icl-u3: [PASS][4] -> [DMESG-WARN][5] ([fdo#107724]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-icl-u3/igt@gem_mmap@basic.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-icl-u3/igt@gem_mmap@basic.html #### Possible fixes #### * igt@gem_ctx_create@basic-files: - {fi-icl-guc}: [INCOMPLETE][6] ([fdo#107713] / [fdo#109100]) -> [PASS][7] [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-icl-guc/igt@gem_ctx_create@basic-files.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-icl-guc/igt@gem_ctx_create@basic-files.html * igt@gem_exec_suspend@basic-s4-devices: - fi-icl-u3: [DMESG-WARN][8] ([fdo#107724]) -> [PASS][9] +1 similar issue [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-icl-u3/igt@gem_exec_suspend@basic-s4-devices.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-icl-u3/igt@gem_exec_suspend@basic-s4-devices.html * igt@i915_selftest@live_gem_contexts: - fi-cfl-8109u: [DMESG-FAIL][10] ([fdo#112050 ]) -> [PASS][11] [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-cfl-8109u/igt@i915_selftest@live_gem_contexts.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-cfl-8109u/igt@i915_selftest@live_gem_contexts.html * {igt@i915_selftest@live_gt_heartbeat}: - fi-kbl-8809g: [DMESG-FAIL][12] ([fdo#112096]) -> [PASS][13] [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-kbl-8809g/igt@i915_selftest@live_gt_heartbeat.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-kbl-8809g/igt@i915_selftest@live_gt_heartbeat.html - fi-kbl-guc: [DMESG-FAIL][14] ([fdo#112096]) -> [PASS][15] [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-kbl-guc/igt@i915_selftest@live_gt_heartbeat.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-kbl-guc/igt@i915_selftest@live_gt_heartbeat.html - {fi-cml-s}: [DMESG-FAIL][16] ([fdo#112096]) -> [PASS][17] [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-cml-s/igt@i915_selftest@live_gt_heartbeat.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-cml-s/igt@i915_selftest@live_gt_heartbeat.html #### Warnings #### * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-7500u: [FAIL][18] ([fdo#111407]) -> [FAIL][19] ([fdo#111045] / [fdo#111096]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100 [fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045 [fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096 [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407 [fdo#112050 ]: https://bugs.freedesktop.org/show_bug.cgi?id=112050 [fdo#112096]: https://bugs.freedesktop.org/show_bug.cgi?id=112096 Participating hosts (52 -> 45) ------------------------------ Additional (1): fi-tgl-u2 Missing (8): fi-ilk-m540 fi-tgl-u fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7154 -> Patchwork_14923 CI-20190529: 20190529 CI_DRM_7154: 90d6ad2cbeadd27e867df4e649d61f14b4ac4b77 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5235: da9abbab69be80dd00812a4607a4ea2dffcc4544 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14923: 49c1502301e46d5a1a32f2ab70242fa62150693c @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 49c1502301e4 drm/i915/selftests: Add coverage of mocs registers 6ddfeb99f8cc drm/i915/gt: Refactor mocs loops into single control macro 552119a6b590 drm/i915/gt: Tidy up debug-warns for the mocs control table 0331f204fce6 drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) @ 2019-10-23 11:00 ` Patchwork 0 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2019-10-23 11:00 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) URL : https://patchwork.freedesktop.org/series/68388/ State : success == Summary == CI Bug Log - changes from CI_DRM_7154_full -> Patchwork_14923_full ==================================================== Summary ------- **SUCCESS** No regressions found. Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_14923_full: ### IGT changes ### #### Possible regressions #### * {igt@i915_selftest@live_gt_mocs} (NEW): - shard-kbl: NOTRUN -> [DMESG-FAIL][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-kbl6/igt@i915_selftest@live_gt_mocs.html #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@kms_vblank@pipe-c-query-busy: - {shard-tglb}: [PASS][2] -> [INCOMPLETE][3] +2 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb2/igt@kms_vblank@pipe-c-query-busy.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb4/igt@kms_vblank@pipe-c-query-busy.html New tests --------- New tests have been introduced between CI_DRM_7154_full and Patchwork_14923_full: ### New IGT tests (1) ### * igt@i915_selftest@live_gt_mocs: - Statuses : 1 dmesg-fail(s) 7 pass(s) - Exec time: [0.33, 2.53] s Known issues ------------ Here are the changes found in Patchwork_14923_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@vcs0-s3: - shard-kbl: [PASS][4] -> [INCOMPLETE][5] ([fdo#103665]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-kbl4/igt@gem_ctx_isolation@vcs0-s3.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-kbl1/igt@gem_ctx_isolation@vcs0-s3.html * igt@gem_ctx_isolation@vcs1-dirty-create: - shard-iclb: [PASS][6] -> [SKIP][7] ([fdo#109276] / [fdo#112080]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_ctx_isolation@vcs1-dirty-create.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb5/igt@gem_ctx_isolation@vcs1-dirty-create.html * igt@gem_exec_schedule@preempt-queue-bsd1: - shard-iclb: [PASS][8] -> [SKIP][9] ([fdo#109276]) +11 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_exec_schedule@preempt-queue-bsd1.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb5/igt@gem_exec_schedule@preempt-queue-bsd1.html * igt@gem_fence_thrash@bo-write-verify-threaded-y: - shard-apl: [PASS][10] -> [INCOMPLETE][11] ([fdo#103927]) +2 similar issues [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl6/igt@gem_fence_thrash@bo-write-verify-threaded-y.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl6/igt@gem_fence_thrash@bo-write-verify-threaded-y.html * igt@gem_userptr_blits@dmabuf-unsync: - shard-snb: [PASS][12] -> [DMESG-WARN][13] ([fdo#111870]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-snb1/igt@gem_userptr_blits@dmabuf-unsync.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-snb5/igt@gem_userptr_blits@dmabuf-unsync.html * igt@gem_userptr_blits@sync-unmap-after-close: - shard-hsw: [PASS][14] -> [DMESG-WARN][15] ([fdo#111870]) +1 similar issue [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-hsw5/igt@gem_userptr_blits@sync-unmap-after-close.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-hsw2/igt@gem_userptr_blits@sync-unmap-after-close.html * igt@i915_selftest@live_hangcheck: - shard-iclb: [PASS][16] -> [INCOMPLETE][17] ([fdo#107713] / [fdo#108569]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb1/igt@i915_selftest@live_hangcheck.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb1/igt@i915_selftest@live_hangcheck.html * igt@i915_suspend@fence-restore-tiled2untiled: - shard-apl: [PASS][18] -> [DMESG-WARN][19] ([fdo#108566]) +3 similar issues [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl3/igt@i915_suspend@fence-restore-tiled2untiled.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl8/igt@i915_suspend@fence-restore-tiled2untiled.html * igt@kms_color@pipe-b-ctm-0-75: - shard-skl: [PASS][20] -> [DMESG-WARN][21] ([fdo#106107]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl2/igt@kms_color@pipe-b-ctm-0-75.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl4/igt@kms_color@pipe-b-ctm-0-75.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render: - shard-iclb: [PASS][22] -> [FAIL][23] ([fdo#103167]) +5 similar issues [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-iclb: [PASS][24] -> [DMESG-WARN][25] ([fdo#111764]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb4/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb1/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc: - shard-skl: [PASS][26] -> [FAIL][27] ([fdo#108145]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl5/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl7/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [PASS][28] -> [FAIL][29] ([fdo#108145] / [fdo#110403]) [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl2/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl1/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping: - shard-iclb: [PASS][30] -> [INCOMPLETE][31] ([fdo#107713] / [fdo#110041]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb7/igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb3/igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping.html * igt@kms_psr@psr2_sprite_plane_move: - shard-iclb: [PASS][32] -> [SKIP][33] ([fdo#109441]) +3 similar issues [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@kms_psr@psr2_sprite_plane_move.html * igt@perf_pmu@init-busy-vcs1: - shard-iclb: [PASS][34] -> [SKIP][35] ([fdo#112080]) +3 similar issues [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@perf_pmu@init-busy-vcs1.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@perf_pmu@init-busy-vcs1.html #### Possible fixes #### * igt@gem_ctx_isolation@bcs0-s3: - shard-apl: [DMESG-WARN][36] ([fdo#108566]) -> [PASS][37] +4 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl3/igt@gem_ctx_isolation@bcs0-s3.html [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl3/igt@gem_ctx_isolation@bcs0-s3.html * igt@gem_ctx_isolation@vcs1-none: - shard-iclb: [SKIP][38] ([fdo#109276] / [fdo#112080]) -> [PASS][39] [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@gem_ctx_isolation@vcs1-none.html [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@gem_ctx_isolation@vcs1-none.html * igt@gem_ctx_isolation@vecs0-s3: - shard-apl: [INCOMPLETE][40] ([fdo#103927]) -> [PASS][41] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl3/igt@gem_ctx_isolation@vecs0-s3.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl7/igt@gem_ctx_isolation@vecs0-s3.html * igt@gem_ctx_shared@exec-single-timeline-bsd: - shard-iclb: [SKIP][42] ([fdo#110841]) -> [PASS][43] [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_ctx_shared@exec-single-timeline-bsd.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@gem_ctx_shared@exec-single-timeline-bsd.html * igt@gem_ctx_switch@vcs1: - shard-iclb: [SKIP][44] ([fdo#112080]) -> [PASS][45] +8 similar issues [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@gem_ctx_switch@vcs1.html [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@gem_ctx_switch@vcs1.html * igt@gem_exec_schedule@preempt-other-chain-bsd: - shard-iclb: [SKIP][46] ([fdo#111325]) -> [PASS][47] +1 similar issue [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_exec_schedule@preempt-other-chain-bsd.html [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@gem_exec_schedule@preempt-other-chain-bsd.html * igt@gem_userptr_blits@map-fixed-invalidate-busy: - shard-snb: [DMESG-WARN][48] ([fdo#111870]) -> [PASS][49] [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-snb6/igt@gem_userptr_blits@map-fixed-invalidate-busy.html [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-snb5/igt@gem_userptr_blits@map-fixed-invalidate-busy.html * igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy-gup: - shard-hsw: [DMESG-WARN][50] ([fdo#111870]) -> [PASS][51] [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-hsw5/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy-gup.html [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-hsw5/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy-gup.html * igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding: - shard-hsw: [DMESG-WARN][52] ([fdo#102614]) -> [PASS][53] +1 similar issue [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-hsw5/igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding.html [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-hsw2/igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding.html * igt@kms_cursor_crc@pipe-c-cursor-suspend: - shard-kbl: [INCOMPLETE][54] ([fdo#103665]) -> [PASS][55] [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-kbl1/igt@kms_cursor_crc@pipe-c-cursor-suspend.html [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-kbl2/igt@kms_cursor_crc@pipe-c-cursor-suspend.html * igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled: - shard-snb: [SKIP][56] ([fdo#109271]) -> [PASS][57] [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-snb1/igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled.html [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-snb1/igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled.html * igt@kms_flip@flip-vs-expired-vblank: - shard-skl: [FAIL][58] ([fdo#105363]) -> [PASS][59] [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl4/igt@kms_flip@flip-vs-expired-vblank.html [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl6/igt@kms_flip@flip-vs-expired-vblank.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt: - shard-iclb: [FAIL][60] ([fdo#103167]) -> [PASS][61] +5 similar issues [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt.html [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt.html * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite: - {shard-tglb}: [FAIL][62] ([fdo#103167]) -> [PASS][63] +2 similar issues [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb5/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite.html [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb7/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes: - shard-skl: [INCOMPLETE][64] ([fdo#104108]) -> [PASS][65] [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html * igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min: - shard-skl: [FAIL][66] ([fdo#108145]) -> [PASS][67] [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl9/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl5/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html * igt@kms_psr@psr2_sprite_plane_onoff: - shard-iclb: [SKIP][68] ([fdo#109441]) -> [PASS][69] [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb1/igt@kms_psr@psr2_sprite_plane_onoff.html [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html * igt@kms_vblank@pipe-c-ts-continuation-idle-hang: - {shard-tglb}: [INCOMPLETE][70] -> [PASS][71] [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb3/igt@kms_vblank@pipe-c-ts-continuation-idle-hang.html [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb6/igt@kms_vblank@pipe-c-ts-continuation-idle-hang.html * igt@kms_vblank@pipe-d-ts-continuation-suspend: - {shard-tglb}: [INCOMPLETE][72] ([fdo#111850]) -> [PASS][73] [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb4/igt@kms_vblank@pipe-d-ts-continuation-suspend.html [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb5/igt@kms_vblank@pipe-d-ts-continuation-suspend.html * igt@prime_busy@hang-bsd2: - shard-iclb: [SKIP][74] ([fdo#109276]) -> [PASS][75] +11 similar issues [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb6/igt@prime_busy@hang-bsd2.html [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb4/igt@prime_busy@hang-bsd2.html #### Warnings #### * igt@gem_ctx_isolation@vcs1-nonpriv: - shard-iclb: [SKIP][76] ([fdo#109276] / [fdo#112080]) -> [FAIL][77] ([fdo#111329]) [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@gem_ctx_isolation@vcs1-nonpriv.html [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@gem_ctx_isolation@vcs1-nonpriv.html * igt@gem_mocs_settings@mocs-reset-bsd2: - shard-iclb: [FAIL][78] ([fdo#111330]) -> [SKIP][79] ([fdo#109276]) [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb1/igt@gem_mocs_settings@mocs-reset-bsd2.html [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb3/igt@gem_mocs_settings@mocs-reset-bsd2.html * igt@kms_psr@psr2_suspend: - shard-iclb: [SKIP][80] ([fdo#109441]) -> [DMESG-WARN][81] ([fdo#107724]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@kms_psr@psr2_suspend.html [81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@kms_psr@psr2_suspend.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108 [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363 [fdo#106107]: https://bugs.freedesktop.org/show_bug.cgi?id=106107 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566 [fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#110041]: https://bugs.freedesktop.org/show_bug.cgi?id=110041 [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403 [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841 [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325 [fdo#111329]: https://bugs.freedesktop.org/show_bug.cgi?id=111329 [fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330 [fdo#111703]: https://bugs.freedesktop.org/show_bug.cgi?id=111703 [fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735 [fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764 [fdo#111832]: https://bugs.freedesktop.org/show_bug.cgi?id=111832 [fdo#111850]: https://bugs.freedesktop.org/show_bug.cgi?id=111850 [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870 [fdo#112068 ]: https://bugs.freedesktop.org/show_bug.cgi?id=112068 [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080 Participating hosts (11 -> 11) ------------------------------ No changes in participating hosts Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7154 -> Patchwork_14923 CI-20190529: 20190529 CI_DRM_7154: 90d6ad2cbeadd27e867df4e649d61f14b4ac4b77 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5235: da9abbab69be80dd00812a4607a4ea2dffcc4544 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14923: 49c1502301e46d5a1a32f2ab70242fa62150693c @ 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_14923/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) @ 2019-10-23 11:00 ` Patchwork 0 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2019-10-23 11:00 UTC (permalink / raw) To: Chris Wilson; +Cc: intel-gfx == Series Details == Series: series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) URL : https://patchwork.freedesktop.org/series/68388/ State : success == Summary == CI Bug Log - changes from CI_DRM_7154_full -> Patchwork_14923_full ==================================================== Summary ------- **SUCCESS** No regressions found. Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_14923_full: ### IGT changes ### #### Possible regressions #### * {igt@i915_selftest@live_gt_mocs} (NEW): - shard-kbl: NOTRUN -> [DMESG-FAIL][1] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-kbl6/igt@i915_selftest@live_gt_mocs.html #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@kms_vblank@pipe-c-query-busy: - {shard-tglb}: [PASS][2] -> [INCOMPLETE][3] +2 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb2/igt@kms_vblank@pipe-c-query-busy.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb4/igt@kms_vblank@pipe-c-query-busy.html New tests --------- New tests have been introduced between CI_DRM_7154_full and Patchwork_14923_full: ### New IGT tests (1) ### * igt@i915_selftest@live_gt_mocs: - Statuses : 1 dmesg-fail(s) 7 pass(s) - Exec time: [0.33, 2.53] s Known issues ------------ Here are the changes found in Patchwork_14923_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@vcs0-s3: - shard-kbl: [PASS][4] -> [INCOMPLETE][5] ([fdo#103665]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-kbl4/igt@gem_ctx_isolation@vcs0-s3.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-kbl1/igt@gem_ctx_isolation@vcs0-s3.html * igt@gem_ctx_isolation@vcs1-dirty-create: - shard-iclb: [PASS][6] -> [SKIP][7] ([fdo#109276] / [fdo#112080]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_ctx_isolation@vcs1-dirty-create.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb5/igt@gem_ctx_isolation@vcs1-dirty-create.html * igt@gem_exec_schedule@preempt-queue-bsd1: - shard-iclb: [PASS][8] -> [SKIP][9] ([fdo#109276]) +11 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_exec_schedule@preempt-queue-bsd1.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb5/igt@gem_exec_schedule@preempt-queue-bsd1.html * igt@gem_fence_thrash@bo-write-verify-threaded-y: - shard-apl: [PASS][10] -> [INCOMPLETE][11] ([fdo#103927]) +2 similar issues [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl6/igt@gem_fence_thrash@bo-write-verify-threaded-y.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl6/igt@gem_fence_thrash@bo-write-verify-threaded-y.html * igt@gem_userptr_blits@dmabuf-unsync: - shard-snb: [PASS][12] -> [DMESG-WARN][13] ([fdo#111870]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-snb1/igt@gem_userptr_blits@dmabuf-unsync.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-snb5/igt@gem_userptr_blits@dmabuf-unsync.html * igt@gem_userptr_blits@sync-unmap-after-close: - shard-hsw: [PASS][14] -> [DMESG-WARN][15] ([fdo#111870]) +1 similar issue [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-hsw5/igt@gem_userptr_blits@sync-unmap-after-close.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-hsw2/igt@gem_userptr_blits@sync-unmap-after-close.html * igt@i915_selftest@live_hangcheck: - shard-iclb: [PASS][16] -> [INCOMPLETE][17] ([fdo#107713] / [fdo#108569]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb1/igt@i915_selftest@live_hangcheck.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb1/igt@i915_selftest@live_hangcheck.html * igt@i915_suspend@fence-restore-tiled2untiled: - shard-apl: [PASS][18] -> [DMESG-WARN][19] ([fdo#108566]) +3 similar issues [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl3/igt@i915_suspend@fence-restore-tiled2untiled.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl8/igt@i915_suspend@fence-restore-tiled2untiled.html * igt@kms_color@pipe-b-ctm-0-75: - shard-skl: [PASS][20] -> [DMESG-WARN][21] ([fdo#106107]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl2/igt@kms_color@pipe-b-ctm-0-75.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl4/igt@kms_color@pipe-b-ctm-0-75.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render: - shard-iclb: [PASS][22] -> [FAIL][23] ([fdo#103167]) +5 similar issues [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-iclb: [PASS][24] -> [DMESG-WARN][25] ([fdo#111764]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb4/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb1/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc: - shard-skl: [PASS][26] -> [FAIL][27] ([fdo#108145]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl5/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl7/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [PASS][28] -> [FAIL][29] ([fdo#108145] / [fdo#110403]) [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl2/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl1/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping: - shard-iclb: [PASS][30] -> [INCOMPLETE][31] ([fdo#107713] / [fdo#110041]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb7/igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb3/igt@kms_plane_scaling@pipe-a-scaler-with-clipping-clamping.html * igt@kms_psr@psr2_sprite_plane_move: - shard-iclb: [PASS][32] -> [SKIP][33] ([fdo#109441]) +3 similar issues [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@kms_psr@psr2_sprite_plane_move.html * igt@perf_pmu@init-busy-vcs1: - shard-iclb: [PASS][34] -> [SKIP][35] ([fdo#112080]) +3 similar issues [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@perf_pmu@init-busy-vcs1.html [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@perf_pmu@init-busy-vcs1.html #### Possible fixes #### * igt@gem_ctx_isolation@bcs0-s3: - shard-apl: [DMESG-WARN][36] ([fdo#108566]) -> [PASS][37] +4 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl3/igt@gem_ctx_isolation@bcs0-s3.html [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl3/igt@gem_ctx_isolation@bcs0-s3.html * igt@gem_ctx_isolation@vcs1-none: - shard-iclb: [SKIP][38] ([fdo#109276] / [fdo#112080]) -> [PASS][39] [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@gem_ctx_isolation@vcs1-none.html [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@gem_ctx_isolation@vcs1-none.html * igt@gem_ctx_isolation@vecs0-s3: - shard-apl: [INCOMPLETE][40] ([fdo#103927]) -> [PASS][41] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-apl3/igt@gem_ctx_isolation@vecs0-s3.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-apl7/igt@gem_ctx_isolation@vecs0-s3.html * igt@gem_ctx_shared@exec-single-timeline-bsd: - shard-iclb: [SKIP][42] ([fdo#110841]) -> [PASS][43] [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_ctx_shared@exec-single-timeline-bsd.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@gem_ctx_shared@exec-single-timeline-bsd.html * igt@gem_ctx_switch@vcs1: - shard-iclb: [SKIP][44] ([fdo#112080]) -> [PASS][45] +8 similar issues [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@gem_ctx_switch@vcs1.html [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@gem_ctx_switch@vcs1.html * igt@gem_exec_schedule@preempt-other-chain-bsd: - shard-iclb: [SKIP][46] ([fdo#111325]) -> [PASS][47] +1 similar issue [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb2/igt@gem_exec_schedule@preempt-other-chain-bsd.html [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb8/igt@gem_exec_schedule@preempt-other-chain-bsd.html * igt@gem_userptr_blits@map-fixed-invalidate-busy: - shard-snb: [DMESG-WARN][48] ([fdo#111870]) -> [PASS][49] [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-snb6/igt@gem_userptr_blits@map-fixed-invalidate-busy.html [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-snb5/igt@gem_userptr_blits@map-fixed-invalidate-busy.html * igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy-gup: - shard-hsw: [DMESG-WARN][50] ([fdo#111870]) -> [PASS][51] [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-hsw5/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy-gup.html [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-hsw5/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy-gup.html * igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding: - shard-hsw: [DMESG-WARN][52] ([fdo#102614]) -> [PASS][53] +1 similar issue [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-hsw5/igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding.html [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-hsw2/igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding.html * igt@kms_cursor_crc@pipe-c-cursor-suspend: - shard-kbl: [INCOMPLETE][54] ([fdo#103665]) -> [PASS][55] [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-kbl1/igt@kms_cursor_crc@pipe-c-cursor-suspend.html [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-kbl2/igt@kms_cursor_crc@pipe-c-cursor-suspend.html * igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled: - shard-snb: [SKIP][56] ([fdo#109271]) -> [PASS][57] [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-snb1/igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled.html [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-snb1/igt@kms_draw_crc@draw-method-xrgb8888-mmap-gtt-xtiled.html * igt@kms_flip@flip-vs-expired-vblank: - shard-skl: [FAIL][58] ([fdo#105363]) -> [PASS][59] [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl4/igt@kms_flip@flip-vs-expired-vblank.html [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl6/igt@kms_flip@flip-vs-expired-vblank.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt: - shard-iclb: [FAIL][60] ([fdo#103167]) -> [PASS][61] +5 similar issues [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt.html [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt.html * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite: - {shard-tglb}: [FAIL][62] ([fdo#103167]) -> [PASS][63] +2 similar issues [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb5/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite.html [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb7/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes: - shard-skl: [INCOMPLETE][64] ([fdo#104108]) -> [PASS][65] [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html * igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min: - shard-skl: [FAIL][66] ([fdo#108145]) -> [PASS][67] [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-skl9/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-skl5/igt@kms_plane_alpha_blend@pipe-c-constant-alpha-min.html * igt@kms_psr@psr2_sprite_plane_onoff: - shard-iclb: [SKIP][68] ([fdo#109441]) -> [PASS][69] [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb1/igt@kms_psr@psr2_sprite_plane_onoff.html [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html * igt@kms_vblank@pipe-c-ts-continuation-idle-hang: - {shard-tglb}: [INCOMPLETE][70] -> [PASS][71] [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb3/igt@kms_vblank@pipe-c-ts-continuation-idle-hang.html [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb6/igt@kms_vblank@pipe-c-ts-continuation-idle-hang.html * igt@kms_vblank@pipe-d-ts-continuation-suspend: - {shard-tglb}: [INCOMPLETE][72] ([fdo#111850]) -> [PASS][73] [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-tglb4/igt@kms_vblank@pipe-d-ts-continuation-suspend.html [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-tglb5/igt@kms_vblank@pipe-d-ts-continuation-suspend.html * igt@prime_busy@hang-bsd2: - shard-iclb: [SKIP][74] ([fdo#109276]) -> [PASS][75] +11 similar issues [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb6/igt@prime_busy@hang-bsd2.html [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb4/igt@prime_busy@hang-bsd2.html #### Warnings #### * igt@gem_ctx_isolation@vcs1-nonpriv: - shard-iclb: [SKIP][76] ([fdo#109276] / [fdo#112080]) -> [FAIL][77] ([fdo#111329]) [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@gem_ctx_isolation@vcs1-nonpriv.html [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@gem_ctx_isolation@vcs1-nonpriv.html * igt@gem_mocs_settings@mocs-reset-bsd2: - shard-iclb: [FAIL][78] ([fdo#111330]) -> [SKIP][79] ([fdo#109276]) [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb1/igt@gem_mocs_settings@mocs-reset-bsd2.html [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb3/igt@gem_mocs_settings@mocs-reset-bsd2.html * igt@kms_psr@psr2_suspend: - shard-iclb: [SKIP][80] ([fdo#109441]) -> [DMESG-WARN][81] ([fdo#107724]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7154/shard-iclb3/igt@kms_psr@psr2_suspend.html [81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14923/shard-iclb2/igt@kms_psr@psr2_suspend.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108 [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363 [fdo#106107]: https://bugs.freedesktop.org/show_bug.cgi?id=106107 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566 [fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#110041]: https://bugs.freedesktop.org/show_bug.cgi?id=110041 [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403 [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841 [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325 [fdo#111329]: https://bugs.freedesktop.org/show_bug.cgi?id=111329 [fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330 [fdo#111703]: https://bugs.freedesktop.org/show_bug.cgi?id=111703 [fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735 [fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764 [fdo#111832]: https://bugs.freedesktop.org/show_bug.cgi?id=111832 [fdo#111850]: https://bugs.freedesktop.org/show_bug.cgi?id=111850 [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870 [fdo#112068 ]: https://bugs.freedesktop.org/show_bug.cgi?id=112068 [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080 Participating hosts (11 -> 11) ------------------------------ No changes in participating hosts Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7154 -> Patchwork_14923 CI-20190529: 20190529 CI_DRM_7154: 90d6ad2cbeadd27e867df4e649d61f14b4ac4b77 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5235: da9abbab69be80dd00812a4607a4ea2dffcc4544 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14923: 49c1502301e46d5a1a32f2ab70242fa62150693c @ 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_14923/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2019-10-24 16:44 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-10-22 11:51 [PATCH 1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others Chris Wilson 2019-10-22 11:51 ` [PATCH 2/4] drm/i915/gt: Tidy up debug-warns for the mocs control table Chris Wilson 2019-10-22 11:51 ` [PATCH 3/4] drm/i915/gt: Refactor mocs loops into single control macro Chris Wilson 2019-10-22 11:51 ` [PATCH 4/4] drm/i915/selftests: Add coverage of mocs registers Chris Wilson 2019-10-22 11:57 ` [PATCH] " Chris Wilson 2019-10-23 21:03 ` Kumar Valsan, Prathap 2019-10-23 21:03 ` [Intel-gfx] " Kumar Valsan, Prathap 2019-10-24 7:13 ` Chris Wilson 2019-10-24 7:13 ` [Intel-gfx] " Chris Wilson 2019-10-24 17:01 ` Kumar Valsan, Prathap 2019-10-24 17:01 ` [Intel-gfx] " Kumar Valsan, Prathap 2019-10-22 19:28 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/4] drm/i915/gt: Set unused mocs entry to follow PTE on tgl as on all others (rev2) Patchwork 2019-10-22 19:55 ` ✓ Fi.CI.BAT: success " Patchwork 2019-10-23 11:00 ` ✓ Fi.CI.IGT: " Patchwork 2019-10-23 11:00 ` [Intel-gfx] " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.