c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2026-05-24 19:28:53 +03:00
Code Issues Packages Projects Releases Wiki Activity

Linux: Refactor zpl_fadvise()

Browse Source 
Similar to FreeBSD stop issuing prefetches on POSIX_FADV_SEQUENTIAL.
It should not have this semantics, only hint speculative prefetcher,
if access ever happen later.  Instead after POSIX_FADV_WILLNEED
handling call generic_fadvise(), if available, to do all the generic
stuff, including setting f_mode in struct file, that we could later
use to control prefetcher as part of read/write operations.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Alexander Motin <alexander.motin@TrueNAS.com>
Closes #18395
This commit is contained in:
Alexander Motin
2026-04-06 18:57:18 -04:00
committed by Tony Hutter
parent 26e9a69fea
commit 63b8da8ff7
1 changed files with 10 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
module/os/linux/zfs/zpl_file.c
+10 -27
View File
@@ -779,38 +779,21 @@ zpl_fadvise(struct file *filp, loff_t offset, loff_t len, int advice)
if ((error = zpl_enter_verify_zp(zfsvfs, zp, FTAG)) != 0) if ((error = zpl_enter_verify_zp(zfsvfs, zp, FTAG)) != 0)
return (error); return (error);
switch (advice) { if (advice == POSIX_FADV_WILLNEED) {
case POSIX_FADV_SEQUENTIAL: loff_t rlen = len ? len : i_size_read(ip) - offset;
case POSIX_FADV_WILLNEED: dmu_prefetch(os, zp->z_id, 0, offset, rlen,
#ifdef HAVE_GENERIC_FADVISE
if (zn_has_cached_data(zp, offset, offset + len - 1))
error = generic_fadvise(filp, offset, len, advice);
#endif
/*
* Pass on the caller's size directly, but note that
* dmu_prefetch_max will effectively cap it. If there
* really is a larger sequential access pattern, perhaps
* dmu_zfetch will detect it.
*/
if (len == 0)
len = i_size_read(ip) - offset;
dmu_prefetch(os, zp->z_id, 0, offset, len,
ZIO_PRIORITY_ASYNC_READ); ZIO_PRIORITY_ASYNC_READ);
break; if (!zn_has_cached_data(zp, offset, offset + rlen - 1)) {
case POSIX_FADV_NORMAL: zfs_exit(zfsvfs, FTAG);
case POSIX_FADV_RANDOM: return (error);
case POSIX_FADV_DONTNEED: }
case POSIX_FADV_NOREUSE:
/* ignored for now */
break;
default:
error = -EINVAL;
break;
} }
zfs_exit(zfsvfs, FTAG); zfs_exit(zfsvfs, FTAG);
#ifdef HAVE_GENERIC_FADVISE
error = generic_fadvise(filp, offset, len, advice);
#endif
return (error); return (error);
} }