diff options
| author | Chris Mason <chris.mason@oracle.com> | 2009-04-24 20:39:24 +0200 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2009-04-24 21:46:05 +0200 |
| commit | e980b50cda1610f1c17978d9b7fd311a9dd93877 (patch) | |
| tree | a5ab72fa4b791758e25fd6ece6f1aedd29954e7b /fs/btrfs/ctree.h | |
| parent | Btrfs: kill btrfs_cache_create (diff) | |
| download | linux-e980b50cda1610f1c17978d9b7fd311a9dd93877.tar.xz linux-e980b50cda1610f1c17978d9b7fd311a9dd93877.zip | |
Btrfs: fix fallocate deadlock on inode extent lock
The btrfs fallocate call takes an extent lock on the entire range
being fallocated, and then runs through insert_reserved_extent on each
extent as they are allocated.
The problem with this is that btrfs_drop_extents may decide to try
and take the same extent lock fallocate was already holding. The solution
used here is to push down knowledge of the range that is already locked
going into btrfs_drop_extents.
It turns out that at least one other caller had the same bug.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index 213535f45da2..4414a5d9983a 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -2177,7 +2177,8 @@ int btrfs_check_file(struct btrfs_root *root, struct inode *inode); extern struct file_operations btrfs_file_operations; int btrfs_drop_extents(struct btrfs_trans_handle *trans, struct btrfs_root *root, struct inode *inode, - u64 start, u64 end, u64 inline_limit, u64 *hint_block); + u64 start, u64 end, u64 locked_end, + u64 inline_limit, u64 *hint_block); int btrfs_mark_extent_written(struct btrfs_trans_handle *trans, struct btrfs_root *root, struct inode *inode, u64 start, u64 end); |