Re: [PATCH] mm/damon/stat: use secs_to_jiffies() instead of msecs_to_jiffies()

[SeongJae Park <sj@kernel.org>]

On Mon, 15 Jun 2026 21:41:43 +0000 Philippe Laferriere <plafer@proton.me> wrote:

> The conversion of a duration expressed in seconds reads as
> msecs_to_jiffies(5 * MSEC_PER_SEC), which obscures the intent and
> needlessly goes through milliseconds. Use the dedicated
> secs_to_jiffies() helper, which expresses the 5-second refresh
> interval directly. No functional change.

Thank you for this patch!

> 
> Found using Coccinelle (scripts/coccinelle/misc/secs_to_jiffies.cocci).
> 
> Signed-off-by: Philippe Laferriere <plafer@proton.me>

Reviewed-by: SeongJae Park <sj@kernel.org>

I applied  this patch to damon/next [1] tree.  We are now in the merge window.
We (mm community) want to focus on making the stabilized mm changes be merged
into the mainline until the merge window is closed, rather than adding more
changes that are not really urgent.  I understand this patch is not really
urgent.  Hence, Andrew might not add this patch until next -rc1 release.  In
the case, I will request adding this to mm.git after next -rc1 release.  So, no
action from your side is needed for now.  Let me know if you think this is
really  urgent.

[1] https://origin.kernel.org/doc/html/latest/mm/damon/maintainer-profile.html#scm-trees


Thanks,
SJ

[...]