From nobody Fri Dec 19 21:57:55 2025 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.223.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9109C22B8D5 for ; Fri, 2 May 2025 08:36:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.135.223.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746175015; cv=none; b=Ya5YAfdE45GbXDhn3nfK4Rty6NdXIDzxB38LwJCzNexA2eir0bTqctOtwUj9V3xzdorvAEjg9ZvsKLCkIG2dhXrzOGFlMUC70xQ5v4WvFF7EPkPmZ9LalSDBOS4CjltP4WY5C4eVgiN4C8dPp8JD0PcmXr1SHoLVsAvtVTlW4r4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746175015; c=relaxed/simple; bh=0UTWqwOTYd8MnaGhf5vFiZKCrivphaJiJeYZpOO16U0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=DNeADoxKVSBFOs0OdcERbZSQv59hcIGjpWpytnrWQkVvVsmis3tz3wwDPTjLLARxeQRCFvLBXfsVTelfPE5mBCjk1xEuSbrAPs8+ToEeTDXqbwyAVIJ3YepJVB5Ulq530cfgvqfvCbbxG78sLF8DrEu2SCqaZet0QdLuT9mAXUI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de; spf=pass smtp.mailfrom=suse.de; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=deBg1re9; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=bOAu/k21; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=deBg1re9; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=bOAu/k21; arc=none smtp.client-ip=195.135.223.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="deBg1re9"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="bOAu/k21"; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="deBg1re9"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="bOAu/k21" Received: from imap1.dmz-prg2.suse.org (unknown [10.150.64.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 0B3C31F7C0; Fri, 2 May 2025 08:36:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1746175006; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RF1txh0lNcRcIYqklkvU0lHaUfn+UjfmEi23eEsi2hA=; b=deBg1re90BTlCw/LOGhobMySte4pi+vUlEaRLRgKTmjAoW0F8ryPlqD/qYdprzNm5lKaWE u0ORpqvtyXhcdBWrcs5cPS1t3L9xoJu3CH7GL+dIgSUB67t7xsScn7MojWuKm1WH/U28KG aFLA+qFOJbyVymGGtOtS7oni/OEdnrs= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1746175006; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RF1txh0lNcRcIYqklkvU0lHaUfn+UjfmEi23eEsi2hA=; b=bOAu/k21P5OEVOt0pUdj6RnAyEnnkmn16GdP/3/zuVfGzFiDdcy4L3zkvJAoJHj+eCQ2fk 9/s0EIbSEpfCIXAQ== Authentication-Results: smtp-out2.suse.de; none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1746175006; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RF1txh0lNcRcIYqklkvU0lHaUfn+UjfmEi23eEsi2hA=; b=deBg1re90BTlCw/LOGhobMySte4pi+vUlEaRLRgKTmjAoW0F8ryPlqD/qYdprzNm5lKaWE u0ORpqvtyXhcdBWrcs5cPS1t3L9xoJu3CH7GL+dIgSUB67t7xsScn7MojWuKm1WH/U28KG aFLA+qFOJbyVymGGtOtS7oni/OEdnrs= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1746175006; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RF1txh0lNcRcIYqklkvU0lHaUfn+UjfmEi23eEsi2hA=; b=bOAu/k21P5OEVOt0pUdj6RnAyEnnkmn16GdP/3/zuVfGzFiDdcy4L3zkvJAoJHj+eCQ2fk 9/s0EIbSEpfCIXAQ== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id 7EDD91399F; Fri, 2 May 2025 08:36:45 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id WDQ5HB2EFGhVSwAAD6G6ig (envelope-from ); Fri, 02 May 2025 08:36:45 +0000 From: Oscar Salvador To: Andrew Morton Cc: David Hildenbrand , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Vlastimil Babka , Hyeonggon Yoo <42.hyeyoo@gmail.com>, Jonathan Cameron , Rakie Kim , Oscar Salvador Subject: [PATCH v3 1/3] mm,slub: Do not special case N_NORMAL nodes for slab_nodes Date: Fri, 2 May 2025 10:36:22 +0200 Message-ID: <20250502083624.49849-2-osalvador@suse.de> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250502083624.49849-1-osalvador@suse.de> References: <20250502083624.49849-1-osalvador@suse.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Spam-Level: X-Spamd-Result: default: False [-1.30 / 50.00]; BAYES_HAM(-3.00)[100.00%]; SUSPICIOUS_RECIPS(1.50)[]; MID_CONTAINS_FROM(1.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; R_MISSING_CHARSET(0.50)[]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; DBL_BLOCKED_OPENRESOLVER(0.00)[suse.de:email,suse.de:mid]; FREEMAIL_CC(0.00)[redhat.com,kvack.org,vger.kernel.org,suse.cz,gmail.com,huawei.com,sk.com,suse.de]; TAGGED_RCPT(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; MIME_TRACE(0.00)[0:+]; ARC_NA(0.00)[]; FROM_HAS_DN(0.00)[]; RCVD_TLS_ALL(0.00)[]; RCPT_COUNT_SEVEN(0.00)[9]; RCVD_COUNT_TWO(0.00)[2]; TO_MATCH_ENVRCPT_ALL(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; FUZZY_BLOCKED(0.00)[rspamd.com]; TO_DN_SOME(0.00)[]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; FREEMAIL_ENVRCPT(0.00)[gmail.com] X-Spam-Score: -1.30 X-Spam-Flag: NO Content-Type: text/plain; charset="utf-8" Currently, slab_mem_going_going_callback() checks whether the node has N_NORMAL memory in order to be set in slab_nodes. While it is true that gettind rid of that enforcing would mean ending up with movables nodes in slab_nodes, the memory waste that comes with that is negligible. So stop checking for status_change_nid_normal and just use status_change_nid instead which works for both types of memory. Also, once we allocate the kmem_cache_node cache for the node in slab_mem_online_callback(), we never deallocate it in slab_mem_off_callback() when the node goes memoryless, so we can just get rid of it. The only side effect is that we will stop clearing the node from slab_nodes. Suggested-by: David Hildenbrand Signed-off-by: Oscar Salvador Reviewed-by: Vlastimil Babka --- mm/slub.c | 34 +++------------------------------- 1 file changed, 3 insertions(+), 31 deletions(-) diff --git a/mm/slub.c b/mm/slub.c index dc9e729e1d26..236d679e6a1d 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -447,7 +447,7 @@ static inline struct kmem_cache_node *get_node(struct k= mem_cache *s, int node) =20 /* * Tracks for which NUMA nodes we have kmem_cache_nodes allocated. - * Corresponds to node_state[N_NORMAL_MEMORY], but can temporarily + * Corresponds to node_state[N_MEMORY], but can temporarily * differ during memory hotplug/hotremove operations. * Protected by slab_mutex. */ @@ -6174,36 +6174,12 @@ static int slab_mem_going_offline_callback(void *ar= g) return 0; } =20 -static void slab_mem_offline_callback(void *arg) -{ - struct memory_notify *marg =3D arg; - int offline_node; - - offline_node =3D marg->status_change_nid_normal; - - /* - * If the node still has available memory. we need kmem_cache_node - * for it yet. - */ - if (offline_node < 0) - return; - - mutex_lock(&slab_mutex); - node_clear(offline_node, slab_nodes); - /* - * We no longer free kmem_cache_node structures here, as it would be - * racy with all get_node() users, and infeasible to protect them with - * slab_mutex. - */ - mutex_unlock(&slab_mutex); -} - static int slab_mem_going_online_callback(void *arg) { struct kmem_cache_node *n; struct kmem_cache *s; struct memory_notify *marg =3D arg; - int nid =3D marg->status_change_nid_normal; + int nid =3D marg->status_change_nid; int ret =3D 0; =20 /* @@ -6261,10 +6237,6 @@ static int slab_memory_callback(struct notifier_bloc= k *self, case MEM_GOING_OFFLINE: ret =3D slab_mem_going_offline_callback(arg); break; - case MEM_OFFLINE: - case MEM_CANCEL_ONLINE: - slab_mem_offline_callback(arg); - break; case MEM_ONLINE: case MEM_CANCEL_OFFLINE: break; @@ -6335,7 +6307,7 @@ void __init kmem_cache_init(void) * Initialize the nodemask for which we will allocate per node * structures. Here we don't need taking slab_mutex yet. */ - for_each_node_state(node, N_NORMAL_MEMORY) + for_each_node_state(node, N_MEMORY) node_set(node, slab_nodes); =20 create_boot_cache(kmem_cache_node, "kmem_cache_node", --=20 2.49.0 From nobody Fri Dec 19 21:57:55 2025 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.223.130]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1021322A7ED for ; Fri, 2 May 2025 08:36:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.135.223.130 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746175011; cv=none; b=i6dtjnumXz+SXZ09HKYIkNA11xhUglWWGNNd/d+8KyuZpMLNmEIwY8ee85XqBGWmP6JP/J439p5STOWstbl6DPvTkUBHr5W2F4lYHdMDTNIyKS35FoYXy+EGTXLPqGB1hmuJomtQy6unvFeY4ctFSbgeeuuqLBQRO/l97OpA4HM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746175011; c=relaxed/simple; bh=CPcKJ5GKh3nx2AjO8i1ZWdqozSS9cflwuPLGci6Vick=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Kp12mS+WLrTBNB7ZnX73WcDwxQo7oxi0SHkyuFT+7aEvaNdG4jfOxGj3Pgdly+N6QF6ACVAFqfkTYAle6ujJB6JjCxcmxUyxTQDemUH4GrNzmo+x9ZT8MErJgXfslwoCUXDLQpC+Ib2bGCd/fjMi6iSOAi0SHjU++E1uSTRBfhw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de; spf=pass smtp.mailfrom=suse.de; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=hWEzDVyR; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=fnX1zO7+; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=1xVAF7wf; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=bhlBZeqd; arc=none smtp.client-ip=195.135.223.130 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="hWEzDVyR"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="fnX1zO7+"; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="1xVAF7wf"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="bhlBZeqd" Received: from imap1.dmz-prg2.suse.org (unknown [10.150.64.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id ACC402116E; Fri, 2 May 2025 08:36:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1746175007; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=isa0zxdidhzOJ/B3vHKf7DxtoluhEXh9YyWcI8v7Fq8=; b=hWEzDVyRkAfJTnsYc+QzMtxz/BSlwYoJRYZ1j/fdWT3jCu5bgXuBbUTZCKlNMCngut21ez 5rX/EZ5SYZiV4ujKgiPsY8+BzW21Oci0VMKhcQhr1G1vXTlb7/PBkMEo8S+Z+beGS7EE+e Gs2ASNFG/d7BO4bPyTA8fZmyDFFdOaY= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1746175007; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=isa0zxdidhzOJ/B3vHKf7DxtoluhEXh9YyWcI8v7Fq8=; b=fnX1zO7+SChBPPgwShObbIzwoc9fJXRHJhVpceoAcsszXQj/VeB4ITyyFmTAD7p7k+V4a5 t1StWyYwLELuT1Ag== Authentication-Results: smtp-out1.suse.de; none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1746175006; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=isa0zxdidhzOJ/B3vHKf7DxtoluhEXh9YyWcI8v7Fq8=; b=1xVAF7wfGJ25GoKrRJK1dvS+mDxqvYrRqjO1AjDkh5obDYO2tXpgJb+tUfl8o0dUYjHx/1 a7T2G89efLkxcYnM4zs656UQ4q3c99HVyMm27XEh7XPVKOSX+5P1S4Y6QJkx4jhE/IrMYs yXES76UR+HrbxGwF2Zgk2pvqgQxbOhE= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1746175006; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=isa0zxdidhzOJ/B3vHKf7DxtoluhEXh9YyWcI8v7Fq8=; b=bhlBZeqd8ihCnsX/+dGmof8gmR3+x9sNumdLBdF7+6Ii/Xv+oHKWCXFEurCj4E7kOBJ5kh ORYA+ZHgFEknQNBg== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id 1EE2413687; Fri, 2 May 2025 08:36:46 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id KMvGBB6EFGhVSwAAD6G6ig (envelope-from ); Fri, 02 May 2025 08:36:46 +0000 From: Oscar Salvador To: Andrew Morton Cc: David Hildenbrand , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Vlastimil Babka , Hyeonggon Yoo <42.hyeyoo@gmail.com>, Jonathan Cameron , Rakie Kim , Oscar Salvador , Harry Yoo Subject: [PATCH v3 2/3] mm,memory_hotplug: Implement numa node notifier Date: Fri, 2 May 2025 10:36:23 +0200 Message-ID: <20250502083624.49849-3-osalvador@suse.de> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250502083624.49849-1-osalvador@suse.de> References: <20250502083624.49849-1-osalvador@suse.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Spam-Score: -1.30 X-Spamd-Result: default: False [-1.30 / 50.00]; BAYES_HAM(-3.00)[100.00%]; SUSPICIOUS_RECIPS(1.50)[]; MID_CONTAINS_FROM(1.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; R_MISSING_CHARSET(0.50)[]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; DBL_BLOCKED_OPENRESOLVER(0.00)[oracle.com:email,huawei.com:email,suse.de:mid,suse.de:email]; FREEMAIL_CC(0.00)[redhat.com,kvack.org,vger.kernel.org,suse.cz,gmail.com,huawei.com,sk.com,suse.de,oracle.com]; TAGGED_RCPT(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; MIME_TRACE(0.00)[0:+]; ARC_NA(0.00)[]; FROM_HAS_DN(0.00)[]; RCVD_TLS_ALL(0.00)[]; RCPT_COUNT_SEVEN(0.00)[10]; RCVD_COUNT_TWO(0.00)[2]; TO_MATCH_ENVRCPT_ALL(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; FUZZY_BLOCKED(0.00)[rspamd.com]; TO_DN_SOME(0.00)[]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; FREEMAIL_ENVRCPT(0.00)[gmail.com] X-Spam-Flag: NO X-Spam-Level: Content-Type: text/plain; charset="utf-8" There are at least six consumers of hotplug_memory_notifier that what they really are interested in is whether any numa node changed its state, e.g: g= oing from being memory aware to becoming memoryless and vice versa. Implement a specific notifier for numa nodes when their state gets changed, and have those consumers that only care about numa node state changes use i= t. Signed-off-by: Oscar Salvador Reviewed-by: Harry Yoo Reviewed-by: Jonathan Cameron Reviewed-by: Vlastimil Babka --- drivers/acpi/numa/hmat.c | 6 +- drivers/base/node.c | 21 +++++++ drivers/cxl/core/region.c | 14 ++--- drivers/cxl/cxl.h | 4 +- include/linux/memory.h | 38 ++++++++++++- kernel/cgroup/cpuset.c | 2 +- mm/memory-tiers.c | 8 +-- mm/memory_hotplug.c | 117 +++++++++++++++++++++----------------- mm/mempolicy.c | 8 +-- mm/slub.c | 13 ++--- 10 files changed, 150 insertions(+), 81 deletions(-) diff --git a/drivers/acpi/numa/hmat.c b/drivers/acpi/numa/hmat.c index 9d9052258e92..9ac82a767daf 100644 --- a/drivers/acpi/numa/hmat.c +++ b/drivers/acpi/numa/hmat.c @@ -962,10 +962,10 @@ static int hmat_callback(struct notifier_block *self, unsigned long action, void *arg) { struct memory_target *target; - struct memory_notify *mnb =3D arg; + struct node_notify *mnb =3D arg; int pxm, nid =3D mnb->status_change_nid; =20 - if (nid =3D=3D NUMA_NO_NODE || action !=3D MEM_ONLINE) + if (nid =3D=3D NUMA_NO_NODE || action !=3D NODE_BECAME_MEM_AWARE) return NOTIFY_OK; =20 pxm =3D node_to_pxm(nid); @@ -1118,7 +1118,7 @@ static __init int hmat_init(void) hmat_register_targets(); =20 /* Keep the table and structures if the notifier may use them */ - if (hotplug_memory_notifier(hmat_callback, HMAT_CALLBACK_PRI)) + if (hotplug_node_notifier(hmat_callback, HMAT_CALLBACK_PRI)) goto out_put; =20 if (!hmat_set_default_dram_perf()) diff --git a/drivers/base/node.c b/drivers/base/node.c index cd13ef287011..ca635848ee68 100644 --- a/drivers/base/node.c +++ b/drivers/base/node.c @@ -110,6 +110,27 @@ static const struct attribute_group *node_access_node_= groups[] =3D { NULL, }; =20 +#ifdef CONFIG_MEMORY_HOTPLUG +static BLOCKING_NOTIFIER_HEAD(node_chain); + +int register_node_notifier(struct notifier_block *nb) +{ + return blocking_notifier_chain_register(&node_chain, nb); +} +EXPORT_SYMBOL(register_node_notifier); + +void unregister_node_notifier(struct notifier_block *nb) +{ + blocking_notifier_chain_unregister(&node_chain, nb); +} +EXPORT_SYMBOL(unregister_node_notifier); + +int node_notify(unsigned long val, void *v) +{ + return blocking_notifier_call_chain(&node_chain, val, v); +} +#endif + static void node_remove_accesses(struct node *node) { struct node_access_nodes *c, *cnext; diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c index c3f4dc244df7..c43770d6834c 100644 --- a/drivers/cxl/core/region.c +++ b/drivers/cxl/core/region.c @@ -2432,12 +2432,12 @@ static int cxl_region_perf_attrs_callback(struct no= tifier_block *nb, unsigned long action, void *arg) { struct cxl_region *cxlr =3D container_of(nb, struct cxl_region, - memory_notifier); - struct memory_notify *mnb =3D arg; + node_notifier); + struct node_notify *mnb =3D arg; int nid =3D mnb->status_change_nid; int region_nid; =20 - if (nid =3D=3D NUMA_NO_NODE || action !=3D MEM_ONLINE) + if (nid =3D=3D NUMA_NO_NODE || action !=3D NODE_BECAME_MEM_AWARE) return NOTIFY_DONE; =20 /* @@ -3484,7 +3484,7 @@ static void shutdown_notifiers(void *_cxlr) { struct cxl_region *cxlr =3D _cxlr; =20 - unregister_memory_notifier(&cxlr->memory_notifier); + unregister_node_notifier(&cxlr->node_notifier); unregister_mt_adistance_algorithm(&cxlr->adist_notifier); } =20 @@ -3523,9 +3523,9 @@ static int cxl_region_probe(struct device *dev) if (rc) return rc; =20 - cxlr->memory_notifier.notifier_call =3D cxl_region_perf_attrs_callback; - cxlr->memory_notifier.priority =3D CXL_CALLBACK_PRI; - register_memory_notifier(&cxlr->memory_notifier); + cxlr->node_notifier.notifier_call =3D cxl_region_perf_attrs_callback; + cxlr->node_notifier.priority =3D CXL_CALLBACK_PRI; + register_node_notifier(&cxlr->node_notifier); =20 cxlr->adist_notifier.notifier_call =3D cxl_region_calculate_adistance; cxlr->adist_notifier.priority =3D 100; diff --git a/drivers/cxl/cxl.h b/drivers/cxl/cxl.h index be8a7dc77719..c4bf8eb3c952 100644 --- a/drivers/cxl/cxl.h +++ b/drivers/cxl/cxl.h @@ -513,7 +513,7 @@ enum cxl_partition_mode { * @flags: Region state flags * @params: active + config params for the region * @coord: QoS access coordinates for the region - * @memory_notifier: notifier for setting the access coordinates to node + * @node_notifier: notifier for setting the access coordinates to node * @adist_notifier: notifier for calculating the abstract distance of node */ struct cxl_region { @@ -526,7 +526,7 @@ struct cxl_region { unsigned long flags; struct cxl_region_params params; struct access_coordinate coord[ACCESS_COORDINATE_MAX]; - struct notifier_block memory_notifier; + struct notifier_block node_notifier; struct notifier_block adist_notifier; }; =20 diff --git a/include/linux/memory.h b/include/linux/memory.h index 5ec4e6d209b9..8c5c88eaffb3 100644 --- a/include/linux/memory.h +++ b/include/linux/memory.h @@ -99,6 +99,14 @@ int set_memory_block_size_order(unsigned int order); #define MEM_PREPARE_ONLINE (1<<6) #define MEM_FINISH_OFFLINE (1<<7) =20 +/* These states are used for numa node notifiers */ +#define NODE_BECOMING_MEM_AWARE (1<<0) +#define NODE_BECAME_MEM_AWARE (1<<1) +#define NODE_BECOMING_MEMORYLESS (1<<2) +#define NODE_BECAME_MEMORYLESS (1<<3) +#define NODE_CANCEL_MEM_AWARE (1<<4) +#define NODE_CANCEL_MEMORYLESS (1<<5) + struct memory_notify { /* * The altmap_start_pfn and altmap_nr_pages fields are designated for @@ -109,7 +117,10 @@ struct memory_notify { unsigned long altmap_nr_pages; unsigned long start_pfn; unsigned long nr_pages; - int status_change_nid_normal; + int status_change_nid; +}; + +struct node_notify { int status_change_nid; }; =20 @@ -157,15 +168,34 @@ static inline unsigned long memory_block_advised_max_= size(void) { return 0; } + +static inline int register_node_notifier(struct notifier_block *nb) +{ + return 0; +} +static inline void unregister_node_notifier(struct notifier_block *nb) +{ +} +static inline int node_notify(unsigned long val, void *v) +{ + return 0; +} +static inline int hotplug_node_notifier(notifier_fn_t fn, int pri) +{ + return 0; +} #else /* CONFIG_MEMORY_HOTPLUG */ extern int register_memory_notifier(struct notifier_block *nb); +extern int register_node_notifier(struct notifier_block *nb); extern void unregister_memory_notifier(struct notifier_block *nb); +extern void unregister_node_notifier(struct notifier_block *nb); int create_memory_block_devices(unsigned long start, unsigned long size, struct vmem_altmap *altmap, struct memory_group *group); void remove_memory_block_devices(unsigned long start, unsigned long size); extern void memory_dev_init(void); extern int memory_notify(unsigned long val, void *v); +extern int node_notify(unsigned long val, void *v); extern struct memory_block *find_memory_block(unsigned long section_nr); typedef int (*walk_memory_blocks_func_t)(struct memory_block *, void *); extern int walk_memory_blocks(unsigned long start, unsigned long size, @@ -185,6 +215,12 @@ int walk_dynamic_memory_groups(int nid, walk_memory_gr= oups_func_t func, register_memory_notifier(&fn##_mem_nb); \ }) =20 +#define hotplug_node_notifier(fn, pri) ({ \ + static __meminitdata struct notifier_block fn##_node_nb =3D\ + { .notifier_call =3D fn, .priority =3D pri };\ + register_node_notifier(&fn##_node_nb); \ +}) + #ifdef CONFIG_NUMA void memory_block_add_nid(struct memory_block *mem, int nid, enum meminit_context context); diff --git a/kernel/cgroup/cpuset.c b/kernel/cgroup/cpuset.c index 83639a12883d..66c84024f217 100644 --- a/kernel/cgroup/cpuset.c +++ b/kernel/cgroup/cpuset.c @@ -4013,7 +4013,7 @@ void __init cpuset_init_smp(void) cpumask_copy(top_cpuset.effective_cpus, cpu_active_mask); top_cpuset.effective_mems =3D node_states[N_MEMORY]; =20 - hotplug_memory_notifier(cpuset_track_online_nodes, CPUSET_CALLBACK_PRI); + hotplug_node_notifier(cpuset_track_online_nodes, CPUSET_CALLBACK_PRI); =20 cpuset_migrate_mm_wq =3D alloc_ordered_workqueue("cpuset_migrate_mm", 0); BUG_ON(!cpuset_migrate_mm_wq); diff --git a/mm/memory-tiers.c b/mm/memory-tiers.c index fc14fe53e9b7..dfe6c28c8352 100644 --- a/mm/memory-tiers.c +++ b/mm/memory-tiers.c @@ -872,7 +872,7 @@ static int __meminit memtier_hotplug_callback(struct no= tifier_block *self, unsigned long action, void *_arg) { struct memory_tier *memtier; - struct memory_notify *arg =3D _arg; + struct node_notify *arg =3D _arg; =20 /* * Only update the node migration order when a node is @@ -882,13 +882,13 @@ static int __meminit memtier_hotplug_callback(struct = notifier_block *self, return notifier_from_errno(0); =20 switch (action) { - case MEM_OFFLINE: + case NODE_BECAME_MEMORYLESS: mutex_lock(&memory_tier_lock); if (clear_node_memory_tier(arg->status_change_nid)) establish_demotion_targets(); mutex_unlock(&memory_tier_lock); break; - case MEM_ONLINE: + case NODE_BECAME_MEM_AWARE: mutex_lock(&memory_tier_lock); memtier =3D set_node_memory_tier(arg->status_change_nid); if (!IS_ERR(memtier)) @@ -929,7 +929,7 @@ static int __init memory_tier_init(void) nodes_and(default_dram_nodes, node_states[N_MEMORY], node_states[N_CPU]); =20 - hotplug_memory_notifier(memtier_hotplug_callback, MEMTIER_HOTPLUG_PRI); + hotplug_node_notifier(memtier_hotplug_callback, MEMTIER_HOTPLUG_PRI); return 0; } subsys_initcall(memory_tier_init); diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index b1caedbade5b..777f98326908 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c @@ -701,24 +701,18 @@ static void online_pages_range(unsigned long start_pf= n, unsigned long nr_pages) =20 /* check which state of node_states will be changed when online memory */ static void node_states_check_changes_online(unsigned long nr_pages, - struct zone *zone, struct memory_notify *arg) + struct zone *zone, struct node_notify *arg) { int nid =3D zone_to_nid(zone); =20 arg->status_change_nid =3D NUMA_NO_NODE; - arg->status_change_nid_normal =3D NUMA_NO_NODE; =20 if (!node_state(nid, N_MEMORY)) arg->status_change_nid =3D nid; - if (zone_idx(zone) <=3D ZONE_NORMAL && !node_state(nid, N_NORMAL_MEMORY)) - arg->status_change_nid_normal =3D nid; } =20 -static void node_states_set_node(int node, struct memory_notify *arg) +static void node_states_set_node(int node, struct node_notify *arg) { - if (arg->status_change_nid_normal >=3D 0) - node_set_state(node, N_NORMAL_MEMORY); - if (arg->status_change_nid >=3D 0) node_set_state(node, N_MEMORY); } @@ -1177,7 +1171,9 @@ int online_pages(unsigned long pfn, unsigned long nr_= pages, int need_zonelists_rebuild =3D 0; const int nid =3D zone_to_nid(zone); int ret; - struct memory_notify arg; + struct memory_notify mem_arg; + struct node_notify node_arg; + bool cancel_mem_notifier_on_err =3D false, cancel_node_notifier_on_err = =3D false; =20 /* * {on,off}lining is constrained to full memory sections (or more @@ -1194,11 +1190,22 @@ int online_pages(unsigned long pfn, unsigned long n= r_pages, /* associate pfn range with the zone */ move_pfn_range_to_zone(zone, pfn, nr_pages, NULL, MIGRATE_ISOLATE); =20 - arg.start_pfn =3D pfn; - arg.nr_pages =3D nr_pages; - node_states_check_changes_online(nr_pages, zone, &arg); + mem_arg.start_pfn =3D pfn; + mem_arg.nr_pages =3D nr_pages; + node_states_check_changes_online(nr_pages, zone, &node_arg); + + if (node_arg.status_change_nid >=3D 0) { + /* Node is becoming memory aware. Notify consumers */ + cancel_node_notifier_on_err =3D true; + ret =3D node_notify(NODE_BECOMING_MEM_AWARE, &node_arg); + ret =3D notifier_to_errno(ret); + if (ret) + goto failed_addition; + } =20 - ret =3D memory_notify(MEM_GOING_ONLINE, &arg); + cancel_mem_notifier_on_err =3D true; + mem_arg.status_change_nid =3D node_arg.status_change_nid; + ret =3D memory_notify(MEM_GOING_ONLINE, &mem_arg); ret =3D notifier_to_errno(ret); if (ret) goto failed_addition; @@ -1224,7 +1231,7 @@ int online_pages(unsigned long pfn, unsigned long nr_= pages, online_pages_range(pfn, nr_pages); adjust_present_page_count(pfn_to_page(pfn), group, nr_pages); =20 - node_states_set_node(nid, &arg); + node_states_set_node(nid, &node_arg); if (need_zonelists_rebuild) build_all_zonelists(NULL); =20 @@ -1245,16 +1252,26 @@ int online_pages(unsigned long pfn, unsigned long n= r_pages, kswapd_run(nid); kcompactd_run(nid); =20 + if (node_arg.status_change_nid >=3D 0) + /* + * Node went from memoryless to have memory. Notifiy interested + * consumers + */ + node_notify(NODE_BECAME_MEM_AWARE, &node_arg); + writeback_set_ratelimit(); =20 - memory_notify(MEM_ONLINE, &arg); + memory_notify(MEM_ONLINE, &mem_arg); return 0; =20 failed_addition: pr_debug("online_pages [mem %#010llx-%#010llx] failed\n", (unsigned long long) pfn << PAGE_SHIFT, (((unsigned long long) pfn + nr_pages) << PAGE_SHIFT) - 1); - memory_notify(MEM_CANCEL_ONLINE, &arg); + if (cancel_mem_notifier_on_err) + memory_notify(MEM_CANCEL_ONLINE, &mem_arg); + if (cancel_node_notifier_on_err) + node_notify(NODE_CANCEL_MEM_AWARE, &node_arg); remove_pfn_range_from_zone(zone, pfn, nr_pages); return ret; } @@ -1888,48 +1905,29 @@ early_param("movable_node", cmdline_parse_movable_n= ode); =20 /* check which state of node_states will be changed when offline memory */ static void node_states_check_changes_offline(unsigned long nr_pages, - struct zone *zone, struct memory_notify *arg) + struct zone *zone, struct node_notify *arg) { struct pglist_data *pgdat =3D zone->zone_pgdat; unsigned long present_pages =3D 0; enum zone_type zt; =20 arg->status_change_nid =3D NUMA_NO_NODE; - arg->status_change_nid_normal =3D NUMA_NO_NODE; =20 /* - * Check whether node_states[N_NORMAL_MEMORY] will be changed. - * If the memory to be offline is within the range - * [0..ZONE_NORMAL], and it is the last present memory there, - * the zones in that range will become empty after the offlining, - * thus we can determine that we need to clear the node from - * node_states[N_NORMAL_MEMORY]. + * Here we count the possible pages within the range [0..ZONE_MOVABLE]. + * If after having accounted all the pages, we see that the nr_pages to + * be offlined is over or equal to the accounted pages, we know that the + * node will become empty, ans so, we can clear it for N_MEMORY. */ - for (zt =3D 0; zt <=3D ZONE_NORMAL; zt++) + for (zt =3D 0; zt <=3D ZONE_MOVABLE; zt++) present_pages +=3D pgdat->node_zones[zt].present_pages; - if (zone_idx(zone) <=3D ZONE_NORMAL && nr_pages >=3D present_pages) - arg->status_change_nid_normal =3D zone_to_nid(zone); - - /* - * We have accounted the pages from [0..ZONE_NORMAL); ZONE_HIGHMEM - * does not apply as we don't support 32bit. - * Here we count the possible pages from ZONE_MOVABLE. - * If after having accounted all the pages, we see that the nr_pages - * to be offlined is over or equal to the accounted pages, - * we know that the node will become empty, and so, we can clear - * it for N_MEMORY as well. - */ - present_pages +=3D pgdat->node_zones[ZONE_MOVABLE].present_pages; =20 if (nr_pages >=3D present_pages) arg->status_change_nid =3D zone_to_nid(zone); } =20 -static void node_states_clear_node(int node, struct memory_notify *arg) +static void node_states_clear_node(int node, struct node_notify *arg) { - if (arg->status_change_nid_normal >=3D 0) - node_clear_state(node, N_NORMAL_MEMORY); - if (arg->status_change_nid >=3D 0) node_clear_state(node, N_MEMORY); } @@ -1953,7 +1951,9 @@ int offline_pages(unsigned long start_pfn, unsigned l= ong nr_pages, unsigned long pfn, managed_pages, system_ram_pages =3D 0; const int node =3D zone_to_nid(zone); unsigned long flags; - struct memory_notify arg; + struct memory_notify mem_arg; + struct node_notify node_arg; + bool cancel_mem_notifier_on_err =3D false, cancel_node_notifier_on_err = =3D false; char *reason; int ret; =20 @@ -2012,11 +2012,21 @@ int offline_pages(unsigned long start_pfn, unsigned= long nr_pages, goto failed_removal_pcplists_disabled; } =20 - arg.start_pfn =3D start_pfn; - arg.nr_pages =3D nr_pages; - node_states_check_changes_offline(nr_pages, zone, &arg); + mem_arg.start_pfn =3D start_pfn; + mem_arg.nr_pages =3D nr_pages; + node_states_check_changes_offline(nr_pages, zone, &node_arg); + + if (node_arg.status_change_nid >=3D 0) { + cancel_node_notifier_on_err =3D true; + ret =3D node_notify(NODE_BECOMING_MEMORYLESS, &node_arg); + ret =3D notifier_to_errno(ret); + if (ret) + goto failed_removal_isolated; + } =20 - ret =3D memory_notify(MEM_GOING_OFFLINE, &arg); + cancel_mem_notifier_on_err =3D true; + mem_arg.status_change_nid =3D node_arg.status_change_nid; + ret =3D memory_notify(MEM_GOING_OFFLINE, &mem_arg); ret =3D notifier_to_errno(ret); if (ret) { reason =3D "notifier failure"; @@ -2096,27 +2106,32 @@ int offline_pages(unsigned long start_pfn, unsigned= long nr_pages, * Make sure to mark the node as memory-less before rebuilding the zone * list. Otherwise this node would still appear in the fallback lists. */ - node_states_clear_node(node, &arg); + node_states_clear_node(node, &node_arg); if (!populated_zone(zone)) { zone_pcp_reset(zone); build_all_zonelists(NULL); } =20 - if (arg.status_change_nid >=3D 0) { + if (node_arg.status_change_nid >=3D 0) { kcompactd_stop(node); kswapd_stop(node); + /* Node went memoryless. Notifiy interested consumers */ + node_notify(NODE_BECAME_MEMORYLESS, &node_arg); } =20 writeback_set_ratelimit(); =20 - memory_notify(MEM_OFFLINE, &arg); + memory_notify(MEM_OFFLINE, &mem_arg); remove_pfn_range_from_zone(zone, start_pfn, nr_pages); return 0; =20 failed_removal_isolated: /* pushback to free area */ undo_isolate_page_range(start_pfn, end_pfn, MIGRATE_MOVABLE); - memory_notify(MEM_CANCEL_OFFLINE, &arg); + if (cancel_mem_notifier_on_err) + memory_notify(MEM_CANCEL_OFFLINE, &mem_arg); + if (cancel_node_notifier_on_err) + node_notify(NODE_CANCEL_MEMORYLESS, &node_arg); failed_removal_pcplists_disabled: lru_cache_enable(); zone_pcp_enable(zone); diff --git a/mm/mempolicy.c b/mm/mempolicy.c index f43951668c41..b3ad63fb3a2b 100644 --- a/mm/mempolicy.c +++ b/mm/mempolicy.c @@ -3591,20 +3591,20 @@ static int wi_node_notifier(struct notifier_block *= nb, unsigned long action, void *data) { int err; - struct memory_notify *arg =3D data; + struct node_notify *arg =3D data; int nid =3D arg->status_change_nid; =20 if (nid < 0) return NOTIFY_OK; =20 switch (action) { - case MEM_ONLINE: + case NODE_BECAME_MEM_AWARE: err =3D sysfs_wi_node_add(nid); if (err) pr_err("failed to add sysfs for node%d during hotplug: %d\n", nid, err); break; - case MEM_OFFLINE: + case NODE_BECAME_MEMORYLESS: sysfs_wi_node_delete(nid); break; } @@ -3639,7 +3639,7 @@ static int __init add_weighted_interleave_group(struc= t kobject *mempolicy_kobj) } } =20 - hotplug_memory_notifier(wi_node_notifier, DEFAULT_CALLBACK_PRI); + hotplug_node_notifier(wi_node_notifier, DEFAULT_CALLBACK_PRI); return 0; =20 err_cleanup_kobj: diff --git a/mm/slub.c b/mm/slub.c index 236d679e6a1d..72515ab4aaf8 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -6178,8 +6178,8 @@ static int slab_mem_going_online_callback(void *arg) { struct kmem_cache_node *n; struct kmem_cache *s; - struct memory_notify *marg =3D arg; - int nid =3D marg->status_change_nid; + struct node_notify *narg =3D arg; + int nid =3D narg->status_change_nid; int ret =3D 0; =20 /* @@ -6231,15 +6231,12 @@ static int slab_memory_callback(struct notifier_blo= ck *self, int ret =3D 0; =20 switch (action) { - case MEM_GOING_ONLINE: + case NODE_BECOMING_MEM_AWARE: ret =3D slab_mem_going_online_callback(arg); break; - case MEM_GOING_OFFLINE: + case NODE_BECOMING_MEMORYLESS: ret =3D slab_mem_going_offline_callback(arg); break; - case MEM_ONLINE: - case MEM_CANCEL_OFFLINE: - break; } if (ret) ret =3D notifier_from_errno(ret); @@ -6314,7 +6311,7 @@ void __init kmem_cache_init(void) sizeof(struct kmem_cache_node), SLAB_HWCACHE_ALIGN | SLAB_NO_OBJ_EXT, 0, 0); =20 - hotplug_memory_notifier(slab_memory_callback, SLAB_CALLBACK_PRI); + hotplug_node_notifier(slab_memory_callback, SLAB_CALLBACK_PRI); =20 /* Able to allocate the per node structures */ slab_state =3D PARTIAL; --=20 2.49.0 From nobody Fri Dec 19 21:57:55 2025 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.223.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7533822AE7B for ; Fri, 2 May 2025 08:37:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.135.223.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746175022; cv=none; b=hvDprIm85mH+3+YznI9VltYF8Hip4ZbW1O3ANkbXLt2oyREi3IJ+RllwqnPRZd1Q8qZ9KiGiZ1EA7VowFTkf+6mtAmX1YAzOqD9NLrV9b3k0yfyQ7E5kac526BV3dXMhf4+DdiG6d9ExsyGTtQAJMlfa70gIOh7+NqqL7xaa4oc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746175022; c=relaxed/simple; bh=zNUe0Pj5dkyMuxu1Q6WqSp1fT4n1YqCD8YdUazcTFgE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=B3nHFgL+pBD9rECDANbdGA/jCBFK/j5Euu7KqeHXlPslKI7pNPVGnmfxZwBgeA6s7jvypMkWEIhh0gmk4flU/74le9mlBBjBybO0MAIOExpPBB0Der0GaziXt0bJTX3zcjmqxra1+TTgkVjmqSwX+iO2KoLEPshnmK7CGTSME2E= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de; spf=pass smtp.mailfrom=suse.de; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=E407EIsB; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=dhTksgVL; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b=E407EIsB; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b=dhTksgVL; arc=none smtp.client-ip=195.135.223.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=suse.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="E407EIsB"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="dhTksgVL"; dkim=pass (1024-bit key) header.d=suse.de header.i=@suse.de header.b="E407EIsB"; dkim=permerror (0-bit key) header.d=suse.de header.i=@suse.de header.b="dhTksgVL" Received: from imap1.dmz-prg2.suse.org (imap1.dmz-prg2.suse.org [IPv6:2a07:de40:b281:104:10:150:64:97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 569E01F385; Fri, 2 May 2025 08:36:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1746175007; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=UDf9u3uKTejLxqlFBo+30RhKdS2KOtzyesZwbietrzs=; b=E407EIsBUWip8Z6Cwmgh3Wl2kaa0j7sxZ1hltythl7SUS6D2WT96YDDh9+c9fr40WfOVyo 3IGv7Q+d7ayql4Zq5GYndUiti11mNpddW0MPMCBf3lMF6orjC2otMELnZ7Msq+IKyMa5SU 6F5Hhwp0/n40OrGI/Es6L7RP18hYzCM= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1746175007; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=UDf9u3uKTejLxqlFBo+30RhKdS2KOtzyesZwbietrzs=; b=dhTksgVL5QO8F9XwJ2/P1tItL1hidLjb2ESMRXIkrtbHg+lsCLhNb5i9aiWrFbrUI2Hj1/ 5BRmW3lq2kX19PCQ== Authentication-Results: smtp-out2.suse.de; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=E407EIsB; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=dhTksgVL DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1746175007; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=UDf9u3uKTejLxqlFBo+30RhKdS2KOtzyesZwbietrzs=; b=E407EIsBUWip8Z6Cwmgh3Wl2kaa0j7sxZ1hltythl7SUS6D2WT96YDDh9+c9fr40WfOVyo 3IGv7Q+d7ayql4Zq5GYndUiti11mNpddW0MPMCBf3lMF6orjC2otMELnZ7Msq+IKyMa5SU 6F5Hhwp0/n40OrGI/Es6L7RP18hYzCM= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1746175007; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=UDf9u3uKTejLxqlFBo+30RhKdS2KOtzyesZwbietrzs=; b=dhTksgVL5QO8F9XwJ2/P1tItL1hidLjb2ESMRXIkrtbHg+lsCLhNb5i9aiWrFbrUI2Hj1/ 5BRmW3lq2kX19PCQ== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id BF50F1399F; Fri, 2 May 2025 08:36:46 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id cJTyKx6EFGhVSwAAD6G6ig (envelope-from ); Fri, 02 May 2025 08:36:46 +0000 From: Oscar Salvador To: Andrew Morton Cc: David Hildenbrand , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Vlastimil Babka , Hyeonggon Yoo <42.hyeyoo@gmail.com>, Jonathan Cameron , Rakie Kim , Oscar Salvador , Harry Yoo Subject: [PATCH v3 3/3] mm,memory_hotplug: Rename status_change_nid parameter in memory_notify Date: Fri, 2 May 2025 10:36:24 +0200 Message-ID: <20250502083624.49849-4-osalvador@suse.de> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250502083624.49849-1-osalvador@suse.de> References: <20250502083624.49849-1-osalvador@suse.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: 569E01F385 X-Spam-Level: X-Spamd-Result: default: False [-1.51 / 50.00]; BAYES_HAM(-3.00)[100.00%]; SUSPICIOUS_RECIPS(1.50)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; MID_CONTAINS_FROM(1.00)[]; R_MISSING_CHARSET(0.50)[]; R_DKIM_ALLOW(-0.20)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; MX_GOOD(-0.01)[]; MIME_TRACE(0.00)[0:+]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; ARC_NA(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; FUZZY_BLOCKED(0.00)[rspamd.com]; FREEMAIL_CC(0.00)[redhat.com,kvack.org,vger.kernel.org,suse.cz,gmail.com,huawei.com,sk.com,suse.de,oracle.com]; DKIM_TRACE(0.00)[suse.de:+]; TO_DN_SOME(0.00)[]; RCVD_COUNT_TWO(0.00)[2]; DBL_BLOCKED_OPENRESOLVER(0.00)[oracle.com:email,suse.de:email,suse.de:dkim,suse.de:mid,suse.cz:email]; FROM_EQ_ENVFROM(0.00)[]; FROM_HAS_DN(0.00)[]; RCVD_TLS_ALL(0.00)[]; DNSWL_BLOCKED(0.00)[2a07:de40:b281:106:10:150:64:167:received,2a07:de40:b281:104:10:150:64:97:from]; TAGGED_RCPT(0.00)[]; RCPT_COUNT_SEVEN(0.00)[10]; ASN(0.00)[asn:25478, ipnet:::/0, country:RU]; RCVD_VIA_SMTP_AUTH(0.00)[]; FREEMAIL_ENVRCPT(0.00)[gmail.com] X-Rspamd-Server: rspamd2.dmz-prg2.suse.org X-Rspamd-Action: no action X-Spam-Score: -1.51 X-Spam-Flag: NO Content-Type: text/plain; charset="utf-8" The 'status_change_nid' field was used to track changes in the memory state of a numa node, but that funcionality has been decoupled from memory_notify and moved to node_notify. Current consumers of memory_notify are only interested in which node the memory we are adding belongs to, so rename current 'status_change_nid' to 'nid'. Signed-off-by: Oscar Salvador Reviewed-by: Harry Yoo Reviewed-by: Vlastimil Babka --- include/linux/memory.h | 2 +- mm/memory_hotplug.c | 4 ++-- mm/page_ext.c | 12 +----------- 3 files changed, 4 insertions(+), 14 deletions(-) diff --git a/include/linux/memory.h b/include/linux/memory.h index 8c5c88eaffb3..3c482f189d14 100644 --- a/include/linux/memory.h +++ b/include/linux/memory.h @@ -117,7 +117,7 @@ struct memory_notify { unsigned long altmap_nr_pages; unsigned long start_pfn; unsigned long nr_pages; - int status_change_nid; + int nid; }; =20 struct node_notify { diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index 777f98326908..9b57b58e88bc 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c @@ -1192,6 +1192,7 @@ int online_pages(unsigned long pfn, unsigned long nr_= pages, =20 mem_arg.start_pfn =3D pfn; mem_arg.nr_pages =3D nr_pages; + mem_arg.nid =3D nid; node_states_check_changes_online(nr_pages, zone, &node_arg); =20 if (node_arg.status_change_nid >=3D 0) { @@ -1204,7 +1205,6 @@ int online_pages(unsigned long pfn, unsigned long nr_= pages, } =20 cancel_mem_notifier_on_err =3D true; - mem_arg.status_change_nid =3D node_arg.status_change_nid; ret =3D memory_notify(MEM_GOING_ONLINE, &mem_arg); ret =3D notifier_to_errno(ret); if (ret) @@ -2014,6 +2014,7 @@ int offline_pages(unsigned long start_pfn, unsigned l= ong nr_pages, =20 mem_arg.start_pfn =3D start_pfn; mem_arg.nr_pages =3D nr_pages; + mem_arg.nid =3D node; node_states_check_changes_offline(nr_pages, zone, &node_arg); =20 if (node_arg.status_change_nid >=3D 0) { @@ -2025,7 +2026,6 @@ int offline_pages(unsigned long start_pfn, unsigned l= ong nr_pages, } =20 cancel_mem_notifier_on_err =3D true; - mem_arg.status_change_nid =3D node_arg.status_change_nid; ret =3D memory_notify(MEM_GOING_OFFLINE, &mem_arg); ret =3D notifier_to_errno(ret); if (ret) { diff --git a/mm/page_ext.c b/mm/page_ext.c index c351fdfe9e9a..477e6f24b7ab 100644 --- a/mm/page_ext.c +++ b/mm/page_ext.c @@ -378,16 +378,6 @@ static int __meminit online_page_ext(unsigned long sta= rt_pfn, start =3D SECTION_ALIGN_DOWN(start_pfn); end =3D SECTION_ALIGN_UP(start_pfn + nr_pages); =20 - if (nid =3D=3D NUMA_NO_NODE) { - /* - * In this case, "nid" already exists and contains valid memory. - * "start_pfn" passed to us is a pfn which is an arg for - * online__pages(), and start_pfn should exist. - */ - nid =3D pfn_to_nid(start_pfn); - VM_BUG_ON(!node_online(nid)); - } - for (pfn =3D start; !fail && pfn < end; pfn +=3D PAGES_PER_SECTION) fail =3D init_section_page_ext(pfn, nid); if (!fail) @@ -436,7 +426,7 @@ static int __meminit page_ext_callback(struct notifier_= block *self, switch (action) { case MEM_GOING_ONLINE: ret =3D online_page_ext(mn->start_pfn, - mn->nr_pages, mn->status_change_nid); + mn->nr_pages, mn->nid); break; case MEM_OFFLINE: offline_page_ext(mn->start_pfn, --=20 2.49.0