osd: shutdown recovery_request_timer earlier

recovery_request_timer may hold some QueuePeeringEvts which PGRef,
if we dont shutdown it earlier, it potentially cause the PGRef leak
when kicking pg.

Fixes: https://tracker.ceph.com/issues/38945
Signed-off-by: Zengran Zhang <zhangzengran@sangfor.com.cn>
(cherry picked from commit bb41bdfdb14768b5d46b7b1334bba1a3909e2618)

Conflicts:
	src/osd/OSD.cc : Resolved in shutdown and start_shutdown

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 4f37077a1cd25d0dde0c35f7a53f97f60430696e..49de7ba5030b3c543c6c2d307a88990a0b73b855 100644 (file)
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -374,6 +374,11 @@ void OSDService::start_shutdown()
     Mutex::Locker l(sleep_lock);
     sleep_timer.shutdown();
   }
+
+  {
+    Mutex::Locker l(recovery_request_lock);
+    recovery_request_timer.shutdown();
+  }
 }
 
 void OSDService::shutdown_reserver()
@@ -395,11 +400,6 @@ void OSDService::shutdown()
     f->stop();
   }
 
-  {
-    Mutex::Locker l(recovery_request_lock);
-    recovery_request_timer.shutdown();
-  }
-
   osdmap = OSDMapRef();
   next_osdmap = OSDMapRef();
 }