From 8d0b09d4f2e55a786da1adff07ed6ce02740c504 Mon Sep 17 00:00:00 2001 From: "Isaac J. Manjarres" Date: Wed, 15 Jan 2020 23:53:01 -0800 Subject: [PATCH] Revert "mm: cma: make writeable CMA debugfs optional" This reverts commit 099c0c9a642569edfe672eeb983876b0e6662a75. CONFIG_CMA_DEBUGFS is only enabled in debug builds on 5.4+ and the purpose of this commit is to disallow CMA allocations through debugfs on production builds. Since, this is no longer possible, revert this change. Change-Id: If1f322533660bfc7609fb09a5b13ff2ecbbe3135 Signed-off-by: Isaac J. Manjarres --- mm/Kconfig | 9 --------- mm/cma_debug.c | 10 ---------- 2 files changed, 19 deletions(-) diff --git a/mm/Kconfig b/mm/Kconfig index 880a5221a5ea..183096ddd744 100644 --- a/mm/Kconfig +++ b/mm/Kconfig @@ -519,15 +519,6 @@ config CMA_DEBUGFS help Turns on the DebugFS interface for CMA. -config CMA_ALLOW_WRITE_DEBUGFS - depends on CMA_DEBUGFS - bool "Allow CMA debugfs write" - help - Say 'y' here to allow the CMA debugfs write. - CMA debugfs write could be risky as it allows - cma allocation, so it is not recommended to - enable this option on any production device. - config CMA_AREAS int "Maximum count of the CMA areas" depends on CMA diff --git a/mm/cma_debug.c b/mm/cma_debug.c index 42ad17b64001..bc9a7d681b7a 100644 --- a/mm/cma_debug.c +++ b/mm/cma_debug.c @@ -74,7 +74,6 @@ static int cma_maxchunk_get(void *data, u64 *val) } DEFINE_SIMPLE_ATTRIBUTE(cma_maxchunk_fops, cma_maxchunk_get, NULL, "%llu\n"); -#ifdef CONFIG_CMA_ALLOW_WRITE_DEBUGFS static void cma_add_to_cma_mem_list(struct cma *cma, struct cma_mem *mem) { spin_lock(&cma->mem_head_lock); @@ -133,13 +132,8 @@ static int cma_free_write(void *data, u64 val) return cma_free_mem(cma, pages); } -#else -#define cma_free_write NULL -#endif - DEFINE_SIMPLE_ATTRIBUTE(cma_free_fops, NULL, cma_free_write, "%llu\n"); -#ifdef CONFIG_CMA_ALLOW_WRITE_DEBUGFS static int cma_alloc_mem(struct cma *cma, int count) { struct cma_mem *mem; @@ -170,10 +164,6 @@ static int cma_alloc_write(void *data, u64 val) return cma_alloc_mem(cma, pages); } -#else -#define cma_alloc_write NULL -#endif - DEFINE_SIMPLE_ATTRIBUTE(cma_alloc_fops, NULL, cma_alloc_write, "%llu\n"); static void cma_debugfs_add_one(struct cma *cma, struct dentry *root_dentry)