Skip to content

Backport patches for DAMON merge regions fix

SeongJae Park damon linux-mm lkml stable 1 version Applied

Status from this message: This is a note to let you know that I've just added the patch titled

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest7 patches 3 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/7minmax: sanity check constant bounds when clampinginclude/linux/minmax.h+24 -20 replies
2/7minmax: clamp more efficiently by avoiding extra comparisoninclude/linux/minmax.h+1 -10 replies
3/7minmax: fix header inclusionsinclude/linux/minmax.h+2 -00 replies
4/7minmax: allow min()/max()/clamp() if the arguments have the same signedness.include/linux/minmax.h+32 -280 replies
5/7minmax: allow comparisons of 'int' against 'unsigned char/short'include/linux/minmax.h+3 -20 replies
6/7minmax: relax check to allow comparison between unsigned arguments and signed constantsinclude/linux/minmax.h+17 -70 replies
7/7mm/damon/core: merge regions aggressively when max_nr_regions is unmetmm/damon/core.c+19 -21 reply

Reviews trailers per patch

2 of 7 patches have a Reviewed-by or Acked-by.

PatchAndy ShevchenkoHerve Codina
1/7R-b
2/7
3/7R-b
4/7
5/7
6/7
7/7

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 6.1.y 0/7] Backport patches for DAMON merge regions fix

Commit 310d6c15e910 ("mm/damon/core: merge regions aggressively when max_nr_regions") causes a build warning [1] on 6.1.y. That was due to unnecessarily strict type check from max().

Fix the warning by backporting a minmax.h upstream commit that made the type check less strict for unnecessary case, and upstream commits that it depends on.

Note that all patches except the third one ("minmax: fix header inclusions") are clean cherry-picks of upstream commit. For the third one, a minor conflict fix was needed.

[1] https://lore.kernel.org/2024071519-janitor-robe-779f@gregkh

Andy Shevchenko (1):
minmax: fix header inclusions

David Laight (3):
minmax: allow min()/max()/clamp() if the arguments have the same
signedness.
minmax: allow comparisons of 'int' against 'unsigned char/short'
minmax: relax check to allow comparison between unsigned arguments and
signed constants

Jason A. Donenfeld (2):
minmax: sanity check constant bounds when clamping
minmax: clamp more efficiently by avoiding extra comparison

SeongJae Park (1):
mm/damon/core: merge regions aggressively when max_nr_regions is unmet

 include/linux/minmax.h | 89 ++++++++++++++++++++++++++++++------------
 mm/damon/core.c        | 21 +++++++++-
 2 files changed, 83 insertions(+), 27 deletions(-)

base-commit: 291e563ecab1ea89c70172ecf0d6bff7b725d3cb

Signature · 1 line
2.39.2