From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4013EC369A1 for ; Wed, 9 Apr 2025 11:39:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 05835280077; Wed, 9 Apr 2025 07:39:55 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id F2252280076; Wed, 9 Apr 2025 07:39:54 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D4D25280077; Wed, 9 Apr 2025 07:39:54 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id B2841280076 for ; Wed, 9 Apr 2025 07:39:54 -0400 (EDT) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id CFEBEAC6BF for ; Wed, 9 Apr 2025 11:39:54 +0000 (UTC) X-FDA: 83314311108.20.D6DF816 Received: from invmail4.hynix.com (exvmail4.skhynix.com [166.125.252.92]) by imf30.hostedemail.com (Postfix) with ESMTP id 203DA8000C for ; Wed, 9 Apr 2025 11:39:51 +0000 (UTC) Authentication-Results: imf30.hostedemail.com; dkim=none; spf=pass (imf30.hostedemail.com: domain of honggyu.kim@sk.com designates 166.125.252.92 as permitted sender) smtp.mailfrom=honggyu.kim@sk.com; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1744198793; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=oMoef1igxXER94rSqKMuHlcLAjRYDUKQnkRjaG6W0hg=; b=3lGkQUxvbWOxLpleHpdP/OikjGCjYarHdKGQByWP9KUB2YItuuDGxSETrXtFMI6fo7/KCk 92PzsaJaBO5OiHX2TI1TQRsf21yYiVQxtwnT3J/uFmZpkyy8RQQMZ5ueTVtFUm6SX19Dcc xfvEaL+pWmjeSYE0z1Z7o2ffhSYrrYI= ARC-Authentication-Results: i=1; imf30.hostedemail.com; dkim=none; spf=pass (imf30.hostedemail.com: domain of honggyu.kim@sk.com designates 166.125.252.92 as permitted sender) smtp.mailfrom=honggyu.kim@sk.com; dmarc=none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1744198793; a=rsa-sha256; cv=none; b=qZdHYqsqWSJt2umbzG/Dpim4Lofid6QiNJ4OzXO6qTPoSV1IPsjTvRIC7pJzIc/watHCh3 DKo/bt/MB9r75IpsT6+S/zrbj1DRqzGawwAld1Y9iP4LjMo9MU152W0T7PA1bMVFfaXcS5 9U1KFhzYJficJZ+1xVHgJzpWLJLg+O4= X-AuditID: a67dfc5b-681ff7000002311f-93-67f65c8475e1 Message-ID: Date: Wed, 9 Apr 2025 20:39:47 +0900 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: kernel_team@skhynix.com, gourry@gourry.net, linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-cxl@vger.kernel.org, joshua.hahnjy@gmail.com, dan.j.williams@intel.com, ying.huang@linux.alibaba.com, Jonathan.Cameron@huawei.com, osalvador@suse.de, yunjeong.mun@sk.com Subject: Re: [PATCH v7 3/3] mm/mempolicy: Support memory hotplug in weighted interleave To: David Hildenbrand , Rakie Kim , akpm@linux-foundation.org References: <20250408073243.488-1-rakie.kim@sk.com> <20250408073243.488-4-rakie.kim@sk.com> <19562f7e-38ce-41fc-8dfc-bfd6b1259291@redhat.com> Content-Language: ko From: Honggyu Kim In-Reply-To: <19562f7e-38ce-41fc-8dfc-bfd6b1259291@redhat.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprOIsWRmVeSWpSXmKPExsXC9ZZnoW5LzLd0g5fnuC3mrF/DZjF96gVG i6/rfzFb/Lx7nN1i1cJrbBbHt85jtzg/6xSLxeVdc9gs7q35z2pxZlqRxeo1GQ7cHjtn3WX3 6G67zO7RcuQtq8fiPS+ZPDZ9msTucWLGbxaPnQ8tPd7vu8rmsfl0tcfnTXIBXFFcNimpOZll qUX6dglcGSsmphdsEqm417edsYHxgkAXIyeHhICJxOHGqWxdjBxg9sZPoSBhXgFLiWPzrzCD 2CwCKhKLbk9ghIgLSpyc+YQFxBYVkJe4f2sGexcjFwezwEwmiRdH17KCzBEWiJT42e8MUiMi kCxx+MwZZpAaIYGJjBKTLvxgB0kwC4hIzO5sA1vAJqAmceXlJCaQXk4BO4l5vbYQJWYSXVu7 GCFseYnmrbPB5kgITGaXWLN1IjvE/ZISB1fcYJnAKDgLyX2zkKyYhWTWLCSzFjCyrGIUyswr y03MzDHRy6jMy6zQS87P3cQIjK5ltX+idzB+uhB8iFGAg1GJh/eGytd0IdbEsuLK3EOMEhzM SiK8byd+SRfiTUmsrEotyo8vKs1JLT7EKM3BoiTOa/StPEVIID2xJDU7NbUgtQgmy8TBKdXA qFASYHO1WrzQxe39z/T+1vtWM1PbfTtP39j17vvtSw/O8+qqClw+0aTk+8KlU23K8zJdz7Oy VZGus+dl3xadYcak+qmCJ/X/zYbfd0OD7QU97R4sb32+ZdmOea3TtddG51z7eP/awtcnD5Y9 C3W7vTMs12KvVPF78c287HwVC1PtlzqfT57ircRSnJFoqMVcVJwIADixI0eqAgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFuphkeLIzCtJLcpLzFFi42LhmqGlp9sc8y3dYOcNS4s569ewWUyfeoHR 4uv6X8wWP+8eZ7dYtfAam8XxrfPYLQ7PPclqcX7WKRaLy7vmsFncW/Of1eLMtCKLQ9ees1qs XpNh8XvbCjYHPo+ds+6ye3S3XWb3aDnyltVj8Z6XTB6bPk1i9zgx4zeLx86Hlh7v911l8/h2 28Nj8YsPTB6bT1d7fN4kF8ATxWWTkpqTWZZapG+XwJWxYmJ6wSaRint92xkbGC8IdDFycEgI mEhs/BTaxcjJwStgKXFs/hVmEJtFQEVi0e0JjBBxQYmTM5+wgNiiAvIS92/NYO9i5OJgFpjJ JPHi6FpWkDnCApESP/udQWpEBJIlDp85wwxSIyQwkVFi0oUf7CAJZgERidmdbWAL2ATUJK68 nMQE0sspYCcxr9cWosRMomtrFyOELS/RvHU28wRGvllIzpiFZNIsJC2zkLQsYGRZxSiSmVeW m5iZY6pXnJ1RmZdZoZecn7uJERhHy2r/TNzB+OWy+yFGAQ5GJR7eGypf04VYE8uKK3MPMUpw MCuJ8L6d+CVdiDclsbIqtSg/vqg0J7X4EKM0B4uSOK9XeGqCkEB6YklqdmpqQWoRTJaJg1Oq gVHx++d+Ef31LP83b7i8R/aApmqLb/bT0GPCfqdmC274zLNFvMjuVrjNndZ9u73kc063rnKb d1hlz5f1q3ZwVL2SkmZ9mmm0xP9aqsCDnS77tUOS/3zaunP36kOPLth6/+f6oafocqbZ79FZ DVmhRb2/X00Lt9Wr8t5huKB9x0mH5aVTT/mfnbdKiaU4I9FQi7moOBEA2u1VLZ8CAAA= X-CFilter-Loop: Reflected X-Rspamd-Server: rspam11 X-Rspamd-Queue-Id: 203DA8000C X-Stat-Signature: guyrtm3tnktaw4apyjzn8rx9u6j5t1hm X-Rspam-User: X-HE-Tag: 1744198791-579706 X-HE-Meta: U2FsdGVkX1/ERGMQvrPACo4Hng6e2hdPXqrYNiblHM+XIsa/M5ue+lR67u0QC+vhO10d73KaVxRCrf8g08zyXXwMgG66o7/up7MaMZ5JsiPm9LGqDBnuxBNj12J7y251+cYg/VMwSMiU9N3AABZNntgoYdwhxH9HWIkmod0taClGK7+xyV1qFZWqAqFuzx3cU5sF/YJbUuOSjgKKhf5KAsgijPB5e+2kx8YE7s5cIWOPSta+5QqtGPjYiMAKYxfzNsjEq2IvKXl9plbgqlRUd0Th1SWevUCwYfUiMKsB9SBaVkZGQ7MvUZMlOmjQlfgVL8KdAp7rEb5sQ4SIw/MJuQdxz78xvsdI/Xl7bUjknvtjOb7NUqWZsIz6xk+W1EGU1bC6ucFUN6pWGNbMzWf+M4NYVFpriM++cvnvuQSaVBZL7uHVwJ5+7epvLixUKDOHYAommjN70UqfAOW6OtXPWWLIXNXXdKkWQQO1QILOxa2y7bL6k2d/CWb4sEZoRQe0Am7K8j/HRExJ8T4isDsIi4pCWH7K8hKKEX6/GymatKMdsmrUuzplJ5h49imTqfem4qPu3ZFUh1WqypCgQ4XrmiwYKWrDLWEbfqZ/DtiXLKspMp0IvgKXk66vjJ8Xy3pRCYkSAVgvcl/SXDSbLzzzsbHXVNrDxoB7w2Turh5q/pMMr7UsNH5Y1OuJqM6k1wflLhkI1tu9wE30riGoi5QMFUQEDsux/33Fv5YqkYoEaMyJoLQWB6vhualMmKi9wPMj+Ua0DDMBJJfSH7PYU8YbezrOBu+QbbTcyJ+mJ2wcHDdNUHcAbzivUW0WshAxwndxMFBGWKt/8udEnBkAhYQYPVpN3TGZhPJUTYuJDDtAwoky9WhOZdRodGMer1r3NskvLRhl7s71+HSThLSBg8oOfwc3/lWPo5tZVI5Y7DkdjPiRalICdXJggh2INElbhQBE6eRfExmLBeoQxQArOuT uEtWK/yb smiTvvX1cGYvSY/tLOMOyqsad7xBQBEHkItxJypDeHwnp62ied/2JMhw4vgvTkjcydWI6uuv84+PB4Z0Z2sQ2KF4k8akb0iRX8la+f0zeWH8hStVD2bbpoM/a4/El9EmBSKoHA/Gs3GsJj56QVpMdGfUJK+SGdidZjYsT9gCf2b9sTMSkva8ADf6IKLaLOlar6urJr3tYt7Gr+Qo= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Hi David, On 4/9/2025 6:05 PM, David Hildenbrand wrote: > On 08.04.25 09:32, Rakie Kim wrote: >> The weighted interleave policy distributes page allocations across multiple >> NUMA nodes based on their performance weight, thereby improving memory >> bandwidth utilization. The weight values for each node are configured >> through sysfs. >> >> Previously, sysfs entries for configuring weighted interleave were created >> for all possible nodes (N_POSSIBLE) at initialization, including nodes that >> might not have memory. However, not all nodes in N_POSSIBLE are usable at >> runtime, as some may remain memoryless or offline. >> This led to sysfs entries being created for unusable nodes, causing >> potential misconfiguration issues. >> >> To address this issue, this patch modifies the sysfs creation logic to: >> 1) Limit sysfs entries to nodes that are online and have memory, avoiding >>     the creation of sysfs entries for nodes that cannot be used. >> 2) Support memory hotplug by dynamically adding and removing sysfs entries >>     based on whether a node transitions into or out of the N_MEMORY state. >> >> Additionally, the patch ensures that sysfs attributes are properly managed >> when nodes go offline, preventing stale or redundant entries from persisting >> in the system. >> >> By making these changes, the weighted interleave policy now manages its >> sysfs entries more efficiently, ensuring that only relevant nodes are >> considered for interleaving, and dynamically adapting to memory hotplug >> events. >> >> Signed-off-by: Rakie Kim >> Signed-off-by: Honggyu Kim >> Signed-off-by: Yunjeong Mun > > > Why are the other SOF in there? Are there Co-developed-by missing? I initially found the problem and fixed it with my internal implementation but Rakie also had his idea so he started working on it. His initial implementation has almost been similar to mine. I thought Signed-off-by is a way to express the patch series contains our contribution, but if you think it's unusual, then I can add this. Co-developed-by: Honggyu Kim Signed-off-by: Honggyu Kim For Yunjeong, the following can be added. Tested-by: Yunjeong Mun However, this patch series is already in Andrew's mm-new so I don't want to bother him again unless we need to update this patches for other reasons. Is this okay? Thanks, Honggyu > > > Acked-by: David Hildenbrand >