Josef Bacik 7bddf18f47 btrfs: don't drop extent_map for free space inode on write error ... commit 5571e41ec6 upstream. While running the CI for an unrelated change I hit the following panic with generic/648 on btrfs_holes_spacecache. assertion failed: block_start != EXTENT_MAP_HOLE, in fs/btrfs/extent_io.c:1385 ------------[ cut here ]------------ kernel BUG at fs/btrfs/extent_io.c:1385! invalid opcode: 0000 [#1] PREEMPT SMP NOPTI CPU: 1 PID: 2695096 Comm: fsstress Kdump: loaded Tainted: G W 6.8.0-rc2+ #1 RIP: 0010:__extent_writepage_io.constprop.0+0x4c1/0x5c0 Call Trace: <TASK> extent_write_cache_pages+0x2ac/0x8f0 extent_writepages+0x87/0x110 do_writepages+0xd5/0x1f0 filemap_fdatawrite_wbc+0x63/0x90 __filemap_fdatawrite_range+0x5c/0x80 btrfs_fdatawrite_range+0x1f/0x50 btrfs_write_out_cache+0x507/0x560 btrfs_write_dirty_block_groups+0x32a/0x420 commit_cowonly_roots+0x21b/0x290 btrfs_commit_transaction+0x813/0x1360 btrfs_sync_file+0x51a/0x640 __x64_sys_fdatasync+0x52/0x90 do_syscall_64+0x9c/0x190 entry_SYSCALL_64_after_hwframe+0x6e/0x76 This happens because we fail to write out the free space cache in one instance, come back around and attempt to write it again. However on the second pass through we go to call btrfs_get_extent() on the inode to get the extent mapping. Because this is a new block group, and with the free space inode we always search the commit root to avoid deadlocking with the tree, we find nothing and return a EXTENT_MAP_HOLE for the requested range. This happens because the first time we try to write the space cache out we hit an error, and on an error we drop the extent mapping. This is normal for normal files, but the free space cache inode is special. We always expect the extent map to be correct. Thus the second time through we end up with a bogus extent map. Since we're deprecating this feature, the most straightforward way to fix this is to simply skip dropping the extent map range for this failed range. I shortened the test by using error injection to stress the area to make it easier to reproduce. With this patch in place we no longer panic with my error injection test. CC: stable@vger.kernel.org # 4.14+ Reviewed-by: Filipe Manana <fdmanana@suse.com> Signed-off-by: Josef Bacik <josef@toxicpanda.com> Signed-off-by: David Sterba <dsterba@suse.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2024-02-23 09:24:48 +01:00
..
tests
accessors.c
accessors.h
acl.c
acl.h
async-thread.c
async-thread.h
backref.c
backref.h
bio.c
bio.h
block-group.c	btrfs: do not delete unused block group if it may be used soon	2024-02-23 09:24:47 +01:00
block-group.h	btrfs: add and use helper to check if block group is used	2024-02-23 09:24:47 +01:00
block-rsv.c
block-rsv.h
btrfs_inode.h
check-integrity.c
check-integrity.h
compression.c
compression.h
ctree.c
ctree.h
defrag.c
defrag.h
delalloc-space.c	btrfs: don't reserve space for checksums when writing to nocow files	2024-02-23 09:24:48 +01:00
delalloc-space.h
delayed-inode.c
delayed-inode.h
delayed-ref.c
delayed-ref.h
dev-replace.c
dev-replace.h
dir-item.c
dir-item.h
discard.c
discard.h
disk-io.c	btrfs: do not ASSERT() if the newly created subvolume already got read	2024-02-23 09:24:47 +01:00
disk-io.h
export.c
export.h
extent_io.c
extent_io.h
extent_map.c
extent_map.h
extent-io-tree.c
extent-io-tree.h
extent-tree.c	btrfs: zoned: optimize hint byte for zoned allocator	2024-01-31 16:19:10 -08:00
extent-tree.h
file-item.c
file-item.h
file.c
file.h
free-space-cache.c
free-space-cache.h
free-space-tree.c
free-space-tree.h
fs.c
fs.h
inode-item.c
inode-item.h
inode.c	btrfs: don't drop extent_map for free space inode on write error	2024-02-23 09:24:48 +01:00
ioctl.c	btrfs: forbid creating subvol qgroups	2024-02-23 09:24:47 +01:00
ioctl.h
Kconfig
locking.c
locking.h
lru_cache.c
lru_cache.h
lzo.c
Makefile
messages.c
messages.h
misc.h
ordered-data.c
ordered-data.h
orphan.c
orphan.h
print-tree.c
print-tree.h
props.c
props.h
qgroup.c	btrfs: forbid deleting live subvol qgroup	2024-02-23 09:24:48 +01:00
qgroup.h
raid56.c
raid56.h
rcu-string.h
ref-verify.c	btrfs: ref-verify: free ref cache before clearing mount opt	2024-01-31 16:19:06 -08:00
ref-verify.h
reflink.c
reflink.h
relocation.c
relocation.h
root-tree.c
root-tree.h
scrub.c
scrub.h
send.c	btrfs: send: return EOPNOTSUPP on unknown flags	2024-02-23 09:24:48 +01:00
send.h
space-info.c
space-info.h
subpage.c
subpage.h
super.c
super.h
sysfs.c
sysfs.h
transaction.c
transaction.h
tree-checker.c	btrfs: tree-checker: fix inline ref size in error messages	2024-01-31 16:19:06 -08:00
tree-checker.h
tree-log.c
tree-log.h
tree-mod-log.c
tree-mod-log.h
ulist.c
ulist.h
uuid-tree.c
uuid-tree.h
verity.c
verity.h
volumes.c
volumes.h
xattr.c
xattr.h
zlib.c
zoned.c	btrfs: zoned: fix lock ordering in btrfs_zone_activate()	2024-01-31 16:19:05 -08:00
zoned.h
zstd.c