]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgwlc: remove magic debug blocks for clearing stale lc entries 44206/head
authorMatt Benjamin <mbenjamin@redhat.com>
Fri, 3 Dec 2021 18:06:10 +0000 (13:06 -0500)
committerMatt Benjamin <mbenjamin@redhat.com>
Fri, 3 Dec 2021 18:06:10 +0000 (13:06 -0500)
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
src/rgw/rgw_lc.cc

index 1556124be4a44717a0c6a0a795a2255e2bec06d7..3ad0f85d6079f09bf2307505c57e507b2232e26b 100644 (file)
@@ -2002,23 +2002,21 @@ int RGWLC::process_bucket(int index, int max_lock_secs, LCWorker* worker,
   std::unique_lock<rgw::sal::LCSerializer> lock(
     *(serializer.get()), std::adopt_lock);
 
-  if (! (cct->_conf->rgw_lc_lock_max_time == 9969)) {
-    ret = sal_lc->get_entry(obj_names[index], bucket_entry_marker, entry);
-    if (ret >= 0) {
-      if (entry.status == lc_processing) {
-       if (expired_session(entry.start_time)) {
-         ldpp_dout(this, 5) << "RGWLC::process_bucket(): STALE lc session found for: " << entry
-                            << " index: " << index << " worker ix: " << worker->ix
-                            << " (clearing)"
-                            << dendl;
-       } else {
-         ldpp_dout(this, 5) << "RGWLC::process_bucket(): ACTIVE entry: "
-                            << entry
-                            << " index: " << index
-                            << " worker ix: " << worker->ix
-                            << dendl;
-         return ret;
-       }
+  ret = sal_lc->get_entry(obj_names[index], bucket_entry_marker, entry);
+  if (ret >= 0) {
+    if (entry.status == lc_processing) {
+      if (expired_session(entry.start_time)) {
+       ldpp_dout(this, 5) << "RGWLC::process_bucket(): STALE lc session found for: " << entry
+                          << " index: " << index << " worker ix: " << worker->ix
+                          << " (clearing)"
+                          << dendl;
+      } else {
+       ldpp_dout(this, 5) << "RGWLC::process_bucket(): ACTIVE entry: "
+                          << entry
+                          << " index: " << index
+                          << " worker ix: " << worker->ix
+                          << dendl;
+       return ret;
       }
     }
   }
@@ -2090,21 +2088,19 @@ int RGWLC::process(int index, int max_lock_secs, LCWorker* worker,
       goto exit;
     }
 
-    if (! (cct->_conf->rgw_lc_lock_max_time == 9969)) {
-      ret = sal_lc->get_entry(obj_names[index], head.marker, entry);
-      if (ret >= 0) {
-       if (entry.status == lc_processing) {
-         if (expired_session(entry.start_time)) {
-           ldpp_dout(this, 5) << "RGWLC::process(): STALE lc session found for: " << entry
-                   << " index: " << index << " worker ix: " << worker->ix
-                   << " (clearing)"
-                   << dendl;
-         } else {
-           ldpp_dout(this, 5) << "RGWLC::process(): ACTIVE entry: " << entry
-                   << " index: " << index << " worker ix: " << worker->ix
-                 << dendl;
-           goto exit;
-         }
+    ret = sal_lc->get_entry(obj_names[index], head.marker, entry);
+    if (ret >= 0) {
+      if (entry.status == lc_processing) {
+       if (expired_session(entry.start_time)) {
+         ldpp_dout(this, 5) << "RGWLC::process(): STALE lc session found for: " << entry
+                            << " index: " << index << " worker ix: " << worker->ix
+                            << " (clearing)"
+                            << dendl;
+       } else {
+         ldpp_dout(this, 5) << "RGWLC::process(): ACTIVE entry: " << entry
+                            << " index: " << index << " worker ix: " << worker->ix
+                            << dendl;
+         goto exit;
        }
       }
     }