From: Igor Fedotov <ifedotov@mirantis.com>
Date: Thu, 12 May 2016 13:26:10 +0000 (+0300)
Subject: os/bluestore: read patch code cleanup
X-Git-Tag: v11.0.0~359^2~85
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=60576789be374dab4047c40dba8778705521f113;p=ceph.git

os/bluestore: read patch code cleanup

Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
---

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 5c3f9fd1fdbf..7805babf9502 100644
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -2766,8 +2766,6 @@ int BlueStore::_do_read(
   map<uint64_t,bluestore_lextent_t>::iterator ep, eend;
   uint64_t block_size = bdev->get_block_size();
   int r = 0;
-//  IOContext ioc(NULL);   // FIXME?
-//  BnodeRef bnode;
 
   // generally, don't buffer anything, unless the client explicitly requests
   // it.
@@ -2952,7 +2950,6 @@ int BlueStore::_read_extent_sparse(
   //FIXME: this is a trivial implementation that reads each region independently - can be improved to read neighboring and/or close enough regions together.
 
   IOContext ioc(NULL);   // FIXME?
-  //uint64_t block_size = get_read_block_size(blob);
   uint64_t block_size = bdev->get_block_size();
   if (blob->csum_type != bluestore_blob_t::CSUM_NONE)
     block_size = MAX(blob->get_csum_block_size(), block_size);
@@ -3070,8 +3067,6 @@ int BlueStore::_verify_csum(const bluestore_blob_t* blob, uint64_t blob_xoffset,
   start += block0 * csum_len;
   end += (block0 + blocks) * csum_len;
 
-//  std::copy(start, end, csum_data.begin());
-
   checksummer->calculate(
     blob->csum_type,
     blob->get_csum_block_size(),