linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: SeongJae Park <sj@kernel.org>
Cc: SeongJae Park <sj@kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Jonathan Corbet <corbet@lwn.net>, Shuah Khan <shuah@kernel.org>,
	Brendan Higgins <brendan.higgins@linux.dev>,
	David Gow <davidgow@google.com>, Rae Moar <rmoar@google.com>,
	damon@lists.linux.dev, linux-mm@kvack.org,
	linux-kselftest@vger.kernel.org, linux-doc@vger.kernel.org,
	kunit-dev@googlegroups.com, linux-kernel@vger.kernel.org
Subject: [RFC PATCH 0/7] mm/damon: remove DAMON debugfs interface
Date: Tue, 15 Oct 2024 10:54:05 -0700	[thread overview]
Message-ID: <20241015175412.60563-1-sj@kernel.org> (raw)

DAMON debugfs interface was the only user interface of DAMON at the
beginning[1].  However, it turned out the interface would be not good
enough for long-term flexibility and stability.

In Feb 2022[2], we therefore introduced DAMON sysfs interface as an
alternative user interface that aims long-term flexibility and
stability.  With its introduction, DAMON debugfs interface has announced
to be deprecated in near future.

In Feb 2023[3], we announced the official deprecation of DAMON debugfs
interface.  In Jan 2024[4], we further made the deprecation difficult to
be ignored.

And as of this writing (2024-10-14), no problem or concerns about the
deprecation have reported.  Apparently users are already moved to the
alternative, or made good plans for the change.

Remove the DAMON debugfs interface code from the tree.  Given the past
timeline and the absence of reported problems or concerns, it is safe
enough to be done.  That said, we will not drop the RFC tag of this
patch series at least until the end of this year, to use this as the
real last call for users.

[1] https://lore.kernel.org/20210716081449.22187-1-sj38.park@gmail.com
[2] https://lore.kernel.org/20220228081314.5770-1-sj@kernel.org
[3] https://lore.kernel.org/20230209192009.7885-1-sj@kernel.org
[4] https://lore.kernel.org/20240130013549.89538-1-sj@kernel.org

SeongJae Park (7):
  Docs/admin-guide/mm/damon/usage: remove DAMON debugfs interface
    documentation
  Docs/mm/damon/design: update for removal of DAMON debugfs interface
  selftests/damon/config: remove configs for DAMON debugfs interface
    selftests
  selftests/damon: remove tests for DAMON debugfs interface
  kunit: configs: remove configs for DAMON debugfs interface tests
  mm/damon: remove DAMON debugfs interface kunit tests
  mm/damon: remove DAMON debugfs interface

 Documentation/admin-guide/mm/damon/usage.rst  |  309 -----
 Documentation/mm/damon/design.rst             |   23 +-
 mm/damon/Kconfig                              |   30 -
 mm/damon/Makefile                             |    1 -
 mm/damon/dbgfs.c                              | 1148 -----------------
 mm/damon/tests/.kunitconfig                   |    7 -
 mm/damon/tests/dbgfs-kunit.h                  |  173 ---
 tools/testing/kunit/configs/all_tests.config  |    3 -
 tools/testing/selftests/damon/.gitignore      |    3 -
 tools/testing/selftests/damon/Makefile        |   11 +-
 tools/testing/selftests/damon/config          |    1 -
 .../testing/selftests/damon/debugfs_attrs.sh  |   17 -
 .../debugfs_duplicate_context_creation.sh     |   27 -
 .../selftests/damon/debugfs_empty_targets.sh  |   21 -
 .../damon/debugfs_huge_count_read_write.sh    |   22 -
 .../damon/debugfs_rm_non_contexts.sh          |   19 -
 .../selftests/damon/debugfs_schemes.sh        |   19 -
 .../selftests/damon/debugfs_target_ids.sh     |   19 -
 .../damon/debugfs_target_ids_pid_leak.c       |   68 -
 .../damon/debugfs_target_ids_pid_leak.sh      |   22 -
 ...fs_target_ids_read_before_terminate_race.c |   80 --
 ...s_target_ids_read_before_terminate_race.sh |   14 -
 .../selftests/damon/huge_count_read_write.c   |   48 -
 23 files changed, 11 insertions(+), 2074 deletions(-)
 delete mode 100644 mm/damon/dbgfs.c
 delete mode 100644 mm/damon/tests/dbgfs-kunit.h
 delete mode 100755 tools/testing/selftests/damon/debugfs_attrs.sh
 delete mode 100755 tools/testing/selftests/damon/debugfs_duplicate_context_creation.sh
 delete mode 100755 tools/testing/selftests/damon/debugfs_empty_targets.sh
 delete mode 100755 tools/testing/selftests/damon/debugfs_huge_count_read_write.sh
 delete mode 100755 tools/testing/selftests/damon/debugfs_rm_non_contexts.sh
 delete mode 100755 tools/testing/selftests/damon/debugfs_schemes.sh
 delete mode 100755 tools/testing/selftests/damon/debugfs_target_ids.sh
 delete mode 100644 tools/testing/selftests/damon/debugfs_target_ids_pid_leak.c
 delete mode 100755 tools/testing/selftests/damon/debugfs_target_ids_pid_leak.sh
 delete mode 100644 tools/testing/selftests/damon/debugfs_target_ids_read_before_terminate_race.c
 delete mode 100755 tools/testing/selftests/damon/debugfs_target_ids_read_before_terminate_race.sh
 delete mode 100644 tools/testing/selftests/damon/huge_count_read_write.c


base-commit: 5ef943709a1b88304aa6e8cb8683a25bf81874f0
-- 
2.39.5



             reply	other threads:[~2024-10-15 17:54 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-15 17:54 SeongJae Park [this message]
2024-10-15 17:54 ` [RFC PATCH 1/7] Docs/admin-guide/mm/damon/usage: remove DAMON debugfs interface documentation SeongJae Park
2024-10-15 17:54 ` [RFC PATCH 2/7] Docs/mm/damon/design: update for removal of DAMON debugfs interface SeongJae Park
2024-10-15 17:54 ` [RFC PATCH 3/7] selftests/damon/config: remove configs for DAMON debugfs interface selftests SeongJae Park
2024-10-15 17:54 ` [RFC PATCH 4/7] selftests/damon: remove tests for DAMON debugfs interface SeongJae Park
2024-10-15 17:54 ` [RFC PATCH 5/7] kunit: configs: remove configs for DAMON debugfs interface tests SeongJae Park
2024-10-15 17:54 ` [RFC PATCH 6/7] mm/damon: remove DAMON debugfs interface kunit tests SeongJae Park
2024-10-15 17:54 ` [RFC PATCH 7/7] mm/damon: remove DAMON debugfs interface SeongJae 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=20241015175412.60563-1-sj@kernel.org \
    --to=sj@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=brendan.higgins@linux.dev \
    --cc=corbet@lwn.net \
    --cc=damon@lists.linux.dev \
    --cc=davidgow@google.com \
    --cc=kunit-dev@googlegroups.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=rmoar@google.com \
    --cc=shuah@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).