mirror of
https://github.com/torvalds/linux.git
synced 2026-09-23 13:14:02 +02:00
mm/damon/core: allow esz to be set to zero
When the temporal quota goal tuner determines that the goal has been
achieved (score >= 10000), it sets esz_bp to zero so that the esz becomes
zero. However, damos_set_effective_quota() clamps the esz to
min_region_sz when quota->ms is set.
This is a minor issue, the main problem is that it doesn't match the
description in the documentation, which state that if the goal has already
been [over-]achieved, the quota will be set to zero.
Fix this by set quota (esz) as minimum as possible.
Link: https://lore.kernel.org/20260908135413.97570-1-sj@kernel.org
Fixes: 8bbde987c2 ("mm/damon/core: disallow time-quota setting zero esz")
Signed-off-by: SJ Park <sj@kernel.org>
Signed-off-by: Liew Rui Yan <aethernet65535@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Reviewed-by: SJ Park <sj@kernel.org>
Cc: <stable@vger.kernel.org> # v7.1.x
This commit is contained in:
parent
f166586f74
commit
90179da203
|
|
@ -3091,6 +3091,7 @@ static void damos_set_effective_quota(struct damon_ctx *ctx, struct damos *s)
|
|||
struct damos_quota *quota = &s->quota;
|
||||
unsigned long throughput;
|
||||
unsigned long esz = ULONG_MAX;
|
||||
unsigned long esz_time;
|
||||
|
||||
if (!quota->ms && list_empty("a->goals)) {
|
||||
quota->esz = quota->sz;
|
||||
|
|
@ -3111,8 +3112,8 @@ static void damos_set_effective_quota(struct damon_ctx *ctx, struct damos *s)
|
|||
1000000, quota->total_charged_ns);
|
||||
else
|
||||
throughput = PAGE_SIZE * 1024;
|
||||
esz = min(throughput * quota->ms, esz);
|
||||
esz = max(ctx->min_region_sz, esz);
|
||||
esz_time = max(throughput * quota->ms, ctx->min_region_sz);
|
||||
esz = min(esz_time, esz);
|
||||
}
|
||||
|
||||
if (quota->sz && quota->sz < esz)
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user