From: Yehuda Sadeh <yehuda@redhat.com>
Date: Mon, 6 Nov 2017 23:06:20 +0000 (-0800)
Subject: rgw: remove warning
X-Git-Tag: v13.1.0~270^2~60
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=b2d16317ba30b4d59662b79d01ac8f58697b7608;p=ceph.git

rgw: remove warning

No need to clean io_ids when stack finishes, as list of io_ids is kept inside
the stack object, and will be cleaned up anyway. We also have a list of finished
ios in the completion manager (complete_reqs), but this is a fifo, and will be
cleaned up once pulled out (and we identify that io references an obsolete
stack).

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
---

diff --git a/src/rgw/rgw_coroutine.cc b/src/rgw/rgw_coroutine.cc
index 4e3fed07d8d4..c93ff816d19d 100644
--- a/src/rgw/rgw_coroutine.cc
+++ b/src/rgw/rgw_coroutine.cc
@@ -488,7 +488,6 @@ void RGWCoroutinesStack::init_new_io(RGWIOProvider *io_provider)
 bool RGWCoroutinesStack::try_io_unblock(const rgw_io_id& io_id)
 {
   if (!can_io_unblock(io_id)) {
-#warning io_finish_ids needs to be cleaned up when owning stack finishes
     auto p = io_finish_ids.emplace(io_id.id, io_id);
     auto& iter = p.first;
     bool inserted = p.second;