diff options
| author | David Sterba <dsterba@suse.cz> | 2011-06-14 12:52:17 +0200 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2011-10-24 14:48:00 +0200 |
| commit | dff51cd1c60856c28f5d22a571294c2b70b6b322 (patch) | |
| tree | fe310f20368175086107460a247d5e6aa6d82e36 /fs | |
| parent | Merge branch 'hotfixes-20111024/josef/for-chris' into btrfs-next-stable (diff) | |
| download | linux-dff51cd1c60856c28f5d22a571294c2b70b6b322.tar.xz linux-dff51cd1c60856c28f5d22a571294c2b70b6b322.zip | |
btrfs: ratelimit WARN_ON in use_block_rsv
The WARN_ON under some circumstances heavily polute log and slow down
the machine. This is just a safety, as the warning should be fixed by
another patch, nevertheless, it still pops up during testing.
Signed-off-by: David Sterba <dsterba@suse.cz>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/extent-tree.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index cef355f1328a..28c4809851a5 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -23,6 +23,7 @@ #include <linux/rcupdate.h> #include <linux/kthread.h> #include <linux/slab.h> +#include <linux/ratelimit.h> #include "compat.h" #include "hash.h" #include "ctree.h" @@ -5783,7 +5784,13 @@ use_block_rsv(struct btrfs_trans_handle *trans, if (!ret) return block_rsv; if (ret) { - WARN_ON(1); + static DEFINE_RATELIMIT_STATE(_rs, + DEFAULT_RATELIMIT_INTERVAL, + /*DEFAULT_RATELIMIT_BURST*/ 2); + if (__ratelimit(&_rs)) { + printk(KERN_DEBUG "btrfs: block rsv returned %d\n", ret); + WARN_ON(1); + } ret = reserve_metadata_bytes(root, block_rsv, blocksize, 0); if (!ret) { return block_rsv; |