Re: [PATCH 1/3] selftests/proc: test PROCMAP_QUERY ioctl while vma is concurrently modified

2025-08-01 Thread SeongJae Park
On Thu, 31 Jul 2025 15:00:22 -0700 Suren Baghdasaryan wrote: > Extend /proc/pid/maps tearing tests to verify PROCMAP_QUERY ioctl operation > correctness while the vma is being concurrently modified. > > Signed-off-by: Suren Baghdasaryan Acked-by: SeongJae Park Tested-by: S

[PATCH mm-stable] selftests/damon/sysfs.py: stop DAMON for dumping failures

2025-07-21 Thread SeongJae Park
robot Closes: https://lore.kernel.org/oe-lkp/202507220707.9c5d6247-...@intel.com Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 5 + 1 file changed, 5 insertions(+) diff --git a/tools/testing/selftests/damon/sysfs.py b/tools/testing/selftests/damon/sysfs.py index 4ff99db0d247.

Re: [PATCH 16/22] selftests/damon/sysfs.py: generalize DAMOS schemes commit assertion

2025-07-21 Thread SeongJae Park
On Mon, 21 Jul 2025 20:09:22 -0700 Andrew Morton wrote: > On Sun, 20 Jul 2025 10:16:46 -0700 SeongJae Park wrote: > > > DAMOS scheme commitment assertion is hard-coded for a specific test > > case. Split it out into a general version that can be reused for > &

[PATCH 20/22] selftests/damon/sysfs.py: generalize DAMON context commit assertion

2025-07-20 Thread SeongJae Park
DAMON context commitment assertion is hard-coded for a specific test case. Split it out into a general version that can be reused for different test cases. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 28 +++--- 1 file changed, 16 insertions

[PATCH 19/22] selftests/damon/sysfs.py: generalize monitoring attributes commit assertion

2025-07-20 Thread SeongJae Park
DAMON monitoring attributes commitment assertion is hard-coded for a specific test case. Split it out into a general version that can be reused for different test cases. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 42 +++--- 1 file changed, 25

[PATCH 22/22] selftests/damon/sysfs.py: test runtime reduction of DAMON parameters

2025-07-20 Thread SeongJae Park
sysfs.py is testing if non-default additional parameters can be committed. Add a test case for further reducing the parameters to the default set. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 14 ++ 1 file changed, 14 insertions(+) diff --git a/tools

[PATCH 21/22] selftests/damon/sysfs.py: test non-default parameters runtime commit

2025-07-20 Thread SeongJae Park
sysfs.py is testing only the default and minimum DAMON parameters. Add another test case for more non-default additional DAMON parameters commitment on runtime. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 53 ++ 1 file changed, 53

[PATCH 18/22] selftests/damon/sysfs.py: generalize DAMOS schemes commit assertion

2025-07-20 Thread SeongJae Park
DAMOS schemes commitment assertion is hard-coded for a specific test case. Split it out into a general version that can be reused for different test cases. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 10 ++ 1 file changed, 6 insertions(+), 4 deletions

[PATCH 17/22] selftests/damon/sysfs.py: test DAMOS filters commitment

2025-07-20 Thread SeongJae Park
Current DAMOS scheme commitment assertion is not testing DAMOS filters. Add the test. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 20 1 file changed, 20 insertions(+) diff --git a/tools/testing/selftests/damon/sysfs.py b/tools/testing

[PATCH 16/22] selftests/damon/sysfs.py: generalize DAMOS schemes commit assertion

2025-07-20 Thread SeongJae Park
DAMOS scheme commitment assertion is hard-coded for a specific test case. Split it out into a general version that can be reused for different test cases. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 59 -- 1 file changed, 37 insertions

[PATCH 14/22] selftests/damon/sysfs.py: test quota goal commitment

2025-07-20 Thread SeongJae Park
Current DAMOS quota commitment assertion is not testing quota goal commitment. Add the test. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/damon/sysfs.py b

[PATCH 15/22] selftests/damon/sysfs.py: test DAMOS destinations commitment

2025-07-20 Thread SeongJae Park
Current DAMOS commitment assertion is not testing quota destinations commitment. Add the test. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/tools/testing/selftests/damon/sysfs.py b

[PATCH 13/22] selftests/damon/sysfs.py: generalize DamosQuota commit assertion

2025-07-20 Thread SeongJae Park
DamosQuota commitment assertion is hard-coded for a specific test case. Split it out into a general version that can be reused for different test cases. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 31 +- 1 file changed, 20 insertions(+), 11

[PATCH 12/22] selftests/damon/sysfs.py: generalize DAMOS Watermarks commit assertion

2025-07-20 Thread SeongJae Park
DamosWatermarks commitment assertion is hard-coded for a specific test case. Split it out into a general version that can be reused for different test cases. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 26 ++ 1 file changed, 18 insertions

[PATCH 10/22] selftests/damon/drgn_dump_damon_status: dump ctx->ops.id

2025-07-20 Thread SeongJae Park
increasing test coverage difficult. Add ctx->ops.id dumping for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/drgn_dump_damon_status.py | 6 ++ 1 file changed, 6 insertions(+) diff --git a/tools/testing/selftests/damon/drgn_dump_damon_status.py b/tools/test

[PATCH 09/22] selftests/damon/drgn_dump_damon_status: dump damos->migrate_dests

2025-07-20 Thread SeongJae Park
increasing test coverage difficult. Add damos->migrate_dests dumping for more tests. Signed-off-by: SeongJae Park --- .../selftests/damon/drgn_dump_damon_status.py | 14 ++ 1 file changed, 14 insertions(+) diff --git a/tools/testing/selftests/damon/drgn_dump_damon_status.py b/to

[PATCH 11/22] selftests/damon/drgn_dump_damon_status: dump DAMOS filters

2025-07-20 Thread SeongJae Park
increasing test coverage difficult. Add damos filters dumping for more tests. Signed-off-by: SeongJae Park --- .../selftests/damon/drgn_dump_damon_status.py | 43 ++- 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/damon/drgn_dump_damon_status.py

[PATCH 08/22] selftests/damon/_damon_sysfs: use 2**32 - 1 as max nr_accesses and age

2025-07-20 Thread SeongJae Park
nr_accesses and age are unsigned int. Use the proper max value. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/testing/selftests/damon/_damon_sysfs.py b/tools/testing/selftests/damon

[PATCH 07/22] selftests/damon/_damon_sysfs: support DAMOS target_nid setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of DAMOS action destination target_nid setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 9 - 1 file changed, 8 insertions(+), 1 deletion

[PATCH 06/22] selftests/damon/_damon_sysfs: support DAMOS action dests setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of DAMOS action destinations setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 56 ++- 1 file changed, 55 insertions(+), 1 deletion

[PATCH 05/22] selftests/damon/_damon_sysfs: support DAMOS quota goal nid setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of DAMOS quota goal nid setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a

[PATCH 04/22] selftests/damon/_damon_sysfs: support DAMOS quota weights setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of DAMOS quotas prioritization weights setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 24 ++- 1 file changed, 23 insertions(+), 1

[PATCH 03/22] selftests/damon/_damon_sysfs: support monitoring intervals goal setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of the monitoring intervals auto-tune goal setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 43 ++- 1 file changed, 42 insertions

[PATCH 02/22] selftests/damon/_damon_sysfs: support DAMOS filters setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of DAMOS filters setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 115 +- 1 file changed, 111 insertions(+), 4 deletions(-) diff

[PATCH 01/22] selftests/damon/_damon_sysfs: support DAMOS watermarks setup

2025-07-20 Thread SeongJae Park
_damon_sysfs.py contains code for test-purpose DAMON sysfs interface control. Add support of DAMOS watermarks setup for more tests. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 46 +-- 1 file changed, 42 insertions(+), 4 deletions(-) diff

[PATCH 00/22] selftests/damon/sysfs.py: test all parameters

2025-07-20 Thread SeongJae Park
) extend drgn_dump_damon_status.py to dump full DAMON parameters. Following nine patches (12-20) refactor sysfs.py for general testing code reuse, and extend it for full parameters check. Finally, two patches (21 and 22) add test cases in sysfs.py for full parameters testing. SeongJae Park (22

Re: [PATCH v2] selftests/mm: Add process_madvise() tests

2025-06-30 Thread SeongJae Park
; Suggested-by: David Hildenbrand I left a few trivial comments below, but overall this looks useful to me. Thank you :) Acked-by: SeongJae Park FYI, 'git am' of this patch on latest mm-new fails. Maybe this is not based on the latest mm-new? But, I also found Andrew added this to m

[PATCH 6/6] selftests/damon/sysfs.py: test DAMOS schemes parameters setup

2025-06-28 Thread SeongJae Park
Add DAMON sysfs interface functionality tests for basic DAMOS schemes parameters setup. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 46 -- 1 file changed, 43 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/damon/sysfs.py

[PATCH 3/6] selftests/damon: add python and drgn-based DAMON sysfs test

2025-06-28 Thread SeongJae Park
s expected. The test is very minimum at the moment. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/Makefile | 1 + tools/testing/selftests/damon/sysfs.py | 42 ++ 2 files changed, 43 insertions(+) create mode 100755 tools/testing/selftests/damon/sysfs.py

[PATCH 5/6] selftests/damon/sysfs.py: test adaptive targets parameter

2025-06-28 Thread SeongJae Park
Add DAMON sysfs interface functionality tests for setup of basic adaptive targets parameters. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/damon/sysfs.py b/tools

[PATCH 4/6] selftests/damon/sysfs.py: test monitoring attribute parameters

2025-06-28 Thread SeongJae Park
Add DAMON sysfs interface functionality tests for DAMON monitoring attribute parameters, including intervals, intervals tuning goals, and min/max number of regions. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 34 -- 1 file changed, 32

[PATCH 2/6] selftests/damon/_damon_sysfs: set Kdamond.pid in start()

2025-06-28 Thread SeongJae Park
_damon_sysfs.py is a Python module for reading and writing DAMON sysfs for testing. It is not reading resulting kdamond pids. Read and update those when starting kdamonds. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 3 +++ 1 file changed, 3 insertions

[PATCH 1/6] selftests/damon: add drgn script for extracting damon status

2025-06-28 Thread SeongJae Park
an alternative tool in future. Note that the script is tested with 'drgn v0.0.22'. Signed-off-by: SeongJae Park --- .../selftests/damon/drgn_dump_damon_status.py | 161 ++ 1 file changed, 161 insertions(+) create mode 100755 tools/testing/selftests/damon/drgn_dump_damo

[PATCH 0/6] selftests/damon: add python and drgn based DAMON sysfs functionality tests

2025-06-28 Thread SeongJae Park
all DAMON parameters and input combinations but very basic things. Advancing the test infrastructure and adding more tests are future works. Changes from RFC (https://lore.kernel.org/20250622210330.40490-1...@kernel.org) - Rebase on latest mm-new SeongJae Park (6): selftests/damon: add drgn s

[RFC PATCH 6/6] selftests/damon/sysfs.py: test DAMOS schemes parameters setup

2025-06-22 Thread SeongJae Park
Add DAMON sysfs interface functionality tests for basic DAMOS schemes parameters setup. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 46 -- 1 file changed, 43 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/damon/sysfs.py

[RFC PATCH 2/6] selftests/damon/_damon_sysfs: set Kdamond.pid in start()

2025-06-22 Thread SeongJae Park
_damon_sysfs.py is a Python module for reading and writing DAMON sysfs for testing. It is not reading resulting kdamond pids. Read and update those when starting kdamonds. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_damon_sysfs.py | 3 +++ 1 file changed, 3 insertions

[RFC PATCH 3/6] selftests/damon: add python and drgn-based DAMON sysfs test

2025-06-22 Thread SeongJae Park
s expected. The test is very minimum at the moment. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/Makefile | 1 + tools/testing/selftests/damon/sysfs.py | 42 ++ 2 files changed, 43 insertions(+) create mode 100755 tools/testing/selftests/damon/sysfs.py

[RFC PATCH 0/6] selftests/damon: add python and drgn based DAMON sysfs functionality tests

2025-06-22 Thread SeongJae Park
all DAMON parameters and input combinations but very basic things. Advancing the test infrastructure and adding more tests are future works. SeongJae Park (6): selftests/damon: add drgn script for extracting damon status selftests/damon/_damon_sysfs: set Kdamond.pid in start() selftests/damon:

[RFC PATCH 5/6] selftests/damon/sysfs.py: test adaptive targets parameter

2025-06-22 Thread SeongJae Park
Add DAMON sysfs interface functionality tests for setup of basic adaptive targets parameters. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/damon/sysfs.py b/tools

[RFC PATCH 4/6] selftests/damon/sysfs.py: test monitoring attribute parameters

2025-06-22 Thread SeongJae Park
Add DAMON sysfs interface functionality tests for DAMON monitoring attribute parameters, including intervals, intervals tuning goals, and min/max number of regions. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/sysfs.py | 34 -- 1 file changed, 32

[RFC PATCH 1/6] selftests/damon: add drgn script for extracting damon status

2025-06-22 Thread SeongJae Park
an alternative tool in future. Note that the script is tested with 'drgn v0.0.22'. Signed-off-by: SeongJae Park --- .../selftests/damon/drgn_dump_damon_status.py | 161 ++ 1 file changed, 161 insertions(+) create mode 100755 tools/testing/selftests/damon/drgn_dump_damo

[PATCH 2/2] selftets/damon: add a test for memcg_path leak

2025-06-19 Thread SeongJae Park
There was a memory leak bug in DAMOS sysfs memcg_path file. Add a selftest to ensure the bug never comes back. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/Makefile| 1 + .../selftests/damon/sysfs_memcg_path_leak.sh | 43 +++ 2 files changed, 44

[PATCH 0/2] mm/damon: fix memory leak in memcg_path sysfs file

2025-06-19 Thread SeongJae Park
Users can leak memory by repeatedly writing a string to DAMOS sysfs memcg_path file. Fix it (patch 1) and add a selftest (patch 2) to avoid reoccurrance of the bug. SeongJae Park (2): mm/damon/sysfs-schemes: free old damon_sysfs_scheme_filter->memcg_path on write selftets/damon: ad

[PATCH 4/6] mm/damon/tests/core-kunit: add a test for damos_set_filters_default_reject()

2025-05-12 Thread SeongJae Park
DAMOS filters' default reject behavior is not very simple. Actually there was a mistake[1] during the development. Add a kunit test for validating the behavior. [1] https://lore.kernel.org/20250227002913.19359-1...@kernel.org Signed-off-by: SeongJae Park --- mm/damon/tests/core-kunit.h

[PATCH 5/6] selftests/damon/_damon_sysfs: read tried regions directories in order

2025-05-12 Thread SeongJae Park
Kdamond.update_schemes_tried_regions() reads and stores tried regions information out of address order. It makes debugging a test failure difficult. Change the behavior to do the reading and writing in the address order. Signed-off-by: SeongJae Park --- tools/testing/selftests/damon

[PATCH 1/3] selftests/damon/damos_quota: make real expectation of quota exceeds

2025-02-25 Thread SeongJae Park
. Expect how much time the quota will be exceeded by checking the monitoring results, and use it instead of the naive assumption. Fixes: 51f58c9da14b ("selftests/damon: add a test for DAMOS quota") Cc: Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/damos_qu

[PATCH 2/3] selftests/damon/damon_nr_regions: set ops update for merge results check to 100ms

2025-02-25 Thread SeongJae Park
test online-tuned max_nr_regions") Cc: Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/damon_nr_regions.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests/damon/damon_nr_regions.py b/tools/testing/selftests/damon/damon_nr_regions.py index 2e

[PATCH 3/3] selftests/damon/damon_nr_regions: sort collected regiosn before checking with min/max boundaries

2025-02-25 Thread SeongJae Park
list before doing the check. Fixes: 781497347d1b ("selftests/damon: implement test for min/max_nr_regions") Cc: Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/damon_nr_regions.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests/damon/damon_nr_reg

[PATCH 0/3] selftests/damon: three fixes for false results

2025-02-25 Thread SeongJae Park
Fix three DAMON selftest bugs that causes two and one false positive failures and success. SeongJae Park (3): selftests/damon/damos_quota: make real expectation of quota exceeds selftests/damon/damon_nr_regions: set ops update for merge results check to 100ms selftests/damon

[PATCH] selftests/damon/damos_quota_goal: handle minimum quota that cannot be further reduced

2025-02-17 Thread SeongJae Park
DAMOS quota goal") Cc: # 6.10.x Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-lkp/202502171423.b28a918d-...@intel.com Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/damos_quota_goal.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/testing

Re: next-20250210: WARNING: at include/linux/rwsem.h:85 madvise_unlock

2025-02-11 Thread SeongJae Park
seen on next-20250210 > > Good: next-20250207 > > Bad: next-20250210 > > Cc SeongJae Park [...] > This is mmap_write_unlock() triggering > rwsem_assert_held_write(&mm->mmap_lock); > > That code was changed in these commits: > > 415553982a1e > mm-madvi

[PATCH v2 7/8] mm/damon: remove DAMON debugfs interface kunit tests

2025-01-06 Thread SeongJae Park
It's time to remove DAMON debugfs interface, which has deprecated long before in February 2023. Read the cover letter of this patch series for more details. Remove kunit tests for the interface, to prevent unnecessary test failures. Signed-off-by: SeongJae Park --- mm/damon/Kc

[PATCH v2 6/8] kunit: configs: remove configs for DAMON debugfs interface tests

2025-01-06 Thread SeongJae Park
oises from tests. Signed-off-by: SeongJae Park --- tools/testing/kunit/configs/all_tests.config | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/testing/kunit/configs/all_tests.config b/tools/testing/kunit/configs/all_tests.config index b3b00269a52a..b0049be00c70 100644 --- a/tools/te

[PATCH v2 5/8] selftests/damon: remove tests for DAMON debugfs interface

2025-01-06 Thread SeongJae Park
It's time to remove DAMON debugfs interface, which has deprecated long before in February 2023. Read the cover letter of this patch series for more details. Remove selftests for the interface, to prevent causing unnecessary test failures. Signed-off-by: SeongJae Park --- tools/te

[PATCH v2 4/8] selftests/damon/config: remove configs for DAMON debugfs interface selftests

2025-01-06 Thread SeongJae Park
https://lore.kernel.org/20230209192009.7885-1...@kernel.org Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/config | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/testing/selftests/damon/config b/tools/testing/selftests/damon/config index 0daf38974eb0..a68a9fead5dc 100644 --- a/

[PATCH 6/7] mm/damon: remove DAMON debugfs interface kunit tests

2025-01-01 Thread SeongJae Park
It's time to remove DAMON debugfs interface, which has deprecated long before in February 2023. Read the cover letter of this patch series for more details. Remove kunit tests for the interface, to prevent unnecessary test failures. Signed-off-by: SeongJae Park --- mm/damon/Kc

[PATCH 4/7] selftests/damon: remove tests for DAMON debugfs interface

2025-01-01 Thread SeongJae Park
It's time to remove DAMON debugfs interface, which has deprecated long before in February 2023. Read the cover letter of this patch series for more details. Remove selftests for the interface, to prevent causing unnecessary test failures. Signed-off-by: SeongJae Park --- tools/te

[PATCH 5/7] kunit: configs: remove configs for DAMON debugfs interface tests

2025-01-01 Thread SeongJae Park
oises from tests. Signed-off-by: SeongJae Park --- tools/testing/kunit/configs/all_tests.config | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/testing/kunit/configs/all_tests.config b/tools/testing/kunit/configs/all_tests.config index b3b00269a52a..b0049be00c70 100644 --- a/tools/te

[PATCH 3/7] selftests/damon/config: remove configs for DAMON debugfs interface selftests

2025-01-01 Thread SeongJae Park
https://lore.kernel.org/20230209192009.7885-1...@kernel.org Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/config | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/testing/selftests/damon/config b/tools/testing/selftests/damon/config index 0daf38974eb0..a68a9fead5dc 100644 --- a/

Re: [PATCH] selftests/damon: Add _damon_sysfs.py to TEST_FILES

2024-11-27 Thread SeongJae Park
06abb63a8ca ("selftests/damon: implement a python module for > test-purpose DAMON sysfs controls") > Signed-off-by: Maximilian Heyne Reviewed-by: SeongJae Park Thanks, SJ > --- > tools/testing/selftests/damon/Makefile | 2 +- > 1 file changed, 1 insertion(+),

[PATCH 6/6] mm/damon/tests/dbgfs-kunit: fix the header double inclusion guarding ifdef comment

2024-10-28 Thread SeongJae Park
Closing part of double inclusion guarding macro for dbgfs-kunit.h was copy-pasted from somewhere (maybe before the initial mainline merge of DAMON), and not properly updated. Fix it. Fixes: 17ccae8bb5c9 ("mm/damon: add kunit tests") Signed-off-by: SeongJae Park --- mm/damon/tests/dbg

[PATCH 3/6] selftests/damon/_debugfs_common: hide expected error message from test_write_result()

2024-10-28 Thread SeongJae Park
ot;mm/damon: add user space selftests") Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/_debugfs_common.sh | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/damon/_debugfs_common.sh b/tools/testing/selftests/damon/_debugfs_commo

[PATCH 4/6] selftests/damon/debugfs_duplicate_context_creation: hide errors from expected file write failures

2024-10-28 Thread SeongJae Park
ation") Signed-off-by: SeongJae Park --- .../selftests/damon/debugfs_duplicate_context_creation.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/damon/debugfs_duplicate_context_creation.sh b/tools/testing/selft

[PATCH 0/6] damon/{self,kunit}tests: minor fixups for DAMON debugfs interface tests

2024-10-28 Thread SeongJae Park
(patches 3 and 4). Following two patches fix copy-paste mistakes in DAMON Kconfig help message that copied from debugfs kunit test (patch 5) and a comment on the debugfs kunit test code (patch 6). Signed-off-by: SeongJae Park Andrew Paniakin (1): selftests/damon/huge_count_read_write: provide

[PATCH 2/6] selftests/damon/huge_count_read_write: remove unnecessary debugging message

2024-10-28 Thread SeongJae Park
The program prints expected errors from write/read of the files with invalid huge count, for only debugging purpose. It is only making the output noisy. Remove those. Fixes: b4a002889d24 ("selftests/damon: test debugfs file reads/writes with huge count") Signed-off-by: Seo

[PATCH 1/6] selftests/damon/huge_count_read_write: provide sufficiently large buffer for DEPRECATED file read

2024-10-28 Thread SeongJae Park
characters. Fixes: b4a002889d24 ("selftests/damon: test debugfs file reads/writes with huge count") Signed-off-by: Andrew Paniakin Signed-off-by: SeongJae Park --- Note that this fix has originally wrote[1] by Andrew for the downstream version of the same test. Because the downstream

Re: [PATCH v2 2/2] mm/damon/vaddr: Add 'nr_piece == 1' check in damon_va_evenly_split_region()

2024-10-22 Thread SeongJae Park
ces == 1, no actual split is required. Check that case > for better code readability and add a simple kunit testcase. > > [1] https://lore.kernel.org/all/20241021163316.12443-1...@kernel.org/ > > Signed-off-by: Zheng Yejian Reviewed-by: SeongJae Park Thanks, SJ [...]

Re: [PATCH v2 1/2] mm/damon/vaddr: Fix issue in damon_va_evenly_split_region()

2024-10-22 Thread SeongJae Park
y_three_regions4 > [PASSED] damon_test_split_evenly > [PASSED] damon-operations = > > Fixes: 3f49584b262c ("mm/damon: implement primitives for the virtual memory > address spaces") > Signed-off-by: Zheng Yejian Reviewed-by: SeongJae Park Thanks, SJ [...]

[RFC PATCH 6/7] mm/damon: remove DAMON debugfs interface kunit tests

2024-10-15 Thread SeongJae Park
It's time to remove DAMON debugfs interface, which has deprecated long before in February 2023. Read the cover letter of this patch series for more details. Remove kunit tests for the interface, to prevent unnecessary test failures. Signed-off-by: SeongJae Park --- mm/damon/Kc

[RFC PATCH 5/7] kunit: configs: remove configs for DAMON debugfs interface tests

2024-10-15 Thread SeongJae Park
oises from tests. Signed-off-by: SeongJae Park --- tools/testing/kunit/configs/all_tests.config | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/testing/kunit/configs/all_tests.config b/tools/testing/kunit/configs/all_tests.config index b3b00269a52a..b0049be00c70 100644 --- a/tools/te

[RFC PATCH 4/7] selftests/damon: remove tests for DAMON debugfs interface

2024-10-15 Thread SeongJae Park
It's time to remove DAMON debugfs interface, which has deprecated long before in February 2023. Read the cover letter of this patch series for more details. Remove selftests for the interface, to prevent causing unnecessary test failures. Signed-off-by: SeongJae Park --- tools/te

[RFC PATCH 3/7] selftests/damon/config: remove configs for DAMON debugfs interface selftests

2024-10-15 Thread SeongJae Park
https://lore.kernel.org/20230209192009.7885-1...@kernel.org Signed-off-by: SeongJae Park --- tools/testing/selftests/damon/config | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/testing/selftests/damon/config b/tools/testing/selftests/damon/config index 0daf38974eb0..a68a9fead5dc 100644 --- a/

Re: [PATCH v2] selftests/damon/access_memory_even: remove unused variables

2024-09-29 Thread SeongJae Park
From: s...@kernel.org On Mon, 23 Sep 2024 21:08:45 -0700 SeongJae Park wrote: > Hi Ba, > > > Thank you for revising this patch following my comment. > > On Tue, 24 Sep 2024 10:14:26 +0800 Ba Jing > wrote: > > > By reading the code, I found these variables are ne

Re: [PATCH v1 1/1] selftests: set executable bit

2024-09-24 Thread SeongJae Park
Hi David, On Tue, 24 Sep 2024 13:54:57 -0400 David Hunter wrote: > Turn on the executable bit for the following script files. These scripts > are set to TEST_PROGS in their respective Makefiles, but currently, when > these tests are run, a warning occurs: > > # Warning: is not executable T

Re: [PATCH v2] selftests/damon/access_memory_even: remove unused variables

2024-09-23 Thread SeongJae Park
Hi Ba, Thank you for revising this patch following my comment. On Tue, 24 Sep 2024 10:14:26 +0800 Ba Jing wrote: > By reading the code, I found these variables are never > referenced in the code. Just remove them. > > Signed-off-by: Ba Jing Reviewed-by: SeongJae Park

Re: [PATCH] damon: access_memory_even: remove unused variables

2024-09-23 Thread SeongJae Park
g Other than the above trivial nitpick, Reviewed-by: SeongJae Park Thanks, SJ > --- > tools/testing/selftests/damon/access_memory_even.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/tools/testing/selftests/damon/access_memory_even.c > b/tools/testing/se

[PATCH 1/1] mm/damon/tests/core-kunit: skip damon_test_nr_accesses_to_accesses_bp() if aggr_interval is zero

2024-09-05 Thread SeongJae Park
Roeck Closes: https://lore.kernel.org/c771b962-a58f-435b-89e4-1211a9323...@roeck-us.net Fixes: 5e06ad590096 ("mm/damon/core-test: test max_nr_accesses overflow caused divide-by-zero") Signed-off-by: SeongJae Park --- mm/damon/core.c | 8 +++- mm/damon/tests/co

Re: [PATCH] mm/damon/tests/vaddr-kunit: don't use mas_lock for MM_MT_FLAGS-initialized maple tree

2024-09-04 Thread SeongJae Park
On Wed, 4 Sep 2024 12:56:33 -0700 Guenter Roeck wrote: > On 9/4/24 12:26, Liam R. Howlett wrote: > > * Guenter Roeck [240904 00:27]: > >> On 9/3/24 20:36, Liam R. Howlett wrote: > >>> * Guenter Roeck [240903 22:38]: > >>>> On 9/3/24 19:31, L

Re: [PATCH v2] mm/damon/tests/vaddr-kunit: init maple tree without MT_FLAGS_LOCK_EXTERN

2024-09-04 Thread SeongJae Park
On Wed, 4 Sep 2024 11:41:28 -0700 Guenter Roeck wrote: > On 9/4/24 10:29, SeongJae Park wrote: > > damon_test_three_regions_in_vmas() initializes a maple tree with > > MM_MT_FLAGS. The flags contains MT_FLAGS_LOCK_EXTERN, which means > > mt_lock of the maple tree w

Re: [PATCH] mm/damon/tests/vaddr-kunit: don't use mas_lock for MM_MT_FLAGS-initialized maple tree

2024-09-04 Thread SeongJae Park
On Tue, 3 Sep 2024 22:43:40 -0400 "Liam R. Howlett" wrote: > * Guenter Roeck [240903 21:54]: > > On 9/3/24 18:18, SeongJae Park wrote: > > > On Tue, 3 Sep 2024 17:58:15 -0700 SeongJae Park wrote: > > > > > > > On Tue, 3 Sep 2024

[PATCH v2] mm/damon/tests/vaddr-kunit: init maple tree without MT_FLAGS_LOCK_EXTERN

2024-09-04 Thread SeongJae Park
47f0d ("damon: convert __damon_va_three_regions to use the VMA iterator") Suggested-by: Liam R. Howlett Signed-off-by: SeongJae Park --- Changes from v1 (https://lore.kernel.org/20240904004534.1189-1...@kernel.org) - Keep lock usage and update the initialization flags (Liam) - Fix a typo

Re: [PATCH] mm/damon/tests/vaddr-kunit: don't use mas_lock for MM_MT_FLAGS-initialized maple tree

2024-09-03 Thread SeongJae Park
On Tue, 3 Sep 2024 17:58:15 -0700 SeongJae Park wrote: > On Tue, 3 Sep 2024 20:48:53 -0400 "Liam R. Howlett" > wrote: > > > * SeongJae Park [240903 20:45]: > > > damon_test_three_regions_in_vmas() initializes a maple tree with > > > MM_MT_FLAG

Re: [PATCH] mm/damon/tests/vaddr-kunit: don't use mas_lock for MM_MT_FLAGS-initialized maple tree

2024-09-03 Thread SeongJae Park
On Tue, 3 Sep 2024 20:48:53 -0400 "Liam R. Howlett" wrote: > * SeongJae Park [240903 20:45]: > > damon_test_three_regions_in_vmas() initializes a maple tree with > > MM_MT_FLAGS. The flags contains MT_FLAGS_LOCK_EXTERN, which means > > mt_lock of the ma

[PATCH] mm/damon/tests/vaddr-kunit: don't use mas_lock for MM_MT_FLAGS-initialized maple tree

2024-09-03 Thread SeongJae Park
: https://lore.kernel.org/1453b2b2-6119-4082-ad9e-f3c5239bf...@roeck-us.net Fixes: d0cf3dd47f0d ("damon: convert __damon_va_three_regions to use the VMA iterator") Signed-off-by: SeongJae Park --- mm/damon/tests/vaddr-kunit.h | 10 +++--- 1 file changed, 3 insertions(+), 7 deletion

Re: [PATCH v6 0/7] DAMON based tiered memory management for CXL memory

2024-06-14 Thread SeongJae Park
> > Each test result is based on the exeuction environment as follows. Seems the typo is not fixed? I don't want to delay this work for such trivial thing, though. For the patch series, Acked-by: SeongJae Park Thanks, SJ [...]

Re: [PATCH v6 7/7] Docs/damon: document damos_migrate_{hot,cold}

2024-06-14 Thread SeongJae Park
> Signed-off-by: Honggyu Kim Reviewed-by: SeongJae Park Thanks, SJ [...]

Re: [PATCH v4 12/35] kmsan: Support SLAB_POISON

2024-06-13 Thread SeongJae Park
Hi Ilya, On Thu, 13 Jun 2024 17:34:14 +0200 Ilya Leoshkevich wrote: > Avoid false KMSAN negatives with SLUB_DEBUG by allowing > kmsan_slab_free() to poison the freed memory, and by preventing > init_object() from unpoisoning new allocations by using __memset(). > > There are two alternatives to

Re: [PATCH v5 0/8] DAMON based tiered memory management for CXL memory

2024-06-13 Thread SeongJae Park
Hi Honggyu, On Thu, 13 Jun 2024 22:20:47 +0900 Honggyu Kim wrote: > There was an RFC IDEA "DAMOS-based Tiered-Memory Management" previously > posted at [1]. > > It says there is no implementation of the demote/promote DAMOS action > are made. This patch series is about its implementation for p

Re: [PATCH v5 8/8] Docs/.../mm/damon: add more damos actions

2024-06-13 Thread SeongJae Park
On Thu, 13 Jun 2024 07:07:29 -0700 SeongJae Park wrote: > Hi Honggyu, > > On Thu, 13 Jun 2024 22:20:55 +0900 Honggyu Kim wrote: > > > This patch adds damon description for "migrate_hot" and "migrate_cold" > > actions for both usage and design docu

Re: [PATCH v5 2/8] mm: rename alloc_demote_folio to alloc_migrate_folio

2024-06-13 Thread SeongJae Park
On Thu, 13 Jun 2024 22:20:49 +0900 Honggyu Kim wrote: > The alloc_demote_folio can also be used for general migration including > both demotion and promotion so it'd be better to rename it from > alloc_demote_folio to alloc_migrate_folio. > > Signed-off-by: Honggyu Kim R

Re: [PATCH v5 8/8] Docs/.../mm/damon: add more damos actions

2024-06-13 Thread SeongJae Park
Hi Honggyu, On Thu, 13 Jun 2024 22:20:55 +0900 Honggyu Kim wrote: > This patch adds damon description for "migrate_hot" and "migrate_cold" > actions for both usage and design documents as long as a new > "target_nid" knob to set the migration target node. > > Signed-off-by: Honggyu Kim > --- >

Re: [PATCH v5 7/8] Docs/admin-guide/mm/damon/usage: add missing actions

2024-06-13 Thread SeongJae Park
Hi Honggyu, On Thu, 13 Jun 2024 22:20:54 +0900 Honggyu Kim wrote: > "lru_prio" and "lru_deprio" are missing in the damos action list so they > have to be added properly at damon usage document. > > Signed-off-by: Honggyu Kim > --- > Documentation/admin-guide/mm/damon/usage.rst | 4 +++- > 1 f

Re: [PATCH 0/8] DAMON based tiered memory management for CXL memory

2024-06-13 Thread SeongJae Park
On Thu, 13 Jun 2024 22:27:26 +0900 Honggyu Kim wrote: > On Thu, 13 Jun 2024 22:17:31 +0900 Honggyu Kim wrote: > > There was an RFC IDEA "DAMOS-based Tiered-Memory Management" previously > > posted at [1]. > > > > It says there is no implementation of the demote/promote DAMOS action > > are made.

Re: [RFC PATCH v4 0/5] DAMON based tiered memory management for CXL memory

2024-05-15 Thread SeongJae Park
malloc No problem at all. We all work on our own schedule and nobody can chase/push anybody :) > > On Sun, 12 May 2024 10:54:42 -0700 SeongJae Park wrote: > > There was an RFC IDEA "DAMOS-based Tiered-Memory Management" previously > > posted at [1]. > >

[RFC IDEA v2 6/6] drivers/virtio/virtio_balloon: integrate ACMA and ballooning

2024-05-12 Thread SeongJae Park
s and put it into the balloon. As a result, the host can use the requested amount of guest memory, so from the host's perspective, the ballooning just works, but in transparent and access/contiguity-aware way. Signed-off-by: SeongJae Park --- drivers/virtio/virtio_ball

[RFC IDEA v2 0/6] mm/damon: introduce Access/Contiguity-aware Memory Auto-scaling (ACMA)

2024-05-12 Thread SeongJae Park
ging. We could use ACMA-like approach for dynamically allocating a memory pool for contiguous memory allocation. It will be similar to ACMA but do not report DAMOS-alloc-ed pages to the host. Instead, use the regions as contiguous memory allocation pool. DRAM Consuming Power Saving ---

Re: [RFC PATCH v3 1/7] mm/damon/paddr: refactor DAMOS_PAGEOUT with migration_mode

2024-05-12 Thread SeongJae Park
On Sat, 11 May 2024 13:16:17 -0700 SeongJae Park wrote: > On Fri, 5 Apr 2024 12:19:07 -0700 SeongJae Park wrote: > > > On Fri, 5 Apr 2024 15:08:50 +0900 Honggyu Kim wrote: > > > > > This is a preparation patch that introduces migration modes. > > > >

[RFC PATCH v4 3/5] mm/migrate: add MR_DAMON to migrate_reason

2024-05-12 Thread SeongJae Park
From: Honggyu Kim The current patch series introduces DAMON based migration across NUMA nodes so it'd be better to have a new migrate_reason in trace events. Signed-off-by: Honggyu Kim Reviewed-by: SeongJae Park Signed-off-by: SeongJae Park --- include/linux/migrate_mode.h | 1 + in

[RFC PATCH v4 0/5] DAMON based tiered memory management for CXL memory

2024-05-12 Thread SeongJae Park
" memory policy from 17~18% to 4~5% when the system runs with high memory pressure on its fast tier DRAM nodes. Having these DAMOS_MIGRATE_HOT and DAMOS_MIGRATE_COLD actions can make tiered memory systems run more efficiently under high memory pressures. Signed-off-by: Honggyu Kim Signed-off-b

Re: [RFC PATCH v3 1/7] mm/damon/paddr: refactor DAMOS_PAGEOUT with migration_mode

2024-05-11 Thread SeongJae Park
On Fri, 5 Apr 2024 12:19:07 -0700 SeongJae Park wrote: > On Fri, 5 Apr 2024 15:08:50 +0900 Honggyu Kim wrote: > > > This is a preparation patch that introduces migration modes. > > > > The damon_pa_pageout is renamed to damon_pa_migrate and it receives a

  1   2   3   4   5   6   7   8   9   10   >