]> git.baikalelectronics.ru Git - kernel.git/commit
f2fs: Delete f2fs_copy_page() and replace with memcpy_page()
authorFabio M. De Francesco <fmdefrancesco@gmail.com>
Sun, 17 Jul 2022 08:36:13 +0000 (10:36 +0200)
committerJaegeuk Kim <jaegeuk@kernel.org>
Sun, 31 Jul 2022 03:16:57 +0000 (20:16 -0700)
commit805fd0d1a93c181fe955c4cdfe4eee2d165699dd
tree7191f3c5d0e7eb3e806c53550e9e5930238633cc
parent1441aadbc880028c376052829bb29af99ec6e97c
f2fs: Delete f2fs_copy_page() and replace with memcpy_page()

f2fs_copy_page() is a wrapper around two kmap() + one memcpy() from/to
the mapped pages. It unnecessarily duplicates a kernel API and it makes
use of kmap(), which is being deprecated in favor of kmap_local_page().

Two main problems with kmap(): (1) It comes with an overhead as mapping
space is restricted and protected by a global lock for synchronization and
(2) it also requires global TLB invalidation when the kmap’s pool wraps
and it might block when the mapping space is fully utilized until a slot
becomes available.

With kmap_local_page() the mappings are per thread, CPU local, can take
page faults, and can be called from any context (including interrupts).
It is faster than kmap() in kernels with HIGHMEM enabled. Therefore, its
use in __clone_blkaddrs() is safe and should be preferred.

Delete f2fs_copy_page() and use a plain memcpy_page() in the only one
site calling the removed function. memcpy_page() avoids open coding two
kmap_local_page() + one memcpy() between the two kernel virtual addresses.

Suggested-by: Christoph Hellwig <hch@infradead.org>
Suggested-by: Ira Weiny <ira.weiny@intel.com>
Signed-off-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/f2fs.h
fs/f2fs/file.c