summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorSage Weil <sage@inktank.com>2012-05-21 09:45:23 -0700
committerSage Weil <sage@inktank.com>2012-05-21 20:40:42 -0700
commit6bd9adbdf9ca6a052b0b7455ac67b925eb38cfad (patch)
tree61d254846558e23906bbef993092cf372f641509 /net
parent35f9f8a09e1e88e31bd34a1e645ca0e5f070dd5c (diff)
downloadop-kernel-dev-6bd9adbdf9ca6a052b0b7455ac67b925eb38cfad.zip
op-kernel-dev-6bd9adbdf9ca6a052b0b7455ac67b925eb38cfad.tar.gz
libceph: fix pg_temp updates
Usually, we are adding pg_temp entries or removing them. Occasionally they update. In that case, osdmap_apply_incremental() was failing because the rbtree entry already exists. Fix by removing the existing entry before inserting a new one. Fixes http://tracker.newdream.net/issues/2446 Signed-off-by: Sage Weil <sage@inktank.com> Reviewed-by: Alex Elder <elder@inktank.com>
Diffstat (limited to 'net')
-rw-r--r--net/ceph/osdmap.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/net/ceph/osdmap.c b/net/ceph/osdmap.c
index 2592f3c..1892c52 100644
--- a/net/ceph/osdmap.c
+++ b/net/ceph/osdmap.c
@@ -883,8 +883,12 @@ struct ceph_osdmap *osdmap_apply_incremental(void **p, void *end,
pglen = ceph_decode_32(p);
if (pglen) {
- /* insert */
ceph_decode_need(p, end, pglen*sizeof(u32), bad);
+
+ /* removing existing (if any) */
+ (void) __remove_pg_mapping(&map->pg_temp, pgid);
+
+ /* insert */
pg = kmalloc(sizeof(*pg) + sizeof(u32)*pglen, GFP_NOFS);
if (!pg) {
err = -ENOMEM;
OpenPOWER on IntegriCloud