Hi, Sam why we need trim_pushed_data() in handle_pull_response(). I think data_included is always a subset of recovery_info.copy_subset since the logic in build_push_op tells us that, isn't it? Regards Ning Yao -- 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