From patchwork Fri Sep 16 15:20:35 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kaixu Xia X-Patchwork-Id: 12978656 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 7308DECAAD8 for ; Fri, 16 Sep 2022 15:22:57 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id CB71580007; Fri, 16 Sep 2022 11:22:56 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id C66428D0001; Fri, 16 Sep 2022 11:22:56 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B2CEE80007; Fri, 16 Sep 2022 11:22:56 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id A34DB8D0001 for ; Fri, 16 Sep 2022 11:22:56 -0400 (EDT) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 7F32D140DD9 for ; Fri, 16 Sep 2022 15:22:56 +0000 (UTC) X-FDA: 79918316352.04.F21722D Received: from mail-pf1-f170.google.com (mail-pf1-f170.google.com [209.85.210.170]) by imf24.hostedemail.com (Postfix) with ESMTP id 27BF51800D2 for ; Fri, 16 Sep 2022 15:22:56 +0000 (UTC) Received: by mail-pf1-f170.google.com with SMTP id a80so12982042pfa.4 for ; Fri, 16 Sep 2022 08:22:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=message-id:date:subject:cc:to:from:from:to:cc:subject:date; bh=8piKn5FnzNCRTWXKeKltQYBcUotQeP0xAHSm87BtmOg=; b=YKah748oMkqEZtDKOJdy63FjeXS3rfEh1bkzQ9bZPEQyZRWO+kERMHY6vVw6kAhfG3 e1CP8tw9FI/kXPKWTntaMb/QXYAHS5QC+wAOzJtYwfCorb1I+/PBrRkFEEp2qebmiZ3k aEJJqL0JLKwMe7JnitgX+aSZABWgKDbbxvxSvFRgKvnvefN18Ne9RaHUdAPpiOGpZjo9 yl4zR8qfzoinC8toiPtbgHoRmdRwSVr9WxFCrhLUKGCB0/o6Y+z1RvYj7fDPI1892c+5 Q2ddyEuKcKEEoEzCFiWAkUuuRCCnn/rlaBTJ1qu4DMZdbqDqFUmlffJ60paVzJ29gbfb ZBsg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date; bh=8piKn5FnzNCRTWXKeKltQYBcUotQeP0xAHSm87BtmOg=; b=oMyTv92FckHIwMG1xGjO9e/lxYL9PH+l1oabsdp4P4rR68TftEzaRD+g+FKr5meLu5 f8zRUh77ISbNygbQke7l0DnEZaVsC0XHIwOAtpnye6A/QlH3qmAw+59w/XppxFy0QdBC drLfX3O+vT6Ij/CN9WB4dQikxOzCVQ7NlC5lX2sWFEb4qHyw6KTH0UrgWpSQlIbqjJjP v3CbTluNvI8nPIq4qyCxdGC4+sJWfcTxyu3FJSrXPVf3drMP1hlys7pNmFBbGRmAcrla 2oNJ57azMcMMZi0I2vXvhWG0b96PCUX24vaPlg8wpNwiSfwQDTyZubL153dxACpie3Pw cPPA== X-Gm-Message-State: ACrzQf3WO8qAXc0w4XefDdjOt8UI1suI5pu4zULmpdC3XnB4wvgJs7k8 q+YG34K83mG1+Dxk85PJqg== X-Google-Smtp-Source: AMsMyM6xJ68WQvbE7MiwVujE3NnstM8TpzOuUmkp/ve1xHZ+2MhL2p85sgsiPNKOB//JCsfxBtwsaQ== X-Received: by 2002:a05:6a00:b4d:b0:540:cee1:657e with SMTP id p13-20020a056a000b4d00b00540cee1657emr5359168pfo.22.1663341774957; Fri, 16 Sep 2022 08:22:54 -0700 (PDT) Received: from localhost.localdomain ([43.132.141.8]) by smtp.gmail.com with ESMTPSA id 17-20020a630d51000000b0042fe1914e26sm13845173pgn.37.2022.09.16.08.22.52 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 16 Sep 2022 08:22:54 -0700 (PDT) From: xiakaixu1987@gmail.com X-Google-Original-From: kaixuxia@tencent.com To: sj@kernel.org, akpm@linux-foundation.org Cc: damon@lists.linux.dev, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Kaixu Xia Subject: [PATCH] mm/damon: return void from damon_set_schemes() Date: Fri, 16 Sep 2022 23:20:35 +0800 Message-Id: <1663341635-12675-1-git-send-email-kaixuxia@tencent.com> X-Mailer: git-send-email 1.8.3.1 ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1663341776; a=rsa-sha256; cv=none; b=IaNNd6Dbqumyei1WVQkWifOTnb33tX440G0GLehr9OxDaYBkzHugbLvU8fhSk1zi/6mtUr INbmTg1++WT3b/bz1XQWooSNiSpQIfqQ7GtkBpPfq2h/lmqJUD9vdJfvz3A23R56EB8jyZ edVUAk4agSsrPzHk92hUGisk6TkBR5Q= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=YKah748o; spf=pass (imf24.hostedemail.com: domain of xiakaixu1987@gmail.com designates 209.85.210.170 as permitted sender) smtp.mailfrom=xiakaixu1987@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1663341776; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references:dkim-signature; bh=8piKn5FnzNCRTWXKeKltQYBcUotQeP0xAHSm87BtmOg=; b=Bb/sDl2WSFDqTo9sk4esMNbcTyeFi7BAI/1qTQyhmbe/x8Kp4MhI0AaA46nQG/ZceRi4eA B/7grYKS31RWj76B1+72pEzxT+VlNrDAWwJujd8wT/pfz1LuWArmnGUyoecCcJOjBvmChy vWUdLtAD5RisLBaYR1p+UATERlfKqQM= X-Rspam-User: Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=YKah748o; spf=pass (imf24.hostedemail.com: domain of xiakaixu1987@gmail.com designates 209.85.210.170 as permitted sender) smtp.mailfrom=xiakaixu1987@gmail.com; dmarc=pass (policy=none) header.from=gmail.com X-Rspamd-Server: rspam03 X-Stat-Signature: 3tc4k5h6s61igjm1qypbqqphiipyr7c9 X-Rspamd-Queue-Id: 27BF51800D2 X-HE-Tag: 1663341776-506898 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000002, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: From: Kaixu Xia There is no point in returning an int from damon_set_schemes(). It always returns 0 which is meaningless for the caller, so change it to return void directly. Signed-off-by: Kaixu Xia Reviewed-by: SeongJae Park Reviewed-by: Muchun Song --- include/linux/damon.h | 2 +- mm/damon/core.c | 5 +---- mm/damon/dbgfs.c | 8 +++----- mm/damon/lru_sort.c | 4 +--- mm/damon/reclaim.c | 4 +--- 5 files changed, 7 insertions(+), 16 deletions(-) diff --git a/include/linux/damon.h b/include/linux/damon.h index 1dda8d0068e5..e7808a84675f 100644 --- a/include/linux/damon.h +++ b/include/linux/damon.h @@ -541,7 +541,7 @@ unsigned int damon_nr_regions(struct damon_target *t); struct damon_ctx *damon_new_ctx(void); void damon_destroy_ctx(struct damon_ctx *ctx); int damon_set_attrs(struct damon_ctx *ctx, struct damon_attrs *attrs); -int damon_set_schemes(struct damon_ctx *ctx, +void damon_set_schemes(struct damon_ctx *ctx, struct damos **schemes, ssize_t nr_schemes); int damon_nr_running_ctxs(void); bool damon_is_registered_ops(enum damon_ops_id id); diff --git a/mm/damon/core.c b/mm/damon/core.c index a843673c11cf..9c80c6eb00c2 100644 --- a/mm/damon/core.c +++ b/mm/damon/core.c @@ -454,10 +454,8 @@ int damon_set_attrs(struct damon_ctx *ctx, struct damon_attrs *attrs) * * This function should not be called while the kdamond of the context is * running. - * - * Return: 0 if success, or negative error code otherwise. */ -int damon_set_schemes(struct damon_ctx *ctx, struct damos **schemes, +void damon_set_schemes(struct damon_ctx *ctx, struct damos **schemes, ssize_t nr_schemes) { struct damos *s, *next; @@ -467,7 +465,6 @@ int damon_set_schemes(struct damon_ctx *ctx, struct damos **schemes, damon_destroy_scheme(s); for (i = 0; i < nr_schemes; i++) damon_add_scheme(ctx, schemes[i]); - return 0; } /** diff --git a/mm/damon/dbgfs.c b/mm/damon/dbgfs.c index c00eba4448d8..6f0ae7d3ae39 100644 --- a/mm/damon/dbgfs.c +++ b/mm/damon/dbgfs.c @@ -307,11 +307,9 @@ static ssize_t dbgfs_schemes_write(struct file *file, const char __user *buf, goto unlock_out; } - ret = damon_set_schemes(ctx, schemes, nr_schemes); - if (!ret) { - ret = count; - nr_schemes = 0; - } + damon_set_schemes(ctx, schemes, nr_schemes); + ret = count; + nr_schemes = 0; unlock_out: mutex_unlock(&ctx->kdamond_lock); diff --git a/mm/damon/lru_sort.c b/mm/damon/lru_sort.c index abfaf471e3e9..0abb9f8aa9af 100644 --- a/mm/damon/lru_sort.c +++ b/mm/damon/lru_sort.c @@ -203,9 +203,7 @@ static int damon_lru_sort_apply_parameters(void) scheme = damon_lru_sort_new_hot_scheme(hot_thres); if (!scheme) return -ENOMEM; - err = damon_set_schemes(ctx, &scheme, 1); - if (err) - return err; + damon_set_schemes(ctx, &scheme, 1); cold_thres = cold_min_age / damon_lru_sort_mon_attrs.aggr_interval; scheme = damon_lru_sort_new_cold_scheme(cold_thres); diff --git a/mm/damon/reclaim.c b/mm/damon/reclaim.c index 039fa55e0ae9..3d59ab11b7b3 100644 --- a/mm/damon/reclaim.c +++ b/mm/damon/reclaim.c @@ -155,9 +155,7 @@ static int damon_reclaim_apply_parameters(void) scheme = damon_reclaim_new_scheme(); if (!scheme) return -ENOMEM; - err = damon_set_schemes(ctx, &scheme, 1); - if (err) - return err; + damon_set_schemes(ctx, &scheme, 1); if (monitor_region_start > monitor_region_end) return -EINVAL;