Re: [PATCH] highmem: Rename put_and_unmap_page() to unmap_and_put_page()

From: Christian Brauner
Date: Mon Jun 05 2023 - 07:55:19 EST


On Fri, 02 Jun 2023 12:33:07 +0200, Fabio M. De Francesco wrote:
> With commit 849ad04cf562a ("new helper: put_and_unmap_page()"), Al Viro
> introduced the put_and_unmap_page() to use in those many places where we
> have a common pattern consisting of calls to kunmap_local() +
> put_page().
>
> Obviously, first we unmap and then we put pages. Instead, the original
> name of this helper seems to imply that we first put and then unmap.
>
> [...]

Grabbed it now. Please yell if this wants to go another route.

---

Applied to the vfs.misc branch of the vfs/vfs.git tree.
Patches in the vfs.misc branch should appear in linux-next soon.

Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.

It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.

tree: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.misc

[1/1] highmem: Rename put_and_unmap_page() to unmap_and_put_page()
https://git.kernel.org/vfs/vfs/c/01af7e7dd0ea