Fix the PG listing issue which could miss objects for EC pool (where there is object...

Backport: firefly
Signed-off-by: Guang Yang (yguang@yahoo-inc.com)

diff --git a/src/os/ObjectStore.cc b/src/os/ObjectStore.cc
index 4e77af93e261e6e8b293385c1a260999f0876e33..dfd5a33de075bbf77bfedd53550c8ac23cadc412 100644 (file)
--- a/src/os/ObjectStore.cc
+++ b/src/os/ObjectStore.cc
@@ -610,7 +610,11 @@ int ObjectStore::collection_list_range(coll_t c, hobject_t start, hobject_t end,
                            snapid_t seq, vector<hobject_t> *ls)
 {
   vector<ghobject_t> go;
-  ghobject_t gstart(start), gend(end);
+  // Starts with the smallest shard id and generation to
+  // make sure the result list has the marker object
+  ghobject_t gstart(start, 0, shard_id_t(0));
+  // Exclusive end, choose the smallest end ghobject
+  ghobject_t gend(end, 0, shard_id_t(0));
   int ret = collection_list_range(c, gstart, gend, seq, &go);
   if (ret == 0) {
     ls->reserve(go.size());

diff --git a/src/osd/PGBackend.cc b/src/osd/PGBackend.cc
index 09cbe8daa5017da3290b87fc5e2764d7dd1768ce..39e4b7809eb8c47db401e7eafa39987ace26147b 100644 (file)
--- a/src/osd/PGBackend.cc
+++ b/src/osd/PGBackend.cc
@@ -116,7 +116,11 @@ int PGBackend::objects_list_partial(
   hobject_t *next)
 {
   assert(ls);
-  ghobject_t _next(begin);
+  // Starts with the smallest shard id and generation to
+  // make sure the result list has the marker object (
+  // it might have multiple generations though, which would
+  // be filtered).
+  ghobject_t _next(begin, 0, shard_id_t(0));
   ls->reserve(max);
   int r = 0;
   while (!_next.is_max() && ls->size() < (unsigned)min) {