Commit eff1b1bb authored by Bolea Sanchez, Vicente Adolfo's avatar Bolea Sanchez, Vicente Adolfo
Browse files

BP5: fixes memory error with IBM XL 

(cherry picked from commit 50127828)
parent 80a8b4d5

source/adios2/engine/bp5/BP5Writer.cpp

+15 −3
Original line number Diff line number Diff line
@@ -547,12 +547,14 @@ void BP5Writer::EndStep() 
    m_Profiler.Start("AWD");

    // TSInfo destructor would delete the DataBuffer so we need to save it

    // for async IO and let the writer free it up when not needed anymore

    adios2::format::BufferV *databuf = TSInfo.DataBuffer;

    TSInfo.DataBuffer = NULL;

    m_AsyncWriteLock.lock();

    m_flagRush = false;

    m_AsyncWriteLock.unlock();

    WriteData(databuf);



    // WriteData will free TSInfo.DataBuffer

    WriteData(TSInfo.DataBuffer);

    TSInfo.DataBuffer = NULL;



    m_Profiler.Stop("AWD");



    /*
@@ -688,6 +690,16 @@ void BP5Writer::EndStep() 
    m_EndStepEnd = Now();

    /* Seconds ts2 = Now() - m_EngineStart;

     std::cout << "END STEP ended at: " << ts2.count() << std::endl;*/



    if (TSInfo.AttributeEncodeBuffer)

    {

      delete TSInfo.AttributeEncodeBuffer;

    }



    if (TSInfo.MetaEncodeBuffer)

    {

      delete TSInfo.MetaEncodeBuffer;

    }

}



// PRIVATE

source/adios2/toolkit/format/bp5/BP5Serializer.h

+0 −4
Original line number Diff line number Diff line
@@ -42,10 +42,6 @@ public: 


        ~TimestepInfo()

        {

            delete MetaEncodeBuffer;

            if (AttributeEncodeBuffer)

                delete AttributeEncodeBuffer;

            delete DataBuffer;

        }

    };