diff options
| author | Liu Bo <bo.li.liu@oracle.com> | 2014-07-24 22:48:05 +0800 |
|---|---|---|
| committer | Jiri Slaby <jslaby@suse.cz> | 2014-09-17 15:12:20 +0200 |
| commit | 28758933a321c0eb053fa6f85c9426882bf7e44c (patch) | |
| tree | e3aaf5c961317b3eedd43c9aed93ffcd67c024aa /fs | |
| parent | 308d30d163995b77b264982dd463ae05aedeb0f9 (diff) | |
Btrfs: fix compressed write corruption on enospc
commit ce62003f690dff38d3164a632ec69efa15c32cbf upstream.
When failing to allocate space for the whole compressed extent, we'll
fallback to uncompressed IO, but we've forgotten to redirty the pages
which belong to this compressed extent, and these 'clean' pages will
simply skip 'submit' part and go to endio directly, at last we got data
corruption as we write nothing.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Tested-By: Martin Steigerwald <martin@lichtvoll.de>
Signed-off-by: Chris Mason <clm@fb.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/inode.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index fa8010c1b628..7e6758d075ad 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -683,6 +683,18 @@ retry: unlock_extent(io_tree, async_extent->start, async_extent->start + async_extent->ram_size - 1); + + /* + * we need to redirty the pages if we decide to + * fallback to uncompressed IO, otherwise we + * will not submit these pages down to lower + * layers. + */ + extent_range_redirty_for_io(inode, + async_extent->start, + async_extent->start + + async_extent->ram_size - 1); + goto retry; } goto out_free; |