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 A162AC8303C for ; Mon, 7 Jul 2025 16:15:14 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E9AC48D0003; Mon, 7 Jul 2025 12:15:13 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id E72298D0002; Mon, 7 Jul 2025 12:15:13 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D87DA8D0003; Mon, 7 Jul 2025 12:15:13 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id CAC808D0002 for ; Mon, 7 Jul 2025 12:15:13 -0400 (EDT) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 1E81F801DC for ; Mon, 7 Jul 2025 16:15:13 +0000 (UTC) X-FDA: 83637968106.01.16521E6 Received: from tor.source.kernel.org (tor.source.kernel.org [172.105.4.254]) by imf24.hostedemail.com (Postfix) with ESMTP id 6F459180003 for ; Mon, 7 Jul 2025 16:15:11 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=NY3MigCe; spf=pass (imf24.hostedemail.com: domain of sj@kernel.org designates 172.105.4.254 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1751904911; 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-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=uOF07Oy2KrN4qN28xoqaDtrvCY/qnhgpM+9CY+ZxYnY=; b=DETYZv7JMNu4uSNIODnlpKoLxMeTsH6hwfmDbu6X6rBJbZ0J3ZobKW3fSR8sGKt7kBv09S 1oEpl8IFHd3MvLjEKlsvKBV81CwyaZLnhtIqAW/fVVQsT9395UlyoFo3d5Akdf/SXjZxNa KjBIG+x8xaO6kCXIE8SUORqi6PeuUkk= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1751904911; a=rsa-sha256; cv=none; b=C8jUpycEq124IriWXpq8d28hep0DACIVImkMA7vbTvbacxHeaFEGvMEh5WNQauZjiC0re+ g5tp46kvIPxMgZlZoHKJjmd4YMoZFVY2l+e9yT/gOZQyumhJsPk4A6T+/DiV+/VMldhcbj 5UYrCVyF1LW8fxk7mdAnItlOKOWQC24= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=NY3MigCe; spf=pass (imf24.hostedemail.com: domain of sj@kernel.org designates 172.105.4.254 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by tor.source.kernel.org (Postfix) with ESMTP id ACE7B60EDF; Mon, 7 Jul 2025 16:15:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 424A7C4CEE3; Mon, 7 Jul 2025 16:15:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1751904910; bh=T452YEkKfjvjzCiqbAyq6ODzZ4DafggrKpSyaGNIn2g=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=NY3MigCekCHgNC7E0HDea7EkLXDimGWDUnCmerTRZHIOBnIz4r+evfUii6KeBpLI+ MtUyMWB4HQ2c0Q78Ru1b+OpuEykAh8Q1XV46kAxj+QPhMaN/Zs5hWt8UKvmBkjZFre SCvpizuzJoL+Jhrw5vMT+cyyL7kUFtOz7t2036iG9+BID4GuO6eQa60uE+fpUZ246T XF0NhoLPv/T2JeG1pJiv5jvOJB9G7LoJgPKjHsXaphTzVCc/x5tjuno+mDA8wfcpi/ RA974taOjByGXVFhlybKClV68KLaA2sYe3zDKXl4owZAx5jhslZAtSpyR1djleKDRI FokQpCXaZbL6Q== From: SeongJae Park To: Yunjeong Mun Cc: SeongJae Park , akpm@linux-foundation.org, damon@lists.linux.dev, linux-mm@kvack.org, linux-kernel@vger.kernel.org, kernel_team@skhynix.com, honggyu.kim@sk.com Subject: Re: [PATCH v3] samples/damon: support automatic node address detection Date: Mon, 7 Jul 2025 09:15:08 -0700 Message-Id: <20250707161508.85341-1-sj@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20250707023135.453-1-yunjeong.mun@sk.com> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspam-User: X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 6F459180003 X-Stat-Signature: mzt8fh6cr8bxtkx9hk1ek933r3h8p7m5 X-HE-Tag: 1751904911-514126 X-HE-Meta: U2FsdGVkX18OPnuEb/WdH7ts12v2vSLeQP4CIwRLP608g34J6C+AOr2ntq26dtzZ6M3mT8STMGIR9b7l5VMtvqjX8Rz0pQCg+aAYpbvCR4Ps1FcRjeEojREPzLIUFNSSFOly/lR0uJcDftTdjsnlEAjhSLVCpwMhFg7SHhgi6WAcmaEw2PuuyFusHhuzqv6zdqqWyuUGpbkRUdCnNNbUQBW3XmKt+g7/yZYQuQKghZUS8psOLWUDSff0JApXHTecZ44RlHvgN/1pO7nbzvJjj4aWSZlE8KqU2uIqC9d6+tSUcVY5PJ1KNH0IMm2paRFa1hB9gD7deUTQ3T2g7DsbOmxR5qNS7pSc4Ugfe/mYFo21Q28fMx+d3dsEFuN9QjeOKkfvd0p+osAu3D49oi/FxRBfwqFHs0oHx6CQLhat0N6QPL7KUC8+HeYGCFwUGw00aud0Kv/5RppoTaq8NI3b8pZYdyK2lZ1XKnOPo72KkqojV14UYwWC1gpFxEMQyJGeXTs4cn0QTKF5dEhpheR1+m52T+/cxJCsxxL6EAYtofdjfCs1k8PtNSTq+ibk6oPxnsKhN42MN776O6k5ofMqIyNzSpbbRMSuOxDmWPJxH0+6SCTtSZj/YYDCxp3OVGYgAjENoNPoXasHpozNyPT4wZe+k8btHTrXJiLtHUeok7wP7ZtVw7aZgySu58BEI0nnr8pp9I7e/TsU5wxHa6cZ/74eLJzr3S8XRUxiJU2rlgT4M6Jvlo2B78/vqgQrLlqf5f1OLZ/eQfHqDkJi5/5ZKP8uw4ES3b/XVtdop7XQWxV/sqBnEjSHEWo25nYA149cqy44/NF93tYG1CV22h0ibFeVR0ewMWBlaH9Q26bMX2vlv/rQoxEJ6pehKM/XehyTAuxBywkFbCL/22YvrXy/tF63sIbDNarnYJtuDu9UKYGN7n4rrMzihVMgsE4Usm7rc5r5vfQOH3nssCcCjfC /y5F/6Vg KGEqzmBerJJUgh2hw5AmR+VDN+gMBzxjuz0w5W4YqepsPsWp7JxQ6Vp7fxRsuUkEYILJJc+C9rN/8eNZyxFIqgGJfZKgh+wLEzxYbYBzT1CY6BB3G0W1W64NQFDvBFZ0xKZndT87EBHpRaASdus52Qj9H+disBhLp6FiL5elKN6lmJNX+rpmm1iVKhGq+8dbLSBRpM1SIij3KA9XEmTYpDGhVaohHb9+x+8B6G9nC/1bQx0+ttySyr5o/RpAq6D6pJJebLERWwdJ/U74Ulr8aHGOxW0HHwBJEsxvB 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: Hello Yunjeong, On Mon, 7 Jul 2025 11:31:34 +0900 Yunjeong Mun wrote: > This patch adds a new knob `detect_node_addresses`, which determines > whether the physical address range is set manually using the existing > knobs or automatically by the mtier module. When `detect_node_addresses` > set to 'Y', mtier automatically converts node0 and node1 to their > physical addresses. If set to 'N', it uses the existing > 'node#_start_addr' and 'node#_end_addr' to define regions as before. > > Suggested-by: Honggyu Kim > Signed-off-by: Yunjeong Mun > --- > V2 -> V3: Removed an unusable comment > V1 -> V2: Removed second patch, added a new knob, and cleand up code Thank you for revisioning. Let's clearly mention the v1 and v2 were RFC versions. Also, from the next time, it would be greater if you could add links to the previous versions. For other reviewers, RFC v2: https://lore.kernel.org/20250703074423.1771-1-yunjeong.mun@sk.com RFC v1: https://lore.kernel.org/20250701085417.1734-1-yunjeong.mun@sk.com > --- > samples/damon/mtier.c | 41 ++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 38 insertions(+), 3 deletions(-) > > diff --git a/samples/damon/mtier.c b/samples/damon/mtier.c > index f3220d6e6739..3a6ae78efafe 100644 > --- a/samples/damon/mtier.c > +++ b/samples/damon/mtier.c > @@ -42,8 +42,33 @@ static bool enable __read_mostly; > module_param_cb(enable, &enable_param_ops, &enable, 0600); > MODULE_PARM_DESC(enable, "Enable of disable DAMON_SAMPLE_MTIER"); > > +static bool detect_node_addresses __read_mostly; > +module_param(detect_node_addresses, bool, 0600); > + > static struct damon_ctx *ctxs[2]; > > +struct region_range { > + phys_addr_t start; > + phys_addr_t end; > +}; > + > +static int nid_to_phys(int target_node, struct region_range *range) > +{ > + Let's drop this unnecessary blank line. > + if (!node_online(target_node)) { > + pr_err("NUMA node %d is not online\n", target_node); > + return -EINVAL; > + } > + > + unsigned long start_pfn = node_start_pfn(target_node); > + unsigned long end_pfn = node_end_pfn(target_node); Let's put declarations before executable statements, for the coding style consistency. > + > + range->start = PFN_PHYS(start_pfn); > + range->end = PFN_PHYS(end_pfn); Or, you could just skip the declarations and do something like below? range->start = PFN_PHYS(node_start_pfn(target_node)); > + > + return 0; > +} > + > static struct damon_ctx *damon_sample_mtier_build_ctx(bool promote) > { > struct damon_ctx *ctx; > @@ -53,6 +78,8 @@ static struct damon_ctx *damon_sample_mtier_build_ctx(bool promote) > struct damos *scheme; > struct damos_quota_goal *quota_goal; > struct damos_filter *filter; > + struct region_range addr; > + int ret; > > ctx = damon_new_ctx(); > if (!ctx) > @@ -82,9 +109,17 @@ static struct damon_ctx *damon_sample_mtier_build_ctx(bool promote) > if (!target) > goto free_out; > damon_add_target(ctx, target); > - region = damon_new_region( > - promote ? node1_start_addr : node0_start_addr, > - promote ? node1_end_addr : node0_end_addr); > + > + if (detect_node_addresses) { > + ret = promote ? nid_to_phys(1, &addr) : nid_to_phys(0, &addr); > + if (ret) > + goto free_out; > + } else { > + addr.start = promote ? node1_start_addr : node0_start_addr; > + addr.end = promote ? node1_end_addr : node0_end_addr; > + } > + > + region = damon_new_region(addr.start, addr.end); > if (!region) > goto free_out; > damon_add_region(region, target); > > base-commit: db16fe88cdf83a1e7fdf75de282025b6ad61d08f > -- > 2.34.1 Thanks, SJ