Re: [PATCH] lib: use correct format string for find-bit tests

From: Yury Norov
Date: Tue Nov 14 2017 - 04:46:25 EST


On Mon, Nov 13, 2017 at 02:55:45PM +0100, Arnd Bergmann wrote:
> The cycles_t definition is architecture specific, which causes
> a link error on all architectures that use a 'long long' or 'int'
> type for it:
>
> lib/test_find_bit.c: In function 'test_find_last_bit':
> include/linux/kern_levels.h:5:18: error: format '%ld' expects argument of type 'long int', but argument 2 has type 'cycles_t {aka long long unsigned int}' [-Werror=format=]
>
> This adds an explicit cast to 'u64' for it, which lets us use
> '%llu' everywhere.
>
> Fixes: 09588b1f1d58 ("lib: test module for find_*_bit() functions")
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>

Hi Arnd,

patch looks OK. Thank you.

Acked-by: Yury Norov <ynorov@xxxxxxxxxxxxxxxxxx>