Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Fabio M. De Francesco" <fmdefrancesco@gmail.com>
To:     Matthew Wilcox <willy@infradead.org>,
        Randy Dunlap <rdunlap@infradead.org>
Cc:     Jonathan Corbet <corbet@lwn.net>,
        Andrew Morton <akpm@linux-foundation.org>,
        "Mike Rapoport (IBM)" <rppt@kernel.org>,
        Deming Wang <wangdeming@inspur.com>,
        Ira Weiny <ira.weiny@intel.com>, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-mm@kvack.org,
        Catalin Marinas <catalin.marinas@arm.com>,
        Mike Rapoport <rppt@linux.ibm.com>,
        Peter Collingbourne <pcc@google.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
        Thomas Gleixner <tglx@linutronix.de>,
        Vlastimil Babka <vbabka@suse.cz>, Will Deacon <will@kernel.org>
Subject: Re: [PATCH v2] Documentation/mm: Add information about kmap_local_folio()
Date:   Sun, 02 Jul 2023 15:05:09 +0200
Message-ID: <2238844.iZASKD2KPV@suse>
In-Reply-To: <970a881a-cdaf-2568-657a-3b93b0273338@infradead.org>
References: <20230701142041.9992-1-fmdefrancesco@gmail.com>
 <ZKC8uE9fE57PuL91@casper.infradead.org>
 <970a881a-cdaf-2568-657a-3b93b0273338@infradead.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
Precedence: bulk

On domenica 2 luglio 2023 01:59:04 CEST Randy Dunlap wrote:
> On 7/1/23 16:54, Matthew Wilcox wrote:
> > On Sat, Jul 01, 2023 at 08:21:20AM -0700, Randy Dunlap wrote:
> >>> -* kmap_local_page().  This function is used to require short term
> >>> mappings.
> >>> -  It can be invoked from any context (including interrupts) but the
> >>> mappings
> >>> -  can only be used in the context which acquired them.
> >>> -
> >>> -  This function should always be used, whereas kmap_atomic() and kmap()
> >>> have
> >>> +* kmap_local_page(), kmap_local_folio() - These functions are used to
> >>> require
> >>>  
> >> acquire?
> > 
> > "create" might be better?
> 
> Yes, that's good.

Agreed.
However, I can send next version only by week 28th.

Thanks,

Fabio

P.S.: Actually I meant "to request". Unfortunately, "to request" and "to 
require" may have the same translation in Italian, my native language.

I preferred to not use "acquire" because it is re-used few lines below. So I 
thought that "to request short term mappings" was good (although I wrongly 
confused the different meanings between "to require" and "to request").

Matthew's suggestion to use "create" avoids repetition of "acquire(d)".  

> >>> +  short term mappings. They can be invoked from any context (including
> >>> +  interrupts) but the mappings can only be used in the context which
> >>> acquired +  them. The only differences between them consist in the first
> >>> taking a pointer +  to a struct page and the second taking a pointer to
> >>> struct folio and the byte +  offset within the folio which identifies 
the
> >>> page.
> >> 
> >> --
> >> ~Randy
> 
> --
> ~Randy