[1/2] mm: memory_failure: move memory_failure_attr_group under MEMORY_FAILURE

Message ID 20230508114128.37081-1-wangkefeng.wang@huawei.com
State New
Headers
Series [1/2] mm: memory_failure: move memory_failure_attr_group under MEMORY_FAILURE |

Commit Message

Kefeng Wang May 8, 2023, 11:41 a.m. UTC
  The memory_failure_attr_group is only called if MEMORY_FAILURE
enabled, move it under this configuration.

Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
---
 include/linux/mm.h | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)
  

Comments

HORIGUCHI NAOYA(堀口 直也) May 10, 2023, 3:03 a.m. UTC | #1
On Mon, May 08, 2023 at 07:41:27PM +0800, Kefeng Wang wrote:
> The memory_failure_attr_group is only called if MEMORY_FAILURE
> enabled, move it under this configuration.
> 
> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>

Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
  

Patch

diff --git a/include/linux/mm.h b/include/linux/mm.h
index d533ef955dd0..cdb0377f97fa 100644
--- a/include/linux/mm.h
+++ b/include/linux/mm.h
@@ -3687,6 +3687,10 @@  extern void shake_page(struct page *p);
 extern atomic_long_t num_poisoned_pages __read_mostly;
 extern int soft_offline_page(unsigned long pfn, int flags);
 #ifdef CONFIG_MEMORY_FAILURE
+/*
+ * Sysfs entries for memory failure handling statistics.
+ */
+extern const struct attribute_group memory_failure_attr_group;
 extern void memory_failure_queue(unsigned long pfn, int flags);
 extern int __get_huge_page_for_hwpoison(unsigned long pfn, int flags,
 					bool *migratable_cleared);
@@ -3779,11 +3783,6 @@  enum mf_action_page_type {
 	MF_MSG_UNKNOWN,
 };
 
-/*
- * Sysfs entries for memory failure handling statistics.
- */
-extern const struct attribute_group memory_failure_attr_group;
-
 #if defined(CONFIG_TRANSPARENT_HUGEPAGE) || defined(CONFIG_HUGETLBFS)
 extern void clear_huge_page(struct page *page,
 			    unsigned long addr_hint,