diff options
| author | David Gow <davidgow@google.com> | 2024-02-21 17:27:17 +0800 |
|---|---|---|
| committer | Sasha Levin <sashal@kernel.org> | 2024-03-26 18:19:18 -0400 |
| commit | 491ed9f9ebafc62c083187b89c35e46cb5198030 (patch) | |
| tree | 8c07052506e331e681cbd4a44f2f34e4acf544ed /kernel/time | |
| parent | 3c3d394b531fe141ddabc02eed7b7d4a9e49da9c (diff) | |
| download | linux-491ed9f9ebafc62c083187b89c35e46cb5198030.tar.gz linux-491ed9f9ebafc62c083187b89c35e46cb5198030.tar.bz2 linux-491ed9f9ebafc62c083187b89c35e46cb5198030.zip | |
time: test: Fix incorrect format specifier
[ Upstream commit 133e267ef4a26d19c93996a874714e9f3f8c70aa ]
'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: 276010551664 ("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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'kernel/time')
| -rw-r--r-- | kernel/time/time_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/time/time_test.c b/kernel/time/time_test.c index ca058c8af6ba..3e5d422dd15c 100644 --- a/kernel/time/time_test.c +++ b/kernel/time/time_test.c @@ -73,7 +73,7 @@ static void time64_to_tm_test_date_range(struct kunit *test) days = div_s64(secs, 86400); - #define FAIL_MSG "%05ld/%02d/%02d (%2d) : %ld", \ + #define FAIL_MSG "%05ld/%02d/%02d (%2d) : %lld", \ year, month, mdday, yday, days KUNIT_ASSERT_EQ_MSG(test, year - 1900, result.tm_year, FAIL_MSG); |