linux-kernel - Re: [PATCH] tmpfs: fixup Docs table for huge mount options

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <329bd13b-8970-ce41-1068-2928d4dd7b24@google.com>
Date:   Mon, 24 Jul 2023 23:31:31 -0700 (PDT)
From:   Hugh Dickins <hughd@...gle.com>
To:     Randy Dunlap <rdunlap@...radead.org>
cc:     linux-kernel@...r.kernel.org,
        Stephen Rothwell <sfr@...b.auug.org.au>,
        Hugh Dickins <hughd@...gle.com>, linux-mm@...ck.org,
        Andrew Morton <akpm@...ux-foundation.org>,
        Jonathan Corbet <corbet@....net>, linux-doc@...r.kernel.org
Subject: Re: [PATCH] tmpfs: fixup Docs table for huge mount options

On Mon, 24 Jul 2023, Randy Dunlap wrote:

> A table's header and footer lines must match its text columns in
> width, so extend the header/footer for column 1 to match the text.
> 
> Fixes this documentation build error:
> Documentation/filesystems/tmpfs.rst:116: ERROR: Malformed table.
> Text in column margin in table line 4.
> 
> Fixes: a0ebb5aa2de3 ("tmpfs: fix Documentation of noswap and huge mount options")
> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Reported-by: Stephen Rothwell <sfr@...b.auug.org.au>
> Link: https://lore.kernel.org/lkml/3084e97c-3a7d-ace8-2e9c-31642fd663df@google.com/T/#me28ed2124bca2e632eee86ff1d986e5c1f731a7c
> Cc: Hugh Dickins <hughd@...gle.com>
> Cc: linux-mm@...ck.org
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Cc: Jonathan Corbet <corbet@....net>
> Cc: linux-doc@...r.kernel.org
> ---
> I guess that Andrew will merge this since he merged the previous patch.

Acked-by: Hugh Dickins <hughd@...gle.com>

Many thanks, Randy: your rapid rescue service is much appreciated.

> 
>  Documentation/filesystems/tmpfs.rst |   12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff -- a/Documentation/filesystems/tmpfs.rst b/Documentation/filesystems/tmpfs.rst
> --- a/Documentation/filesystems/tmpfs.rst
> +++ b/Documentation/filesystems/tmpfs.rst
> @@ -110,13 +110,13 @@ configured with CONFIG_TRANSPARENT_HUGEP
>  your system (has_transparent_hugepage(), which is architecture specific).
>  The mount options for this are:
>  
> -===========  ==============================================================
> -huge=never   Do not allocate huge pages.  This is the default.
> -huge=always  Attempt to allocate huge page every time a new page is needed.
> +================ ==============================================================
> +huge=never       Do not allocate huge pages.  This is the default.
> +huge=always      Attempt to allocate huge page every time a new page is needed.
>  huge=within_size Only allocate huge page if it will be fully within i_size.
> -             Also respect madvise(2) hints.
> -huge=advise  Only allocate huge page if requested with madvise(2).
> -===========  ==============================================================
> +                 Also respect madvise(2) hints.
> +huge=advise      Only allocate huge page if requested with madvise(2).
> +================ ==============================================================
>  
>  See also Documentation/admin-guide/mm/transhuge.rst, which describes the
>  sysfs file /sys/kernel/mm/transparent_hugepage/shmem_enabled: which can