linux-kernel - Re: [PATCH v10 10/10] erofs: implement .fadvise for page cache share

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5140f1e6-a669-479c-9e59-7bbf2191f546@linux.alibaba.com>
Date: Tue, 23 Dec 2025 16:38:36 +0800
From: Gao Xiang <hsiangkao@...ux.alibaba.com>
To: Hongbo Li <lihongbo22@...wei.com>
Cc: linux-fsdevel@...r.kernel.org, linux-erofs@...ts.ozlabs.org,
 linux-kernel@...r.kernel.org, Chao Yu <chao@...nel.org>, brauner@...nel.org,
 "Darrick J. Wong" <djwong@...nel.org>, Amir Goldstein <amir73il@...il.com>,
 hch@....de
Subject: Re: [PATCH v10 10/10] erofs: implement .fadvise for page cache share



On 2025/12/23 10:00, Hongbo Li wrote:
> From: Hongzhen Luo <hongzhen@...ux.alibaba.com>
> 
> This patch implements the .fadvise interface for page cache share.
> Similar to overlayfs, it drops those clean, unused pages through
> vfs_fadvise().
> 
> Reviewed-by: Gao Xiang <hsiangkao@...ux.alibaba.com>
> Signed-off-by: Hongzhen Luo <hongzhen@...ux.alibaba.com>
> Signed-off-by: Hongbo Li <lihongbo22@...wei.com>

This should be sent out together within the series next time.

> ---
>   fs/erofs/ishare.c | 11 +++++++++++
>   1 file changed, 11 insertions(+)
> 
> diff --git a/fs/erofs/ishare.c b/fs/erofs/ishare.c
> index 269b53b3ed79..d7231953cba2 100644
> --- a/fs/erofs/ishare.c
> +++ b/fs/erofs/ishare.c
> @@ -187,6 +187,16 @@ static int erofs_ishare_mmap(struct file *file, struct vm_area_struct *vma)
>   	return generic_file_readonly_mmap(file, vma);
>   }
>   
> +static int erofs_ishare_fadvise(struct file *file, loff_t offset,
> +				      loff_t len, int advice)
> +{
> +	struct file *realfile = file->private_data;
> +
> +	if (!realfile)
> +		return -EINVAL;

BTW, when file->private_data == NULL here?

I think it can only happen if buggy, so just:

	return vfs_fadvise((struct file *)file->private_data,
			   offset, len, advice);
Thanks,
Gao Xiang

> +	return vfs_fadvise(realfile, offset, len, advice);
> +}
> +
>   const struct file_operations erofs_ishare_fops = {
>   	.open		= erofs_ishare_file_open,
>   	.llseek		= generic_file_llseek,
> @@ -195,6 +205,7 @@ const struct file_operations erofs_ishare_fops = {
>   	.release	= erofs_ishare_file_release,
>   	.get_unmapped_area = thp_get_unmapped_area,
>   	.splice_read	= filemap_splice_read,
> +	.fadvise	= erofs_ishare_fadvise,
>   };
>   
>   /*