From: Greg Farnum <gfarnum@redhat.com>
Date: Wed, 25 Mar 2015 19:03:38 +0000 (-0700)
Subject: ReplicatedPG: fix a signed/unsigned comparison warning
X-Git-Tag: v9.0.0~85^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=e51761233e00f416e661fc957be4d99b0d190e73;p=ceph.git

ReplicatedPG: fix a signed/unsigned comparison warning

This mismatch about whether pool IDs are signed or unsigned is
a persistent annoyance. I'm now casting the unsigned down to signed space
because apparently the OSD is using negative IDs for temporary object
namespaces.

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
---

diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc
index 29e864f391113..a718afbea817c 100644
--- a/src/osd/ReplicatedPG.cc
+++ b/src/osd/ReplicatedPG.cc
@@ -8107,7 +8107,7 @@ int ReplicatedPG::find_object_context(const hobject_t& oid,
 				      bool map_snapid_to_clone,
 				      hobject_t *pmissing)
 {
-  assert(oid.pool == info.pgid.pool());
+  assert(oid.pool == static_cast<int64_t>(info.pgid.pool()));
   // want the head?
   if (oid.snap == CEPH_NOSNAP) {
     ObjectContextRef obc = get_object_context(oid, can_create);