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 799B3C6FA69 for ; Mon, 29 Aug 2022 17:02:26 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B34C5940008; Mon, 29 Aug 2022 13:02:25 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id AE07C940007; Mon, 29 Aug 2022 13:02:25 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9A893940008; Mon, 29 Aug 2022 13:02:25 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 88685940007 for ; Mon, 29 Aug 2022 13:02:25 -0400 (EDT) Received: from smtpin18.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 536BB801CA for ; Mon, 29 Aug 2022 17:02:25 +0000 (UTC) X-FDA: 79853248650.18.96B2705 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf09.hostedemail.com (Postfix) with ESMTP id D66C014005A for ; Mon, 29 Aug 2022 17:02:24 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id DD805612C8; Mon, 29 Aug 2022 17:02:23 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0587EC43145; Mon, 29 Aug 2022 17:02:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1661792543; bh=4pEZlSmJdDF1JN/EGNx6SD2ZgK0UVXZj72Ehq/Kq6Wg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=QkhAyAULGB4RzDU4hu99I6+7TSfXd8I9SkPztsg/c98fPqwc67F2XvyYDwyAjPUov 1bEYlyg7vaOpZda9i/hReJu95XiDpIL+OA1Vco+AzIuSDgIQ2xqE5fg9SyEMurX8Oc C2N1JEGTGvUJoVZhxpS+Op8AM0EEVwHTbXxYCgfPAbFdx92Ka4k6r0PhV3i28SuRLj 6h8uOgF14vr7QDl65zPSiJi7e/78oKKm4Dc6IhIHFxiCAb7krhiuZ+NvGCbkkMucaF xmVSp+lmsH92hOKOIEoscNcetcLmH192xHJPt/oXJLbdM+5Aj3IlHk+13bD5/kNCbh V6TnoYeNBAmNg== From: SeongJae Park To: xiakaixu1987@gmail.com Cc: sj@kernel.org, akpm@linux-foundation.org, damon@lists.linux.dev, linux-mm@kvack.org, linux-kernel@vger.kernel.org, Kaixu Xia Subject: Re: [PATCH] mm/damon: get the hotness from damon_hot_score() in damon_pageout_score() Date: Mon, 29 Aug 2022 17:02:19 +0000 Message-Id: <20220829170219.51527-1-sj@kernel.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <1661766366-20998-1-git-send-email-kaixuxia@tencent.com> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit ARC-Authentication-Results: i=1; imf09.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=QkhAyAUL; spf=pass (imf09.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1661792544; 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=9jL7RsL/H+L/b/mM/LmFxbKQc6UObIKTBcqX24KkRUQ=; b=JvAL60PslNfvrgqILQO+PU9lwZ8ReUeJGDXFcDMb0JebF3vGjrtNp2h0oNtACuup+TR6+T fglBCtNYtsRZKLtIEhU+Oe2ZQ7xtLpG5GM249EVqseYGDy2g+A2BsdeBdPv4dtRam5HYKk budQBqmI32o3Fs0gPUkxIao6STFZmik= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1661792544; a=rsa-sha256; cv=none; b=EgK9E0A0kXpR4rodH4lFTCT72e3g8qmHWSqcN8jKrA434wYkqTdeSxgpIh9hz/nI7nv54S W4Mce/mCyeLwd2qKYQgcKG4UT6OUf/v67EL85kpM/5xN5dA6K6cwq5ihpZgj1V7HeokFnx YRjnK/xugNrXx2wYMM5BlAmmIkhoDKw= X-Stat-Signature: coss941gaixodpp4w3u1oxcc5y1u6e85 X-Rspamd-Queue-Id: D66C014005A Authentication-Results: imf09.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=QkhAyAUL; spf=pass (imf09.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=none) header.from=kernel.org X-Rspam-User: X-Rspamd-Server: rspam01 X-HE-Tag: 1661792544-807001 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: Hi Kaixu, On Mon, 29 Aug 2022 17:46:06 +0800 xiakaixu1987@gmail.com wrote: > From: Kaixu Xia > > We can get the hotness value from damon_hot_score() directly in > damon_pageout_score() function and improve the code readability. > > Signed-off-by: Kaixu Xia Reviewed-by: SeongJae Park Thanks, SJ > --- > mm/damon/ops-common.c | 46 ++++++------------------------------------- > 1 file changed, 6 insertions(+), 40 deletions(-) > > diff --git a/mm/damon/ops-common.c b/mm/damon/ops-common.c > index b1335de200e7..f599838b5f64 100644 > --- a/mm/damon/ops-common.c > +++ b/mm/damon/ops-common.c > @@ -88,7 +88,7 @@ void damon_pmdp_mkold(pmd_t *pmd, struct mm_struct *mm, unsigned long addr) > #define DAMON_MAX_SUBSCORE (100) > #define DAMON_MAX_AGE_IN_LOG (32) > > -int damon_pageout_score(struct damon_ctx *c, struct damon_region *r, > +int damon_hot_score(struct damon_ctx *c, struct damon_region *r, > struct damos *s) > { > unsigned int max_nr_accesses; > @@ -127,48 +127,14 @@ int damon_pageout_score(struct damon_ctx *c, struct damon_region *r, > */ > hotness = hotness * DAMOS_MAX_SCORE / DAMON_MAX_SUBSCORE; > > - /* Return coldness of the region */ > - return DAMOS_MAX_SCORE - hotness; > + return hotness; > } > > -int damon_hot_score(struct damon_ctx *c, struct damon_region *r, > +int damon_pageout_score(struct damon_ctx *c, struct damon_region *r, > struct damos *s) > { > - unsigned int max_nr_accesses; > - int freq_subscore; > - unsigned int age_in_sec; > - int age_in_log, age_subscore; > - unsigned int freq_weight = s->quota.weight_nr_accesses; > - unsigned int age_weight = s->quota.weight_age; > - int hotness; > - > - max_nr_accesses = c->aggr_interval / c->sample_interval; > - freq_subscore = r->nr_accesses * DAMON_MAX_SUBSCORE / max_nr_accesses; > + int hotness = damon_hot_score(c, r, s); > > - age_in_sec = (unsigned long)r->age * c->aggr_interval / 1000000; > - for (age_in_log = 0; age_in_log < DAMON_MAX_AGE_IN_LOG && age_in_sec; > - age_in_log++, age_in_sec >>= 1) > - ; > - > - /* If frequency is 0, higher age means it's colder */ > - if (freq_subscore == 0) > - age_in_log *= -1; > - > - /* > - * Now age_in_log is in [-DAMON_MAX_AGE_IN_LOG, DAMON_MAX_AGE_IN_LOG]. > - * Scale it to be in [0, 100] and set it as age subscore. > - */ > - age_in_log += DAMON_MAX_AGE_IN_LOG; > - age_subscore = age_in_log * DAMON_MAX_SUBSCORE / > - DAMON_MAX_AGE_IN_LOG / 2; > - > - hotness = (freq_weight * freq_subscore + age_weight * age_subscore); > - if (freq_weight + age_weight) > - hotness /= freq_weight + age_weight; > - /* > - * Transform it to fit in [0, DAMOS_MAX_SCORE] > - */ > - hotness = hotness * DAMOS_MAX_SCORE / DAMON_MAX_SUBSCORE; > - > - return hotness; > + /* Return coldness of the region */ > + return DAMOS_MAX_SCORE - hotness; > } > -- > 2.27.0 >