Thread (19 messages) flat view 19 messages, 5 authors, 2012-11-21
STALE5046d

[PATCH v1 0/9] fs: fix up AIO+DIO+O_SYNC to actually do the sync part

From: Darrick J. Wong <hidden>
Date: 2012-11-20 07:41:16
Also in: linux-fsdevel, linux-xfs, lkml

Hi everybody,

On March 29th, Jeff Moyer posted to lkml a patchset with this note:
Currently, AIO+DIO+O_SYNC writes are not actually sync'd (for xfs), or they
are sync'd before the I/O is actually issued (everybody else).  The following
patch series fixes this in two parts.  First, for the file systems that use
the generic routines, Jan has provided some generic infrastructure to perform
the syncs after the I/O is completed.  Second, for those file systems which
require some endio processing of their own for O_DIRECT writes (xfs and
ext4), [Jeff] implemented file system specific syncing.  This passes the
updated xfs-tests 113 test [Jeff] posted earlier, as well as all of the tests
in the aio group.  [Jeff] tested ext3, ext4, xfs, and btrfs only.
Since the original post a few months ago, this patchset doesn't seem to have
made any progress.  An internal testing team here discovered that the issue
also affects O_SYNC+AIO+DIO writes to block devices.  Worse yet, since the
flushes were being issued (and waited upon) directly in the io_submit call
graph, the io_submit calls themselves would take a very long time to complete.
Therefore, I added another patch to move the flush to the io_end processing.

The blockdev patch was written by me.  The ext4 patch had to be updated to
accomodate a rework of the ext4 endio code that landed since March.  Everything
else has been passed through from Jeff's March 30th resend, with few changes.

This patchset has been tested (albeit lightly) against 3.7-rc6 on x64, with
ext4, xfs, btrfs, vfat, jfs, hfsplus, ext2, ext3, and raw block devices.

Comments and questions are, as always, welcome.

--D
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help