All of lore.kernel.org
 help / color / mirror / Atom feed
From: sashiko-bot@kernel.org
To: "SJ Park" <sj@kernel.org>
Cc: damon@lists.linux.dev
Subject: Re: [RFC PATCH v4 3/7] mm/damon/sysfs-schemes: support quota goal complement flag
Date: Sun, 27 Sep 2026 12:19:52 +0000	[thread overview]
Message-ID: <20260927121953.2C2741F000FF@smtp.kernel.org> (raw)
In-Reply-To: <20260927120533.50484-4-sj@kernel.org>

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] damon/tests: build failure due to complementfalse typo
--

commit c13d1579fc49797d69a40d8e22286757e16cdd20
Author: SJ Park <sj@kernel.org>

mm/damon/core: add complement argument to damos_new_quota_goal()

This commit extends the damos_new_quota_goal() constructor to receive and
set the complement flag value. It also updates all API callers to use the
new signature to prevent manual configuration mistakes.

[ ... ]

> diff --git a/mm/damon/tests/core-kunit.h b/mm/damon/tests/core-kunit.h
> index df84d9cc7d204..20bba7421a406 100644
> --- a/mm/damon/tests/core-kunit.h
> +++ b/mm/damon/tests/core-kunit.h
> @@ -930,6 +930,7 @@ static void damos_test_commit_quota_goals_for(struct kunit *test,
>  		 * Make it kfree()-able.
>  		 */
>  		goal = damos_new_quota_goal(dst_goals[i].metric,
> +				dst_goals[i].complementfalse,

[Severity: Low]
Is this a typo? The struct damos_quota_goal does not have a complementfalse
member, which causes a build failure in damos_test_commit_quota_goals_for()
when the damon kunit test configuration is enabled.

>  				dst_goals[i].target_value);
>  		if (!goal)
>  			goto out;

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260927120533.50484-1-sj@kernel.org?part=3

  reply	other threads:[~2026-09-27 12:19 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-27 12:05 [RFC PATCH v4 0/7] snapshot_set_swap_area() unpins the previously selected swap device and SJ Park
2026-09-27 12:05 ` [RFC PATCH v4 1/7] mm/damon/core: introduce damos_quota_goal->complement SJ Park
2026-09-27 12:17   ` sashiko-bot
2026-09-27 12:22     ` SJ Park
2026-09-27 12:05 ` [RFC PATCH v4 2/7] mm/damon/core: add complement argument to damos_new_quota_goal() SJ Park
2026-09-27 12:12   ` sashiko-bot
2026-09-27 12:30     ` SJ Park
2026-09-27 12:05 ` [RFC PATCH v4 3/7] mm/damon/sysfs-schemes: support quota goal complement flag SJ Park
2026-09-27 12:19   ` sashiko-bot [this message]
2026-09-27 12:31     ` SJ Park
2026-09-27 12:05 ` [RFC PATCH v4 4/7] mm/damon/tests/core-kunit: test quota_goal->complement commit SJ Park
2026-09-27 12:13   ` sashiko-bot
2026-09-27 12:05 ` [RFC PATCH v4 5/7] Docs/mm/damon/design: document damos quota goal complement flag SJ Park
2026-09-27 12:13   ` sashiko-bot
2026-09-27 12:35     ` SJ Park
2026-09-27 12:05 ` [RFC PATCH v4 6/7] Docs/admin-guide/mm/damon/usage: update for quota goal complement file SJ Park
2026-09-27 12:13   ` sashiko-bot
2026-09-27 12:36     ` SJ Park
2026-09-27 12:05 ` [RFC PATCH v4 7/7] Docs/ABI/damon: update for quota goal metric complement sysfs file SJ Park
2026-09-27 12:14   ` sashiko-bot
2026-09-27 12:37     ` SJ Park
2026-09-27 13:05 ` [RFC PATCH v4 0/7] mm/damon: introduce damos quota goal target metric complement flag (was: "Re: [RFC PATCH v4 0/7] snapshot_set_swap_area() unpins the previously selected swap device and") SJ Park

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260927121953.2C2741F000FF@smtp.kernel.org \
    --to=sashiko-bot@kernel.org \
    --cc=damon@lists.linux.dev \
    --cc=sashiko-reviews@lists.linux.dev \
    --cc=sj@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.