* [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-16 18:58 ` Chris Wilson
0 siblings, 0 replies; 14+ messages in thread
From: Chris Wilson @ 2019-05-16 18:58 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev
To avoid hitting the same rut on each benchmark run, start with a new
random seed. To allow hitting the same rut again, let it be specified
by the user.
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, 5 insertions(+), 1 deletion(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index 48568ce40..cf2a44746 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
igt_require(fd);
init_clocks();
+ srand(time(NULL));
- while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
+ while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
switch (c) {
case 'W':
if (master_workload >= 0) {
@@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
case 'p':
prio = atoi(optarg);
break;
+ case 's':
+ srand(atoi(optarg));
+ break;
case 'a':
if (append_workload_arg) {
if (verbose)
--
2.20.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [igt-dev] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-16 18:58 ` Chris Wilson
0 siblings, 0 replies; 14+ messages in thread
From: Chris Wilson @ 2019-05-16 18:58 UTC (permalink / raw)
To: intel-gfx; +Cc: igt-dev, Tvrtko Ursulin
To avoid hitting the same rut on each benchmark run, start with a new
random seed. To allow hitting the same rut again, let it be specified
by the user.
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, 5 insertions(+), 1 deletion(-)
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index 48568ce40..cf2a44746 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
igt_require(fd);
init_clocks();
+ srand(time(NULL));
- while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
+ while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
switch (c) {
case 'W':
if (master_workload >= 0) {
@@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
case 'p':
prio = atoi(optarg);
break;
+ case 's':
+ srand(atoi(optarg));
+ break;
case 'a':
if (append_workload_arg) {
if (verbose)
--
2.20.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for benchmarks/gem_wsim: Randomise random seed
2019-05-16 18:58 ` [igt-dev] " Chris Wilson
(?)
@ 2019-05-16 19:37 ` Patchwork
-1 siblings, 0 replies; 14+ messages in thread
From: Patchwork @ 2019-05-16 19:37 UTC (permalink / raw)
To: Chris Wilson; +Cc: igt-dev
== Series Details ==
Series: benchmarks/gem_wsim: Randomise random seed
URL : https://patchwork.freedesktop.org/series/60733/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6091 -> IGTPW_2993
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/60733/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_2993 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_basic@userptr:
- fi-kbl-8809g: [PASS][1] -> [DMESG-WARN][2] ([fdo#108965])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-kbl-8809g/igt@amdgpu/amd_basic@userptr.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-kbl-8809g/igt@amdgpu/amd_basic@userptr.html
* igt@debugfs_test@read_all_entries:
- fi-ilk-650: [PASS][3] -> [DMESG-WARN][4] ([fdo#106387])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-ilk-650/igt@debugfs_test@read_all_entries.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-ilk-650/igt@debugfs_test@read_all_entries.html
* igt@i915_selftest@live_guc:
- fi-apl-guc: [PASS][5] -> [INCOMPLETE][6] ([fdo#103927])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-apl-guc/igt@i915_selftest@live_guc.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-apl-guc/igt@i915_selftest@live_guc.html
* igt@i915_selftest@live_hangcheck:
- fi-apl-guc: [PASS][7] -> [DMESG-FAIL][8] ([fdo#110620])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-apl-guc/igt@i915_selftest@live_hangcheck.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-apl-guc/igt@i915_selftest@live_hangcheck.html
#### Possible fixes ####
* igt@i915_pm_rpm@module-reload:
- fi-skl-6770hq: [FAIL][9] ([fdo#108511]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-skl-6770hq/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live_contexts:
- fi-bdw-gvtdvm: [DMESG-FAIL][11] ([fdo#110235]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-bdw-gvtdvm/igt@i915_selftest@live_contexts.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-bdw-gvtdvm/igt@i915_selftest@live_contexts.html
* igt@kms_frontbuffer_tracking@basic:
- fi-hsw-peppy: [DMESG-WARN][13] ([fdo#102614]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html
* igt@prime_vgem@basic-fence-flip:
- fi-ilk-650: [DMESG-WARN][15] ([fdo#106387]) -> [PASS][16] +1 similar issue
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/fi-ilk-650/igt@prime_vgem@basic-fence-flip.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/fi-ilk-650/igt@prime_vgem@basic-fence-flip.html
[fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#106387]: https://bugs.freedesktop.org/show_bug.cgi?id=106387
[fdo#108511]: https://bugs.freedesktop.org/show_bug.cgi?id=108511
[fdo#108965]: https://bugs.freedesktop.org/show_bug.cgi?id=108965
[fdo#110235]: https://bugs.freedesktop.org/show_bug.cgi?id=110235
[fdo#110620]: https://bugs.freedesktop.org/show_bug.cgi?id=110620
Participating hosts (50 -> 44)
------------------------------
Additional (1): fi-bsw-n3050
Missing (7): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-byt-clapper fi-bdw-samus
Build changes
-------------
* IGT: IGT_4994 -> IGTPW_2993
CI_DRM_6091: 0ad895242a8e957336088625a9a6ba48ab838ec9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2993: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/
IGT_4994: 555019f862c35f1619627761d6da21385be40920 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 14+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for benchmarks/gem_wsim: Randomise random seed
2019-05-16 18:58 ` [igt-dev] " Chris Wilson
(?)
(?)
@ 2019-05-17 1:50 ` Patchwork
-1 siblings, 0 replies; 14+ messages in thread
From: Patchwork @ 2019-05-17 1:50 UTC (permalink / raw)
To: Chris Wilson; +Cc: igt-dev
== Series Details ==
Series: benchmarks/gem_wsim: Randomise random seed
URL : https://patchwork.freedesktop.org/series/60733/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6091_full -> IGTPW_2993_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/60733/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_2993_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_eio@in-flight-suspend:
- shard-apl: [PASS][1] -> [DMESG-WARN][2] ([fdo#108566]) +6 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-apl7/igt@gem_eio@in-flight-suspend.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-apl1/igt@gem_eio@in-flight-suspend.html
- shard-glk: [PASS][3] -> [FAIL][4] ([fdo#110667])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-glk3/igt@gem_eio@in-flight-suspend.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-glk1/igt@gem_eio@in-flight-suspend.html
* igt@gem_softpin@noreloc-s3:
- shard-kbl: [PASS][5] -> [DMESG-WARN][6] ([fdo#108566])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-kbl1/igt@gem_softpin@noreloc-s3.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-kbl7/igt@gem_softpin@noreloc-s3.html
* igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic:
- shard-glk: [PASS][7] -> [FAIL][8] ([fdo#106509] / [fdo#107409])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-glk4/igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-glk4/igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-hsw: [PASS][9] -> [INCOMPLETE][10] ([fdo#103540])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-hsw4/igt@kms_flip@flip-vs-suspend-interruptible.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-hsw5/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw:
- shard-iclb: [PASS][11] -> [FAIL][12] ([fdo#103167]) +5 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb7/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
* igt@kms_psr@psr2_no_drrs:
- shard-iclb: [PASS][13] -> [SKIP][14] ([fdo#109441])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb2/igt@kms_psr@psr2_no_drrs.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb1/igt@kms_psr@psr2_no_drrs.html
* igt@kms_sysfs_edid_timing:
- shard-iclb: [PASS][15] -> [FAIL][16] ([fdo#100047])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb8/igt@kms_sysfs_edid_timing.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb2/igt@kms_sysfs_edid_timing.html
* igt@perf_pmu@rc6-runtime-pm:
- shard-glk: [PASS][17] -> [FAIL][18] ([fdo#105010])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-glk9/igt@perf_pmu@rc6-runtime-pm.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-glk3/igt@perf_pmu@rc6-runtime-pm.html
- shard-apl: [PASS][19] -> [FAIL][20] ([fdo#105010])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-apl4/igt@perf_pmu@rc6-runtime-pm.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-apl1/igt@perf_pmu@rc6-runtime-pm.html
- shard-kbl: [PASS][21] -> [FAIL][22] ([fdo#105010])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-kbl6/igt@perf_pmu@rc6-runtime-pm.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-kbl2/igt@perf_pmu@rc6-runtime-pm.html
#### Possible fixes ####
* igt@i915_pm_rpm@gem-mmap-gtt:
- shard-iclb: [INCOMPLETE][23] ([fdo#107713] / [fdo#108840]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb7/igt@i915_pm_rpm@gem-mmap-gtt.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb1/igt@i915_pm_rpm@gem-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite:
- shard-iclb: [FAIL][25] ([fdo#103167]) -> [PASS][26] +1 similar issue
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
- shard-kbl: [INCOMPLETE][27] ([fdo#103665]) -> [PASS][28]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-kbl2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-kbl7/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c.html
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
- shard-apl: [DMESG-WARN][29] ([fdo#108566]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-apl5/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-apl3/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
- shard-kbl: [DMESG-WARN][31] ([fdo#108566]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-kbl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-kbl2/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html
* igt@kms_plane_cursor@pipe-a-primary-size-64:
- shard-iclb: [INCOMPLETE][33] ([fdo#107713]) -> [PASS][34] +1 similar issue
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb1/igt@kms_plane_cursor@pipe-a-primary-size-64.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb1/igt@kms_plane_cursor@pipe-a-primary-size-64.html
* igt@kms_psr@psr2_sprite_plane_onoff:
- shard-iclb: [SKIP][35] ([fdo#109441]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb3/igt@kms_psr@psr2_sprite_plane_onoff.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html
* igt@perf_pmu@rc6-runtime-pm-long:
- shard-apl: [FAIL][37] ([fdo#105010]) -> [PASS][38]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-apl2/igt@perf_pmu@rc6-runtime-pm-long.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-apl1/igt@perf_pmu@rc6-runtime-pm-long.html
- shard-kbl: [FAIL][39] ([fdo#105010]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-kbl1/igt@perf_pmu@rc6-runtime-pm-long.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-kbl4/igt@perf_pmu@rc6-runtime-pm-long.html
#### Warnings ####
* igt@gem_mmap_gtt@forked-big-copy:
- shard-iclb: [TIMEOUT][41] ([fdo#109673]) -> [INCOMPLETE][42] ([fdo#107713] / [fdo#109100])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb6/igt@gem_mmap_gtt@forked-big-copy.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb4/igt@gem_mmap_gtt@forked-big-copy.html
* igt@kms_chamelium@hdmi-hpd-after-suspend:
- shard-iclb: [INCOMPLETE][43] ([fdo#107713]) -> [SKIP][44] ([fdo#109284])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-iclb7/igt@kms_chamelium@hdmi-hpd-after-suspend.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-iclb2/igt@kms_chamelium@hdmi-hpd-after-suspend.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite:
- shard-apl: [SKIP][45] ([fdo#109271]) -> [INCOMPLETE][46] ([fdo#103927])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6091/shard-apl6/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/shard-apl6/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite.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#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#105010]: https://bugs.freedesktop.org/show_bug.cgi?id=105010
[fdo#106509]: https://bugs.freedesktop.org/show_bug.cgi?id=106509
[fdo#107409]: https://bugs.freedesktop.org/show_bug.cgi?id=107409
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
[fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109673]: https://bugs.freedesktop.org/show_bug.cgi?id=109673
[fdo#110667]: https://bugs.freedesktop.org/show_bug.cgi?id=110667
Participating hosts (10 -> 6)
------------------------------
Missing (4): pig-skl-6260u shard-skl pig-hsw-4770r pig-glk-j5005
Build changes
-------------
* IGT: IGT_4994 -> IGTPW_2993
* Piglit: piglit_4509 -> None
CI_DRM_6091: 0ad895242a8e957336088625a9a6ba48ab838ec9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2993: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/
IGT_4994: 555019f862c35f1619627761d6da21385be40920 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2993/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
2019-05-16 18:58 ` [igt-dev] " Chris Wilson
@ 2019-05-17 8:28 ` Tvrtko Ursulin
-1 siblings, 0 replies; 14+ messages in thread
From: Tvrtko Ursulin @ 2019-05-17 8:28 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 16/05/2019 19:58, Chris Wilson wrote:
> To avoid hitting the same rut on each benchmark run, start with a new
> random seed. To allow hitting the same rut again, let it be specified
> by the user.
>
> 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, 5 insertions(+), 1 deletion(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index 48568ce40..cf2a44746 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
> igt_require(fd);
>
> init_clocks();
> + srand(time(NULL));
>
> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
> switch (c) {
> case 'W':
> if (master_workload >= 0) {
> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
> case 'p':
> prio = atoi(optarg);
> break;
> + case 's':
> + srand(atoi(optarg));
> + break;
> case 'a':
> if (append_workload_arg) {
> if (verbose)
>
Makes sense, just the help text missing.
I was initially confused at srand when I thought I used your
hars_petruska library routines but then saw rand() feeds the seed into
those.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-17 8:28 ` Tvrtko Ursulin
0 siblings, 0 replies; 14+ messages in thread
From: Tvrtko Ursulin @ 2019-05-17 8:28 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 16/05/2019 19:58, Chris Wilson wrote:
> To avoid hitting the same rut on each benchmark run, start with a new
> random seed. To allow hitting the same rut again, let it be specified
> by the user.
>
> 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, 5 insertions(+), 1 deletion(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index 48568ce40..cf2a44746 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
> igt_require(fd);
>
> init_clocks();
> + srand(time(NULL));
>
> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
> switch (c) {
> case 'W':
> if (master_workload >= 0) {
> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
> case 'p':
> prio = atoi(optarg);
> break;
> + case 's':
> + srand(atoi(optarg));
> + break;
> case 'a':
> if (append_workload_arg) {
> if (verbose)
>
Makes sense, just the help text missing.
I was initially confused at srand when I thought I used your
hars_petruska library routines but then saw rand() feeds the seed into
those.
Regards,
Tvrtko
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
2019-05-17 8:28 ` [igt-dev] [Intel-gfx] " Tvrtko Ursulin
@ 2019-05-17 8:37 ` Chris Wilson
-1 siblings, 0 replies; 14+ messages in thread
From: Chris Wilson @ 2019-05-17 8:37 UTC (permalink / raw)
To: Tvrtko Ursulin, intel-gfx; +Cc: igt-dev
Quoting Tvrtko Ursulin (2019-05-17 09:28:46)
>
> On 16/05/2019 19:58, Chris Wilson wrote:
> > To avoid hitting the same rut on each benchmark run, start with a new
> > random seed. To allow hitting the same rut again, let it be specified
> > by the user.
> >
> > 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, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> > index 48568ce40..cf2a44746 100644
> > --- a/benchmarks/gem_wsim.c
> > +++ b/benchmarks/gem_wsim.c
> > @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
> > igt_require(fd);
> >
> > init_clocks();
> > + srand(time(NULL));
> >
> > - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
> > + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
> > switch (c) {
> > case 'W':
> > if (master_workload >= 0) {
> > @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
> > case 'p':
> > prio = atoi(optarg);
> > break;
> > + case 's':
> > + srand(atoi(optarg));
> > + break;
> > case 'a':
> > if (append_workload_arg) {
> > if (verbose)
> >
>
> Makes sense, just the help text missing.
You use 's' for sseu, I think I (Initial random seed) is one of the few
available.
> I was initially confused at srand when I thought I used your
> hars_petruska library routines but then saw rand() feeds the seed into
> those.
Yeah, I had to go and double check that we actually used rand()
somewhere (and not just in my hacks :)
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-17 8:37 ` Chris Wilson
0 siblings, 0 replies; 14+ messages in thread
From: Chris Wilson @ 2019-05-17 8:37 UTC (permalink / raw)
To: Tvrtko Ursulin, intel-gfx; +Cc: igt-dev
Quoting Tvrtko Ursulin (2019-05-17 09:28:46)
>
> On 16/05/2019 19:58, Chris Wilson wrote:
> > To avoid hitting the same rut on each benchmark run, start with a new
> > random seed. To allow hitting the same rut again, let it be specified
> > by the user.
> >
> > 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, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> > index 48568ce40..cf2a44746 100644
> > --- a/benchmarks/gem_wsim.c
> > +++ b/benchmarks/gem_wsim.c
> > @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
> > igt_require(fd);
> >
> > init_clocks();
> > + srand(time(NULL));
> >
> > - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
> > + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
> > switch (c) {
> > case 'W':
> > if (master_workload >= 0) {
> > @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
> > case 'p':
> > prio = atoi(optarg);
> > break;
> > + case 's':
> > + srand(atoi(optarg));
> > + break;
> > case 'a':
> > if (append_workload_arg) {
> > if (verbose)
> >
>
> Makes sense, just the help text missing.
You use 's' for sseu, I think I (Initial random seed) is one of the few
available.
> I was initially confused at srand when I thought I used your
> hars_petruska library routines but then saw rand() feeds the seed into
> those.
Yeah, I had to go and double check that we actually used rand()
somewhere (and not just in my hacks :)
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
2019-05-17 8:37 ` [igt-dev] [Intel-gfx] " Chris Wilson
@ 2019-05-17 10:32 ` Tvrtko Ursulin
-1 siblings, 0 replies; 14+ messages in thread
From: Tvrtko Ursulin @ 2019-05-17 10:32 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 17/05/2019 09:37, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2019-05-17 09:28:46)
>>
>> On 16/05/2019 19:58, Chris Wilson wrote:
>>> To avoid hitting the same rut on each benchmark run, start with a new
>>> random seed. To allow hitting the same rut again, let it be specified
>>> by the user.
>>>
>>> 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, 5 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
>>> index 48568ce40..cf2a44746 100644
>>> --- a/benchmarks/gem_wsim.c
>>> +++ b/benchmarks/gem_wsim.c
>>> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
>>> igt_require(fd);
>>>
>>> init_clocks();
>>> + srand(time(NULL));
>>>
>>> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
>>> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
>>> switch (c) {
>>> case 'W':
>>> if (master_workload >= 0) {
>>> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
>>> case 'p':
>>> prio = atoi(optarg);
>>> break;
>>> + case 's':
>>> + srand(atoi(optarg));
>>> + break;
>>> case 'a':
>>> if (append_workload_arg) {
>>> if (verbose)
>>>
>>
>> Makes sense, just the help text missing.
>
> You use 's' for sseu, I think I (Initial random seed) is one of the few
> available.
No complaints on 'I'.
>> I was initially confused at srand when I thought I used your
>> hars_petruska library routines but then saw rand() feeds the seed into
>> those.
>
> Yeah, I had to go and double check that we actually used rand()
> somewhere (and not just in my hacks :)
Seems I have to separate streams of randomness.
One is relating to random mode load balancing, which this patch influences.
Second is relating to batch buffer duration which is driven from the
global hars_petruska_f54_1_random_seed and wrongly uses the _unsafe
function from unlocked section. This might be from before I replaced
forking with threads. :(
I think the aggregate fix would be to store a second seed per workload
for the latter. Replace hars_pretruska_f54_1_random_seed near beginning
of run_worklaod with wrk->bb_prng = (wrk->flags & SYNCEDCLIENTS) ?
master_prng_seed : time(NULL); And in get_duration use wrk->bb_prng and
safe pnrg function. (You store master_prng_seed in main after it is set.)
I think this should fix all issues and propagate the passed in seed
correctly.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-17 10:32 ` Tvrtko Ursulin
0 siblings, 0 replies; 14+ messages in thread
From: Tvrtko Ursulin @ 2019-05-17 10:32 UTC (permalink / raw)
To: Chris Wilson, intel-gfx; +Cc: igt-dev
On 17/05/2019 09:37, Chris Wilson wrote:
> Quoting Tvrtko Ursulin (2019-05-17 09:28:46)
>>
>> On 16/05/2019 19:58, Chris Wilson wrote:
>>> To avoid hitting the same rut on each benchmark run, start with a new
>>> random seed. To allow hitting the same rut again, let it be specified
>>> by the user.
>>>
>>> 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, 5 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
>>> index 48568ce40..cf2a44746 100644
>>> --- a/benchmarks/gem_wsim.c
>>> +++ b/benchmarks/gem_wsim.c
>>> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
>>> igt_require(fd);
>>>
>>> init_clocks();
>>> + srand(time(NULL));
>>>
>>> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
>>> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
>>> switch (c) {
>>> case 'W':
>>> if (master_workload >= 0) {
>>> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
>>> case 'p':
>>> prio = atoi(optarg);
>>> break;
>>> + case 's':
>>> + srand(atoi(optarg));
>>> + break;
>>> case 'a':
>>> if (append_workload_arg) {
>>> if (verbose)
>>>
>>
>> Makes sense, just the help text missing.
>
> You use 's' for sseu, I think I (Initial random seed) is one of the few
> available.
No complaints on 'I'.
>> I was initially confused at srand when I thought I used your
>> hars_petruska library routines but then saw rand() feeds the seed into
>> those.
>
> Yeah, I had to go and double check that we actually used rand()
> somewhere (and not just in my hacks :)
Seems I have to separate streams of randomness.
One is relating to random mode load balancing, which this patch influences.
Second is relating to batch buffer duration which is driven from the
global hars_petruska_f54_1_random_seed and wrongly uses the _unsafe
function from unlocked section. This might be from before I replaced
forking with threads. :(
I think the aggregate fix would be to store a second seed per workload
for the latter. Replace hars_pretruska_f54_1_random_seed near beginning
of run_worklaod with wrk->bb_prng = (wrk->flags & SYNCEDCLIENTS) ?
master_prng_seed : time(NULL); And in get_duration use wrk->bb_prng and
safe pnrg function. (You store master_prng_seed in main after it is set.)
I think this should fix all issues and propagate the passed in seed
correctly.
Regards,
Tvrtko
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
2019-05-16 18:58 ` [igt-dev] " Chris Wilson
@ 2019-05-17 10:55 ` Ser, Simon
-1 siblings, 0 replies; 14+ messages in thread
From: Ser, Simon @ 2019-05-17 10:55 UTC (permalink / raw)
To: intel-gfx, chris; +Cc: igt-dev
On Thu, 2019-05-16 at 19:58 +0100, Chris Wilson wrote:
> To avoid hitting the same rut on each benchmark run, start with a new
> random seed. To allow hitting the same rut again, let it be specified
> by the user.
>
> 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, 5 insertions(+), 1 deletion(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index 48568ce40..cf2a44746 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
> igt_require(fd);
>
> init_clocks();
> + srand(time(NULL));
Maybe the seed used for the run should be printed to stderr, so that's
it's easy to re-run a failure?
> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
> switch (c) {
> case 'W':
> if (master_workload >= 0) {
> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
> case 'p':
> prio = atoi(optarg);
> break;
> + case 's':
> + srand(atoi(optarg));
> + break;
> case 'a':
> if (append_workload_arg) {
> if (verbose)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-17 10:55 ` Ser, Simon
0 siblings, 0 replies; 14+ messages in thread
From: Ser, Simon @ 2019-05-17 10:55 UTC (permalink / raw)
To: intel-gfx, chris; +Cc: igt-dev, Ursulin, Tvrtko
On Thu, 2019-05-16 at 19:58 +0100, Chris Wilson wrote:
> To avoid hitting the same rut on each benchmark run, start with a new
> random seed. To allow hitting the same rut again, let it be specified
> by the user.
>
> 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, 5 insertions(+), 1 deletion(-)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index 48568ce40..cf2a44746 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
> igt_require(fd);
>
> init_clocks();
> + srand(time(NULL));
Maybe the seed used for the run should be printed to stderr, so that's
it's easy to re-run a failure?
> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
> switch (c) {
> case 'W':
> if (master_workload >= 0) {
> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
> case 'p':
> prio = atoi(optarg);
> break;
> + case 's':
> + srand(atoi(optarg));
> + break;
> case 'a':
> if (append_workload_arg) {
> if (verbose)
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
2019-05-17 10:55 ` Ser, Simon
@ 2019-05-17 12:59 ` Tvrtko Ursulin
-1 siblings, 0 replies; 14+ messages in thread
From: Tvrtko Ursulin @ 2019-05-17 12:59 UTC (permalink / raw)
To: Ser, Simon, intel-gfx, chris; +Cc: igt-dev
On 17/05/2019 11:55, Ser, Simon wrote:
> On Thu, 2019-05-16 at 19:58 +0100, Chris Wilson wrote:
>> To avoid hitting the same rut on each benchmark run, start with a new
>> random seed. To allow hitting the same rut again, let it be specified
>> by the user.
>>
>> 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, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
>> index 48568ce40..cf2a44746 100644
>> --- a/benchmarks/gem_wsim.c
>> +++ b/benchmarks/gem_wsim.c
>> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
>> igt_require(fd);
>>
>> init_clocks();
>> + srand(time(NULL));
>
> Maybe the seed used for the run should be printed to stderr, so that's
> it's easy to re-run a failure?
Sounds right. But I wouldn't say stderr and would make it conditional on
verbosity level:
if (verbose)
printf(...)
Chris, would you like me to take over this jobby of fixing and improving
random seeding/handling? If you got too much on your plate..
Regards,
Tvrtko
>
>> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
>> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
>> switch (c) {
>> case 'W':
>> if (master_workload >= 0) {
>> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
>> case 'p':
>> prio = atoi(optarg);
>> break;
>> + case 's':
>> + srand(atoi(optarg));
>> + break;
>> case 'a':
>> if (append_workload_arg) {
>> if (verbose)
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [igt-dev] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed
@ 2019-05-17 12:59 ` Tvrtko Ursulin
0 siblings, 0 replies; 14+ messages in thread
From: Tvrtko Ursulin @ 2019-05-17 12:59 UTC (permalink / raw)
To: Ser, Simon, intel-gfx, chris; +Cc: igt-dev
On 17/05/2019 11:55, Ser, Simon wrote:
> On Thu, 2019-05-16 at 19:58 +0100, Chris Wilson wrote:
>> To avoid hitting the same rut on each benchmark run, start with a new
>> random seed. To allow hitting the same rut again, let it be specified
>> by the user.
>>
>> 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, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
>> index 48568ce40..cf2a44746 100644
>> --- a/benchmarks/gem_wsim.c
>> +++ b/benchmarks/gem_wsim.c
>> @@ -2282,8 +2282,9 @@ int main(int argc, char **argv)
>> igt_require(fd);
>>
>> init_clocks();
>> + srand(time(NULL));
>
> Maybe the seed used for the run should be printed to stderr, so that's
> it's easy to re-run a failure?
Sounds right. But I wouldn't say stderr and would make it conditional on
verbosity level:
if (verbose)
printf(...)
Chris, would you like me to take over this jobby of fixing and improving
random seeding/handling? If you got too much on your plate..
Regards,
Tvrtko
>
>> - while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:")) != -1) {
>> + while ((c = getopt(argc, argv, "hqv2RSHxGdc:n:r:w:W:a:t:b:p:s:")) != -1) {
>> switch (c) {
>> case 'W':
>> if (master_workload >= 0) {
>> @@ -2300,6 +2301,9 @@ int main(int argc, char **argv)
>> case 'p':
>> prio = atoi(optarg);
>> break;
>> + case 's':
>> + srand(atoi(optarg));
>> + break;
>> case 'a':
>> if (append_workload_arg) {
>> if (verbose)
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-05-17 12:59 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-16 18:58 [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed Chris Wilson
2019-05-16 18:58 ` [igt-dev] " Chris Wilson
2019-05-16 19:37 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2019-05-17 1:50 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2019-05-17 8:28 ` [PATCH i-g-t] " Tvrtko Ursulin
2019-05-17 8:28 ` [igt-dev] [Intel-gfx] " Tvrtko Ursulin
2019-05-17 8:37 ` Chris Wilson
2019-05-17 8:37 ` [igt-dev] [Intel-gfx] " Chris Wilson
2019-05-17 10:32 ` Tvrtko Ursulin
2019-05-17 10:32 ` [igt-dev] [Intel-gfx] " Tvrtko Ursulin
2019-05-17 10:55 ` [igt-dev] " Ser, Simon
2019-05-17 10:55 ` Ser, Simon
2019-05-17 12:59 ` Tvrtko Ursulin
2019-05-17 12:59 ` [Intel-gfx] " Tvrtko Ursulin
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.