]> git.baikalelectronics.ru Git - kernel.git/commit
ext4: fix extent status tree race in writeback error recovery path
authorEric Whitney <enwlinux@gmail.com>
Wed, 15 Jun 2022 16:05:30 +0000 (12:05 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Wed, 3 Aug 2022 03:52:19 +0000 (23:52 -0400)
commit727d526f4e7996c55cae8f6d0fab2e9d21491d5c
tree7e543b54c94ed02e9bb420acb5f467204f1f5967
parent8745a9b8e458d1612cd8af4826c855f11401167d
ext4: fix extent status tree race in writeback error recovery path

A race can occur in the unlikely event ext4 is unable to allocate a
physical cluster for a delayed allocation in a bigalloc file system
during writeback.  Failure to allocate a cluster forces error recovery
that includes a call to mpage_release_unused_pages().  That function
removes any corresponding delayed allocated blocks from the extent
status tree.  If a new delayed write is in progress on the same cluster
simultaneously, resulting in the addition of an new extent containing
one or more blocks in that cluster to the extent status tree, delayed
block accounting can be thrown off if that delayed write then encounters
a similar cluster allocation failure during future writeback.

Write lock the i_data_sem in mpage_release_unused_pages() to fix this
problem.  Ext4's block/cluster accounting code for bigalloc relies on
i_data_sem for mutual exclusion, as is found in the delayed write path,
and the locking in mpage_release_unused_pages() is missing.

Cc: stable@kernel.org
Reported-by: Ye Bin <yebin10@huawei.com>
Signed-off-by: Eric Whitney <enwlinux@gmail.com>
Link: https://lore.kernel.org/r/20220615160530.1928801-1-enwlinux@gmail.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/inode.c