[PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[John Hubbard]

In commit 462a385e9a2 ("tmpfs.5: Document current mount options"), there
is a reference to CONFIG_TRANSPARENT_HUGE_PAGECACHE. However, that
option was removed from the kernel via commit 396bcc5299c2 ("mm: remove
CONFIG_TRANSPARENT_HUGE_PAGECACHE"), a couple of years later.

The net effect is that CONFIG_TRANSPARENT_HUGEPAGE is now used in all
the remaining places in the kernel where
CONFIG_TRANSPARENT_HUGE_PAGECACHE had previously been used.

This has caused some minor confusion at the man page level, though. So
let's fix it by updating the man page to also refer to
CONFIG_TRANSPARENT_HUGEPAGE.

Reported-by: Vahid Noormofidi <vnoormof@nvidia.com>
Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Carsten Grohmann <carstengrohmann@gmx.de>
Cc: Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: John Hubbard <jhubbard@nvidia.com>
---
 man5/tmpfs.5 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/man5/tmpfs.5 b/man5/tmpfs.5
index 09d955898..cdd5f2fd8 100644
--- a/man5/tmpfs.5
+++ b/man5/tmpfs.5
@@ -115,7 +115,7 @@ Set the initial user ID of the root directory.
 .BR huge "=\fIhuge_option\fR (since Linux 4.7.0)"
 .\" commit 5a6e75f8110c97e2a5488894d4e922187e6cb343
 Set the huge table memory allocation policy for all files in this instance (if
-.B CONFIG_TRANSPARENT_HUGE_PAGECACHE
+.B CONFIG_TRANSPARENT_HUGEPAGE
 is enabled).
 .IP
 The

base-commit: c654cde5ec9e9fa975d5d041aa069489ab12a4ce
-- 
2.41.0

Re: [PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[Alejandro Colomar]

[-- Attachment #1.1: Type: text/plain, Size: 1836 bytes --]



On 2023-07-19 04:05, John Hubbard wrote:
> In commit 462a385e9a2 ("tmpfs.5: Document current mount options"), there
> is a reference to CONFIG_TRANSPARENT_HUGE_PAGECACHE. However, that
> option was removed from the kernel via commit 396bcc5299c2 ("mm: remove
> CONFIG_TRANSPARENT_HUGE_PAGECACHE"), a couple of years later.
> 
> The net effect is that CONFIG_TRANSPARENT_HUGEPAGE is now used in all
> the remaining places in the kernel where
> CONFIG_TRANSPARENT_HUGE_PAGECACHE had previously been used.
> 
> This has caused some minor confusion at the man page level, though. So
> let's fix it by updating the man page to also refer to
> CONFIG_TRANSPARENT_HUGEPAGE.
> 
> Reported-by: Vahid Noormofidi <vnoormof@nvidia.com>
> Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
> Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Carsten Grohmann <carstengrohmann@gmx.de>
> Cc: Signed-off-by: Mike Frysinger <vapier@gentoo.org>

Accident here :-)

> Signed-off-by: John Hubbard <jhubbard@nvidia.com>
> ---
>  man5/tmpfs.5 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man5/tmpfs.5 b/man5/tmpfs.5
> index 09d955898..cdd5f2fd8 100644
> --- a/man5/tmpfs.5
> +++ b/man5/tmpfs.5
> @@ -115,7 +115,7 @@ Set the initial user ID of the root directory.
>  .BR huge "=\fIhuge_option\fR (since Linux 4.7.0)"
>  .\" commit 5a6e75f8110c97e2a5488894d4e922187e6cb343
>  Set the huge table memory allocation policy for all files in this instance (if
> -.B CONFIG_TRANSPARENT_HUGE_PAGECACHE
> +.B CONFIG_TRANSPARENT_HUGEPAGE
>  is enabled).
>  .IP
>  The
> 
> base-commit: c654cde5ec9e9fa975d5d041aa069489ab12a4ce

-- 
<http://www.alejandro-colomar.es/>
GPG key fingerprint: A9348594CE31283A826FBDD8D57633D441E25BB5

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[John Hubbard]

On 7/19/23 12:59, Alejandro Colomar wrote:
> On 2023-07-19 04:05, John Hubbard wrote:
>> In commit 462a385e9a2 ("tmpfs.5: Document current mount options"), there
>> is a reference to CONFIG_TRANSPARENT_HUGE_PAGECACHE. However, that
>> option was removed from the kernel via commit 396bcc5299c2 ("mm: remove
>> CONFIG_TRANSPARENT_HUGE_PAGECACHE"), a couple of years later.
>>
>> The net effect is that CONFIG_TRANSPARENT_HUGEPAGE is now used in all
>> the remaining places in the kernel where
>> CONFIG_TRANSPARENT_HUGE_PAGECACHE had previously been used.
>>
>> This has caused some minor confusion at the man page level, though. So
>> let's fix it by updating the man page to also refer to
>> CONFIG_TRANSPARENT_HUGEPAGE.
>>
>> Reported-by: Vahid Noormofidi <vnoormof@nvidia.com>
>> Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
>> Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
>> Cc: Andrew Morton <akpm@linux-foundation.org>
>> Cc: Carsten Grohmann <carstengrohmann@gmx.de>
>> Cc: Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> 
> Accident here :-)
> 

Yes. :)  Michael Kerrisk, could you change that line for me
to just be a Cc, for Mike Frysinger?

Or, let me know if you'd prefer a re-send of the patch instead.


thanks,
-- 
John Hubbard
NVIDIA


>> Signed-off-by: John Hubbard <jhubbard@nvidia.com>
>> ---
>>   man5/tmpfs.5 | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/man5/tmpfs.5 b/man5/tmpfs.5
>> index 09d955898..cdd5f2fd8 100644
>> --- a/man5/tmpfs.5
>> +++ b/man5/tmpfs.5
>> @@ -115,7 +115,7 @@ Set the initial user ID of the root directory.
>>   .BR huge "=\fIhuge_option\fR (since Linux 4.7.0)"
>>   .\" commit 5a6e75f8110c97e2a5488894d4e922187e6cb343
>>   Set the huge table memory allocation policy for all files in this instance (if
>> -.B CONFIG_TRANSPARENT_HUGE_PAGECACHE
>> +.B CONFIG_TRANSPARENT_HUGEPAGE
>>   is enabled).
>>   .IP
>>   The
>>
>> base-commit: c654cde5ec9e9fa975d5d041aa069489ab12a4ce
> 

Re: [PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[Alejandro Colomar]

[-- Attachment #1.1: Type: text/plain, Size: 1556 bytes --]

Hi John,

On 2023-07-19 22:13, John Hubbard wrote:
> On 7/19/23 12:59, Alejandro Colomar wrote:
>> On 2023-07-19 04:05, John Hubbard wrote:
>>> In commit 462a385e9a2 ("tmpfs.5: Document current mount options"), there
>>> is a reference to CONFIG_TRANSPARENT_HUGE_PAGECACHE. However, that
>>> option was removed from the kernel via commit 396bcc5299c2 ("mm: remove
>>> CONFIG_TRANSPARENT_HUGE_PAGECACHE"), a couple of years later.
>>>
>>> The net effect is that CONFIG_TRANSPARENT_HUGEPAGE is now used in all
>>> the remaining places in the kernel where
>>> CONFIG_TRANSPARENT_HUGE_PAGECACHE had previously been used.
>>>
>>> This has caused some minor confusion at the man page level, though. So
>>> let's fix it by updating the man page to also refer to
>>> CONFIG_TRANSPARENT_HUGEPAGE.
>>>
>>> Reported-by: Vahid Noormofidi <vnoormof@nvidia.com>
>>> Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
>>> Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
>>> Cc: Andrew Morton <akpm@linux-foundation.org>
>>> Cc: Carsten Grohmann <carstengrohmann@gmx.de>
>>> Cc: Signed-off-by: Mike Frysinger <vapier@gentoo.org>
>>
>> Accident here :-)
>>
> 
> Yes. :)  Michael Kerrisk, could you change that line for me
> to just be a Cc, for Mike Frysinger?
> 
> Or, let me know if you'd prefer a re-send of the patch instead.

I can change it.  Don't worry.  I'll review it later.

Thanks,
Alex

> 
> 
> thanks,

-- 
<http://www.alejandro-colomar.es/>
GPG key fingerprint: A9348594CE31283A826FBDD8D57633D441E25BB5


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[Alejandro Colomar]

[-- Attachment #1.1: Type: text/plain, Size: 2189 bytes --]

Hi John,

On 2023-07-20 10:03, Alejandro Colomar wrote:
> Hi John,
> 
> On 2023-07-19 22:13, John Hubbard wrote:
>> On 7/19/23 12:59, Alejandro Colomar wrote:
>>> On 2023-07-19 04:05, John Hubbard wrote:
>>>> In commit 462a385e9a2 ("tmpfs.5: Document current mount options"), there
>>>> is a reference to CONFIG_TRANSPARENT_HUGE_PAGECACHE. However, that
>>>> option was removed from the kernel via commit 396bcc5299c2 ("mm: remove
>>>> CONFIG_TRANSPARENT_HUGE_PAGECACHE"), a couple of years later.
>>>>
>>>> The net effect is that CONFIG_TRANSPARENT_HUGEPAGE is now used in all
>>>> the remaining places in the kernel where
>>>> CONFIG_TRANSPARENT_HUGE_PAGECACHE had previously been used.
>>>>
>>>> This has caused some minor confusion at the man page level, though. So
>>>> let's fix it by updating the man page to also refer to
>>>> CONFIG_TRANSPARENT_HUGEPAGE.
>>>>
>>>> Reported-by: Vahid Noormofidi <vnoormof@nvidia.com>
>>>> Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
>>>> Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
>>>> Cc: Andrew Morton <akpm@linux-foundation.org>
>>>> Cc: Carsten Grohmann <carstengrohmann@gmx.de>
>>>> Cc: Signed-off-by: Mike Frysinger <vapier@gentoo.org>
>>>
>>> Accident here :-)
>>>
>>
>> Yes. :)  Michael Kerrisk, could you change that line for me
>> to just be a Cc, for Mike Frysinger?
>>
>> Or, let me know if you'd prefer a re-send of the patch instead.
> 
> I can change it.  Don't worry.  I'll review it later.

Patch applied.  Thanks!
I've slightly reworded the commit message:

-  Use tmpfs.5: instead of man5/tmpfs.5:.  We only specify the last
   filename in the path (see the git log).

-  It really wasn't a typo, because the previous text was written in
   a time when that was the correct spelling.  So, instead, I've
   changed it to talk about an update, rather than a typo fix.

<https://git.kernel.org/pub/scm/docs/man-pages/man-pages.git/commit/?id=80e3024bde68ee3fa55cfd87728b81f94ec931df>

Cheers,
Alex

> 
> Thanks,
> Alex
> 
>>
>>
>> thanks,
> 

-- 
<http://www.alejandro-colomar.es/>
GPG key fingerprint: A9348594CE31283A826FBDD8D57633D441E25BB5


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] man5/tmpfs.5: fix typo in reference to CONFIG_TRANSPARENT_HUGEPAGE

[John Hubbard]

On 7/28/23 12:04, Alejandro Colomar wrote:
...
> I've slightly reworded the commit message:
> 
> -  Use tmpfs.5: instead of man5/tmpfs.5:.  We only specify the last
>     filename in the path (see the git log).
> 
> -  It really wasn't a typo, because the previous text was written in
>     a time when that was the correct spelling.  So, instead, I've
>     changed it to talk about an update, rather than a typo fix.
> 

Oh yes, thanks for fixing that up! That's perfect.

I wrote the subject line early on, when I initially thought it was a
typo. After looking more closely it became clear what happened, but I
failed to update the subject line accordingly, so I really appreciate
that you caught it.


thanks,
-- 
John Hubbard
NVIDIA