[PATCH v1] commands: Add dhrystone
Daniel Schultz
d.schultz at phytec.de
Fri Jul 24 04:41:59 PDT 2015
Am 24.07.2015 um 09:41 schrieb Sascha Hauer:
> On Thu, Jul 23, 2015 at 10:01:32AM +0200, Daniel Schultz wrote:
>> This tool will help to measure the system performance.
>>
>> Some SoCs haven't the possibility to route their clocks to the output pins.
>> So you can use dhrystone to get a feedback about the clock speed.
>>
>> Signed-off-by: Daniel Schultz <d.schultz at phytec.de>
>> ---
>>
>> Changes:
>> v1:
>> applied RFC suggestions:
>> -removed 'default n' in Kconfig
>> -refactored time defines with default time units
>> -only print variable values when values are wrong
>> -added units to printf
>> -the iteration parameter is now optional. The programm starts
>> with 10000 iterations and adjust higher, when execution time
>> is too short.
>
> I can see that the parameter is now optional, but I can't find the code
> which adjusts the number of iterations. Am I missing something?
>
At the end of the do_dhrystone function:
if (user_time < TOO_SMALL_TIME) {
number_of_runs = number_of_runs * 10;
new_argv[0] = argv[0];
sprintf(tmp_str, "%i", number_of_runs);
new_argv[1] = tmp_str;
printf("Measured time too small to obtain meaningful results\n");
printf("or a timer wrap happend. I will increase the number\n");
printf("of runs by *10 to %d\n", number_of_runs);
printf("user_time: %llu ns\n", user_time);
do_dhrystone(2, new_argv);
...
I'm sure there is a better solution.
Kind regards,
Daniel Schultz
More information about the barebox
mailing list