* [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch
@ 2022-02-19 0:10 Jasmine Newsome
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner Jasmine Newsome
` (3 more replies)
0 siblings, 4 replies; 13+ messages in thread
From: Jasmine Newsome @ 2022-02-19 0:10 UTC (permalink / raw)
To: igt-dev; +Cc: jasmine.newsome
Saving more context info for added convenience in spinner.
Also adding a fix to remove context persistence from
gem_spin_batch in order to pass test on GuC enabled
platforms.
Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
Jasmine Newsome (2):
lib/igt_dummyload.c: Save more context info in spinner
tests/i915/gem_spin_batch: Removing context persistence
lib/igt_dummyload.c | 1 +
lib/igt_dummyload.h | 43 ++++++++++++++++++++-----------------
tests/i915/gem_spin_batch.c | 11 +++++-----
3 files changed, 30 insertions(+), 25 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner
2022-02-19 0:10 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
@ 2022-02-19 0:10 ` Jasmine Newsome
2022-02-19 0:23 ` Dixit, Ashutosh
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
` (2 subsequent siblings)
3 siblings, 1 reply; 13+ messages in thread
From: Jasmine Newsome @ 2022-02-19 0:10 UTC (permalink / raw)
To: igt-dev; +Cc: jasmine.newsome
Save igt_spin_factory info in igt_spin. This will allow
for access to the actual context within the spinner.
Previously, only the context id was accessible. This
might help make spinner code cleaner.
Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
---
lib/igt_dummyload.c | 1 +
lib/igt_dummyload.h | 43 +++++++++++++++++++++++--------------------
2 files changed, 24 insertions(+), 20 deletions(-)
diff --git a/lib/igt_dummyload.c b/lib/igt_dummyload.c
index 645db922..0b2be154 100644
--- a/lib/igt_dummyload.c
+++ b/lib/igt_dummyload.c
@@ -421,6 +421,7 @@ emit_recursive_batch(igt_spin_t *spin,
}
spin->cmd_precondition = *spin->condition;
+ spin->opts = *opts;
return fence_fd;
}
diff --git a/lib/igt_dummyload.h b/lib/igt_dummyload.h
index f0205261..af9e6a43 100644
--- a/lib/igt_dummyload.h
+++ b/lib/igt_dummyload.h
@@ -33,6 +33,28 @@
#include "i915_drm.h"
#include "intel_ctx.h"
+
+/**
+ * igt_spin_factory_t:
+ * @ctx_id: GEM context handle
+ * @ctx: intel_ctx_t context wrapper
+ * @dependency: GEM object to depend on
+ * @engine: Flags describing the engine to execute on
+ * @flags: Set of IGT_SPIN_* flags
+ * @fence: In-fence to wait on
+ *
+ * A factory struct which contains creation parameters for an igt_spin_t.
+ */
+typedef struct igt_spin_factory {
+ uint32_t ctx_id;
+ const intel_ctx_t *ctx;
+ uint32_t dependency;
+ unsigned int engine;
+ unsigned int flags;
+ int fence;
+ uint64_t ahnd;
+} igt_spin_factory_t;
+
typedef struct igt_spin {
struct igt_list_head link;
@@ -60,28 +82,9 @@ typedef struct igt_spin {
#define SPIN_CLFLUSH (1 << 0)
uint64_t ahnd;
+ struct igt_spin_factory opts;
} igt_spin_t;
-/**
- * igt_spin_factory_t:
- * @ctx_id: GEM context handle
- * @ctx: intel_ctx_t context wrapper
- * @dependency: GEM object to depend on
- * @engine: Flags describing the engine to execute on
- * @flags: Set of IGT_SPIN_* flags
- * @fence: In-fence to wait on
- *
- * A factory struct which contains creation parameters for an igt_spin_t.
- */
-typedef struct igt_spin_factory {
- uint32_t ctx_id;
- const intel_ctx_t *ctx;
- uint32_t dependency;
- unsigned int engine;
- unsigned int flags;
- int fence;
- uint64_t ahnd;
-} igt_spin_factory_t;
#define IGT_SPIN_FENCE_IN (1 << 0)
#define IGT_SPIN_FENCE_SUBMIT (1 << 1)
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-19 0:10 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner Jasmine Newsome
@ 2022-02-19 0:10 ` Jasmine Newsome
2022-02-19 0:20 ` Dixit, Ashutosh
2022-02-19 1:50 ` [igt-dev] ✓ Fi.CI.BAT: success for Fix for gem_spin_batch (rev2) Patchwork
2022-02-19 19:58 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 1 reply; 13+ messages in thread
From: Jasmine Newsome @ 2022-02-19 0:10 UTC (permalink / raw)
To: igt-dev; +Cc: jasmine.newsome
The spin all test relied on context persistence unecessarily by trying
to destroy contexts while keeping spinners active.
The current implementation of context persistence in i915 can cause
failures, and persistence is not needed for this test.
Moving intel_ctx_destroy after igt_spin_end.
Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
---
tests/i915/gem_spin_batch.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
index 653812c7..0835a122 100644
--- a/tests/i915/gem_spin_batch.c
+++ b/tests/i915/gem_spin_batch.c
@@ -143,6 +143,7 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
const struct intel_execution_engine2 *e;
intel_ctx_cfg_t cfg = ctx->cfg;
struct igt_spin *spin, *n;
+ const intel_ctx_t *local_ctx;
uint64_t ahnd;
IGT_LIST_HEAD(list);
@@ -151,18 +152,16 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
continue;
if (flags & PARALLEL_SPIN_NEW_CTX)
- ctx = intel_ctx_create(i915, &cfg);
+ local_ctx = intel_ctx_create(i915, &cfg);
ahnd = get_reloc_ahnd(i915, ctx->id);
/* Prevent preemption so only one is allowed on each engine */
spin = igt_spin_new(i915,
.ahnd = ahnd,
- .ctx = ctx,
+ .ctx = (flags & PARALLEL_SPIN_NEW_CTX) ? local_ctx : ctx,
.engine = e->flags,
.flags = (IGT_SPIN_POLL_RUN |
IGT_SPIN_NO_PREEMPTION));
- if (flags & PARALLEL_SPIN_NEW_CTX)
- intel_ctx_destroy(i915, ctx);
igt_spin_busywait_until_started(spin);
igt_list_move(&spin->link, &list);
@@ -172,7 +171,9 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
igt_assert(gem_bo_busy(i915, spin->handle));
ahnd = spin->ahnd;
igt_spin_end(spin);
- gem_sync(i915, spin->handle);
+ if (flags & PARALLEL_SPIN_NEW_CTX)
+ intel_ctx_destroy(i915, spin->opts.ctx);
+ gem_sync(i915, spin->handle);
igt_spin_free(i915, spin);
put_ahnd(ahnd);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
@ 2022-02-19 0:20 ` Dixit, Ashutosh
0 siblings, 0 replies; 13+ messages in thread
From: Dixit, Ashutosh @ 2022-02-19 0:20 UTC (permalink / raw)
To: Jasmine Newsome; +Cc: igt-dev
On Fri, 18 Feb 2022 16:10:13 -0800, Jasmine Newsome wrote:
>
> diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
> index 653812c7..0835a122 100644
> --- a/tests/i915/gem_spin_batch.c
> +++ b/tests/i915/gem_spin_batch.c
> @@ -143,6 +143,7 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> const struct intel_execution_engine2 *e;
> intel_ctx_cfg_t cfg = ctx->cfg;
> struct igt_spin *spin, *n;
> + const intel_ctx_t *local_ctx;
Don't need local_ctx, let's overwrite ctx as before.
> uint64_t ahnd;
> IGT_LIST_HEAD(list);
>
> @@ -151,18 +152,16 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> continue;
>
> if (flags & PARALLEL_SPIN_NEW_CTX)
> - ctx = intel_ctx_create(i915, &cfg);
> + local_ctx = intel_ctx_create(i915, &cfg);
Don't need this change.
> ahnd = get_reloc_ahnd(i915, ctx->id);
>
> /* Prevent preemption so only one is allowed on each engine */
> spin = igt_spin_new(i915,
> .ahnd = ahnd,
> - .ctx = ctx,
> + .ctx = (flags & PARALLEL_SPIN_NEW_CTX) ? local_ctx : ctx,
Don't need this change.
> .engine = e->flags,
> .flags = (IGT_SPIN_POLL_RUN |
> IGT_SPIN_NO_PREEMPTION));
> - if (flags & PARALLEL_SPIN_NEW_CTX)
> - intel_ctx_destroy(i915, ctx);
>
> igt_spin_busywait_until_started(spin);
> igt_list_move(&spin->link, &list);
> @@ -172,7 +171,9 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> igt_assert(gem_bo_busy(i915, spin->handle));
> ahnd = spin->ahnd;
> igt_spin_end(spin);
> - gem_sync(i915, spin->handle);
> + if (flags & PARALLEL_SPIN_NEW_CTX)
> + intel_ctx_destroy(i915, spin->opts.ctx);
> + gem_sync(i915, spin->handle);
This line still has spaces not tabs. Rest of the patch is great!
> igt_spin_free(i915, spin);
> put_ahnd(ahnd);
> }
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner Jasmine Newsome
@ 2022-02-19 0:23 ` Dixit, Ashutosh
0 siblings, 0 replies; 13+ messages in thread
From: Dixit, Ashutosh @ 2022-02-19 0:23 UTC (permalink / raw)
To: Jasmine Newsome; +Cc: igt-dev
On Fri, 18 Feb 2022 16:10:12 -0800, Jasmine Newsome wrote:
>
> Save igt_spin_factory info in igt_spin. This will allow
> for access to the actual context within the spinner.
> Previously, only the context id was accessible. This
> might help make spinner code cleaner.
Since we are doing another rev. I'd suggest changing the title of this
patch to something like "lib/igt_dummyload: Save spin->opts as part of the
spinner".
The patch itself is great!
^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-19 1:32 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
@ 2022-02-19 1:32 ` Jasmine Newsome
2022-02-19 2:39 ` Dixit, Ashutosh
2022-02-21 11:54 ` Tvrtko Ursulin
0 siblings, 2 replies; 13+ messages in thread
From: Jasmine Newsome @ 2022-02-19 1:32 UTC (permalink / raw)
To: igt-dev; +Cc: jasmine.newsome
The spin all test relied on context persistence unecessarily by trying
to destroy contexts while keeping spinners active.
The current implementation of context persistence in i915 can cause
failures, and persistence is not needed for this test.
Moving intel_ctx_destroy after igt_spin_end.
Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
---
tests/i915/gem_spin_batch.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
index 653812c7..707d69b6 100644
--- a/tests/i915/gem_spin_batch.c
+++ b/tests/i915/gem_spin_batch.c
@@ -161,8 +161,6 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
.engine = e->flags,
.flags = (IGT_SPIN_POLL_RUN |
IGT_SPIN_NO_PREEMPTION));
- if (flags & PARALLEL_SPIN_NEW_CTX)
- intel_ctx_destroy(i915, ctx);
igt_spin_busywait_until_started(spin);
igt_list_move(&spin->link, &list);
@@ -172,6 +170,8 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
igt_assert(gem_bo_busy(i915, spin->handle));
ahnd = spin->ahnd;
igt_spin_end(spin);
+ if (flags & PARALLEL_SPIN_NEW_CTX)
+ intel_ctx_destroy(i915, spin->opts.ctx);
gem_sync(i915, spin->handle);
igt_spin_free(i915, spin);
put_ahnd(ahnd);
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for Fix for gem_spin_batch (rev2)
2022-02-19 0:10 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner Jasmine Newsome
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
@ 2022-02-19 1:50 ` Patchwork
2022-02-19 19:58 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-02-19 1:50 UTC (permalink / raw)
To: Jasmine Newsome; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 5899 bytes --]
== Series Details ==
Series: Fix for gem_spin_batch (rev2)
URL : https://patchwork.freedesktop.org/series/100161/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11254 -> IGTPW_6655
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/index.html
Participating hosts (44 -> 43)
------------------------------
Additional (1): fi-pnv-d510
Missing (2): fi-bsw-cyan shard-tglu
Known issues
------------
Here are the changes found in IGTPW_6655 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_basic@query-info:
- fi-bsw-kefka: NOTRUN -> [SKIP][1] ([fdo#109271]) +17 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-bsw-kefka/igt@amdgpu/amd_basic@query-info.html
* igt@amdgpu/amd_cs_nop@sync-fork-compute0:
- fi-snb-2600: NOTRUN -> [SKIP][2] ([fdo#109271]) +17 similar issues
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html
* igt@gem_exec_suspend@basic-s0@smem:
- fi-glk-dsi: [PASS][3] -> [DMESG-WARN][4] ([i915#2943])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-glk-dsi/igt@gem_exec_suspend@basic-s0@smem.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-glk-dsi/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gem_exec_suspend@basic-s3:
- fi-skl-6600u: NOTRUN -> [INCOMPLETE][5] ([i915#4547])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-skl-6600u/igt@gem_exec_suspend@basic-s3.html
* igt@gem_huc_copy@huc-copy:
- fi-pnv-d510: NOTRUN -> [SKIP][6] ([fdo#109271]) +57 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-pnv-d510/igt@gem_huc_copy@huc-copy.html
* igt@i915_selftest@live@hangcheck:
- fi-hsw-4770: [PASS][7] -> [INCOMPLETE][8] ([i915#4785])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-hsw-4770/igt@i915_selftest@live@hangcheck.html
* igt@kms_pipe_crc_basic@read-crc-pipe-b:
- fi-cfl-8109u: [PASS][9] -> [DMESG-WARN][10] ([i915#295]) +12 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-cfl-8109u/igt@kms_pipe_crc_basic@read-crc-pipe-b.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-cfl-8109u/igt@kms_pipe_crc_basic@read-crc-pipe-b.html
* igt@runner@aborted:
- fi-hsw-4770: NOTRUN -> [FAIL][11] ([fdo#109271] / [i915#1436] / [i915#4312])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-hsw-4770/igt@runner@aborted.html
- fi-bdw-5557u: NOTRUN -> [FAIL][12] ([i915#2426] / [i915#4312])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-bdw-5557u/igt@runner@aborted.html
#### Possible fixes ####
* igt@i915_selftest@live@execlists:
- fi-bsw-kefka: [INCOMPLETE][13] ([i915#2940]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-bsw-kefka/igt@i915_selftest@live@execlists.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-bsw-kefka/igt@i915_selftest@live@execlists.html
* igt@i915_selftest@live@gt_heartbeat:
- {fi-tgl-dsi}: [DMESG-FAIL][15] ([i915#541]) -> [PASS][16]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-tgl-dsi/igt@i915_selftest@live@gt_heartbeat.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-tgl-dsi/igt@i915_selftest@live@gt_heartbeat.html
* igt@i915_selftest@live@hangcheck:
- fi-snb-2600: [INCOMPLETE][17] ([i915#3921]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
* igt@kms_frontbuffer_tracking@basic:
- fi-cml-u2: [DMESG-WARN][19] ([i915#4269]) -> [PASS][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#2943]: https://gitlab.freedesktop.org/drm/intel/issues/2943
[i915#295]: https://gitlab.freedesktop.org/drm/intel/issues/295
[i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
[i915#4269]: https://gitlab.freedesktop.org/drm/intel/issues/4269
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4547]: https://gitlab.freedesktop.org/drm/intel/issues/4547
[i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
[i915#4897]: https://gitlab.freedesktop.org/drm/intel/issues/4897
[i915#541]: https://gitlab.freedesktop.org/drm/intel/issues/541
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6348 -> IGTPW_6655
CI-20190529: 20190529
CI_DRM_11254: 16159219b34bb0c431b00c019892a561292c81e0 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_6655: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/index.html
IGT_6348: 9cb64a757d2ff1e180b1648e611439d94afd697d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/index.html
[-- Attachment #2: Type: text/html, Size: 7023 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-19 1:32 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
@ 2022-02-19 2:39 ` Dixit, Ashutosh
2022-02-21 11:54 ` Tvrtko Ursulin
1 sibling, 0 replies; 13+ messages in thread
From: Dixit, Ashutosh @ 2022-02-19 2:39 UTC (permalink / raw)
To: Jasmine Newsome; +Cc: igt-dev
On Fri, 18 Feb 2022 17:32:15 -0800, Jasmine Newsome wrote:
>
> diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
> index 653812c7..707d69b6 100644
> --- a/tests/i915/gem_spin_batch.c
> +++ b/tests/i915/gem_spin_batch.c
> @@ -161,8 +161,6 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> .engine = e->flags,
> .flags = (IGT_SPIN_POLL_RUN |
> IGT_SPIN_NO_PREEMPTION));
> - if (flags & PARALLEL_SPIN_NEW_CTX)
> - intel_ctx_destroy(i915, ctx);
>
> igt_spin_busywait_until_started(spin);
> igt_list_move(&spin->link, &list);
> @@ -172,6 +170,8 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> igt_assert(gem_bo_busy(i915, spin->handle));
> ahnd = spin->ahnd;
> igt_spin_end(spin);
> + if (flags & PARALLEL_SPIN_NEW_CTX)
> + intel_ctx_destroy(i915, spin->opts.ctx);
To be safe I think we should move this after gem_sync(). Anyway I am fine
with the patches now. Let's see if anyone else has comments/objections,
otherwise I can fix this up before merging.
> gem_sync(i915, spin->handle);
> igt_spin_free(i915, spin);
> put_ahnd(ahnd);
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for Fix for gem_spin_batch (rev2)
2022-02-19 0:10 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
` (2 preceding siblings ...)
2022-02-19 1:50 ` [igt-dev] ✓ Fi.CI.BAT: success for Fix for gem_spin_batch (rev2) Patchwork
@ 2022-02-19 19:58 ` Patchwork
3 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-02-19 19:58 UTC (permalink / raw)
To: Jasmine Newsome; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 30248 bytes --]
== Series Details ==
Series: Fix for gem_spin_batch (rev2)
URL : https://patchwork.freedesktop.org/series/100161/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11254_full -> IGTPW_6655_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/index.html
Participating hosts (13 -> 8)
------------------------------
Missing (5): pig-kbl-iris pig-glk-j5005 pig-skl-6260u shard-rkl shard-dg1
Known issues
------------
Here are the changes found in IGTPW_6655_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@feature_discovery@display-3x:
- shard-iclb: NOTRUN -> [SKIP][1] ([i915#1839]) +1 similar issue
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb7/igt@feature_discovery@display-3x.html
* igt@gem_ctx_isolation@preservation-s3@bcs0:
- shard-kbl: [PASS][2] -> [DMESG-WARN][3] ([i915#180]) +1 similar issue
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-kbl6/igt@gem_ctx_isolation@preservation-s3@bcs0.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl6/igt@gem_ctx_isolation@preservation-s3@bcs0.html
* igt@gem_ctx_persistence@engines-hostile:
- shard-snb: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#1099])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-snb2/igt@gem_ctx_persistence@engines-hostile.html
* igt@gem_ctx_sseu@invalid-sseu:
- shard-tglb: NOTRUN -> [SKIP][5] ([i915#280])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb5/igt@gem_ctx_sseu@invalid-sseu.html
* igt@gem_exec_balancer@parallel-balancer:
- shard-iclb: [PASS][6] -> [SKIP][7] ([i915#4525])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-iclb1/igt@gem_exec_balancer@parallel-balancer.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb6/igt@gem_exec_balancer@parallel-balancer.html
* igt@gem_exec_balancer@parallel-keep-in-fence:
- shard-kbl: NOTRUN -> [DMESG-WARN][8] ([i915#5076]) +1 similar issue
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl3/igt@gem_exec_balancer@parallel-keep-in-fence.html
* igt@gem_exec_balancer@parallel-keep-submit-fence:
- shard-tglb: NOTRUN -> [DMESG-WARN][9] ([i915#5076])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb5/igt@gem_exec_balancer@parallel-keep-submit-fence.html
* igt@gem_exec_capture@pi@bcs0:
- shard-iclb: [PASS][10] -> [INCOMPLETE][11] ([i915#3371])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-iclb5/igt@gem_exec_capture@pi@bcs0.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb5/igt@gem_exec_capture@pi@bcs0.html
* igt@gem_exec_fair@basic-none-rrul@rcs0:
- shard-iclb: NOTRUN -> [FAIL][12] ([i915#2842]) +1 similar issue
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb1/igt@gem_exec_fair@basic-none-rrul@rcs0.html
- shard-glk: NOTRUN -> [FAIL][13] ([i915#2842])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk1/igt@gem_exec_fair@basic-none-rrul@rcs0.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-tglb: [PASS][14] -> [FAIL][15] ([i915#2842])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-tglb3/igt@gem_exec_fair@basic-none-share@rcs0.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb6/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-tglb: NOTRUN -> [FAIL][16] ([i915#2842]) +5 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_exec_fair@basic-pace-solo@rcs0:
- shard-apl: [PASS][17] -> [FAIL][18] ([i915#2842])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-apl3/igt@gem_exec_fair@basic-pace-solo@rcs0.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl1/igt@gem_exec_fair@basic-pace-solo@rcs0.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-glk: [PASS][19] -> [FAIL][20] ([i915#2842]) +1 similar issue
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-glk5/igt@gem_exec_fair@basic-throttle@rcs0.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk1/igt@gem_exec_fair@basic-throttle@rcs0.html
- shard-iclb: [PASS][21] -> [FAIL][22] ([i915#2842])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-iclb2/igt@gem_exec_fair@basic-throttle@rcs0.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb4/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gem_lmem_swapping@basic:
- shard-kbl: NOTRUN -> [SKIP][23] ([fdo#109271] / [i915#4613]) +1 similar issue
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl3/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@heavy-verify-random:
- shard-tglb: NOTRUN -> [SKIP][24] ([i915#4613]) +2 similar issues
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@gem_lmem_swapping@heavy-verify-random.html
* igt@gem_lmem_swapping@parallel-random-engines:
- shard-iclb: NOTRUN -> [SKIP][25] ([i915#4613])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb4/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@gem_lmem_swapping@random-engines:
- shard-apl: NOTRUN -> [SKIP][26] ([fdo#109271] / [i915#4613])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl3/igt@gem_lmem_swapping@random-engines.html
- shard-glk: NOTRUN -> [SKIP][27] ([fdo#109271] / [i915#4613])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk6/igt@gem_lmem_swapping@random-engines.html
* igt@gem_ppgtt@flink-and-close-vma-leak:
- shard-glk: [PASS][28] -> [FAIL][29] ([i915#644])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-glk2/igt@gem_ppgtt@flink-and-close-vma-leak.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk8/igt@gem_ppgtt@flink-and-close-vma-leak.html
* igt@gem_pxp@protected-encrypted-src-copy-not-readible:
- shard-tglb: NOTRUN -> [SKIP][30] ([i915#4270]) +2 similar issues
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb3/igt@gem_pxp@protected-encrypted-src-copy-not-readible.html
* igt@gem_pxp@reject-modify-context-protection-off-3:
- shard-iclb: NOTRUN -> [SKIP][31] ([i915#4270]) +1 similar issue
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb4/igt@gem_pxp@reject-modify-context-protection-off-3.html
* igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs:
- shard-iclb: NOTRUN -> [SKIP][32] ([i915#768]) +1 similar issue
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb6/igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs.html
* igt@gem_softpin@evict-snoop:
- shard-iclb: NOTRUN -> [SKIP][33] ([fdo#109312])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb6/igt@gem_softpin@evict-snoop.html
- shard-tglb: NOTRUN -> [SKIP][34] ([fdo#109312])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb3/igt@gem_softpin@evict-snoop.html
* igt@gem_userptr_blits@dmabuf-sync:
- shard-kbl: NOTRUN -> [SKIP][35] ([fdo#109271] / [i915#3323])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@gem_userptr_blits@dmabuf-sync.html
* igt@gem_userptr_blits@input-checking:
- shard-tglb: NOTRUN -> [DMESG-WARN][36] ([i915#4991])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb6/igt@gem_userptr_blits@input-checking.html
- shard-kbl: NOTRUN -> [DMESG-WARN][37] ([i915#4991])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@gem_userptr_blits@input-checking.html
* igt@gem_userptr_blits@unsync-overlap:
- shard-iclb: NOTRUN -> [SKIP][38] ([i915#3297]) +1 similar issue
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb3/igt@gem_userptr_blits@unsync-overlap.html
* igt@gem_userptr_blits@unsync-unmap-after-close:
- shard-tglb: NOTRUN -> [SKIP][39] ([i915#3297]) +2 similar issues
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb5/igt@gem_userptr_blits@unsync-unmap-after-close.html
* igt@gen3_mixed_blits:
- shard-iclb: NOTRUN -> [SKIP][40] ([fdo#109289]) +1 similar issue
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb3/igt@gen3_mixed_blits.html
* igt@gen7_exec_parse@basic-rejected:
- shard-tglb: NOTRUN -> [SKIP][41] ([fdo#109289]) +1 similar issue
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb7/igt@gen7_exec_parse@basic-rejected.html
* igt@gen9_exec_parse@bb-start-cmd:
- shard-tglb: NOTRUN -> [SKIP][42] ([i915#2527] / [i915#2856]) +1 similar issue
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb5/igt@gen9_exec_parse@bb-start-cmd.html
* igt@gen9_exec_parse@bb-start-far:
- shard-iclb: NOTRUN -> [SKIP][43] ([i915#2856]) +1 similar issue
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb3/igt@gen9_exec_parse@bb-start-far.html
* igt@i915_pm_dc@dc3co-vpb-simulation:
- shard-tglb: NOTRUN -> [SKIP][44] ([i915#1904])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@i915_pm_dc@dc3co-vpb-simulation.html
- shard-iclb: NOTRUN -> [SKIP][45] ([i915#658])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb7/igt@i915_pm_dc@dc3co-vpb-simulation.html
* igt@i915_pm_lpsp@screens-disabled:
- shard-tglb: NOTRUN -> [SKIP][46] ([i915#1902])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@i915_pm_lpsp@screens-disabled.html
- shard-iclb: NOTRUN -> [SKIP][47] ([i915#1902])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb5/igt@i915_pm_lpsp@screens-disabled.html
* igt@i915_pm_rpm@gem-execbuf-stress-pc8:
- shard-iclb: NOTRUN -> [SKIP][48] ([fdo#109293] / [fdo#109506]) +1 similar issue
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb2/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html
- shard-tglb: NOTRUN -> [SKIP][49] ([fdo#109506] / [i915#2411]) +1 similar issue
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb1/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html
* igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait:
- shard-tglb: NOTRUN -> [SKIP][50] ([fdo#111644] / [i915#1397] / [i915#2411])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb1/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html
- shard-iclb: NOTRUN -> [SKIP][51] ([fdo#110892])
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb4/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html
* igt@i915_selftest@live@gt_lrc:
- shard-tglb: NOTRUN -> [DMESG-FAIL][52] ([i915#2373])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb1/igt@i915_selftest@live@gt_lrc.html
* igt@i915_selftest@live@gt_pm:
- shard-tglb: NOTRUN -> [DMESG-FAIL][53] ([i915#1759])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb1/igt@i915_selftest@live@gt_pm.html
* igt@i915_suspend@sysfs-reader:
- shard-apl: [PASS][54] -> [DMESG-WARN][55] ([i915#180]) +3 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-apl3/igt@i915_suspend@sysfs-reader.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl3/igt@i915_suspend@sysfs-reader.html
* igt@kms_atomic_transition@plane-all-modeset-transition-fencing:
- shard-tglb: NOTRUN -> [SKIP][56] ([i915#1769])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html
* igt@kms_big_fb@linear-16bpp-rotate-90:
- shard-apl: NOTRUN -> [SKIP][57] ([fdo#109271]) +119 similar issues
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl2/igt@kms_big_fb@linear-16bpp-rotate-90.html
- shard-iclb: NOTRUN -> [SKIP][58] ([fdo#110725] / [fdo#111614]) +4 similar issues
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb6/igt@kms_big_fb@linear-16bpp-rotate-90.html
- shard-tglb: NOTRUN -> [SKIP][59] ([fdo#111614]) +4 similar issues
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb3/igt@kms_big_fb@linear-16bpp-rotate-90.html
* igt@kms_big_fb@linear-32bpp-rotate-180:
- shard-glk: [PASS][60] -> [DMESG-WARN][61] ([i915#118]) +1 similar issue
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-glk8/igt@kms_big_fb@linear-32bpp-rotate-180.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk7/igt@kms_big_fb@linear-32bpp-rotate-180.html
* igt@kms_big_fb@x-tiled-32bpp-rotate-180:
- shard-glk: [PASS][62] -> [DMESG-WARN][63] ([i915#118] / [i915#1982])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-glk2/igt@kms_big_fb@x-tiled-32bpp-rotate-180.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk8/igt@kms_big_fb@x-tiled-32bpp-rotate-180.html
* igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip:
- shard-apl: NOTRUN -> [SKIP][64] ([fdo#109271] / [i915#3777]) +1 similar issue
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl8/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip.html
- shard-glk: NOTRUN -> [SKIP][65] ([fdo#109271] / [i915#3777])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk1/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip-async-flip.html
* igt@kms_big_fb@yf-tiled-8bpp-rotate-270:
- shard-iclb: NOTRUN -> [SKIP][66] ([fdo#110723])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb2/igt@kms_big_fb@yf-tiled-8bpp-rotate-270.html
* igt@kms_big_fb@yf-tiled-addfb:
- shard-tglb: NOTRUN -> [SKIP][67] ([fdo#111615]) +6 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb2/igt@kms_big_fb@yf-tiled-addfb.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:
- shard-kbl: NOTRUN -> [SKIP][68] ([fdo#109271] / [i915#3777]) +7 similar issues
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl3/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
* igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_mc_ccs:
- shard-apl: NOTRUN -> [SKIP][69] ([fdo#109271] / [i915#3886]) +7 similar issues
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl4/igt@kms_ccs@pipe-a-ccs-on-another-bo-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc:
- shard-kbl: NOTRUN -> [SKIP][70] ([fdo#109271] / [i915#3886]) +15 similar issues
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl4/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_mc_ccs:
- shard-glk: NOTRUN -> [SKIP][71] ([fdo#109271] / [i915#3886]) +5 similar issues
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk6/igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-b-random-ccs-data-y_tiled_gen12_rc_ccs_cc:
- shard-iclb: NOTRUN -> [SKIP][72] ([fdo#109278] / [i915#3886]) +8 similar issues
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb7/igt@kms_ccs@pipe-b-random-ccs-data-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-c-bad-rotation-90-yf_tiled_ccs:
- shard-tglb: NOTRUN -> [SKIP][73] ([fdo#111615] / [i915#3689]) +4 similar issues
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb2/igt@kms_ccs@pipe-c-bad-rotation-90-yf_tiled_ccs.html
* igt@kms_ccs@pipe-c-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs:
- shard-tglb: NOTRUN -> [SKIP][74] ([i915#3689] / [i915#3886]) +3 similar issues
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb2/igt@kms_ccs@pipe-c-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-d-bad-aux-stride-y_tiled_gen12_mc_ccs:
- shard-tglb: NOTRUN -> [SKIP][75] ([i915#3689]) +7 similar issues
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb7/igt@kms_ccs@pipe-d-bad-aux-stride-y_tiled_gen12_mc_ccs.html
* igt@kms_chamelium@dp-edid-change-during-suspend:
- shard-apl: NOTRUN -> [SKIP][76] ([fdo#109271] / [fdo#111827]) +9 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl7/igt@kms_chamelium@dp-edid-change-during-suspend.html
* igt@kms_chamelium@vga-hpd-after-suspend:
- shard-tglb: NOTRUN -> [SKIP][77] ([fdo#109284] / [fdo#111827]) +16 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@kms_chamelium@vga-hpd-after-suspend.html
* igt@kms_chamelium@vga-hpd-with-enabled-mode:
- shard-iclb: NOTRUN -> [SKIP][78] ([fdo#109284] / [fdo#111827]) +11 similar issues
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb8/igt@kms_chamelium@vga-hpd-with-enabled-mode.html
* igt@kms_color@pipe-d-ctm-0-5:
- shard-iclb: NOTRUN -> [SKIP][79] ([fdo#109278] / [i915#1149]) +2 similar issues
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb2/igt@kms_color@pipe-d-ctm-0-5.html
* igt@kms_color_chamelium@pipe-a-ctm-blue-to-red:
- shard-snb: NOTRUN -> [SKIP][80] ([fdo#109271] / [fdo#111827]) +6 similar issues
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-snb6/igt@kms_color_chamelium@pipe-a-ctm-blue-to-red.html
- shard-kbl: NOTRUN -> [SKIP][81] ([fdo#109271] / [fdo#111827]) +29 similar issues
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@kms_color_chamelium@pipe-a-ctm-blue-to-red.html
- shard-glk: NOTRUN -> [SKIP][82] ([fdo#109271] / [fdo#111827]) +6 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk8/igt@kms_color_chamelium@pipe-a-ctm-blue-to-red.html
* igt@kms_color_chamelium@pipe-d-gamma:
- shard-iclb: NOTRUN -> [SKIP][83] ([fdo#109278] / [fdo#109284] / [fdo#111827])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb1/igt@kms_color_chamelium@pipe-d-gamma.html
* igt@kms_content_protection@content_type_change:
- shard-iclb: NOTRUN -> [SKIP][84] ([fdo#109300] / [fdo#111066]) +1 similar issue
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb4/igt@kms_content_protection@content_type_change.html
* igt@kms_content_protection@srm:
- shard-glk: NOTRUN -> [SKIP][85] ([fdo#109271]) +64 similar issues
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk5/igt@kms_content_protection@srm.html
- shard-kbl: NOTRUN -> [TIMEOUT][86] ([i915#1319])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl4/igt@kms_content_protection@srm.html
- shard-apl: NOTRUN -> [TIMEOUT][87] ([i915#1319])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl7/igt@kms_content_protection@srm.html
* igt@kms_content_protection@uevent:
- shard-tglb: NOTRUN -> [SKIP][88] ([i915#1063]) +2 similar issues
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@kms_content_protection@uevent.html
* igt@kms_cursor_crc@pipe-a-cursor-512x512-sliding:
- shard-iclb: NOTRUN -> [SKIP][89] ([fdo#109278] / [fdo#109279]) +2 similar issues
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb6/igt@kms_cursor_crc@pipe-a-cursor-512x512-sliding.html
* igt@kms_cursor_crc@pipe-a-cursor-suspend:
- shard-kbl: NOTRUN -> [DMESG-WARN][90] ([i915#180])
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
* igt@kms_cursor_crc@pipe-c-cursor-32x32-offscreen:
- shard-tglb: NOTRUN -> [SKIP][91] ([i915#3319]) +1 similar issue
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb6/igt@kms_cursor_crc@pipe-c-cursor-32x32-offscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-512x512-random:
- shard-tglb: NOTRUN -> [SKIP][92] ([fdo#109279] / [i915#3359]) +4 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb3/igt@kms_cursor_crc@pipe-c-cursor-512x512-random.html
* igt@kms_cursor_crc@pipe-d-cursor-max-size-sliding:
- shard-tglb: NOTRUN -> [SKIP][93] ([i915#3359]) +5 similar issues
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb1/igt@kms_cursor_crc@pipe-d-cursor-max-size-sliding.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- shard-tglb: NOTRUN -> [SKIP][94] ([i915#4103])
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb1/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@cursora-vs-flipb-atomic:
- shard-tglb: NOTRUN -> [SKIP][95] ([fdo#109274] / [fdo#111825]) +12 similar issues
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb7/igt@kms_cursor_legacy@cursora-vs-flipb-atomic.html
* igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size:
- shard-iclb: NOTRUN -> [SKIP][96] ([fdo#109274] / [fdo#109278])
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb7/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html
* igt@kms_cursor_legacy@pipe-d-single-move:
- shard-iclb: NOTRUN -> [SKIP][97] ([fdo#109278]) +29 similar issues
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb3/igt@kms_cursor_legacy@pipe-d-single-move.html
* igt@kms_flip@2x-flip-vs-modeset-vs-hang:
- shard-iclb: NOTRUN -> [SKIP][98] ([fdo#109274]) +7 similar issues
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb8/igt@kms_flip@2x-flip-vs-modeset-vs-hang.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-blt:
- shard-kbl: NOTRUN -> [SKIP][99] ([fdo#109271]) +316 similar issues
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl4/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-shrfb-fliptrack-mmap-gtt:
- shard-iclb: NOTRUN -> [SKIP][100] ([fdo#109280]) +18 similar issues
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb8/igt@kms_frontbuffer_tracking@fbc-2p-shrfb-fliptrack-mmap-gtt.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-onoff:
- shard-snb: NOTRUN -> [SKIP][101] ([fdo#109271]) +123 similar issues
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-snb6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-onoff.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-mmap-cpu:
- shard-tglb: NOTRUN -> [SKIP][102] ([fdo#109280] / [fdo#111825]) +30 similar issues
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb5/igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-mmap-cpu.html
* igt@kms_hdr@static-toggle-dpms:
- shard-tglb: NOTRUN -> [SKIP][103] ([i915#1187])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@kms_hdr@static-toggle-dpms.html
- shard-iclb: NOTRUN -> [SKIP][104] ([i915#1187])
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb5/igt@kms_hdr@static-toggle-dpms.html
* igt@kms_multipipe_modeset@basic-max-pipe-crc-check:
- shard-tglb: NOTRUN -> [SKIP][105] ([i915#1839]) +1 similar issue
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb2/igt@kms_multipipe_modeset@basic-max-pipe-crc-check.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-d:
- shard-kbl: NOTRUN -> [SKIP][106] ([fdo#109271] / [i915#533]) +1 similar issue
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl6/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html
* igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence:
- shard-apl: NOTRUN -> [SKIP][107] ([fdo#109271] / [i915#533])
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl7/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-basic:
- shard-apl: NOTRUN -> [FAIL][108] ([fdo#108145] / [i915#265]) +1 similar issue
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl4/igt@kms_plane_alpha_blend@pipe-a-alpha-basic.html
- shard-glk: NOTRUN -> [FAIL][109] ([fdo#108145] / [i915#265])
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk2/igt@kms_plane_alpha_blend@pipe-a-alpha-basic.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb:
- shard-kbl: NOTRUN -> [FAIL][110] ([i915#265])
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-basic:
- shard-kbl: NOTRUN -> [FAIL][111] ([fdo#108145] / [i915#265]) +2 similar issues
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl3/igt@kms_plane_alpha_blend@pipe-c-alpha-basic.html
* igt@kms_plane_lowres@pipe-c-tiling-yf:
- shard-iclb: NOTRUN -> [SKIP][112] ([i915#3536])
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb3/igt@kms_plane_lowres@pipe-c-tiling-yf.html
* igt@kms_plane_lowres@pipe-d-tiling-none:
- shard-tglb: NOTRUN -> [SKIP][113] ([i915#3536])
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb8/igt@kms_plane_lowres@pipe-d-tiling-none.html
* igt@kms_plane_lowres@pipe-d-tiling-yf:
- shard-tglb: NOTRUN -> [SKIP][114] ([fdo#111615] / [fdo#112054]) +1 similar issue
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb6/igt@kms_plane_lowres@pipe-d-tiling-yf.html
* igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping:
- shard-kbl: NOTRUN -> [SKIP][115] ([fdo#109271] / [i915#2733])
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@kms_plane_scaling@scaler-with-clipping-clamping@pipe-c-scaler-with-clipping-clamping.html
* igt@kms_psr2_su@page_flip-nv12:
- shard-tglb: NOTRUN -> [SKIP][116] ([i915#1911])
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb7/igt@kms_psr2_su@page_flip-nv12.html
- shard-apl: NOTRUN -> [SKIP][117] ([fdo#109271] / [i915#658]) +1 similar issue
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl8/igt@kms_psr2_su@page_flip-nv12.html
- shard-glk: NOTRUN -> [SKIP][118] ([fdo#109271] / [i915#658]) +1 similar issue
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk3/igt@kms_psr2_su@page_flip-nv12.html
- shard-kbl: NOTRUN -> [SKIP][119] ([fdo#109271] / [i915#658]) +1 similar issue
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl3/igt@kms_psr2_su@page_flip-nv12.html
* igt@kms_psr@psr2_basic:
- shard-tglb: NOTRUN -> [FAIL][120] ([i915#132] / [i915#3467]) +5 similar issues
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb3/igt@kms_psr@psr2_basic.html
* igt@kms_psr@psr2_cursor_plane_move:
- shard-iclb: [PASS][121] -> [SKIP][122] ([fdo#109441]) +2 similar issues
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11254/shard-iclb2/igt@kms_psr@psr2_cursor_plane_move.html
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb3/igt@kms_psr@psr2_cursor_plane_move.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-iclb: NOTRUN -> [SKIP][123] ([fdo#109441]) +3 similar issues
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb4/igt@kms_psr@psr2_sprite_plane_move.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-kbl: NOTRUN -> [DMESG-WARN][124] ([i915#180] / [i915#295])
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl1/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
* igt@kms_writeback@writeback-check-output:
- shard-apl: NOTRUN -> [SKIP][125] ([fdo#109271] / [i915#2437])
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-apl8/igt@kms_writeback@writeback-check-output.html
- shard-iclb: NOTRUN -> [SKIP][126] ([i915#2437])
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-iclb6/igt@kms_writeback@writeback-check-output.html
- shard-kbl: NOTRUN -> [SKIP][127] ([fdo#109271] / [i915#2437])
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-kbl7/igt@kms_writeback@writeback-check-output.html
- shard-tglb: NOTRUN -> [SKIP][128] ([i915#2437])
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-tglb5/igt@kms_writeback@writeback-check-output.html
- shard-glk: NOTRUN -> [SKIP][129] ([fdo#109271] / [i915#2437])
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/shard-glk7/igt@kms_writeback@writeback-check-output.html
* igt@nouveau_crc@pipe-a-ctx-flip-skip-current-frame:
- shard-tglb: NOTRUN -> [SKIP][130] ([i915#2530]) +2 similar issues
[130]: ht
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6655/index.html
[-- Attachment #2: Type: text/html, Size: 33921 bytes --]
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-19 1:32 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
2022-02-19 2:39 ` Dixit, Ashutosh
@ 2022-02-21 11:54 ` Tvrtko Ursulin
2022-02-23 15:34 ` Newsome, Jasmine
1 sibling, 1 reply; 13+ messages in thread
From: Tvrtko Ursulin @ 2022-02-21 11:54 UTC (permalink / raw)
To: Jasmine Newsome, igt-dev
On 19/02/2022 01:32, Jasmine Newsome wrote:
> The spin all test relied on context persistence unecessarily by trying
> to destroy contexts while keeping spinners active.
> The current implementation of context persistence in i915 can cause
> failures, and persistence is not needed for this test.
Could you please expand a bit on "current implementation" and "can cause
failures"?
Also from the subject of "Removing context persistence" I was expecting
to see usage of I915_CONTEXT_PARAM_PERSISTENCE to actually change the mode.
My concern is that the pattern of destroying contexts while keeping
things active on the GPU is very wide spread in IGT and possibly exists
in userspace as well.
Has the wider story been analysed by the architects here and what is the
plan? Do we actually know no userspace actually depends on it?
[Comes back later, after spotting the cover letter.]
So it's only GuC, not i915, so please say that in this commit message
since cover letters are not saved in git history.
Regards,
Tvrtko
>
> Moving intel_ctx_destroy after igt_spin_end.
>
> Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
> ---
> tests/i915/gem_spin_batch.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
> index 653812c7..707d69b6 100644
> --- a/tests/i915/gem_spin_batch.c
> +++ b/tests/i915/gem_spin_batch.c
> @@ -161,8 +161,6 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> .engine = e->flags,
> .flags = (IGT_SPIN_POLL_RUN |
> IGT_SPIN_NO_PREEMPTION));
> - if (flags & PARALLEL_SPIN_NEW_CTX)
> - intel_ctx_destroy(i915, ctx);
>
> igt_spin_busywait_until_started(spin);
> igt_list_move(&spin->link, &list);
> @@ -172,6 +170,8 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> igt_assert(gem_bo_busy(i915, spin->handle));
> ahnd = spin->ahnd;
> igt_spin_end(spin);
> + if (flags & PARALLEL_SPIN_NEW_CTX)
> + intel_ctx_destroy(i915, spin->opts.ctx);
> gem_sync(i915, spin->handle);
> igt_spin_free(i915, spin);
> put_ahnd(ahnd);
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-21 11:54 ` Tvrtko Ursulin
@ 2022-02-23 15:34 ` Newsome, Jasmine
0 siblings, 0 replies; 13+ messages in thread
From: Newsome, Jasmine @ 2022-02-23 15:34 UTC (permalink / raw)
To: Tvrtko Ursulin, igt-dev@lists.freedesktop.org
On 19/02/2022 01:32, Jasmine Newsome wrote:
> The spin all test relied on context persistence unecessarily by trying
> to destroy contexts while keeping spinners active.
> The current implementation of context persistence in i915 can cause
> failures, and persistence is not needed for this test.
Could you please expand a bit on "current implementation" and "can cause failures"?
Also from the subject of "Removing context persistence" I was expecting to see usage of I915_CONTEXT_PARAM_PERSISTENCE to actually change the mode.
My concern is that the pattern of destroying contexts while keeping things active on the GPU is very wide spread in IGT and possibly exists in userspace as well.
Has the wider story been analysed by the architects here and what is the plan? Do we actually know no userspace actually depends on it?
[Comes back later, after spotting the cover letter.]
So it's only GuC, not i915, so please say that in this commit message since cover letters are not saved in git history.
Regards,
Tvrtko
----------------------------------------------
Ok thanks. I will update the commit message and resend.
Regards,
Jasmine
>
> Moving intel_ctx_destroy after igt_spin_end.
>
> Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
> ---
> tests/i915/gem_spin_batch.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
> index 653812c7..707d69b6 100644
> --- a/tests/i915/gem_spin_batch.c
> +++ b/tests/i915/gem_spin_batch.c
> @@ -161,8 +161,6 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> .engine = e->flags,
> .flags = (IGT_SPIN_POLL_RUN |
> IGT_SPIN_NO_PREEMPTION));
> - if (flags & PARALLEL_SPIN_NEW_CTX)
> - intel_ctx_destroy(i915, ctx);
>
> igt_spin_busywait_until_started(spin);
> igt_list_move(&spin->link, &list); @@ -172,6 +170,8 @@ static void
> spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
> igt_assert(gem_bo_busy(i915, spin->handle));
> ahnd = spin->ahnd;
> igt_spin_end(spin);
> + if (flags & PARALLEL_SPIN_NEW_CTX)
> + intel_ctx_destroy(i915, spin->opts.ctx);
> gem_sync(i915, spin->handle);
> igt_spin_free(i915, spin);
> put_ahnd(ahnd);
^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-24 0:42 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
@ 2022-02-24 0:42 ` Jasmine Newsome
2022-02-24 2:51 ` Dixit, Ashutosh
0 siblings, 1 reply; 13+ messages in thread
From: Jasmine Newsome @ 2022-02-24 0:42 UTC (permalink / raw)
To: igt-dev; +Cc: jasmine.newsome
The spin all test relied on context persistence unecessarily by trying
to destroy contexts while keeping spinners active.
The current implementation of context persistence in i915 can cause
failures with GuC enabled, and persistence is not needed for this test.
Moving intel_ctx_destroy after igt_spin_end.
Signed-off-by: Jasmine Newsome <jasmine.newsome@intel.com>
---
tests/i915/gem_spin_batch.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c
index 653812c7..707d69b6 100644
--- a/tests/i915/gem_spin_batch.c
+++ b/tests/i915/gem_spin_batch.c
@@ -161,8 +161,6 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
.engine = e->flags,
.flags = (IGT_SPIN_POLL_RUN |
IGT_SPIN_NO_PREEMPTION));
- if (flags & PARALLEL_SPIN_NEW_CTX)
- intel_ctx_destroy(i915, ctx);
igt_spin_busywait_until_started(spin);
igt_list_move(&spin->link, &list);
@@ -172,6 +170,8 @@ static void spin_all(int i915, const intel_ctx_t *ctx, unsigned int flags)
igt_assert(gem_bo_busy(i915, spin->handle));
ahnd = spin->ahnd;
igt_spin_end(spin);
+ if (flags & PARALLEL_SPIN_NEW_CTX)
+ intel_ctx_destroy(i915, spin->opts.ctx);
gem_sync(i915, spin->handle);
igt_spin_free(i915, spin);
put_ahnd(ahnd);
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence
2022-02-24 0:42 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
@ 2022-02-24 2:51 ` Dixit, Ashutosh
0 siblings, 0 replies; 13+ messages in thread
From: Dixit, Ashutosh @ 2022-02-24 2:51 UTC (permalink / raw)
To: Jasmine Newsome; +Cc: igt-dev
On Wed, 23 Feb 2022 16:42:20 -0800, Jasmine Newsome wrote:
>
> The spin all test relied on context persistence unecessarily by trying
> to destroy contexts while keeping spinners active.
> The current implementation of context persistence in i915 can cause
> failures with GuC enabled, and persistence is not needed for this test.
>
> Moving intel_ctx_destroy after igt_spin_end.
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2022-02-24 2:52 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-19 0:10 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 1/2] lib/igt_dummyload.c: Save more context info in spinner Jasmine Newsome
2022-02-19 0:23 ` Dixit, Ashutosh
2022-02-19 0:10 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
2022-02-19 0:20 ` Dixit, Ashutosh
2022-02-19 1:50 ` [igt-dev] ✓ Fi.CI.BAT: success for Fix for gem_spin_batch (rev2) Patchwork
2022-02-19 19:58 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2022-02-19 1:32 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
2022-02-19 1:32 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
2022-02-19 2:39 ` Dixit, Ashutosh
2022-02-21 11:54 ` Tvrtko Ursulin
2022-02-23 15:34 ` Newsome, Jasmine
2022-02-24 0:42 [igt-dev] [PATCH i-g-t 0/2] Fix for gem_spin_batch Jasmine Newsome
2022-02-24 0:42 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_spin_batch: Removing context persistence Jasmine Newsome
2022-02-24 2:51 ` Dixit, Ashutosh
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox