Re: [PATCH] docs/vm: clarify overcommit amount sysctl behavior

[Matthew Wilcox]

On Sat, Dec 11, 2021 at 09:41:59PM +0200, Anssi Hannula wrote:
> Documentation/vm/overcommit-accounting.rst says that the overcommit
> amount can be set via vm.overcommit_ratio and vm.overcommit_kbytes.
> 
> Add a clarification that those only take effect in overcommit handling
> mode 2 ("Don't overcommit"), i.e. they do not act as an "additional"
> limit that is always enforced.
> 
> Signed-off-by: Anssi Hannula <anssi.hannula@xxxxxx>
> ---

Simple and clear improvement, IMO.  Cc'ing linux-mm to get more eyes on
it.

> I've had to look this one up in the code enough times already :)
> 
> 
>  Documentation/vm/overcommit-accounting.rst | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/vm/overcommit-accounting.rst b/Documentation/vm/overcommit-accounting.rst
> index 0dd54bbe4afa..1addb0c374a4 100644
> --- a/Documentation/vm/overcommit-accounting.rst
> +++ b/Documentation/vm/overcommit-accounting.rst
> @@ -34,7 +34,8 @@ The Linux kernel supports the following overcommit handling modes
>  The overcommit policy is set via the sysctl ``vm.overcommit_memory``.
>  
>  The overcommit amount can be set via ``vm.overcommit_ratio`` (percentage)
> -or ``vm.overcommit_kbytes`` (absolute value).
> +or ``vm.overcommit_kbytes`` (absolute value). These only have an effect
> +when ``vm.overcommit_memory`` is set to 2.
>  
>  The current overcommit limit and amount committed are viewable in
>  ``/proc/meminfo`` as CommitLimit and Committed_AS respectively.
> -- 
> 2.31.1
>