Does this make sense ? I can prepare a patch. diff --git a/src/crush/builder.c b/src/crush/builder.c index c319620..5700a88 100644 --- a/src/crush/builder.c +++ b/src/crush/builder.c @@ -651,7 +651,7 @@ int crush_add_tree_bucket_item(struct crush_bucket_tree *bucket, int item, int w for (j=1; j<depth; j++) { node = parent(node); - if (!crush_addition_is_unsafe(bucket->node_weights[node], weight)) + if (crush_addition_is_unsafe(bucket->node_weights[node], weight)) return -ERANGE; bucket->node_weights[node] += weight; -- VKM -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html