mm/damon/dbgfs: make debugfs interface deprecation message a macro
authorSeongJae Park <sj@kernel.org>
Tue, 30 Jan 2024 01:35:43 +0000 (17:35 -0800)
committerAndrew Morton <akpm@linux-foundation.org>
Thu, 22 Feb 2024 18:24:45 +0000 (10:24 -0800)
DAMON debugfs interface deprecation message is written twice, once for the
warning, and again for DEPRECATED file's read output.  De-duplicate those
by defining the message as a macro and reuse.

[akpm@linux-foundation.org: s/comnst/const/]
Link: https://lkml.kernel.org/r/20240130013549.89538-5-sj@kernel.org
Signed-off-by: SeongJae Park <sj@kernel.org>
Cc: Alex Shi <alexs@kernel.org>
Cc: Hu Haowen <2023002089@link.tyut.edu.cn>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Shuah Khan <shuah@kernel.org>
Cc: Yanteng Si <siyanteng@loongson.cn>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/damon/dbgfs.c

index f66865a..45d7699 100644 (file)
 #include <linux/page_idle.h>
 #include <linux/slab.h>
 
+#define DAMON_DBGFS_DEPRECATION_NOTICE                                 \
+       "DAMON debugfs interface is deprecated, so users should move "  \
+       "to DAMON_SYSFS. If you cannot, please report your usecase to " \
+       "damon@lists.linux.dev and linux-mm@kvack.org.\n"
+
 static struct damon_ctx **dbgfs_ctxs;
 static int dbgfs_nr_ctxs;
 static struct dentry **dbgfs_dirs;
@@ -22,10 +27,7 @@ static DEFINE_MUTEX(damon_dbgfs_lock);
 
 static void damon_dbgfs_warn_deprecation(void)
 {
-       pr_warn_once("DAMON debugfs interface is deprecated, "
-                    "so users should move to DAMON_SYSFS. If you cannot, "
-                    "please report your usecase to damon@lists.linux.dev and "
-                    "linux-mm@kvack.org.\n");
+       pr_warn_once(DAMON_DBGFS_DEPRECATION_NOTICE);
 }
 
 /*
@@ -808,10 +810,7 @@ static void dbgfs_destroy_ctx(struct damon_ctx *ctx)
 static ssize_t damon_dbgfs_deprecated_read(struct file *file,
                char __user *buf, size_t count, loff_t *ppos)
 {
-       static const char kbuf[512] = "DAMON debugfs interface is deprecated, "
-                    "so users should move to DAMON_SYSFS. If you cannot, "
-                    "please report your usecase to damon@lists.linux.dev and "
-                    "linux-mm@kvack.org.\n";
+       static const char kbuf[512] = DAMON_DBGFS_DEPRECATION_NOTICE;
 
        return simple_read_from_buffer(buf, count, ppos, kbuf, strlen(kbuf));
 }