linux-kernel - Re: [PATCH v2] Docs/mm: Fix a mistake for pfn in page

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <87msj6slrp.fsf@trenco.lwn.net>
Date: Mon, 14 Oct 2024 10:16:10 -0600
From: Jonathan Corbet <corbet@....net>
To: Pengyu Zhang <zpenya1314@...il.com>
Cc: rppt@...nel.org, linus.walleij@...aro.org, fmdefrancesco@...il.com,
 linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
 yaxin_wang_uestc@....com, Pengyu Zhang <zpenya1314@...il.com>
Subject: Re: [PATCH v2] Docs/mm: Fix a mistake for pfn in page_tables.rst

Pengyu Zhang <zpenya1314@...il.com> writes:

> The documentation incorrectly calculate the pfn value as 0x3fffff,
> which should be 0x3ffff instead. It is obtained by right-shifting
> 0xffffc000 by 14 bits.
>
> This patch corrects the value to prevent any potential confusion
> for developers referencing this document.
>
> Signed-off-by: Pengyu Zhang <zpenya1314@...il.com>
> ---
>  Documentation/mm/page_tables.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/mm/page_tables.rst b/Documentation/mm/page_tables.rst
> index be47b192a596..e7c69cc32493 100644
> --- a/Documentation/mm/page_tables.rst
> +++ b/Documentation/mm/page_tables.rst
> @@ -29,7 +29,7 @@ address.
>  With a page granularity of 4KB and a address range of 32 bits, pfn 0 is at
>  address 0x00000000, pfn 1 is at address 0x00001000, pfn 2 is at 0x00002000
>  and so on until we reach pfn 0xfffff at 0xfffff000. With 16KB pages pfs are
> -at 0x00004000, 0x00008000 ... 0xffffc000 and pfn goes from 0 to 0x3fffff.
> +at 0x00004000, 0x00008000 ... 0xffffc000 and pfn goes from 0 to 0x3ffff.
>  

Applied, thanks.

For future reference, when you submit an updated version of a patch,
please make a note of what has changed below the "---" line.  Also, you
didn't pick up Mike's Reviewed-by from the first version; I've added it.

Thanks,

jon