lkml.org 
[lkml]   [2018]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.9 050/329] Btrfs: set plug for fsync
    Date
    4.9-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Liu Bo <bo.li.liu@oracle.com>

    [ Upstream commit 343e4fc1c60971b0734de26dbbd475d433950982 ]

    Setting plug can merge adjacent IOs before dispatching IOs to the disk
    driver.

    Without plug, it'd not be a problem for single disk usecases, but for
    multiple disks using raid profile, a large IO can be split to several
    IOs of stripe length, and plug can be helpful to bring them together
    for each disk so that we can save several disk access.

    Moreover, fsync issues synchronous writes, so plug can really take
    effect.

    Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
    Reviewed-by: David Sterba <dsterba@suse.com>
    Signed-off-by: David Sterba <dsterba@suse.com>
    Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    fs/btrfs/file.c | 9 +++++++++
    1 file changed, 9 insertions(+)

    --- a/fs/btrfs/file.c
    +++ b/fs/btrfs/file.c
    @@ -1912,10 +1912,19 @@ int btrfs_release_file(struct inode *ino
    static int start_ordered_ops(struct inode *inode, loff_t start, loff_t end)
    {
    int ret;
    + struct blk_plug plug;

    + /*
    + * This is only called in fsync, which would do synchronous writes, so
    + * a plug can merge adjacent IOs as much as possible. Esp. in case of
    + * multiple disks using raid profile, a large IO can be split to
    + * several segments of stripe length (currently 64K).
    + */
    + blk_start_plug(&plug);
    atomic_inc(&BTRFS_I(inode)->sync_writers);
    ret = btrfs_fdatawrite_range(inode, start, end);
    atomic_dec(&BTRFS_I(inode)->sync_writers);
    + blk_finish_plug(&plug);

    return ret;
    }

    \
     
     \ /
      Last update: 2018-05-28 16:46    [W:4.083 / U:0.304 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site