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@xxxxxxxxxxxxxxxxxx>
Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
---
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@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
