Re: [PATCHv2 1/1] ext4/f2fs: don't put symlink in pagecache into highmem

From: Greg KH
Date: Wed Feb 07 2018 - 16:18:47 EST


Some minor nits for when you send your next patch for stable:

On Tue, Feb 06, 2018 at 03:53:53PM -0800, Jin Qian wrote:
> From: Jin Qian <jinqian@xxxxxxxxxx>

The original patch was written by Al Viro, please do not take the
original authorship off, that's not good.

> partial backport from 21fc61c73c3903c4c312d0802da01ec2b323d174 upstream
> to v4.4 to prevent virt_to_page on highmem.
>
> ext4_encrypted_follow_link uses kmap() for cpage
> caddr = kmap(cpage);
>
> _ext4_fname_disk_to_usr calls virt_to_page on the kmapped address.
> _ext4_fname_disk_to_usr()
> ext4_fname_decrypt()
> sg_init_one()
> sg_init_one(&src_sg, iname->name, iname->len);
> sg_set_page(sg, virt_to_page(buf), buflen, offset_in_page(buf));
>
> Cc: linux-stable <stable@xxxxxxxxxxxxxxx> # 4.4.y
> Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Jin Qian <jinqian@xxxxxxxxxx>
> Signed-off-by: Jin Qian <jinqian@xxxxxxxxxxx>

Why did you sign off on this twice?

thanks,

greg k-h