* [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable
@ 2018-08-22 22:21 Antonio Argenziano
2018-08-22 22:39 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Antonio Argenziano @ 2018-08-22 22:21 UTC (permalink / raw)
To: igt-dev
'long-*' subtests didn't make any sense now so removed those as well.
Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
---
tests/gem_storedw_loop.c | 31 +++++++++++++------------------
1 file changed, 13 insertions(+), 18 deletions(-)
diff --git a/tests/gem_storedw_loop.c b/tests/gem_storedw_loop.c
index 8e57abe2..b00555e0 100644
--- a/tests/gem_storedw_loop.c
+++ b/tests/gem_storedw_loop.c
@@ -65,7 +65,7 @@ mmap_coherent(int fd, uint32_t handle, int size)
}
static void
-store_dword_loop(int fd, int ring, int count, int divider)
+store_dword_loop(int fd, int ring, int divider)
{
int i, val = 0;
struct drm_i915_gem_execbuffer2 execbuf;
@@ -143,19 +143,19 @@ store_dword_loop(int fd, int ring, int count, int divider)
}
static void
-store_test(int fd, int ring, int count)
+store_test(int fd, int ring)
{
gem_require_ring(fd, ring);
- store_dword_loop(fd, ring, count, 1);
- store_dword_loop(fd, ring, count, 2);
+ store_dword_loop(fd, ring, 1);
+ store_dword_loop(fd, ring, 2);
if (!igt_run_in_simulation()) {
- store_dword_loop(fd, ring, count, 3);
- store_dword_loop(fd, ring, count, 5);
- store_dword_loop(fd, ring, count, 7);
- store_dword_loop(fd, ring, count, 11);
- store_dword_loop(fd, ring, count, 13);
- store_dword_loop(fd, ring, count, 17);
- store_dword_loop(fd, ring, count, 19);
+ store_dword_loop(fd, ring, 3);
+ store_dword_loop(fd, ring, 5);
+ store_dword_loop(fd, ring, 7);
+ store_dword_loop(fd, ring, 11);
+ store_dword_loop(fd, ring, 13);
+ store_dword_loop(fd, ring, 17);
+ store_dword_loop(fd, ring, 19);
}
}
@@ -184,14 +184,9 @@ igt_main
}
for (e = intel_execution_engines; e->name; e++) {
- igt_subtest_f("short-%s", e->name) {
+ igt_subtest_f("store-%s", e->name) {
check_test_requirements(fd, e->exec_id);
- store_test(fd, e->exec_id | e->flags, 4*1024);
- }
-
- igt_subtest_f("long-%s", e->name) {
- check_test_requirements(fd, e->exec_id);
- store_test(fd, e->exec_id | e->flags, 4*1024*1024);
+ store_test(fd, e->exec_id | e->flags);
}
}
--
2.16.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for gem_storedw_loop: Remove unused variable
2018-08-22 22:21 [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable Antonio Argenziano
@ 2018-08-22 22:39 ` Patchwork
2018-08-22 23:54 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2018-08-23 7:23 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2018-08-22 22:39 UTC (permalink / raw)
To: Antonio Argenziano; +Cc: igt-dev
== Series Details ==
Series: gem_storedw_loop: Remove unused variable
URL : https://patchwork.freedesktop.org/series/48591/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4693 -> IGTPW_1737 =
== Summary - SUCCESS ==
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/48591/revisions/1/mbox/
== Known issues ==
Here are the changes found in IGTPW_1737 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@drv_selftest@live_hangcheck:
fi-kbl-7567u: PASS -> DMESG-FAIL (fdo#106947)
igt@gem_exec_suspend@basic-s3:
{fi-kbl-soraka}: NOTRUN -> INCOMPLETE (fdo#107556)
igt@kms_pipe_crc_basic@hang-read-crc-pipe-b:
{fi-byt-clapper}: PASS -> FAIL (fdo#107362, fdo#103191)
==== Possible fixes ====
igt@drv_selftest@live_hangcheck:
fi-kbl-7560u: DMESG-FAIL (fdo#106560, fdo#106947) -> PASS
igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b:
{fi-byt-clapper}: FAIL (fdo#107362) -> PASS
igt@kms_pipe_crc_basic@nonblocking-crc-pipe-b-frame-sequence:
{fi-byt-clapper}: FAIL (fdo#107362, fdo#103191) -> PASS
igt@pm_rpm@basic-pci-d3-state:
fi-skl-6600u: FAIL -> PASS
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#106560 https://bugs.freedesktop.org/show_bug.cgi?id=106560
fdo#106947 https://bugs.freedesktop.org/show_bug.cgi?id=106947
fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
fdo#107556 https://bugs.freedesktop.org/show_bug.cgi?id=107556
== Participating hosts (53 -> 48) ==
Additional (1): fi-kbl-soraka
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-icl-u
== Build changes ==
* IGT: IGT_4607 -> IGTPW_1737
CI_DRM_4693: f22275363ffa04dbd719032810154e857110cd75 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_1737: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1737/
IGT_4607: 6e0b3e7a2d241af36f8c6b1cc335aa1db3532d29 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Testlist changes ==
+igt@gem_storedw_loop@store-blt
+igt@gem_storedw_loop@store-bsd
+igt@gem_storedw_loop@store-bsd1
+igt@gem_storedw_loop@store-bsd2
+igt@gem_storedw_loop@store-default
+igt@gem_storedw_loop@store-render
+igt@gem_storedw_loop@store-vebox
-igt@gem_storedw_loop@long-blt
-igt@gem_storedw_loop@long-bsd
-igt@gem_storedw_loop@long-bsd1
-igt@gem_storedw_loop@long-bsd2
-igt@gem_storedw_loop@long-default
-igt@gem_storedw_loop@long-render
-igt@gem_storedw_loop@long-vebox
-igt@gem_storedw_loop@short-blt
-igt@gem_storedw_loop@short-bsd
-igt@gem_storedw_loop@short-bsd1
-igt@gem_storedw_loop@short-bsd2
-igt@gem_storedw_loop@short-default
-igt@gem_storedw_loop@short-render
-igt@gem_storedw_loop@short-vebox
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1737/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for gem_storedw_loop: Remove unused variable
2018-08-22 22:21 [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable Antonio Argenziano
2018-08-22 22:39 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
@ 2018-08-22 23:54 ` Patchwork
2018-08-23 7:23 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2018-08-22 23:54 UTC (permalink / raw)
To: Antonio Argenziano; +Cc: igt-dev
== Series Details ==
Series: gem_storedw_loop: Remove unused variable
URL : https://patchwork.freedesktop.org/series/48591/
State : success
== Summary ==
= CI Bug Log - changes from IGT_4607_full -> IGTPW_1737_full =
== Summary - WARNING ==
Minor unknown changes coming with IGTPW_1737_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_1737_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/48591/revisions/1/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in IGTPW_1737_full:
=== IGT changes ===
==== Warnings ====
igt@pm_rc6_residency@rc6-accuracy:
shard-kbl: SKIP -> PASS
== Known issues ==
Here are the changes found in IGTPW_1737_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@kms_busy@extended-modeset-hang-oldfb-render-c:
shard-snb: SKIP -> INCOMPLETE (fdo#105411)
igt@kms_plane@pixel-format-pipe-a-planes:
shard-snb: PASS -> FAIL (fdo#107161)
==== Possible fixes ====
igt@drv_suspend@debugfs-reader:
shard-snb: INCOMPLETE (fdo#105411) -> PASS +1
igt@drv_suspend@fence-restore-tiled2untiled:
shard-kbl: DMESG-WARN (fdo#103313) -> PASS
igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-wc:
shard-glk: FAIL (fdo#103167) -> PASS
igt@perf@polling:
shard-hsw: FAIL (fdo#102252) -> PASS
igt@testdisplay:
shard-glk: INCOMPLETE (fdo#103359, fdo#107093, k.org#198133) -> PASS
fdo#102252 https://bugs.freedesktop.org/show_bug.cgi?id=102252
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#103313 https://bugs.freedesktop.org/show_bug.cgi?id=103313
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
fdo#107093 https://bugs.freedesktop.org/show_bug.cgi?id=107093
fdo#107161 https://bugs.freedesktop.org/show_bug.cgi?id=107161
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (5 -> 5) ==
No changes in participating hosts
== Build changes ==
* IGT: IGT_4607 -> IGTPW_1737
* Linux: CI_DRM_4690 -> CI_DRM_4693
CI_DRM_4690: 5b8042159e2b05425c098868ed6fac9518ee638d @ git://anongit.freedesktop.org/gfx-ci/linux
CI_DRM_4693: f22275363ffa04dbd719032810154e857110cd75 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_1737: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1737/
IGT_4607: 6e0b3e7a2d241af36f8c6b1cc335aa1db3532d29 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1737/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable
2018-08-22 22:21 [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable Antonio Argenziano
2018-08-22 22:39 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2018-08-22 23:54 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
@ 2018-08-23 7:23 ` Chris Wilson
2018-08-23 16:49 ` Antonio Argenziano
2 siblings, 1 reply; 5+ messages in thread
From: Chris Wilson @ 2018-08-23 7:23 UTC (permalink / raw)
To: Antonio Argenziano, igt-dev
Quoting Antonio Argenziano (2018-08-22 23:21:10)
> 'long-*' subtests didn't make any sense now so removed those as well.
Even more confusing, it was never used.
> Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable
2018-08-23 7:23 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
@ 2018-08-23 16:49 ` Antonio Argenziano
0 siblings, 0 replies; 5+ messages in thread
From: Antonio Argenziano @ 2018-08-23 16:49 UTC (permalink / raw)
To: Chris Wilson, igt-dev
On 23/08/18 00:23, Chris Wilson wrote:
> Quoting Antonio Argenziano (2018-08-22 23:21:10)
>> 'long-*' subtests didn't make any sense now so removed those as well.
>
> Even more confusing, it was never used.
Confusing indeed :).
>
>> Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
>> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Pushed, thanks!
Antonio
> -Chris
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-08-23 16:49 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-22 22:21 [igt-dev] [PATCH i-g-t] gem_storedw_loop: Remove unused variable Antonio Argenziano
2018-08-22 22:39 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2018-08-22 23:54 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2018-08-23 7:23 ` [igt-dev] [PATCH i-g-t] " Chris Wilson
2018-08-23 16:49 ` Antonio Argenziano
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.