[Intel-gfx] [igt-dev] [PATCH i-g-t] benchmarks/gem_wsim: Randomise random seed

[Tvrtko Ursulin]

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 at chris-wilson.co.uk>
>> Cc: Tvrtko Ursulin <tvrtko.ursulin at 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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>