]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: clean up use of hex for digests
authorSage Weil <sage@redhat.com>
Mon, 22 Dec 2014 15:27:10 +0000 (07:27 -0800)
committerSage Weil <sage@redhat.com>
Mon, 22 Dec 2014 15:27:10 +0000 (07:27 -0800)
Signed-off-by: Sage Weil <sage@redhat.com>
src/osd/PGBackend.cc
src/osd/ReplicatedPG.cc

index f0a73ff335c9f8b6e3527a009f0a8fd56a0b566e..eac5dbf5a2ff4f41bfbf6e6e4a7f1d844088fa62 100644 (file)
@@ -390,10 +390,10 @@ enum scrub_error_type PGBackend::be_compare_scrub_objects(
       if (error != CLEAN)
         errorstream << ", ";
       error = DEEP_ERROR;
-      errorstream << "data_digest " << candidate.digest
+      errorstream << "data_digest 0x" << std::hex << candidate.digest
                  << " != "
                  << (auth_oi.is_data_digest() && okseed ? "known" : "best guess")
-                 << " data_digest " << auth.digest
+                 << " data_digest 0x" << auth.digest << std::dec
                  << " from auth shard " << auth_shard;
     }
   }
@@ -402,10 +402,10 @@ enum scrub_error_type PGBackend::be_compare_scrub_objects(
       if (error != CLEAN)
         errorstream << ", ";
       error = DEEP_ERROR;
-      errorstream << "omap_digest " << candidate.omap_digest
+      errorstream << "omap_digest 0x" << std::hex << candidate.omap_digest
                  << " != "
                  << (auth_oi.is_omap_digest() && okseed ? "known" : "best guess")
-                 << " omap_digest " << auth.omap_digest
+                 << " omap_digest 0x" << auth.omap_digest << std::dec
                  << " from auth shard " << auth_shard;
     }
   }
@@ -508,18 +508,18 @@ map<pg_shard_t, ScrubMap *>::const_iterator
        oi.data_digest != i->second.digest) {
       dout(10) << __func__ << ": rejecting osd " << j->first
               << " for obj " << obj
-              << ", data digest mismatch "
-              << i->second.digest << " != " << oi.data_digest
-              << dendl;
+              << ", data digest mismatch 0x" << std::hex
+              << i->second.digest << " != 0x" << oi.data_digest
+              << std::dec << dendl;
       continue;
     }
     if (okseed && oi.is_omap_digest() && i->second.omap_digest_present &&
-       oi.omap_digest != i->second.omap_digest) {
+         oi.omap_digest != i->second.omap_digest) {
       dout(10) << __func__ << ": rejecting osd " << j->first
               << " for obj " << obj
-              << ", omap digest mismatch "
-              << i->second.omap_digest << " != " << oi.omap_digest
-              << dendl;
+              << ", omap digest mismatch 0x" << std::hex
+              << i->second.omap_digest << " != 0x" << oi.omap_digest
+              << std::dec << dendl;
       continue;
     }
     dout(10) << __func__ << ": selecting osd " << j->first
index 7c74eaa24348eb2bff3d4cd4816fcf4df6bd4ea1..c2e56586deb06de3678cdb194720f4d1914229a1 100644 (file)
@@ -3350,8 +3350,8 @@ int ReplicatedPG::do_osd_ops(OpContext *ctx, vector<OSDOp>& ops)
            uint32_t crc = osd_op.outdata.crc32c(-1);
            if (oi.data_digest != crc) {
              osd->clog->error() << info.pgid << std::hex
-                                << " full-object read crc " << crc
-                                << " != expected " << oi.data_digest
+                                << " full-object read crc 0x" << crc
+                                << " != expected 0x" << oi.data_digest
                                 << std::dec << " on " << soid;
              // FIXME fall back to replica or something?
              if (g_conf->osd_read_eio_on_bad_digest)