| From c4427191b5fc32fa04a212e145479707d155009a Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Wed, 16 Nov 2022 22:23:54 +0800 |
| Subject: btrfs: qgroup: fix sleep from invalid context bug in |
| btrfs_qgroup_inherit() |
| |
| From: ChenXiaoSong <chenxiaosong2@huawei.com> |
| |
| [ Upstream commit f7e942b5bb35d8e3af54053d19a6bf04143a3955 ] |
| |
| Syzkaller reported BUG as follows: |
| |
| BUG: sleeping function called from invalid context at |
| include/linux/sched/mm.h:274 |
| Call Trace: |
| <TASK> |
| dump_stack_lvl+0xcd/0x134 |
| __might_resched.cold+0x222/0x26b |
| kmem_cache_alloc+0x2e7/0x3c0 |
| update_qgroup_limit_item+0xe1/0x390 |
| btrfs_qgroup_inherit+0x147b/0x1ee0 |
| create_subvol+0x4eb/0x1710 |
| btrfs_mksubvol+0xfe5/0x13f0 |
| __btrfs_ioctl_snap_create+0x2b0/0x430 |
| btrfs_ioctl_snap_create_v2+0x25a/0x520 |
| btrfs_ioctl+0x2a1c/0x5ce0 |
| __x64_sys_ioctl+0x193/0x200 |
| do_syscall_64+0x35/0x80 |
| |
| Fix this by calling qgroup_dirty() on @dstqgroup, and update limit item in |
| btrfs_run_qgroups() later outside of the spinlock context. |
| |
| CC: stable@vger.kernel.org # 4.9+ |
| Reviewed-by: Qu Wenruo <wqu@suse.com> |
| Signed-off-by: ChenXiaoSong <chenxiaosong2@huawei.com> |
| Reviewed-by: David Sterba <dsterba@suse.com> |
| Signed-off-by: David Sterba <dsterba@suse.com> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| fs/btrfs/qgroup.c | 9 +-------- |
| 1 file changed, 1 insertion(+), 8 deletions(-) |
| |
| diff --git a/fs/btrfs/qgroup.c b/fs/btrfs/qgroup.c |
| index ba323dcb0a0b..db56e0c0e9ac 100644 |
| --- a/fs/btrfs/qgroup.c |
| +++ b/fs/btrfs/qgroup.c |
| @@ -2920,14 +2920,7 @@ int btrfs_qgroup_inherit(struct btrfs_trans_handle *trans, u64 srcid, |
| dstgroup->rsv_rfer = inherit->lim.rsv_rfer; |
| dstgroup->rsv_excl = inherit->lim.rsv_excl; |
| |
| - ret = update_qgroup_limit_item(trans, dstgroup); |
| - if (ret) { |
| - fs_info->qgroup_flags |= BTRFS_QGROUP_STATUS_FLAG_INCONSISTENT; |
| - btrfs_info(fs_info, |
| - "unable to update quota limit for %llu", |
| - dstgroup->qgroupid); |
| - goto unlock; |
| - } |
| + qgroup_dirty(fs_info, dstgroup); |
| } |
| |
| if (srcid) { |
| -- |
| 2.35.1 |
| |