From patchwork Thu Feb 20 19:35:01 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: SeongJae Park X-Patchwork-Id: 13984434 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 6AC01C021B3 for ; Thu, 20 Feb 2025 19:35:19 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id D4A432802F9; Thu, 20 Feb 2025 14:35:18 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id CD2682802F3; Thu, 20 Feb 2025 14:35:18 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id AFCA02802F9; Thu, 20 Feb 2025 14:35:18 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 86BDB2802F3 for ; Thu, 20 Feb 2025 14:35:18 -0500 (EST) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 48FC282C2B for ; Thu, 20 Feb 2025 19:35:18 +0000 (UTC) X-FDA: 83141326716.09.819D0A3 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf24.hostedemail.com (Postfix) with ESMTP id A62E918000C for ; Thu, 20 Feb 2025 19:35:16 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="P8MMfv/t"; spf=pass (imf24.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 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=1740080116; 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=CU4FfTcGiLqlCmueCyOixm7z+uvbFV2ofQxB1flMyH4=; b=OtZz3RohSy6YDfunu94rXDMqiWtnO+yIYBs/TrwwxdkrjrhRNppW71dK/iLRFxfLWxqkot lzuYfytaTYK8edRgBpkcibHm9pOo1b5zC4ipNEimiUw8DjzVbTblsJjd1+FRwwWQnJZBEK T8PE0mWD9FHR90QQ18zMt/3kGj1dG2Q= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1740080116; a=rsa-sha256; cv=none; b=bb22HFDgwoAUFrOgDsPR5ux+l/FZ8Z4RHt2ylZw6VbtwgeITa3BMALZX9V96KzfumB10R/ kK6rUFi6MIkkAHBC6GdTIsGUyKVuyM/QLQm5n72ixq1752wEHZROaZNs/s+EYKvTYulekI 4J4H9YCT2CCWfchkugAleMP5olJZjHQ= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="P8MMfv/t"; spf=pass (imf24.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 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 dfw.source.kernel.org (Postfix) with ESMTP id 951B65C59B0; Thu, 20 Feb 2025 19:34:36 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3C4DCC4CED1; Thu, 20 Feb 2025 19:35:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1740080115; bh=vASrAL/pnHBXCkrPC/c/PavcC1+wtiSIVa/v3Hmh35w=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=P8MMfv/tOGNBm1TAxJCJDAx4QYLjeYuaj6aAYaqtgN0vsN33ugXGEIHaijFDoIquG hLi2NLkdNHlU/CFkARa3eUpElnZLE1UdG9kRTP7K0eHH9qBlhme2MEp0+wAEfP7+I0 t9G5Uy34RRTBzJSCbKuKIKfwqtoVj/Us5EY+LGCbn7qT2JbrkRV3Is/No+Iwq1lJVk ln3aK/2A4z2vLOp8Tik53VVrLURf9TjF9L3U1buOI51CMrFaRNs1w+9A+Xo2UCY5fh +ZTGB87uLddNh2LGGQCm133hhclagskCW+4JjKQ5t5U+lK4ajkxzxNNF8ibAfb1JWr u2qF/Qu63N8NA== From: SeongJae Park To: Cc: SeongJae Park , Andrew Morton , damon@lists.linux.dev, kernel-team@meta.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: [RFC PATCH 1/9] mm/damon/core: introduce damos->ops_filters Date: Thu, 20 Feb 2025 11:35:01 -0800 Message-Id: <20250220193509.36379-2-sj@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20250220193509.36379-1-sj@kernel.org> References: <20250220193509.36379-1-sj@kernel.org> MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Queue-Id: A62E918000C X-Rspamd-Server: rspam07 X-Stat-Signature: 34qnnyqqu1kxkf5rwat8cy66bfh6k6px X-HE-Tag: 1740080116-743528 X-HE-Meta: U2FsdGVkX180uiK4/1ayUz47KE5aNA+vTBAQPYEiN+zU2mM5enYWZeThuSpKt5a64F5daTXRvC6x7gQiu1ffqNpufrtTeytHvnZQFJ89f2u4eYvH+5Vc5ypbXwIxZqeHFUKy5i7BI7zmHY/TzUxpl4gq2mV7hAnzezZM+sEVmfxc3rLRoXphGpxdUzQA2CwBjMaCBd1sDNDv565KtdUC6ygBkwTa7PFJ9NweoHF04xYvnSFTXBytIUQhB6vzVR8NXyaJry1g4AaNy1HX4ZR4pjx+emfzstg2lFVwci2bDYlF6AGWgr7NJzb5BMVkqeyD4lDfPVZBhB6fU67YNQcvyEigG8LB6JxjPWEMfRGmoKch5NuB1GpOICMLropu+POaKm/sY8Wlc9N15cFk7muMSp5mqPycvQfTLVbFp6dhxvjD9alka0Lozuy3SX8Nb+s83PRAMnm9fElcN1AyePclDd7dwjW0b3FJ5Qj4himyLSw5BpInajJ5G9Jc2tp6odnG3P8GPVXUIj6MPTfPS6ncGkRCAz+lhBnyDlyxs84IIpxx6HJyqxxLamA+8pZOozd69kMa/g/Z8Vd2AVdtUKO7mvK3Ozdpb7zdMP3l1Ek199Gc7856TZdtZeUQXWRi5pQS9CWjABUqmSUjBbyfYyxJzBCJKemecEPUyuCjXQ9NAidi9PgxkKXqO7mXR23JpBPj4Vs9HQ5Uy2ItrvEozu+xN39dw5rtcoIjGW9ppEOl1PBa4dQi2c8zR7ePSG2H75cx7EXfgsI8xRFLaD0YkYZ7PJp4mJQqPUc9IGPX22GmUSz/BSnhb4+hR9L8LUF+9UdmSGadHk0y7qUamnq8i6wyBHCebE9oX6txmKQl0F2kN95pp8BRaasMu8aoKxLykIlQupqhHaEasDudJqmOuWhsxqf+QJqlfiBcm0zndRyhPWtXB8ZK9E0ObZ0jok1LRgbS3eIa8S5AUpktS/kQkR/ mKBdEfH9 zk0rWhntOUC5kz5Y0YWktOViU+TxwPii3fokQEAecMQHtQx756f+QMZo+4/1F8RZYoqOyajbm6GrUBQe8i2qhTyV7NgJF4T24FQVdELENVV066vz9SRZuk2sjvKTBMjAVtiiJw1J5EudMYL8BlgbMtA8S3rnZujgXF0ZZ5+GaTJRNmADYa4fAW0jXI+1yrMx2GwuCXXXck//Cd3J9cftrFUU0aU9OYGaGJe7ZNtJrfM0R8pJmGgt/1JD34nJWnZR3e0spgTAMJlPpkS+uM/Zvv7PG9Q== 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: DAMOS filters can be categorized into two groups depending on which layer they are handled, namely core layer and ops layer. Also their evaluation sequence is decided by the categorization. Currently, all filters are maintained in single list in mix. It makes the filters evaluation loop inefficient since it should do the categorization in the loop for every iteration. Introduce another list that will be used for having all operations layer-handled DAMOS filters. Note that this change simply adds the list and does not change any DAMON code to use it in real. It will be done by following changes. Signed-off-by: SeongJae Park --- include/linux/damon.h | 8 ++++++++ mm/damon/core.c | 1 + 2 files changed, 9 insertions(+) diff --git a/include/linux/damon.h b/include/linux/damon.h index 795ca09b1107..add82fdc1117 100644 --- a/include/linux/damon.h +++ b/include/linux/damon.h @@ -448,6 +448,7 @@ struct damos_access_pattern { * @wmarks: Watermarks for automated (in)activation of this scheme. * @target_nid: Destination node if @action is "migrate_{hot,cold}". * @filters: Additional set of &struct damos_filter for &action. + * @ops_filters: ops layer handling &struct damos_filter objects list. * @last_applied: Last @action applied ops-managing entity. * @stat: Statistics of this scheme. * @list: List head for siblings. @@ -503,6 +504,7 @@ struct damos { int target_nid; }; struct list_head filters; + struct list_head ops_filters; void *last_applied; struct damos_stat stat; struct list_head list; @@ -810,6 +812,12 @@ static inline unsigned long damon_sz_region(struct damon_region *r) #define damos_for_each_filter_safe(f, next, scheme) \ list_for_each_entry_safe(f, next, &(scheme)->filters, list) +#define damos_for_each_ops_filter(f, scheme) \ + list_for_each_entry(f, &(scheme)->ops_filters, list) + +#define damos_for_each_ops_filter_safe(f, next, scheme) \ + list_for_each_entry_safe(f, next, &(scheme)->ops_filters, list) + #ifdef CONFIG_DAMON struct damon_region *damon_new_region(unsigned long start, unsigned long end); diff --git a/mm/damon/core.c b/mm/damon/core.c index 38f545fea585..bcb7e42098dc 100644 --- a/mm/damon/core.c +++ b/mm/damon/core.c @@ -374,6 +374,7 @@ struct damos *damon_new_scheme(struct damos_access_pattern *pattern, */ scheme->next_apply_sis = 0; INIT_LIST_HEAD(&scheme->filters); + INIT_LIST_HEAD(&scheme->ops_filters); scheme->stat = (struct damos_stat){}; INIT_LIST_HEAD(&scheme->list);