diff options
| author | Goldwyn Rodrigues <rgoldwyn@suse.com> | 2017-03-14 05:25:09 -0500 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2017-03-29 14:29:08 +0200 |
| commit | ce0dcee626c482183b42d45b6ea43198c7223fc7 (patch) | |
| tree | c3f9bf304edd911f25f5c26f73039f3e471afed2 /fs/btrfs/ctree.h | |
| parent | 9d0d1c8b1c9d80b17cfa86ecd50c8933a742585c (diff) | |
| download | linux-ce0dcee626c482183b42d45b6ea43198c7223fc7.tar.gz linux-ce0dcee626c482183b42d45b6ea43198c7223fc7.tar.bz2 linux-ce0dcee626c482183b42d45b6ea43198c7223fc7.zip | |
btrfs: Change qgroup_meta_rsv to 64bit
Using an int value is causing qg->reserved to become negative and
exclusive -EDQUOT to be reached prematurely.
This affects exclusive qgroups only.
TEST CASE:
DEVICE=/dev/vdb
MOUNTPOINT=/mnt
SUBVOL=$MOUNTPOINT/tmp
umount $SUBVOL
umount $MOUNTPOINT
mkfs.btrfs -f $DEVICE
mount /dev/vdb $MOUNTPOINT
btrfs quota enable $MOUNTPOINT
btrfs subvol create $SUBVOL
umount $MOUNTPOINT
mount /dev/vdb $MOUNTPOINT
mount -o subvol=tmp $DEVICE $SUBVOL
btrfs qgroup limit -e 3G $SUBVOL
btrfs quota rescan /mnt -w
for i in `seq 1 44000`; do
dd if=/dev/zero of=/mnt/tmp/test_$i bs=10k count=1
if [[ $? > 0 ]]; then
btrfs qgroup show -pcref $SUBVOL
exit 1
fi
done
Signed-off-by: Goldwyn Rodrigues <rgoldwyn@suse.com>
[ add reproducer to changelog ]
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index f03c2f285eb1..660d485b6e8b 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -1258,7 +1258,7 @@ struct btrfs_root { atomic_t will_be_snapshoted; /* For qgroup metadata space reserve */ - atomic_t qgroup_meta_rsv; + atomic64_t qgroup_meta_rsv; }; static inline u32 btrfs_inode_sectorsize(const struct inode *inode) { |