Re: [tip: timers/core] time/kunit: Use correct format specifier

From: Shuah Khan
Date: Fri Feb 23 2024 - 18:06:13 EST


On 2/23/24 00:01, David Gow wrote:
Hi,

On Thu, 22 Feb 2024 at 02:21, tip-bot2 for David Gow
<tip-bot2@xxxxxxxxxxxxx> wrote:

The following commit has been merged into the timers/core branch of tip:

Commit-ID: e0a1284b293bdf91a68a6d1a0479ad476d0d8ec2
Gitweb: https://git.kernel.org/tip/e0a1284b293bdf91a68a6d1a0479ad476d0d8ec2
Author: David Gow <davidgow@xxxxxxxxxx>
AuthorDate: Wed, 21 Feb 2024 17:27:17 +08:00
Committer: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
CommitterDate: Wed, 21 Feb 2024 12:00:42 +01:00

time/kunit: Use correct 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: 276010551664 ("time: Improve performance of time64_to_tm()")
Signed-off-by: David Gow <davidgow@xxxxxxxxxx>
Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
Link: https://lore.kernel.org/r/20240221092728.1281499-5-davidgow@xxxxxxxxxx


We're hoping to take this series in via the KUnit tree, so that we can
enable the warning in one place without annoying Linus with lots of
dependencies between PRs.
(See, e.g,. https://lore.kernel.org/linux-kselftest/CAHk-=wgafXXX17eKx9wH_uHg=UgvXkngxGhPcZwhpj7Uz=_0Pw@xxxxxxxxxxxxxx/
)

Would that cause a problem?


Linus wants this series to go through kunit tree. I will take care.

I will pick these up for Linux 6.9-rc1 as per Linus's request.

thanks,
-- Shuah