]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
os/bluestore: use pextent_t for releases
authorSage Weil <sage@redhat.com>
Tue, 3 May 2016 20:22:24 +0000 (16:22 -0400)
committerSage Weil <sage@redhat.com>
Wed, 1 Jun 2016 15:38:42 +0000 (11:38 -0400)
Trying to remove the old extent_t

Signed-off-by: Sage Weil <sage@redhat.com>
src/os/bluestore/BlueStore.cc
src/os/bluestore/bluestore_types.cc
src/os/bluestore/bluestore_types.h
src/test/objectstore/test_bluestore_types.cc

index a6ef7c9aab8f43c8fc253287adc878cd44c6fdba..3fc8b1fc45392e3ec1e4178a49ddcc06aa2f2896 100644 (file)
@@ -3664,7 +3664,7 @@ void BlueStore::_txc_release(
   bool shared)
 {
   if (shared) {
-    vector<bluestore_extent_t> release;
+    vector<bluestore_pextent_t> release;
     if (!o->enode)
       o->enode = c->get_enode(o->oid.hobj.get_hash());
     o->enode->ref_map.put(offset, length, &release);
index 579e4077927ba249ecc1347870ab9c6cb21ac606..1f4c5dc784f978ff5fc1140c48a3be1902be7a34 100644 (file)
@@ -213,8 +213,9 @@ void bluestore_extent_ref_map_t::get(uint64_t offset, uint32_t length)
   _check();
 }
 
-void bluestore_extent_ref_map_t::put(uint64_t offset, uint32_t length,
-                          vector<bluestore_extent_t> *release)
+void bluestore_extent_ref_map_t::put(
+  uint64_t offset, uint32_t length,
+  vector<bluestore_pextent_t> *release)
 {
   map<uint64_t,record_t>::iterator p = ref_map.lower_bound(offset);
   if (p == ref_map.end() || p->first > offset) {
@@ -243,7 +244,7 @@ void bluestore_extent_ref_map_t::put(uint64_t offset, uint32_t length,
        --p->second.refs;
        _maybe_merge_left(p);
       } else {
-       release->push_back(bluestore_extent_t(p->first, length));
+       release->push_back(bluestore_pextent_t(p->first, length));
        ref_map.erase(p);
       }
       return;
@@ -255,7 +256,7 @@ void bluestore_extent_ref_map_t::put(uint64_t offset, uint32_t length,
       _maybe_merge_left(p);
       ++p;
     } else {
-      release->push_back(bluestore_extent_t(p->first, p->second.length));
+      release->push_back(bluestore_pextent_t(p->first, p->second.length));
       ref_map.erase(p++);
     }
   }
index 6f7f4b56d291cd4932ed15bd5c8d53d4bea550f5..480be8121e4ffbca07f633a6d7c134d303dc1928 100644 (file)
@@ -158,7 +158,7 @@ struct bluestore_extent_ref_map_t {
 
   void add(uint64_t offset, uint32_t len, unsigned ref=2);
   void get(uint64_t offset, uint32_t len);
-  void put(uint64_t offset, uint32_t len, vector<bluestore_extent_t> *release);
+  void put(uint64_t offset, uint32_t len, vector<bluestore_pextent_t> *release);
 
   bool contains(uint64_t offset, uint32_t len) const;
 
index bfe46d41e70cbdfa2564f39a48a973b45501604d..abed0b4d07335b78fb5dd485adc8e77588d56308 100644 (file)
@@ -80,7 +80,7 @@ TEST(bluestore_extent_ref_map_t, get)
 TEST(bluestore_extent_ref_map_t, put)
 {
   bluestore_extent_ref_map_t m;
-  vector<bluestore_extent_t> r;
+  vector<bluestore_pextent_t> r;
   m.add(10, 30, 1);
   m.put(10, 30, &r);
   cout << m << " " << r << std::endl;