Patch "btrfs: Handle btrfs_set_extent_delalloc failure in fixup worker" has been added to the 4.14-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
btrfs: Handle btrfs_set_extent_delalloc failure in fixup worker
to the 4.14-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
btrfs-handle-btrfs_set_extent_delalloc-failure-in-fixup-worker.patch
and it can be found in the queue-4.14 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From
f3038ee3a3f1017a1cbe9907e31fa12d366c5dcbMon Sep 17 00:00:00 2001
From: Nikolay Borisov <[email protected]>
Date: Tue, 5 Dec 2017 09:29:19 +0200
Subject: btrfs: Handle btrfs_set_extent_delalloc failure in fixup worker
From: Nikolay Borisov <[email protected]>
commit f3038ee3a3f1017a1cbe9907e31fa12d366c5dcb upstream.
This function was introduced by 247e743cbe6e ("Btrfs: Use async helpers to deal with pages that have been improperly dirtied") and it didn't do any error handling then. This function might very well fail in ENOMEM situation, yet it's not handled, this could lead to inconsistent state. So let's handle the failure by setting the mapping error bit.
--- fs/btrfs/inode.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-)
fs/btrfs/inode.c +9 -2
@@ -2101,8 +2101,15 @@ again: goto out; } -btrfs_set_extent_delalloc (inode, page_start, page_end, &cached_state, -0 ); + ret =btrfs_set_extent_delalloc (inode, page_start, page_end, + &cached_state,0 ); +if (ret) { +mapping_set_error (page->mapping, ret); +end_extent_writepage (page, ret, page_start, page_end); +ClearPageChecked (page); + goto out; + } +ClearPageChecked (page);set_page_dirty (page); out:
Patches currently in stable-queue which might be from [email protected] are
queue-4.14/btrfs-handle-btrfs_set_extent_delalloc-failure-in-fixup-worker.patch