]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
Merge pull request #9988 from ifed01/wip-bluestore-reduce-blob4
authorMark Nelson <mark.a.nelson@gmail.com>
Thu, 28 Jul 2016 14:53:01 +0000 (09:53 -0500)
committerGitHub <noreply@github.com>
Thu, 28 Jul 2016 14:53:01 +0000 (09:53 -0500)
commitf8c0e344abd157b04e72b4e7f6f7b31e2ee9957d
tree6de2f577a037ca13e49c0f4d18878b51d9e1f049
parentef394161730b0b66b9dea53c3d9497d775a99716
parent9deee08920681f593773a96507ea2a5abde4ad66
Merge pull request #9988 from ifed01/wip-bluestore-reduce-blob4

os/bluestore: get rid off blob's ref_map for non-shared objects

Mark's Comments:

This passed Jenkins checks.
This PR did not appear to have a significant impact on performance tests.
This initially failed "ceph_test_objectstore --gtest_filter=*/2" segfaulting in one of the tests.  See the attached log in the comments for the PR for details.  Subsequent reruns passed, and a run through valgrind's memcheck also passed.  Igor indicated that he has seen this particular segfault previously and believes it is related to running out of memory.  Given that the PR has passed the object store tests multiple times and the segfault has been observed in the past, I believe this PR is unrelated.

Reviewed-by: Mark Nelson <mnelson@redhat.com>
src/os/bluestore/BlueStore.cc
src/os/bluestore/BlueStore.h
src/os/bluestore/bluestore_types.cc
src/os/bluestore/bluestore_types.h