From 42ce128855218b38aabd83d6f093b31544cb5b83 Mon Sep 17 00:00:00 2001
From: Sage Weil <sage@redhat.com>
Date: Fri, 6 May 2016 14:36:04 -0400
Subject: [PATCH] os/bluestore: remove block_map from onode

Signed-off-by: Sage Weil <sage@redhat.com>
---
 src/os/bluestore/bluestore_types.cc | 11 -----------
 src/os/bluestore/bluestore_types.h  | 25 -------------------------
 2 files changed, 36 deletions(-)

diff --git a/src/os/bluestore/bluestore_types.cc b/src/os/bluestore/bluestore_types.cc
index cf7889050550f..039149f27f973 100644
--- a/src/os/bluestore/bluestore_types.cc
+++ b/src/os/bluestore/bluestore_types.cc
@@ -539,7 +539,6 @@ void bluestore_onode_t::encode(bufferlist& bl) const
   ::encode(size, bl);
   ::encode(attrs, bl);
   ::encode(blob_map, bl);
-  ::encode(block_map, bl);
   ::encode(extent_map, bl);
   ::encode(overlay_map, bl);
   ::encode(overlay_refs, bl);
@@ -558,7 +557,6 @@ void bluestore_onode_t::decode(bufferlist::iterator& p)
   ::decode(size, p);
   ::decode(attrs, p);
   ::decode(blob_map, p);
-  ::decode(block_map, p);
   ::decode(extent_map, p);
   ::decode(overlay_map, p);
   ::decode(overlay_refs, p);
@@ -583,15 +581,6 @@ void bluestore_onode_t::dump(Formatter *f) const
     f->close_section();
   }
   f->close_section();
-  f->open_object_section("block_map");
-  for (map<uint64_t, bluestore_extent_t>::const_iterator p = block_map.begin();
-       p != block_map.end(); ++p) {
-    f->open_object_section("extent");
-    f->dump_unsigned("extent_offset", p->first);
-    p->second.dump(f);
-    f->close_section();
-  }
-  f->close_section();
   f->open_object_section("blob_map");
   for (const auto& p : blob_map) {
     f->open_object_section("blob");
diff --git a/src/os/bluestore/bluestore_types.h b/src/os/bluestore/bluestore_types.h
index d751d37dbc36e..fcc488cba8829 100644
--- a/src/os/bluestore/bluestore_types.h
+++ b/src/os/bluestore/bluestore_types.h
@@ -447,7 +447,6 @@ struct bluestore_onode_t {
   uint64_t nid;                        ///< numeric id (locally unique)
   uint64_t size;                       ///< object size
   map<string, bufferptr> attrs;        ///< attrs
-  map<uint64_t, bluestore_extent_t> block_map;   ///< block data
   map<uint64_t,bluestore_lextent_t> extent_map;  ///< extent refs
   map<uint64_t,bluestore_overlay_t> overlay_map; ///< overlay data (stored in db)
   map<uint64_t,uint16_t> overlay_refs; ///< overlay keys ref counts (if >1)
@@ -468,30 +467,6 @@ struct bluestore_onode_t {
       expected_write_size(0),
       alloc_hint_flags(0) {}
 
-  map<uint64_t,bluestore_extent_t>::iterator find_extent(uint64_t offset) {
-    map<uint64_t,bluestore_extent_t>::iterator fp = block_map.lower_bound(offset);
-    if (fp != block_map.begin()) {
-      --fp;
-      if (fp->first + fp->second.length <= offset) {
-	++fp;
-      }
-    }
-    if (fp != block_map.end() && fp->first > offset)
-      return block_map.end();  // extent is past offset
-    return fp;
-  }
-
-  map<uint64_t,bluestore_extent_t>::iterator seek_extent(uint64_t offset) {
-    map<uint64_t,bluestore_extent_t>::iterator fp = block_map.lower_bound(offset);
-    if (fp != block_map.begin()) {
-      --fp;
-      if (fp->first + fp->second.length <= offset) {
-	++fp;
-      }
-    }
-    return fp;
-  }
-
   map<uint64_t,bluestore_lextent_t>::iterator find_lextent(uint64_t offset) {
     map<uint64_t,bluestore_lextent_t>::iterator fp =
       extent_map.lower_bound(offset);
-- 
2.39.5