]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
Merge pull request #10431 from ifed01/wip-bluestore-fix-csum-disable2
authorSage Weil <sage@redhat.com>
Sat, 13 Aug 2016 20:33:14 +0000 (16:33 -0400)
committerGitHub <noreply@github.com>
Sat, 13 Aug 2016 20:33:14 +0000 (16:33 -0400)
os/bluestore: proper handling for csum enable/disable settings

Reviewed-by: Sage Weil <sage@redhat.com>
1  2 
src/os/bluestore/BlueStore.cc
src/os/bluestore/bluestore_types.h

Simple merge
index 2c2446a2f5f3d7ae451785aa1a0d89f87e8a6873,c445f06d552987617738f4d1485df3467212baae..de7dcd0767a683712ed0e95ca093bfc22e1be6e2
@@@ -337,13 -326,10 +337,13 @@@ struct bluestore_blob_t 
    bool has_unused() const {
      return has_flag(FLAG_HAS_UNUSED);
    }
 +  bool has_refmap() const {
 +    return has_flag(FLAG_HAS_REFMAP);
 +  }
  
    /// return chunk (i.e. min readable block) size for the blob
-   uint64_t get_chunk_size(uint64_t dev_block_size) {
-     return has_csum() ? MAX(dev_block_size, get_csum_chunk_size()) : dev_block_size;
+   uint64_t get_chunk_size(bool csum_enabled, uint64_t dev_block_size) {
+     return csum_enabled && has_csum() ? MAX(dev_block_size, get_csum_chunk_size()) : dev_block_size;
    }
    uint32_t get_csum_chunk_size() const {
      return 1 << csum_chunk_order;