* [PATCH i-g-t] benchmarks/gem_wsim: Perturb static_vcs selection across clients
@ 2019-05-14 10:05 ` Chris Wilson
0 siblings, 0 replies; 11+ messages in thread
From: Chris Wilson @ 2019-05-14 10:05 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev
Use the client id to alternate the static_vcs balancer (-b context)
across clients - otherwise all clients end up on vcs0 and do not match
the context balancing employed by media-driver.
This may want to be behind the -R flag, but I felt it was a fundamental
property of static context balancing that to keep it disabled by default
causes unfair comparisons and poor workload scheduling, defeating the
purpose of testing.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
benchmarks/gem_wsim.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index afb9644dd..8c7e30eb4 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
static void
prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
{
- unsigned int ctx_vcs = 0;
+ unsigned int ctx_vcs = id & 1;
int max_ctx = -1;
struct w_step *w;
int i;
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Intel-gfx] [PATCH i-g-t] benchmarks/gem_wsim: Perturb static_vcs selection across clients
@ 2019-05-14 10:05 ` Chris Wilson
0 siblings, 0 replies; 11+ messages in thread
From: Chris Wilson @ 2019-05-14 10:05 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev
Use the client id to alternate the static_vcs balancer (-b context)
across clients - otherwise all clients end up on vcs0 and do not match
the context balancing employed by media-driver.
This may want to be behind the -R flag, but I felt it was a fundamental
property of static context balancing that to keep it disabled by default
causes unfair comparisons and poor workload scheduling, defeating the
purpose of testing.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
benchmarks/gem_wsim.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index afb9644dd..8c7e30eb4 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
static void
prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
{
- unsigned int ctx_vcs = 0;
+ unsigned int ctx_vcs = id & 1;
int max_ctx = -1;
struct w_step *w;
int i;
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 10:05 ` [Intel-gfx] " Chris Wilson
(?)
@ 2019-05-14 10:32 ` Patchwork
-1 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-05-14 10:32 UTC (permalink / raw)
To: Chris Wilson; +Cc: igt-dev
== Series Details ==
Series: benchmarks/gem_wsim: Perturb static_vcs selection across clients
URL : https://patchwork.freedesktop.org/series/60620/
State : success
== Summary ==
CI Bug Log - changes from IGT_4985 -> IGTPW_2978
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/60620/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_2978 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s3:
- fi-blb-e6850: [PASS][1] -> [INCOMPLETE][2] ([fdo#107718])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-blb-e6850/igt@gem_exec_suspend@basic-s3.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-blb-e6850/igt@gem_exec_suspend@basic-s3.html
* igt@i915_pm_rpm@module-reload:
- fi-skl-6770hq: [PASS][3] -> [FAIL][4] ([fdo#108511])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live_hangcheck:
- fi-apl-guc: [PASS][5] -> [DMESG-FAIL][6] ([fdo#110620])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-apl-guc/igt@i915_selftest@live_hangcheck.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-apl-guc/igt@i915_selftest@live_hangcheck.html
* igt@kms_frontbuffer_tracking@basic:
- fi-hsw-peppy: [PASS][7] -> [DMESG-WARN][8] ([fdo#102614])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
#### Possible fixes ####
* igt@i915_selftest@live_contexts:
- fi-skl-gvtdvm: [DMESG-FAIL][9] ([fdo#110235]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-skl-gvtdvm/igt@i915_selftest@live_contexts.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-skl-gvtdvm/igt@i915_selftest@live_contexts.html
* igt@i915_selftest@live_evict:
- fi-bsw-kefka: [DMESG-WARN][11] ([fdo#107709]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-bsw-kefka/igt@i915_selftest@live_evict.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-bsw-kefka/igt@i915_selftest@live_evict.html
* igt@i915_selftest@live_execlists:
- fi-apl-guc: [INCOMPLETE][13] ([fdo#103927] / [fdo#109720]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-apl-guc/igt@i915_selftest@live_execlists.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-apl-guc/igt@i915_selftest@live_execlists.html
#### Warnings ####
* igt@runner@aborted:
- fi-apl-guc: [FAIL][15] ([fdo#108622] / [fdo#109720]) -> [FAIL][16] ([fdo#110622])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/fi-apl-guc/igt@runner@aborted.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/fi-apl-guc/igt@runner@aborted.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#107709]: https://bugs.freedesktop.org/show_bug.cgi?id=107709
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
[fdo#108511]: https://bugs.freedesktop.org/show_bug.cgi?id=108511
[fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
[fdo#108622]: https://bugs.freedesktop.org/show_bug.cgi?id=108622
[fdo#109720]: https://bugs.freedesktop.org/show_bug.cgi?id=109720
[fdo#110235]: https://bugs.freedesktop.org/show_bug.cgi?id=110235
[fdo#110620]: https://bugs.freedesktop.org/show_bug.cgi?id=110620
[fdo#110622]: https://bugs.freedesktop.org/show_bug.cgi?id=110622
Participating hosts (53 -> 46)
------------------------------
Additional (1): fi-byt-j1900
Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus
Build changes
-------------
* IGT: IGT_4985 -> IGTPW_2978
CI_DRM_6078: c8c778558fd52abd3303d8ea324df788062adc97 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2978: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/
IGT_4985: dc0598498a16d43b0bc9cbc654491d4e6dc56626 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH i-g-t] benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 10:05 ` [Intel-gfx] " Chris Wilson
@ 2019-05-14 10:51 ` Tvrtko Ursulin
-1 siblings, 0 replies; 11+ messages in thread
From: Tvrtko Ursulin @ 2019-05-14 10:51 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 14/05/2019 11:05, Chris Wilson wrote:
> Use the client id to alternate the static_vcs balancer (-b context)
> across clients - otherwise all clients end up on vcs0 and do not match
> the context balancing employed by media-driver.
>
> This may want to be behind the -R flag, but I felt it was a fundamental
> property of static context balancing that to keep it disabled by default
> causes unfair comparisons and poor workload scheduling, defeating the
> purpose of testing.
I see your reasoning but it also completely matches the design of other
balancers to keep it under control of -R switch. It can also already be
achieved with the -G switch. Which is perhaps a bit confusing.. Having
both would still make sense I think. (-G gives out engines rr to
contexts sequentially across all clients, -R start each client contexts
by rr.)
But I wouldn't enable it unconditionally. Because consider another
balancer like rr and a two same workload instances of a long context
followed by short second context batch. If suffers the same problem of
poor scheduling until -R is added.
So I think we want to have the two balancers compatible in behaviour in
this respect.
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
> benchmarks/gem_wsim.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index afb9644dd..8c7e30eb4 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
> static void
> prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
> {
> - unsigned int ctx_vcs = 0;
> + unsigned int ctx_vcs = id & 1;
Therefore I think "ctx_vcs = (flags & INITVCSRR) ? id & 1 : 0" here.
> int max_ctx = -1;
> struct w_step *w;
> int i;
>
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] benchmarks/gem_wsim: Perturb static_vcs selection across clients
@ 2019-05-14 10:51 ` Tvrtko Ursulin
0 siblings, 0 replies; 11+ messages in thread
From: Tvrtko Ursulin @ 2019-05-14 10:51 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 14/05/2019 11:05, Chris Wilson wrote:
> Use the client id to alternate the static_vcs balancer (-b context)
> across clients - otherwise all clients end up on vcs0 and do not match
> the context balancing employed by media-driver.
>
> This may want to be behind the -R flag, but I felt it was a fundamental
> property of static context balancing that to keep it disabled by default
> causes unfair comparisons and poor workload scheduling, defeating the
> purpose of testing.
I see your reasoning but it also completely matches the design of other
balancers to keep it under control of -R switch. It can also already be
achieved with the -G switch. Which is perhaps a bit confusing.. Having
both would still make sense I think. (-G gives out engines rr to
contexts sequentially across all clients, -R start each client contexts
by rr.)
But I wouldn't enable it unconditionally. Because consider another
balancer like rr and a two same workload instances of a long context
followed by short second context batch. If suffers the same problem of
poor scheduling until -R is added.
So I think we want to have the two balancers compatible in behaviour in
this respect.
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
> benchmarks/gem_wsim.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index afb9644dd..8c7e30eb4 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
> static void
> prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
> {
> - unsigned int ctx_vcs = 0;
> + unsigned int ctx_vcs = id & 1;
Therefore I think "ctx_vcs = (flags & INITVCSRR) ? id & 1 : 0" here.
> int max_ctx = -1;
> struct w_step *w;
> int i;
>
Regards,
Tvrtko
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH i-g-t v2] benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 10:51 ` [igt-dev] [Intel-gfx] " Tvrtko Ursulin
(?)
@ 2019-05-14 10:59 ` Chris Wilson
-1 siblings, 0 replies; 11+ messages in thread
From: Chris Wilson @ 2019-05-14 10:59 UTC (permalink / raw)
To: intel-gfx
Use the client id to alternate the static_vcs balancer (-b context)
across clients with the round robin flag (-R) - otherwise all clients
end up on vcs0 and do not match the context balancing employed by
media-driver.
v2: Put it behind the -R flag.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
benchmarks/gem_wsim.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index afb9644dd..48568ce40 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
static void
prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
{
- unsigned int ctx_vcs = 0;
+ unsigned int ctx_vcs;
int max_ctx = -1;
struct w_step *w;
int i;
@@ -948,8 +948,10 @@ prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
wrk->prng = rand();
wrk->run = true;
+ ctx_vcs = 0;
if (flags & INITVCSRR)
- wrk->vcs_rr = id & 1;
+ ctx_vcs = id & 1;
+ wrk->vcs_rr = ctx_vcs;
if (flags & GLOBAL_BALANCE) {
int ret = pthread_mutex_init(&wrk->mutex, NULL);
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH i-g-t v3] benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 10:51 ` [igt-dev] [Intel-gfx] " Tvrtko Ursulin
(?)
(?)
@ 2019-05-14 11:04 ` Chris Wilson
2019-05-14 11:30 ` Tvrtko Ursulin
-1 siblings, 1 reply; 11+ messages in thread
From: Chris Wilson @ 2019-05-14 11:04 UTC (permalink / raw)
To: intel-gfx
Use the client id to alternate the static_vcs balancer (-b context)
across clients with the round robin flag (-R) - otherwise all clients
end up on vcs0 and do not match the context balancing employed by
media-driver.
v2: Put it behind the -R flag.
v3: Don't skip -R flag for -b context in scripts/media-bench.pl
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
benchmarks/gem_wsim.c | 6 ++++--
scripts/media-bench.pl | 2 +-
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index afb9644dd..48568ce40 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
static void
prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
{
- unsigned int ctx_vcs = 0;
+ unsigned int ctx_vcs;
int max_ctx = -1;
struct w_step *w;
int i;
@@ -948,8 +948,10 @@ prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
wrk->prng = rand();
wrk->run = true;
+ ctx_vcs = 0;
if (flags & INITVCSRR)
- wrk->vcs_rr = id & 1;
+ ctx_vcs = id & 1;
+ wrk->vcs_rr = ctx_vcs;
if (flags & GLOBAL_BALANCE) {
int ret = pthread_mutex_init(&wrk->mutex, NULL);
diff --git a/scripts/media-bench.pl b/scripts/media-bench.pl
index 066b542f9..f1cd59a25 100755
--- a/scripts/media-bench.pl
+++ b/scripts/media-bench.pl
@@ -52,7 +52,7 @@ my @balancers = ( 'rr', 'rand', 'qd', 'qdr', 'qdavg', 'rt', 'rtr', 'rtavg',
'context', 'busy', 'busy-avg' );
my %bal_skip_H = ( 'rr' => 1, 'rand' => 1, 'context' => 1, , 'busy' => 1,
'busy-avg' => 1 );
-my %bal_skip_R = ( 'context' => 1 );
+my %bal_skip_R = ();
my @workloads = (
'media_load_balance_17i7.wsim',
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH i-g-t v3] benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 11:04 ` [PATCH i-g-t v3] " Chris Wilson
@ 2019-05-14 11:30 ` Tvrtko Ursulin
2019-05-14 11:35 ` Chris Wilson
0 siblings, 1 reply; 11+ messages in thread
From: Tvrtko Ursulin @ 2019-05-14 11:30 UTC (permalink / raw)
To: Chris Wilson, intel-gfx
On 14/05/2019 12:04, Chris Wilson wrote:
> Use the client id to alternate the static_vcs balancer (-b context)
> across clients with the round robin flag (-R) - otherwise all clients
> end up on vcs0 and do not match the context balancing employed by
> media-driver.
>
> v2: Put it behind the -R flag.
> v3: Don't skip -R flag for -b context in scripts/media-bench.pl
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
> benchmarks/gem_wsim.c | 6 ++++--
> scripts/media-bench.pl | 2 +-
> 2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index afb9644dd..48568ce40 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
> static void
> prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
> {
> - unsigned int ctx_vcs = 0;
> + unsigned int ctx_vcs;
> int max_ctx = -1;
> struct w_step *w;
> int i;
> @@ -948,8 +948,10 @@ prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
> wrk->prng = rand();
> wrk->run = true;
>
> + ctx_vcs = 0;
> if (flags & INITVCSRR)
> - wrk->vcs_rr = id & 1;
> + ctx_vcs = id & 1;
> + wrk->vcs_rr = ctx_vcs;
>
> if (flags & GLOBAL_BALANCE) {
> int ret = pthread_mutex_init(&wrk->mutex, NULL);
> diff --git a/scripts/media-bench.pl b/scripts/media-bench.pl
> index 066b542f9..f1cd59a25 100755
> --- a/scripts/media-bench.pl
> +++ b/scripts/media-bench.pl
> @@ -52,7 +52,7 @@ my @balancers = ( 'rr', 'rand', 'qd', 'qdr', 'qdavg', 'rt', 'rtr', 'rtavg',
> 'context', 'busy', 'busy-avg' );
> my %bal_skip_H = ( 'rr' => 1, 'rand' => 1, 'context' => 1, , 'busy' => 1,
> 'busy-avg' => 1 );
> -my %bal_skip_R = ( 'context' => 1 );
> +my %bal_skip_R = ();
>
> my @workloads = (
> 'media_load_balance_17i7.wsim',
>
This probably means I was thinking -G covers this for -b context, which
it does. Difference between -R and -G there seems purely in wording
since clients are initialized sequentially and in deterministic order.
Hm I guess for heterogeneous clients they would be different. Okay,
makes sense then.
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH i-g-t v3] benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 11:30 ` Tvrtko Ursulin
@ 2019-05-14 11:35 ` Chris Wilson
0 siblings, 0 replies; 11+ messages in thread
From: Chris Wilson @ 2019-05-14 11:35 UTC (permalink / raw)
To: Tvrtko Ursulin, intel-gfx
Quoting Tvrtko Ursulin (2019-05-14 12:30:10)
>
> On 14/05/2019 12:04, Chris Wilson wrote:
> > Use the client id to alternate the static_vcs balancer (-b context)
> > across clients with the round robin flag (-R) - otherwise all clients
> > end up on vcs0 and do not match the context balancing employed by
> > media-driver.
> >
> > v2: Put it behind the -R flag.
> > v3: Don't skip -R flag for -b context in scripts/media-bench.pl
> >
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> > ---
> > benchmarks/gem_wsim.c | 6 ++++--
> > scripts/media-bench.pl | 2 +-
> > 2 files changed, 5 insertions(+), 3 deletions(-)
> >
> > diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> > index afb9644dd..48568ce40 100644
> > --- a/benchmarks/gem_wsim.c
> > +++ b/benchmarks/gem_wsim.c
> > @@ -939,7 +939,7 @@ alloc_step_batch(struct workload *wrk, struct w_step *w, unsigned int flags)
> > static void
> > prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
> > {
> > - unsigned int ctx_vcs = 0;
> > + unsigned int ctx_vcs;
> > int max_ctx = -1;
> > struct w_step *w;
> > int i;
> > @@ -948,8 +948,10 @@ prepare_workload(unsigned int id, struct workload *wrk, unsigned int flags)
> > wrk->prng = rand();
> > wrk->run = true;
> >
> > + ctx_vcs = 0;
> > if (flags & INITVCSRR)
> > - wrk->vcs_rr = id & 1;
> > + ctx_vcs = id & 1;
> > + wrk->vcs_rr = ctx_vcs;
> >
> > if (flags & GLOBAL_BALANCE) {
> > int ret = pthread_mutex_init(&wrk->mutex, NULL);
> > diff --git a/scripts/media-bench.pl b/scripts/media-bench.pl
> > index 066b542f9..f1cd59a25 100755
> > --- a/scripts/media-bench.pl
> > +++ b/scripts/media-bench.pl
> > @@ -52,7 +52,7 @@ my @balancers = ( 'rr', 'rand', 'qd', 'qdr', 'qdavg', 'rt', 'rtr', 'rtavg',
> > 'context', 'busy', 'busy-avg' );
> > my %bal_skip_H = ( 'rr' => 1, 'rand' => 1, 'context' => 1, , 'busy' => 1,
> > 'busy-avg' => 1 );
> > -my %bal_skip_R = ( 'context' => 1 );
> > +my %bal_skip_R = ();
> >
> > my @workloads = (
> > 'media_load_balance_17i7.wsim',
> >
>
> This probably means I was thinking -G covers this for -b context, which
> it does. Difference between -R and -G there seems purely in wording
> since clients are initialized sequentially and in deterministic order.
> Hm I guess for heterogeneous clients they would be different. Okay,
> makes sense then.
Hmm, right there is a subtle difference between context_vcs_rr and
ctx_vcs here. If a client creates an odd number of contexts, but doesn't
use all the VCS engines, it is still possible for -G to end up with all
active workloads on VCS0 and -R avoids that.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✗ Fi.CI.BAT: failure for benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 10:05 ` [Intel-gfx] " Chris Wilson
` (2 preceding siblings ...)
(?)
@ 2019-05-14 12:31 ` Patchwork
-1 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-05-14 12:31 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: benchmarks/gem_wsim: Perturb static_vcs selection across clients
URL : https://patchwork.freedesktop.org/series/60624/
State : failure
== Summary ==
Series 60624 revision 1 was fully merged or fully failed: no git log
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* [igt-dev] ✗ Fi.CI.IGT: failure for benchmarks/gem_wsim: Perturb static_vcs selection across clients
2019-05-14 10:05 ` [Intel-gfx] " Chris Wilson
` (3 preceding siblings ...)
(?)
@ 2019-05-14 13:10 ` Patchwork
-1 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2019-05-14 13:10 UTC (permalink / raw)
To: Chris Wilson; +Cc: igt-dev
== Series Details ==
Series: benchmarks/gem_wsim: Perturb static_vcs selection across clients
URL : https://patchwork.freedesktop.org/series/60620/
State : failure
== Summary ==
CI Bug Log - changes from IGT_4985_full -> IGTPW_2978_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_2978_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_2978_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/60620/revisions/1/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_2978_full:
### IGT changes ###
#### Possible regressions ####
* igt@gem_eio@in-flight-suspend:
- shard-snb: [PASS][1] -> [FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-snb1/igt@gem_eio@in-flight-suspend.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-snb7/igt@gem_eio@in-flight-suspend.html
- shard-glk: [PASS][3] -> [FAIL][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-glk2/igt@gem_eio@in-flight-suspend.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-glk3/igt@gem_eio@in-flight-suspend.html
Known issues
------------
Here are the changes found in IGTPW_2978_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@system-suspend-execbuf:
- shard-hsw: [PASS][5] -> [INCOMPLETE][6] ([fdo#103540] / [fdo#107807])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-hsw1/igt@i915_pm_rpm@system-suspend-execbuf.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-hsw8/igt@i915_pm_rpm@system-suspend-execbuf.html
* igt@kms_flip@2x-flip-vs-expired-vblank:
- shard-glk: [PASS][7] -> [FAIL][8] ([fdo#105363])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-glk9/igt@kms_flip@2x-flip-vs-expired-vblank.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-glk3/igt@kms_flip@2x-flip-vs-expired-vblank.html
* igt@kms_flip@modeset-vs-vblank-race:
- shard-kbl: [PASS][9] -> [INCOMPLETE][10] ([fdo#103665])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-kbl3/igt@kms_flip@modeset-vs-vblank-race.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-kbl1/igt@kms_flip@modeset-vs-vblank-race.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-move:
- shard-glk: [PASS][11] -> [FAIL][12] ([fdo#103167])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-glk3/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-move.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-glk2/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-move.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite:
- shard-iclb: [PASS][13] -> [FAIL][14] ([fdo#103167]) +4 similar issues
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-iclb: [PASS][15] -> [SKIP][16] ([fdo#109441]) +1 similar issue
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb5/igt@kms_psr@psr2_sprite_plane_move.html
* igt@kms_setmode@basic:
- shard-apl: [PASS][17] -> [FAIL][18] ([fdo#99912])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-apl4/igt@kms_setmode@basic.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-apl2/igt@kms_setmode@basic.html
* igt@kms_vblank@pipe-c-ts-continuation-suspend:
- shard-apl: [PASS][19] -> [DMESG-WARN][20] ([fdo#108566])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-apl2/igt@kms_vblank@pipe-c-ts-continuation-suspend.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-apl6/igt@kms_vblank@pipe-c-ts-continuation-suspend.html
#### Possible fixes ####
* igt@gem_eio@unwedge-stress:
- shard-glk: [FAIL][21] ([fdo#109661]) -> [PASS][22]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-glk6/igt@gem_eio@unwedge-stress.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-glk4/igt@gem_eio@unwedge-stress.html
* igt@gem_tiled_swapping@non-threaded:
- shard-kbl: [DMESG-WARN][23] ([fdo#108686]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-kbl4/igt@gem_tiled_swapping@non-threaded.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-kbl6/igt@gem_tiled_swapping@non-threaded.html
* igt@i915_pm_rpm@i2c:
- shard-iclb: [DMESG-WARN][25] ([fdo#109982]) -> [PASS][26]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb2/igt@i915_pm_rpm@i2c.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb3/igt@i915_pm_rpm@i2c.html
* igt@i915_pm_rpm@legacy-planes-dpms:
- shard-iclb: [INCOMPLETE][27] ([fdo#107713] / [fdo#108840] / [fdo#109960]) -> [PASS][28]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb4/igt@i915_pm_rpm@legacy-planes-dpms.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb7/igt@i915_pm_rpm@legacy-planes-dpms.html
* igt@i915_suspend@sysfs-reader:
- shard-apl: [DMESG-WARN][29] ([fdo#108566]) -> [PASS][30] +6 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-apl5/igt@i915_suspend@sysfs-reader.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-apl8/igt@i915_suspend@sysfs-reader.html
* {igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen}:
- shard-kbl: [FAIL][31] ([fdo#103232]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-kbl1/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-kbl7/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
- shard-apl: [FAIL][33] ([fdo#103232]) -> [PASS][34]
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-apl4/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-apl5/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-pwrite:
- shard-glk: [FAIL][35] ([fdo#103167]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-glk2/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-pwrite.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-glk5/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-draw-pwrite.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-pwrite:
- shard-iclb: [FAIL][37] ([fdo#103167]) -> [PASS][38] +4 similar issues
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-pwrite.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-pwrite.html
* igt@kms_psr@psr2_no_drrs:
- shard-iclb: [SKIP][39] ([fdo#109441]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb3/igt@kms_psr@psr2_no_drrs.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb2/igt@kms_psr@psr2_no_drrs.html
* igt@kms_sysfs_edid_timing:
- shard-iclb: [FAIL][41] ([fdo#100047]) -> [PASS][42]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-iclb2/igt@kms_sysfs_edid_timing.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-iclb5/igt@kms_sysfs_edid_timing.html
* igt@tools_test@tools_test:
- shard-snb: [SKIP][43] ([fdo#109271]) -> [PASS][44]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4985/shard-snb4/igt@tools_test@tools_test.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/shard-snb4/igt@tools_test@tools_test.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
[fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
[fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109661]: https://bugs.freedesktop.org/show_bug.cgi?id=109661
[fdo#109960]: https://bugs.freedesktop.org/show_bug.cgi?id=109960
[fdo#109982]: https://bugs.freedesktop.org/show_bug.cgi?id=109982
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
Participating hosts (7 -> 6)
------------------------------
Missing (1): shard-skl
Build changes
-------------
* IGT: IGT_4985 -> IGTPW_2978
CI_DRM_6078: c8c778558fd52abd3303d8ea324df788062adc97 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2978: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/
IGT_4985: dc0598498a16d43b0bc9cbc654491d4e6dc56626 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2978/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-05-14 13:10 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-14 10:05 [PATCH i-g-t] benchmarks/gem_wsim: Perturb static_vcs selection across clients Chris Wilson
2019-05-14 10:05 ` [Intel-gfx] " Chris Wilson
2019-05-14 10:32 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2019-05-14 10:51 ` [PATCH i-g-t] " Tvrtko Ursulin
2019-05-14 10:51 ` [igt-dev] [Intel-gfx] " Tvrtko Ursulin
2019-05-14 10:59 ` [PATCH i-g-t v2] " Chris Wilson
2019-05-14 11:04 ` [PATCH i-g-t v3] " Chris Wilson
2019-05-14 11:30 ` Tvrtko Ursulin
2019-05-14 11:35 ` Chris Wilson
2019-05-14 12:31 ` ✗ Fi.CI.BAT: failure for " Patchwork
2019-05-14 13:10 ` [igt-dev] ✗ Fi.CI.IGT: " 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.