Re: [PATCH] Documentation/sysctl: Add timer_migration to kernel.rst

[Jonathan Corbet <corbet@lwn.net>]

Phil Auld <pauld@redhat.com> writes:

> There is no mention of timer_migration in the docs. Add
> a short description.
>
> Signed-off-by: Phil Auld <pauld@redhat.com>
> Cc: Jonathan Corbet <corbet@lwn.net>
> Cc: linux-doc@vger.kernel.org
> ---
>  Documentation/admin-guide/sysctl/kernel.rst | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/Documentation/admin-guide/sysctl/kernel.rst b/Documentation/admin-guide/sysctl/kernel.rst
> index b2b36d0c3094..e03691e2cf4a 100644
> --- a/Documentation/admin-guide/sysctl/kernel.rst
> +++ b/Documentation/admin-guide/sysctl/kernel.rst
> @@ -1544,6 +1544,13 @@ constant ``FUTEX_TID_MASK`` (0x3fffffff).
>  If a value outside of this range is written to ``threads-max`` an
>  ``EINVAL`` error occurs.
>  
> +timer_migration
> +===============
> +
> +When set, attempt to migrate timers away from idle cpus to allow them to remain
> +in low power states longer.
> +
> +Default is set.

When set to *what*?  It will always be set to *some* value, right?  So
we should really say what the specific values mean.

Thanks,

jon