On 7/22/25 13:20, yohan.joung wrote:
> read for the pinfile using Direct I/O do not wait for dio write.
>
> Signed-off-by: yohan.joung <yohan.jo...@sk.com>
> ---
> fs/f2fs/file.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
> index 4039ccb5022c..2b7e5c46c1ae 100644
> --- a/fs/f2fs/file.c
> +++ b/fs/f2fs/file.c
> @@ -4834,6 +4834,7 @@ static ssize_t f2fs_file_read_iter(struct kiocb *iocb,
> struct iov_iter *to)
> struct inode *inode = file_inode(iocb->ki_filp);
> const loff_t pos = iocb->ki_pos;
> ssize_t ret;
> + bool dio;
>
> if (!f2fs_is_compress_backend_ready(inode))
> return -EOPNOTSUPP;
> @@ -4842,12 +4843,15 @@ static ssize_t f2fs_file_read_iter(struct kiocb
> *iocb, struct iov_iter *to)
> f2fs_trace_rw_file_path(iocb->ki_filp, iocb->ki_pos,
> iov_iter_count(to), READ);
>
> + dio = f2fs_should_use_dio(inode, iocb, to);
> +
> /* In LFS mode, if there is inflight dio, wait for its completion */
> if (f2fs_lfs_mode(F2FS_I_SB(inode)) &&
> - get_pages(F2FS_I_SB(inode), F2FS_DIO_WRITE))
> + get_pages(F2FS_I_SB(inode), F2FS_DIO_WRITE) &&
> + !(f2fs_is_pinned_file(inode) && dio))
How about?
(!f2fs_is_pinned_file(inode) || !dio)
Thanks,
> inode_dio_wait(inode);
>
> - if (f2fs_should_use_dio(inode, iocb, to)) {
> + if (dio) {
> ret = f2fs_dio_read_iter(iocb, to);
> } else {
> ret = filemap_read(iocb, to, 0);
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
