From 8b8b4d535e56d5b58ba8628a0206300eb178ba9c Mon Sep 17 00:00:00 2001 From: Kai Germaschewski Date: Thu, 6 Jun 2019 22:36:40 -0400 Subject: [PATCH] mpiwrap: get rid of unused m_RankMPI, m_SizeMPI in Transport They are unused, since all transports are serial. If someone were to add a MPI-I/O Transport in the future, it might make sense to save rank/size within that new derived class, but it's not really a generically needed thing. --- source/adios2/toolkit/transport/Transport.cpp | 2 -- source/adios2/toolkit/transport/Transport.h | 2 -- 2 files changed, 4 deletions(-) diff --git a/source/adios2/toolkit/transport/Transport.cpp b/source/adios2/toolkit/transport/Transport.cpp index 745e3014f5..bb6c33d07a 100644 --- a/source/adios2/toolkit/transport/Transport.cpp +++ b/source/adios2/toolkit/transport/Transport.cpp @@ -20,8 +20,6 @@ Transport::Transport(const std::string type, const std::string library, MPI_Comm mpiComm, const bool debugMode) : m_Type(type), m_Library(library), m_MPIComm(mpiComm), m_DebugMode(debugMode) { - SMPI_Comm_rank(m_MPIComm, &m_RankMPI); - SMPI_Comm_size(m_MPIComm, &m_SizeMPI); } void Transport::IWrite(const char *buffer, size_t size, Status &status, diff --git a/source/adios2/toolkit/transport/Transport.h b/source/adios2/toolkit/transport/Transport.h index 1da6b7e742..695c1e0062 100644 --- a/source/adios2/toolkit/transport/Transport.h +++ b/source/adios2/toolkit/transport/Transport.h @@ -34,8 +34,6 @@ class Transport Mode m_OpenMode = Mode::Undefined; ///< at Open from ADIOSTypes.h bool m_IsOpen = false; ///< true: open for communication, false: unreachable MPI_Comm m_MPIComm; ///< current MPI communicator - int m_RankMPI = 0; ///< from MPI_Comm_Rank - int m_SizeMPI = 1; ///< from MPI_Comm_Size profiling::IOChrono m_Profiler; ///< profiles Open, Write/Read, Close struct Status