Skip to content

Commit 133e267

Browse files
sulixshuahkh
authored andcommitted
time: test: Fix incorrect format specifier
'days' is a s64 (from div_s64), and so should use a %lld specifier. This was found by extending KUnit's assertion macros to use gcc's __printf attribute. Fixes: 2760105 ("time: Improve performance of time64_to_tm()") Signed-off-by: David Gow <davidgow@google.com> Tested-by: Guenter Roeck <linux@roeck-us.net> Reviewed-by: Justin Stitt <justinstitt@google.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
1 parent 0a549ed commit 133e267

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

kernel/time/time_test.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ static void time64_to_tm_test_date_range(struct kunit *test)
7373

7474
days = div_s64(secs, 86400);
7575

76-
#define FAIL_MSG "%05ld/%02d/%02d (%2d) : %ld", \
76+
#define FAIL_MSG "%05ld/%02d/%02d (%2d) : %lld", \
7777
year, month, mdday, yday, days
7878

7979
KUNIT_ASSERT_EQ_MSG(test, year - 1900, result.tm_year, FAIL_MSG);

0 commit comments

Comments
 (0)