osd: do not check ContextQueue for empty before taking the content.

This saves one lock acquisition. And empty() call is useless as move_to
checks for emptyness anyway()

Signed-off-by: Igor Fedotov <ifedotov@suse.com>

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 052768b672f6ba8d82b061224e30521aeb001992..5988c2826060f2e24aa07e702cb55030966064ed 100644 (file)
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -10493,7 +10493,7 @@ void OSD::ShardedOpWQ::_process(uint32_t thread_index, heartbeat_handle_d *hb)
   }
 
   list<Context *> oncommits;
-  if (is_smallest_thread_index && !sdata->context_queue.empty()) {
+  if (is_smallest_thread_index) {
     sdata->context_queue.move_to(oncommits);
   }