From: Mykola Golub <mgolub@suse.com>
Date: Fri, 31 Jul 2020 15:05:10 +0000 (+0100)
Subject: os/bluestore: fix collection_list properly set next if end reached
X-Git-Tag: v16.1.0~1160^2~6
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=46d73a806c90b944c8596ad3d9dae3f5cf78d915;p=ceph.git

os/bluestore: fix collection_list properly set next if end reached

Previously it was setting it to GMAX (happened when one had end
set to not GMAX and max set to INT_MAX).

Signed-off-by: Mykola Golub <mgolub@suse.com>
---

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 2220eb3d66d..e4549b0026f 100644
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -10395,6 +10395,10 @@ int BlueStore::_collection_list(
 	dout(20) << __func__ << " oid " << it->oid() << " >= " << pend << dendl;
       if (temp) {
 	if (end.hobj.is_temp()) {
+          if (it->valid() && it->key() < temp_end_key) {
+            *pnext = it->oid();
+            set_next = true;
+          }
 	  break;
 	}
 	dout(30) << __func__ << " switch to non-temp namespace" << dendl;
@@ -10407,6 +10411,10 @@ int BlueStore::_collection_list(
 	dout(30) << __func__ << " pend " << pend << dendl;
 	continue;
       }
+      if (it->valid() && it->key() < end_key) {
+        *pnext = it->oid();
+        set_next = true;
+      }
       break;
     }
     dout(30) << __func__ << " key " << pretty_binary_string(it->key()) << dendl;