Changeset 2565

Timestamp:

09/14/23 09:06:45 (8 months ago)

Author:

jderouillat

Message:

Clean buffers of servers in One Sided protocol

Location:

XIOS3/trunk/src/transport

Files:

• one_sided_server_buffer.cpp (modified) (2 diffs)
• one_sided_server_buffer.hpp (modified) (2 diffs)

XIOS3/trunk/src/transport/one_sided_server_buffer.cpp

@@ -113 +113 @@
 
     if (buffers_.size()>1) 
-     if (buffers_.front()->getCount()==0) buffers_.pop_front() ; // if buffer is empty free buffer
+    {
+      if (buffers_.front()->getCount()==0) {
+        delete buffers_.front();
+        buffers_.pop_front() ; // if buffer is empty free buffer
+      }
+    }
   }
 
@@ -408 +413 @@
       bloc.buffer->free(bloc.start, bloc.count) ; // free bloc
       addr=bloc.addr ;
-      if (bloc.buffer->getCount()==0) if (buffers_.size() > 1) buffers_.pop_front() ; // if buffer is empty free buffer
+      if (bloc.buffer->getCount()==0)
+      {
+        if (buffers_.size() > 1)
+        {
+          delete buffers_.front();
+          buffers_.pop_front() ; // if buffer is empty free buffer
+        }
+      }      
     }
     event.push(clientRank_, nullptr, buffer, size) ;

XIOS3/trunk/src/transport/one_sided_server_buffer.hpp

@@ -36 +36 @@
           { 
             if (count_>0) ERROR("COneSidedServerBuffer::~CBuffer()",<<"Try to delete buffer that is not empty"<<std::endl) ;
-            MPI_Free_mem(&buffer_) ;
+            MPI_Free_mem(buffer_) ;
           }
        
@@ -104 +104 @@
                              map<size_t, SPendingEvent>& completedEvents, vector<char>& buffer)  ;
       
+      ~COneSidedServerBuffer()
+      {
+          while (!buffers_.empty()) {
+              delete buffers_.front();
+              buffers_.pop_front() ; // if buffer is empty free buffer
+          }
+      };
+
       void receivedRequest(vector<char>& buffer) ;
       void eventLoop(void) ;