summaryrefslogtreecommitdiff
path: root/fs/erofs
diff options
context:
space:
mode:
authorDavid Sterba <dsterba@suse.com>2020-06-09 19:56:06 +0200
committerDavid Sterba <dsterba@suse.com>2020-06-14 01:19:02 +0200
commit55e20bd12a56e06c38b953177bb162cbbaa96004 (patch)
treeb19873031657b8f2c5870cfbc683c67c29547733 /fs/erofs
parentf1084bc60a1ec592011a805a0f925f2f1205897d (diff)
Revert "btrfs: switch to iomap_dio_rw() for dio"
This reverts commit a43a67a2d715540c1368b9501a22b0373b5874c0. This patch reverts the main part of switching direct io implementation to iomap infrastructure. There's a problem in invalidate page that couldn't be solved as regression in this development cycle. The problem occurs when buffered and direct io are mixed, and the ranges overlap. Although this is not recommended, filesystems implement measures or fallbacks to make it somehow work. In this case, fallback to buffered IO would be an option for btrfs (this already happens when direct io is done on compressed data), but the change would be needed in the iomap code, bringing new semantics to other filesystems. Another problem arises when again the buffered and direct ios are mixed, invalidation fails, then -EIO is set on the mapping and fsync will fail, though there's no real error. There have been discussions how to fix that, but revert seems to be the least intrusive option. Link: https://lore.kernel.org/linux-btrfs/20200528192103.xm45qoxqmkw7i5yl@fiona/ Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/erofs')
0 files changed, 0 insertions, 0 deletions