Commit c8791350 authored by Ma Wupeng's avatar Ma Wupeng Committed by Wupeng Ma
Browse files

mm: mem_reliable: Initialize reliable_nr_page when mm_init() 

hulk inclusion
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I6RKHX



--------------------------------

After the fork operation, it is erroneous for the child process to have a
reliable page size twice that of its parent process.

Upon examining the mm_struct structure, it was discovered that
reliable_nr_page should be initialized to 0, similar to how RSS is
initialized during mm_init(). This particular problem that arises during
forking is merely one such example.

To resolve this issue, it is recommended to set reliable_nr_page to 0
during the mm_init() operation.

Fixes: 8fc2546f ("proc: mem_reliable: Count reliable memory usage of reliable tasks")
Signed-off-by: default avatarMa Wupeng <mawupeng1@huawei.com>
parent 980f4346

include/linux/mem_reliable.h

+9 −0
Original line number Diff line number Diff line
@@ -216,6 +216,14 @@ static inline void add_reliable_page_counter(struct page *page, 


	reliable_page_counter_inner(mm, val);

}



static inline void reliable_clear_page_counter(struct mm_struct *mm)

{

	if (!mem_reliable_is_enabled())

		return;



	atomic_long_set(&mm->reliable_nr_page, 0);

}

#else

#define reliable_enabled 0
@@ -259,6 +267,7 @@ static inline void add_reliable_folio_counter(struct folio *folio, 
		struct mm_struct *mm, int val) {}

static inline void reliable_report_usage(struct seq_file *m,

		struct mm_struct *mm) {}

static inline void reliable_clear_page_counter(struct mm_struct *mm) {}

#endif



#endif

kernel/fork.c

+1 −0
Original line number Diff line number Diff line
@@ -1326,6 +1326,7 @@ static struct mm_struct *mm_init(struct mm_struct *mm, struct task_struct *p, 
	mm->locked_vm = 0;

	atomic64_set(&mm->pinned_vm, 0);

	memset(&mm->rss_stat, 0, sizeof(mm->rss_stat));

	reliable_clear_page_counter(mm);

	spin_lock_init(&mm->page_table_lock);

	spin_lock_init(&mm->arg_lock);

	mm_init_cpumask(mm);