[Devel] [PATCH RHEL7 COMMIT] fs: More use iov_iter in ext4_file_write()

Konstantin Khorenko khorenko at virtuozzo.com
Mon May 25 17:52:33 MSK 2020


The commit is pushed to "branch-rh7-3.10.0-1127.8.2.vz7.161.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git
after rh7-3.10.0-1127.8.2.vz7.161.1
------>
commit ea3e15dc5854e937ade593884d760f3d2807ea46
Author: Kirill Tkhai <ktkhai at virtuozzo.com>
Date:   Mon May 25 17:52:33 2020 +0300

    fs: More use iov_iter in ext4_file_write()
    
    1)After iov_iter is added as local variable at start
     of this function, we should not introduce it again
     for ext4_file_dio_write().
    
    2)Instead of generic_file_aio_write() we use its
    iov_iter variant generic_file_write_iter()
    
    Signed-off-by: Kirill Tkhai <ktkhai at virtuozzo.com>
    
    =====================
    Patchset description:
    [00/30] fs,direct_IO: Switch to iov_iter and allow bio_vec for ext4
    
    This patchset transforms direct_IO callbacks, blockdev_direct_IO
    and its underlining functions to iov_iter, and introduces complete
    support of iov_iter for ext4.
    
    Supported iov_iter subtypes for ext4 is iovec and bio_vec. The first
    is for traditional user-submitted aio, while bio_vec is the type,
    which is important for us, since we use it in ploop.
    
    bio_vec operates with pages instead of user addresses (like iovec
    does), so it requires specific callbacks in do_blockdev_direct_IO()
    and in the functions it calls.
    
    The patchset reworks do_blockdev_direct_IO() in the same manner
    as in mainstrean. The most of rest patches are prepared manually,
    since we have significant differences to ms (RHEL7 patches, our
    direct IO patches for FUSE; all they have changed many functions).
    At the end, kaio engine (resulting in direct_IO) became possible
    to be enabled for ext4.
    
    https://jira.sw.ru/browse/PSBM-99793
---
 fs/ext4/file.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/fs/ext4/file.c b/fs/ext4/file.c
index 36622e82d152f..43c37e05cdb34 100644
--- a/fs/ext4/file.c
+++ b/fs/ext4/file.c
@@ -273,14 +273,10 @@ ext4_file_write(struct kiocb *iocb, const struct iovec *iov,
 #endif
 
 	iocb->private = &overwrite; /* RHEL7 only - prevent DIO race */
-	if (unlikely(io_is_direct(iocb->ki_filp))) {
-		struct iov_iter iter;
-
-		iov_iter_init(&iter, iov, nr_segs, iov_length(iov, nr_segs), 0);
-
+	if (unlikely(io_is_direct(iocb->ki_filp)))
 		ret = ext4_file_dio_write(iocb, &iter, pos);
-	} else
-		ret = generic_file_aio_write(iocb, iov, nr_segs, pos);
+	else
+		ret = generic_file_write_iter(iocb, &iter, pos);
 
 	return ret;
 }


More information about the Devel mailing list