Re: [PATCH] pip_stress: increase usleep for ARM devices

[Pavel Vasilyev <pavel@pavlinux.ru>]

May be #defines?

#ifndef (ARM)
		usleep(500);
#else
		usleep(3500);
#endif


10.12.2014 11:25, chase.qi@linaro.org пишет:
> From: Chase Qi <chase.qi@linaro.org>
>
> Hello,
>
> pip_stress works out of the box on my x86 based laptop, but
> doesn't work on ARM devices, returned 'no inversion incurred'.
> Follow the comment to increase usleep value, 2500 worked for
> pandaboard and 3000 worked for Beaglebone Black board.
>
> I propose that increase the usleep value to 3500 from upstream,
> so that we can use pip_stress right out of the box.
>
> Please let me know if this is acceptable.
>
> Regards,
> Chase
>
> Signed-off-by: Chase Qi <chase.qi@linaro.org>
> ---
>   src/pi_tests/pip_stress.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/pi_tests/pip_stress.c b/src/pi_tests/pip_stress.c
> index 2b42b8f..553290b 100644
> --- a/src/pi_tests/pip_stress.c
> +++ b/src/pi_tests/pip_stress.c
> @@ -162,7 +162,7 @@ void low(pid_t pid)
>   				statep->inversion = 0;
>   			}
>   		Pthread_mutex_unlock(statep->mutex);
> -		usleep(500);
> +		usleep(3500);
>   	Pthread_mutex_unlock(resource);
>   	waitpid(pid, &status, 0);
>   }
>


-- 

                                                          Pavel.
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html