Re: [PATCH v1] commands: Add dhrystone

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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@xxxxxxxxx>
---

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

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux