Skip to content

introduce kernel file mapped folios

Boris Burkov linux-btrfs linux-fsdevel linux-mm oe-kbuild-all 4 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest3 patches 3 repliesIn review
  2. v34 patches 17 repliesSuperseded
  3. v23 patches 6 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v13 patches3 repliesSuperseded

Changes in v4 from the cover letter

- change the concept from "uncharged" to "kernel_file"
- no longer violates the invariant that each mapped folio has a memcg when CONFIG_MEMCG=y
- no longer really tied to memcg conceptually, so simplify build/helpers v3:
- use mod_node_page_state since we will never count cgroup stats
- include Shakeel's patch that removes a WARNING triggered by this series v2:
- switch from filemap_add_folio_nocharge() to AS_UNCHARGED on the address_space.
- fix an interrupt safety bug in the vmstat patch.
- fix some foolish build errors for CONFIG_MEMCG=n

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3mm/filemap: add AS_KERNEL_FILEinclude/linux/pagemap.h mm/filemap.c+8 -02 replies
2/3mm: add vmstat for kernel_file pagesinclude/linux/mmzone.h mm/filemap.c mm/vmstat.c+9 -00 replies
3/3btrfs: set AS_KERNEL_FILE on the btree_inodefs/btrfs/disk-io.c+1 -00 replies

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

Patch[email protected]Shakeel ButtDavid Sterba
SeriesA-b
1/3T-b A-b
2/3T-b A-b
3/3T-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v4 thread

Cover letter [PATCH v4 0/3] introduce kernel file mapped folios

I would like to revisit Qu's proposal to not charge btrfs extent_buffer allocations to the user's cgroup.

https://lore.kernel.org/linux-mm/b5fef5372ae454a7b6da4f2f75c427aeab6a07d6.1727498749.git.wqu@suse.com/

I believe it is detrimental to account these global pages to the cgroup using them, basically at random. A bit more justification and explanation in the patches themselves.

---
Changelog:
v4:
- change the concept from "uncharged" to "kernel_file"
- no longer violates the invariant that each mapped folio has a memcg when CONFIG_MEMCG=y
- no longer really tied to memcg conceptually, so simplify build/helpers v3:
- use mod_node_page_state since we will never count cgroup stats
- include Shakeel's patch that removes a WARNING triggered by this series v2:
- switch from filemap_add_folio_nocharge() to AS_UNCHARGED on the address_space.
- fix an interrupt safety bug in the vmstat patch.
- fix some foolish build errors for CONFIG_MEMCG=n

Boris Burkov (3):
mm/filemap: add AS_KERNEL_FILE
mm: add vmstat for kernel_file pages
btrfs: set AS_KERNEL_FILE on the btree_inode

 fs/btrfs/disk-io.c      |  1 +
 include/linux/mmzone.h  |  1 +
 include/linux/pagemap.h |  2 ++
 mm/filemap.c            | 13 +++++++++++++
 mm/vmstat.c             |  1 +
 5 files changed, 18 insertions(+)
Signature · 1 line
2.50.1