diff options
| author | Kent Overstreet | 2013-12-17 01:38:49 +0100 |
|---|---|---|
| committer | Kent Overstreet | 2014-01-08 22:05:09 +0100 |
| commit | 5f5837d2d650db25b9153b91535e67a96b265f58 (patch) | |
| tree | 48afdf71783484950632a95972cad6e9608332c7 /drivers/md/bcache | |
| parent | bcache: Rework allocator reserves (diff) | |
| download | kernel-qcow2-linux-5f5837d2d650db25b9153b91535e67a96b265f58.tar.gz kernel-qcow2-linux-5f5837d2d650db25b9153b91535e67a96b265f58.tar.xz kernel-qcow2-linux-5f5837d2d650db25b9153b91535e67a96b265f58.zip | |
bcache: Do bkey_put() in btree_split() error path
This error path shouldn't have been hit in practice.. and we've got reworked
reserve code coming soon so that it shouldn't _ever_ be bit... but if we've got
code for this error path it should be correct.
Signed-off-by: Kent Overstreet <kmo@daterainc.com>
Diffstat (limited to 'drivers/md/bcache')
| -rw-r--r-- | drivers/md/bcache/btree.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/md/bcache/btree.c b/drivers/md/bcache/btree.c index 6a0f5faf0bed..5a6b7522eb20 100644 --- a/drivers/md/bcache/btree.c +++ b/drivers/md/bcache/btree.c @@ -2191,18 +2191,21 @@ static int btree_split(struct btree *b, struct btree_op *op, return 0; err_free2: + bkey_put(b->c, &n2->key); btree_node_free(n2); rw_unlock(true, n2); err_free1: + bkey_put(b->c, &n1->key); btree_node_free(n1); rw_unlock(true, n1); err: + WARN(1, "bcache: btree split failed"); + if (n3 == ERR_PTR(-EAGAIN) || n2 == ERR_PTR(-EAGAIN) || n1 == ERR_PTR(-EAGAIN)) return -EAGAIN; - pr_warn("couldn't split"); return -ENOMEM; } |