linux-kernel - Re: [PATCH v2] Documentation/page_tables: Add info about MMU/TLB and Page Faults

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <3179607.5fSG56mABF@suse>
Date:   Tue, 15 Aug 2023 14:27:42 +0200
From:   "Fabio M. De Francesco" <fmdefrancesco@...il.com>
To:     Linus Walleij <linus.walleij@...aro.org>
Cc:     Jonathan Corbet <corbet@....net>,
        Jonathan Cameron <Jonathan.Cameron@...wei.com>,
        Mike Rapoport <rppt@...nel.org>, linux-doc@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-mm@...ck.org,
        Andrew Morton <akpm@...ux-foundation.org>,
        Ira Weiny <ira.weiny@...el.com>,
        Matthew Wilcox <willy@...radead.org>,
        Randy Dunlap <rdunlap@...radead.org>
Subject: Re: [PATCH v2] Documentation/page_tables: Add info about MMU/TLB and Page
 Faults

On martedì 15 agosto 2023 10:51:24 CEST Linus Walleij wrote:
> Hi Fabio,
> 
> overall this v2 looks good!

Hi Linus,

Thanks for your review. I appreciated it.

I'm counting at least ten mistakes. Well my poor English should still improve 
in order to work on documentation.

I agree with you on all changes you are proposing, so I won't agree line by 
line. Instead I'll send a v3 and forward your tag. 

I have only a doubt and a questions. 
I'll jump directly to the relevant parts.

> 
> The below are my grammar and spelling nitpicks.
>
> [snip] 
> 
> > +If the above-mentioned conditions happen in user-space, the kernel sends 
a
> > +`Segmentation Fault` (SIGSEGV) signal to the current thread. That signal
> > usually +causes the termination of the thread and of the process it 
belongs
> > to. +
> > +Instead, there are also common and expected other causes of page faults.
> > These
> The word you are looking for is "Additionally" right?
> 
> "Additionally, there are..."

I was only able to use "Instead" to express that, contrary to the former 
conditions that is unexpected and uncommon, there are other expected and 
common causes of page faults. I thought that "Instead" stresses that the 
latter causes carry with them opposite and wanted consequences.

I think of "additionally" as a means to introduce less important and less 
frequently occurring conditions.

Nevertheless, I'll change it to "Additionally" as you are asking for.

Everything that follows from here onward should surely be changed as you are 
suggesting.  

[snip]

> > +Swapping can't work for memory mapped by kernel logical addresses. These
> > are a
> "kernel logical addresses" -> "kernel-internal logical addresses"

My only question is about why you prefer "kernel-internal" to a straight 
"kernel". Can you please say more about this?

[snip]

> With
> or without the above suggestions:

I'll do the v3 _with_ the above suggestions.

> Reviewed-by: Linus Walleij <linus.walleij@...aro.org>
> 
> Yours,
> Linus Walleij

Again thanks,

Fabio