[PATCH 0/6] treewide: remove unnecessary invalid range checks in memblock iteration loops

Sang-Heon Jeon ekffu200098 at gmail.com
Mon Jun 29 09:48:27 PDT 2026


On Mon, Jun 29, 2026 at 8:44 PM Mike Rapoport <rppt at kernel.org> wrote:
>
> On Fri, Jun 26, 2026 at 07:59:22PM +0900, Sang-Heon Jeon wrote:
> > On Fri, Jun 26, 2026 at 5:23 PM Mike Rapoport <rppt at kernel.org> wrote:
> > >
> > > On Sun, Jun 21, 2026 at 11:59:10PM +0900, Sang-Heon Jeon wrote:
> > > > The memblock API guarantees that for_each_mem_range() and
> > > > for_each_mem_pfn_range() never return an invalid range, meaning start is
> > > > always less than end.
> > > >
> > > > Several memblock callers still have unnecessary invalid range checks in
> > > > their loop bodies, so remove them.
> > > >
> > > > Sang-Heon Jeon (6):
> > > >   arm64: mm: remove unreachable invalid range check in
> > > >     kasan_init_shadow()
> > > >   LoongArch: remove unreachable invalid range check in kasan_init()
> > > >   riscv: remove unreachable invalid range check in
> > > >     create_linear_mapping_page_table()
> > > >   riscv: remove unreachable invalid range check in kasan_init()
> > > >   mm: remove unnecessary empty range check in
> > > >     early_calculate_totalpages()
> > > >   mm/hugetlb: remove unnecessary empty range check in
> > > >     hugetlb_bootmem_set_nodes()
> > >
> > > I queued this for inclusion into memblock tree.
> >
> > Thank you, Mike.
> >
> > Could you please review and queue this patch [1] as well? It does the
> > same kind of clean up, I just missed it at the time.
>
> Can you please resend them all as a single set?

Thanks for considering, Mike.
I've sent v2 patch series [1] that includes the missing patches.

[1] https://lore.kernel.org/all/20260629163736.1606688-1-ekffu200098@gmail.com/#t

> > [1] https://lore.kernel.org/all/20260626032902.703944-1-ekffu200098@gmail.com/
> >
> > > >  arch/arm64/mm/kasan_init.c     | 3 ---
> > > >  arch/loongarch/mm/kasan_init.c | 3 ---
> > > >  arch/riscv/mm/init.c           | 2 --
> > > >  arch/riscv/mm/kasan_init.c     | 3 ---
> > > >  mm/hugetlb.c                   | 3 +--
> > > >  mm/mm_init.c                   | 3 +--
> > > >  6 files changed, 2 insertions(+), 15 deletions(-)
> > > >
> > > > --
> > > > 2.43.0
> > > >
> > >
> > > --
> > > Sincerely yours,
> > > Mike.
> >
> > Best Regards,
> > Sang-Heon Jeon
>
> --
> Sincerely yours,
> Mike.

Best Regards,
Sang-Heon Jeon



More information about the linux-arm-kernel mailing list