Commit 35520a71 authored by Aaron Yang's avatar Aaron Yang Committed by Andrew Morton
Browse files

mm/damon/paddr: initialize 'folio' variables to NULL for clarity 

In damon_pa_mark_accessed_or_deactivate(), damon_pa_pageout(),
damon_pa_migrate(), and damon_pa_stat(), the local variable 'folio' is
declared but not initialized.

Initialize 'folio' to NULL to improve code readability and
maintainability.

Link: https://patch.msgid.link/20260104013255.16962-1-yangqixiao@inspur.com
Link: https://lkml.kernel.org/r/20260108013041.80601-1-sj@kernel.org


Signed-off-by: default avatarAaron Yang <yangqixiao@inspur.com>
Reviewed-by: default avatarSeongJae Park <sj@kernel.org>
Signed-off-by: default avatarSeongJae Park <sj@kernel.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 19c4707b

mm/damon/paddr.c

+4 −4
Original line number Diff line number Diff line
@@ -156,7 +156,7 @@ static unsigned long damon_pa_pageout(struct damon_region *r, 
	LIST_HEAD(folio_list);

	bool install_young_filter = true;

	struct damos_filter *filter;

	struct folio *folio;

	struct folio *folio = NULL;



	/* check access in page level again by default */

	damos_for_each_ops_filter(filter, s) {
@@ -212,7 +212,7 @@ static inline unsigned long damon_pa_mark_accessed_or_deactivate( 
		unsigned long *sz_filter_passed)

{

	phys_addr_t addr, applied = 0;

	struct folio *folio;

	struct folio *folio = NULL;



	addr = damon_pa_phys_addr(r->ar.start, addr_unit);

	while (addr < damon_pa_phys_addr(r->ar.end, addr_unit)) {
@@ -262,7 +262,7 @@ static unsigned long damon_pa_migrate(struct damon_region *r, 
{

	phys_addr_t addr, applied;

	LIST_HEAD(folio_list);

	struct folio *folio;

	struct folio *folio = NULL;



	addr = damon_pa_phys_addr(r->ar.start, addr_unit);

	while (addr < damon_pa_phys_addr(r->ar.end, addr_unit)) {
@@ -295,7 +295,7 @@ static unsigned long damon_pa_stat(struct damon_region *r, 
		unsigned long *sz_filter_passed)

{

	phys_addr_t addr;

	struct folio *folio;

	struct folio *folio = NULL;



	if (!damos_ops_has_filter(s))

		return 0;