Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Date:   Wed, 18 May 2022 00:33:44 -0700
From:   Christoph Hellwig <hch@infradead.org>
To:     Ira Weiny <ira.weiny@intel.com>
Cc:     Christoph Hellwig <hch@infradead.org>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        "H. Peter Anvin" <hpa@zytor.com>,
        Dan Williams <dan.j.williams@intel.com>,
        Fenghua Yu <fenghua.yu@intel.com>,
        Rick Edgecombe <rick.p.edgecombe@intel.com>,
        "Shankar, Ravi V" <ravi.v.shankar@intel.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH V10 33/44] kmap: Make kmap work for devmap protected pages
Message-ID: <YoShWMGVj9ZJbWqT@infradead.org>
References: <20220419170649.1022246-1-ira.weiny@intel.com>
 <20220419170649.1022246-34-ira.weiny@intel.com>
 <Ymq3yrQ3rmgnbX1o@infradead.org>
 <Ynxh/5jewIuGwb5d@iweiny-desk3>
 <YoQl3yLa4iDCjicO@iweiny-desk3>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YoQl3yLa4iDCjicO@iweiny-desk3>
Precedence: bulk

On Tue, May 17, 2022 at 03:46:55PM -0700, Ira Weiny wrote:
> Ok there is a trade off here which I'm not sure is a good idea.
> 
> In order to make kmap faster I carefully placed the devmap_protected() call
> completely inline.[1]  This specifically checks the PGMAP_PROTECTION flag in
> dev_pagemap.[1]
> 
> I see only 2 ways of not including memremap.h in highmem-internal.h.
> 
> 1) Make this check a function call and place it in memremap.c
> 2) Move struct dev_pagemap (and it's dependencies) to another header.
> 
> Number 2 negates any positive effect of splitting the header file.
> 
> Number 1 is going to force some overhead on all ZONE_DEVICE pages if PMEM is in
> the system.
> 
> Do you think the extra run time overhead is a reasonable trade off to
> eliminating the header?

Well, given how big these devmap helpes are it seems like they should be
out of line anyway due to the code size impact.  It would be great to
compare the code size for the cases of:

 1) baseline kernel
 2) devmap protection inline
 3) devmap protection out of line

And maybe you need to add linux-mm and Thomas to get a few more
opinions.