Re: + timers-implement-usleep_idle_range.patch added to -mm tree

[SeongJae Park <sj@kernel.org>]

Hello Andrew,

On Sat, 27 Nov 2021 15:56:53 -0800 akpm@linux-foundation.org wrote:

> 
> The patch titled
>      Subject: timers: implement usleep_idle_range()
> has been added to the -mm tree.  Its filename is
>      timers-implement-usleep_idle_range.patch
> 
> This patch should soon appear at
>     https://ozlabs.org/~akpm/mmots/broken-out/timers-implement-usleep_idle_range.patch
> and later at
>     https://ozlabs.org/~akpm/mmotm/broken-out/timers-implement-usleep_idle_range.patch
> 
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
> 
> *** Remember to use Documentation/process/submit-checklist.rst when testing your code ***
> 
> The -mm tree is included into linux-next and is updated
> there every 3-4 working days
> 
> ------------------------------------------------------
> From: SeongJae Park <sj@kernel.org>
> Subject: timers: implement usleep_idle_range()
> 
> Patch series "mm/damon: Fix fake /proc/loadavg reports", v3.
> 
> This patchset fixes DAMON's fake load report issue.  The first patch makes
> yet another variant of usleep_range() for this fix, and the second patch
> fixes the issue of DAMON by making it using the newly introduced function.
> 
> 
> This patch (of 2):
> 
> Some kernel threads such as DAMON could need to repeatedly sleep in micro
> seconds level.  Because usleep_range() sleeps in uninterruptible state,
> however, such threads would make /proc/loadavg reports fake load.
> 
> To help such cases, this commit implements a variant of usleep_range()
> called usleep_idle_range().  It is same to usleep_range() but sets the
> state of the current task as TASK_IDLE while sleeping.
> 
> Link: https://lkml.kernel.org/r/20211126145015.15862-1-sj@kernel.org
> Link: https://lkml.kernel.org/r/20211126145015.15862-2-sj@kernel.org
> Signed-off-by: SeongJae Park <sj@kernel.org>
> Suggested-by: Andrew Morton <akpm@linux-foundation.org>
> Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
> Cc: John Stultz <john.stultz@linaro.org>
> Cc: Oleksandr Natalenko <oleksandr@natalenko.name>
> Cc: <stable@vger.kernel.org>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

Thank you for picking this patch series.  Could you please also add
'Tested-by:' tags for Oleksandr[1]?

[1] https://lore.kernel.org/linux-mm/51190182.RbnaydPRjS@natalenko.name/


Thanks,
SJ

> ---
> 
>  include/linux/delay.h |   14 +++++++++++++-
>  kernel/time/timer.c   |   16 +++++++++-------
>  2 files changed, 22 insertions(+), 8 deletions(-)