Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 2 Jul 2021 15:58:02 +0300
From:   Mike Rapoport <rppt@linux.ibm.com>
To:     Ohhoon Kwon <ohoono.kwon@samsung.com>
Cc:     david@redhat.com, akpm@linux-foundation.org, mhocko@suse.com,
        bhe@redhat.com, ohkwon1043@gmail.com, linux-mm@kvack.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3] mm: sparse: remove __section_nr() function
Message-ID: <YN8NWgf5yTHbsVlG@linux.ibm.com>
References: <CGME20210702094457epcas1p295611b5799befffd016b8fccf3adceff@epcas1p2.samsung.com>
 <20210702094132.6276-1-ohoono.kwon@samsung.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210702094132.6276-1-ohoono.kwon@samsung.com>
Precedence: bulk

On Fri, Jul 02, 2021 at 06:41:29PM +0900, Ohhoon Kwon wrote:
> This series contains cleanups to remove __section_nr().
> 
> When CONFIG_SPARSEMEM_EXTREME is enabled, __section_nr() could be
> costly since it iterates all section roots to check if the given
> mem_section is in its range.
> 
> On the other hand, __nr_to_section which converts section_nr to
> mem_section can be done in O(1).
> 
> The only users of __section_nr() was section_mark_present() and
> find_memory_block().
> 
> PATCH 1 & 2 changes both functions to use section_nr instead of
> mem_section.
> PATCH 3 finally removes __section_nr() function.
> 
> More details can be found in each changelogs.
> 
> Ohhoon Kwon (3):
>   mm: sparse: pass section_nr to section_mark_present
>   mm: sparse: pass section_nr to find_memory_block
>   mm: sparse: remove __section_nr() function
> 
>  .../platforms/pseries/hotplug-memory.c        |  4 +--
>  drivers/base/memory.c                         |  4 +--
>  include/linux/memory.h                        |  2 +-
>  include/linux/mmzone.h                        |  1 -
>  mm/sparse.c                                   | 35 +++----------------
>  5 files changed, 9 insertions(+), 37 deletions(-)
> 
> -- 
> 2.17.1
 
For the series:

Acked-by: Mike Rapoport <rppt@linux.ibm.com>

-- 
Sincerely yours,
Mike.