]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
msg: drop unnecessary polling_stop() 17079/head
authorJos Collin <jcollin@redhat.com>
Fri, 18 Aug 2017 01:30:31 +0000 (07:00 +0530)
committerJos Collin <jcollin@redhat.com>
Fri, 18 Aug 2017 01:32:40 +0000 (07:02 +0530)
Drop the unnecessary polling_stop() in the ~RDMAStack(). The RDMADispatcher object contained in
RDMAStack will get distroyed when RDMAStack gets distroyed. Thus ~RDMADispatcher() will be
called automatically and executes the polling_stop().

Fixes:
The below coverity scan reports.
1. CID 1416590:  Memory - corruptions
2. CID 1416593:  Memory - illegal accesses
3. CID 1416595:  Memory - corruptions
4. CID 1416597:  Memory - illegal accesses

Signed-off-by: Jos Collin <jcollin@redhat.com>
src/msg/async/rdma/RDMAStack.cc

index 7e5c9d3e9bf3716c371785e1deaa2493ce49b3d4..b3fefcda1afe66328d1a68fdb08d8f36c1f875ce 100644 (file)
@@ -547,8 +547,6 @@ RDMAStack::~RDMAStack()
   if (cct->_conf->ms_async_rdma_enable_hugepage) {
     unsetenv("RDMAV_HUGEPAGES_SAFE");  //remove env variable on destruction
   }
-
-  dispatcher.polling_stop();
 }
 
 void RDMAStack::spawn_worker(unsigned i, std::function<void ()> &&func)