Re: [PATCH] mm/slub: remove meaningless node check in ___slab_alloc()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: JaeSang Yoo <js.yoo.5b@xxxxxxxxx>
Subject: Re: [PATCH] mm/slub: remove meaningless node check in ___slab_alloc()
From: Muchun Song <songmuchun@xxxxxxxxxxxxx>
Date: Mon, 11 Apr 2022 11:50:14 +0800
Cc: Christoph Lameter <cl@xxxxxxxxx>, Pekka Enberg <penberg@xxxxxxxxxx>, David Rientjes <rientjes@xxxxxxxxxx>, Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Vlastimil Babka <vbabka@xxxxxxx>, Roman Gushchin <roman.gushchin@xxxxxxxxx>, Ohhoon Kwon <ohkwon1043@xxxxxxxxx>, Wonhyuk Yang <vvghjk1234@xxxxxxxxx>, Jiyoup Kim <lakroforce@xxxxxxxxx>, Donghyeok Kim <dthex5d@xxxxxxxxx>, JaeSang Yoo <jsyoo5b@xxxxxxxxx>, Linux Memory Management List <linux-mm@xxxxxxxxx>, LKML <linux-kernel@xxxxxxxxxxxxxxx>
In-reply-to: <20220409144239.2649257-1-jsyoo5b@gmail.com>
References: <20220409144239.2649257-1-jsyoo5b@gmail.com>

On Sat, Apr 9, 2022 at 10:43 PM JaeSang Yoo <js.yoo.5b@xxxxxxxxx> wrote:
>
> node_match() with node=NUMA_NO_NODE always returns 1.
> Duplicate check by goto statement is meaningless. Remove it.
>
> Signed-off-by: JaeSang Yoo <jsyoo5b@xxxxxxxxx>

LGTM.

Reviewed-by: Muchun Song <songmuchun@xxxxxxxxxxxxx>

References:
- [PATCH] mm/slub: remove meaningless node check in ___slab_alloc()
  - From: JaeSang Yoo

Prev by Date: Re: [RFC PATCH 00/47] Address Space Isolation for KVM
Next by Date: Re: [PATCH] mm/slub: remove duplicate flag in allocate_slab()
Previous by thread: Re: [PATCH] mm/slub: remove meaningless node check in ___slab_alloc()
Next by thread: Re: [PATCH] mm/slub: remove meaningless node check in ___slab_alloc()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]

Powered by Linux