Re: [PATCH v9 2/3] mm: rename swap.c to folio.c

"Lorenzo Stoakes (ARM)" <[email protected]>
Newsgroups gmane.linux.kernel,gmane.linux.kernel.mm,gmane.linux.documentation
Message-ID <anG0QwQddFNo8PKJ@lucifer>
On Wed, Jul 08, 2026 at 07:35:44PM +0800, Jianyue Wu wrote:
> Rename mm/swap.c to mm/folio.c so the filename better matches
> the code's main responsibility.
>
> This keeps the implementation split from swap-specific code without
> changing the published LRU helper interfaces.
>
> Update MAINTAINERS and the remaining mm/swap.c documentation references
> after the rename.
>
> Suggested-by: Baoquan He <[email protected]>
> Suggested-by: David Hildenbrand <[email protected]>
> Suggested-by: Matthew Wilcox <[email protected]>
> Reviewed-by: Lorenzo Stoakes <[email protected]>
> Acked-by: Johannes Weiner <[email protected]>
> Acked-by: David Hildenbrand (Arm) <[email protected]>
> Signed-off-by: Jianyue Wu <[email protected]>
> ---
>  Documentation/admin-guide/sysctl/vm.rst |  3 ---
>  Documentation/core-api/mm-api.rst       |  2 +-
>  MAINTAINERS                             |  3 +--
>  mm/Makefile                             |  2 +-
>  mm/{swap.c => folio.c}                  | 10 +++-------
>  5 files changed, 6 insertions(+), 14 deletions(-)
>
> diff --git a/Documentation/admin-guide/sysctl/vm.rst b/Documentation/admin-guide/sysctl/vm.rst
> index 22cc54cac3b2..5b318d17aa4b 100644
> --- a/Documentation/admin-guide/sysctl/vm.rst
> +++ b/Documentation/admin-guide/sysctl/vm.rst
> @@ -19,9 +19,6 @@ The files in this directory can be used to tune the operation
>  of the virtual memory (VM) subsystem of the Linux kernel and
>  the writeout of dirty data to disk.
>
> -Default values and initialization routines for most of these
> -files can be found in mm/swap.c.
> -
>  Currently, these files are in /proc/sys/vm:
>
>  - admin_reserve_kbytes
> diff --git a/Documentation/core-api/mm-api.rst b/Documentation/core-api/mm-api.rst
> index 4df7d5edbee5..c1d03a5a2a19 100644
> --- a/Documentation/core-api/mm-api.rst
> +++ b/Documentation/core-api/mm-api.rst
> @@ -118,7 +118,7 @@ More Memory Management Functions
>  .. #kernel-doc:: mm/hmm.c (build warnings)
>  .. kernel-doc:: mm/memremap.c
>  .. kernel-doc:: mm/hugetlb.c
> -.. kernel-doc:: mm/swap.c
> +.. kernel-doc:: mm/folio.c
>  .. kernel-doc:: mm/memcontrol.c
>  .. #kernel-doc:: mm/memory-tiers.c (build warnings)
>  .. kernel-doc:: mm/shmem.c
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d19822d574fd..c3e535ca4bef 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17085,7 +17085,7 @@ F:	Documentation/admin-guide/mm/multigen_lru.rst
>  F:	Documentation/mm/multigen_lru.rst
>  F:	include/linux/mm_inline.h
>  F:	include/linux/mmzone.h
> -F:	mm/swap.c
> +F:	mm/folio.c

Hmm noticed today that mm/folio.c is in the MGLRU section :)

This doesn't seem correct - shouldn't this be moved to the core mm section?

If this is already in mm-stable can send a follow up patch for that?

>  F:	mm/vmscan.c
>  F:	mm/workingset.c
>
> @@ -17237,7 +17237,6 @@ F:	include/linux/swap.h
>  F:	include/linux/swapfile.h
>  F:	include/linux/swapops.h
>  F:	mm/page_io.c
> -F:	mm/swap.c
>  F:	mm/swap.h
>  F:	mm/swap_table.h
>  F:	mm/swap_state.c
> diff --git a/mm/Makefile b/mm/Makefile
> index 4fc713867b9b..8e7e964b99a4 100644
> --- a/mm/Makefile
> +++ b/mm/Makefile
> @@ -50,7 +50,7 @@ endif
>
>  obj-y			:= filemap.o mempool.o oom_kill.o fadvise.o \
>  			   maccess.o page-writeback.o folio-compat.o \
> -			   readahead.o swap.o truncate.o vmscan.o shrinker.o \
> +			   readahead.o folio.o truncate.o vmscan.o shrinker.o \
>  			   shmem.o util.o mmzone.o vmstat.o backing-dev.o \
>  			   mm_init.o percpu.o slab_common.o \
>  			   compaction.o show_mem.o \
> diff --git a/mm/swap.c b/mm/folio.c
> similarity index 99%
> rename from mm/swap.c
> rename to mm/folio.c
> index c88643e56af7..d2937600cf72 100644
> --- a/mm/swap.c
> +++ b/mm/folio.c
> @@ -1,17 +1,13 @@
>  // SPDX-License-Identifier: GPL-2.0-only
>  /*
> - *  linux/mm/swap.c
> + *  linux/mm/folio.c
>   *
>   *  Copyright (C) 1991, 1992, 1993, 1994  Linus Torvalds
>   */
>
>  /*
> - * This file contains the default values for the operation of the
> - * Linux VM subsystem. Fine-tuning documentation can be found in
> - * Documentation/admin-guide/sysctl/vm.rst.
> - * Started 18.12.91
> - * Swap aging added 23.2.95, Stephen Tweedie.
> - * Buffermem limits added 12.3.98, Rik van Riel.
> + * Folio LRU helpers: add/remove folios from LRU lists, batching,
> + * activation/deactivation, and page cache release paths.
>   */
>
>  #include <linux/mm.h>
>
> --
> 2.43.0
>

--
Cheers, Lorenzo
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.