All callers of mftb() expect 'unsigned long', and the function itself only returns lower part of the TB so it really is 'unsigned long' not 'unsigned long long'
Signed-off-by: Christophe Leroy christophe.leroy@c-s.fr --- tools/testing/selftests/powerpc/benchmarks/null_syscall.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/powerpc/benchmarks/null_syscall.c b/tools/testing/selftests/powerpc/benchmarks/null_syscall.c index ecc14d68e101..908de689a902 100644 --- a/tools/testing/selftests/powerpc/benchmarks/null_syscall.c +++ b/tools/testing/selftests/powerpc/benchmarks/null_syscall.c @@ -25,7 +25,7 @@ unsigned long long clock_frequency; unsigned long long timebase_frequency; double timebase_multiplier;
-static inline unsigned long long mftb(void) +static inline unsigned long mftb(void) { unsigned long low;
On Tue, 2019-01-22 at 13:54:57 UTC, Christophe Leroy wrote:
All callers of mftb() expect 'unsigned long', and the function itself only returns lower part of the TB so it really is 'unsigned long' not 'unsigned long long'
Signed-off-by: Christophe Leroy christophe.leroy@c-s.fr
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/beb4f4722cf60d9f0803054dec4eb502
cheers
linux-kselftest-mirror@lists.linaro.org