Received-SPF: pass (google.com: domain of linux-kernel+bounces-92063-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <80269a92-1242-49be-97c5-315c0621a1d3@suse.cz>
Date: Tue, 5 Mar 2024 11:14:32 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] slab: remove PARTIAL_NODE slab_state
Content-Language: en-US
To: Chengming Zhou <chengming.zhou@linux.dev>
Cc: akpm@linux-foundation.org, cl@linux.com, penberg@kernel.org,
 rientjes@google.com, iamjoonsoo.kim@lge.com, roman.gushchin@linux.dev,
 42.hyeyoo@gmail.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org
References: <20240305083913.1494784-1-chengming.zhou@linux.dev>
From: Vlastimil Babka <vbabka@suse.cz>
In-Reply-To: <20240305083913.1494784-1-chengming.zhou@linux.dev>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 3/5/24 09:39, Chengming Zhou wrote:
> The PARTIAL_NODE slab_state has gone with SLAB removed, so just
> remove it.
> 
> Signed-off-by: Chengming Zhou <chengming.zhou@linux.dev>

Thanks, queued.

> ---
>  mm/slab.h                  | 1 -
>  tools/include/linux/slab.h | 1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/mm/slab.h b/mm/slab.h
> index 96cb74be4b69..d2bc9b191222 100644
> --- a/mm/slab.h
> +++ b/mm/slab.h
> @@ -363,7 +363,6 @@ static inline int objs_per_slab(const struct kmem_cache *cache,
>  enum slab_state {
>  	DOWN,			/* No slab functionality yet */
>  	PARTIAL,		/* SLUB: kmem_cache_node available */
> -	PARTIAL_NODE,		/* SLAB: kmalloc size for node struct available */
>  	UP,			/* Slab caches usable but not all extras yet */
>  	FULL			/* Everything is working */
>  };
> diff --git a/tools/include/linux/slab.h b/tools/include/linux/slab.h
> index 311759ea25e9..51b25e9c4ec7 100644
> --- a/tools/include/linux/slab.h
> +++ b/tools/include/linux/slab.h
> @@ -18,7 +18,6 @@ bool slab_is_available(void);
>  enum slab_state {
>  	DOWN,
>  	PARTIAL,
> -	PARTIAL_NODE,
>  	UP,
>  	FULL
>  };