Re: [PATCH v2] erofs: use kmap_local_page() only for erofs_bread()

From: Fabio M. De Francesco
Date: Tue Oct 18 2022 - 15:18:44 EST


On Tuesday, October 18, 2022 12:53:13 PM CEST Gao Xiang wrote:
> Convert all mapped erofs_bread() users to use kmap_local_page()
> instead of kmap() or kmap_atomic().
>
> Signed-off-by: Gao Xiang <hsiangkao@xxxxxxxxxxxxxxxxx>
> ---
> fs/erofs/data.c | 8 ++------
> fs/erofs/internal.h | 3 +--
> fs/erofs/xattr.c | 8 ++++----
> fs/erofs/zmap.c | 4 ++--
> 4 files changed, 9 insertions(+), 14 deletions(-)
>

I just realized that you know the code of fs/erofs very well. I saw a Gao
Xiang in MAINTAINERS, although having a different email address.

Therefore, I'm sure that everybody can trust that you checked everything is
needed to assure the safety of the conversions.

However, an extended commit message would have prevented me to send you the
previous email with all those questions / objections.

Thanks,

Fabio