Commit f04cc63d authored by Ye Liu's avatar Ye Liu Committed by Andrew Morton
Browse files

mm/rmap: rename page__anon_vma to anon_vma for consistency 

Patch series "mm: minor cleanups in rmap", v3.

Minor cleanups in mm/rmap.c:

- Rename a local variable for consistency
- Fix a typo in a comment

No functional changes.


This patch (of 2):

Rename local variable page__anon_vma in page_address_in_vma() to anon_vma.
The previous naming convention of using double underscores (__) is
unnecessary and inconsistent with typical kernel style, which uses single
underscores to denote local variables.  Also updated comments to reflect
the new variable name.

Functionality unchanged.

Link: https://lkml.kernel.org/r/20250421085729.127845-1-ye.liu@linux.dev
Link: https://lkml.kernel.org/r/20250421085729.127845-2-ye.liu@linux.dev


Signed-off-by: default avatarYe Liu <liuye@kylinos.cn>
Reviewed-by: default avatarLorenzo Stoakes <lorenzo.stoakes@oracle.com>
Acked-by: default avatarDavid Hildenbrand <david@redhat.com>
Reviewed-by: default avatarHarry Yoo <harry.yoo@oracle.com>
Cc: Liam Howlett <liam.howlett@oracle.com>
Cc: Liu Ye <liuye@kylinos.cn>
Cc: Miaohe Lin <linmiaohe@huawei.com>
Cc: Naoya Horiguchi <nao.horiguchi@gmail.com>
Cc: Rik van Riel <riel@surriel.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent a73dbc85

mm/rmap.c

+4 −4
Original line number Diff line number Diff line
@@ -789,13 +789,13 @@ unsigned long page_address_in_vma(const struct folio *folio, 
		const struct page *page, const struct vm_area_struct *vma)

{

	if (folio_test_anon(folio)) {

		struct anon_vma *page__anon_vma = folio_anon_vma(folio);

		struct anon_vma *anon_vma = folio_anon_vma(folio);

		/*

		 * Note: swapoff's unuse_vma() is more efficient with this

		 * check, and needs it to match anon_vma when KSM is active.

		 */

		if (!vma->anon_vma || !page__anon_vma ||

		    vma->anon_vma->root != page__anon_vma->root)

		if (!vma->anon_vma || !anon_vma ||

		    vma->anon_vma->root != anon_vma->root)

			return -EFAULT;

	} else if (!vma->vm_file) {

		return -EFAULT;
@@ -803,7 +803,7 @@ unsigned long page_address_in_vma(const struct folio *folio, 
		return -EFAULT;

	}



	/* KSM folios don't reach here because of the !page__anon_vma check */

	/* KSM folios don't reach here because of the !anon_vma check */

	return vma_address(vma, page_pgoff(folio, page), 1);

}