X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/e3fd4ced0204c7d9334a68482e1b9730c8293a35..49609678d7329f0824ca7d5c6d5421fc9d358cbb:/src/smpi/mpi/smpi_file.cpp diff --git a/src/smpi/mpi/smpi_file.cpp b/src/smpi/mpi/smpi_file.cpp index 8d841ef4f7..c743cf9b89 100644 --- a/src/smpi/mpi/smpi_file.cpp +++ b/src/smpi/mpi/smpi_file.cpp @@ -13,21 +13,43 @@ #include "smpi_file.hpp" #include "smpi_status.hpp" +#include "simgrid/s4u/Disk.hpp" +#include "simgrid/s4u/Host.hpp" #include "simgrid/plugins/file_system.h" #define FP_SIZE sizeof(MPI_Offset) XBT_LOG_NEW_DEFAULT_SUBCATEGORY(smpi_io, smpi, "Logging specific to SMPI (RMA operations)"); +MPI_Errhandler SMPI_default_File_Errhandler = MPI_ERRORS_RETURN; + namespace simgrid{ namespace smpi{ - File::File(MPI_Comm comm, char *filename, int amode, MPI_Info info): comm_(comm), flags_(amode), info_(info) { - file_= new simgrid::s4u::File(filename, nullptr); + File::File(MPI_Comm comm, const char *filename, int amode, MPI_Info info): comm_(comm), flags_(amode), info_(info) { + std::string fullname=filename; + if (simgrid::s4u::Host::current()->get_disks().empty()) + xbt_die("SMPI/IO : Trying to open file on a diskless host ! Add one to your platform file"); + + size_t found=fullname.find('/'); + //in case no fullpath is provided ... just pick the first mountpoint. + if(found==std::string::npos){ + auto disk = simgrid::s4u::Host::current()->get_disks().front(); + std::string mount; + if (disk->get_host() != simgrid::s4u::Host::current()) + mount = disk->extension()->get_mount_point(disk->get_host()); + else + mount = disk->extension()->get_mount_point(); + XBT_DEBUG("No absolute path given for file opening, use '%s'", mount.c_str()); + fullname.insert(0, mount); + } + + file_= new simgrid::s4u::File(fullname, nullptr); list_=nullptr; if (comm_->rank() == 0) { int size= comm_->size() + FP_SIZE; list_ = new char[size]; + errhandler_= SMPI_default_File_Errhandler; memset(list_, 0, size); shared_file_pointer_ = new MPI_Offset(); shared_mutex_ = s4u::Mutex::create(); @@ -36,8 +58,8 @@ namespace smpi{ }else{ win_=new Win(list_, 0, 1, MPI_INFO_NULL, comm_); } - simgrid::smpi::Colls::bcast(&shared_file_pointer_, 1, MPI_AINT, 0, comm); - simgrid::smpi::Colls::bcast(&shared_mutex_, 1, MPI_AINT, 0, comm); + simgrid::smpi::colls::bcast(&shared_file_pointer_, 1, MPI_AINT, 0, comm); + simgrid::smpi::colls::bcast(&shared_mutex_, 1, MPI_AINT, 0, comm); if(comm_->rank() != 0) intrusive_ptr_add_ref(&*shared_mutex_); } @@ -60,7 +82,8 @@ namespace smpi{ return MPI_SUCCESS; } - int File::del(char *filename, MPI_Info info){ + int File::del(const char* filename, MPI_Info) + { //get the file with MPI_MODE_DELETE_ON_CLOSE and then close it File* f = new File(MPI_COMM_SELF,filename,MPI_MODE_DELETE_ON_CLOSE|MPI_MODE_RDWR, nullptr); close(&f); @@ -107,7 +130,8 @@ namespace smpi{ return MPI_SUCCESS; } - int File::read(MPI_File fh, void *buf, int count, MPI_Datatype datatype, MPI_Status *status){ + int File::read(MPI_File fh, void* /*buf*/, int count, MPI_Datatype datatype, MPI_Status* status) + { //get position first as we may be doing non contiguous reads and it will probably be updated badly MPI_Offset position = fh->file_->tell(); MPI_Offset movesize = datatype->get_extent()*count; @@ -152,7 +176,7 @@ namespace smpi{ } MPI_Offset result; - simgrid::smpi::Colls::scan(&val, &result, 1, MPI_OFFSET, MPI_SUM, fh->comm_); + simgrid::smpi::colls::scan(&val, &result, 1, MPI_OFFSET, MPI_SUM, fh->comm_); fh->seek(result, MPI_SEEK_SET); int ret = fh->op_all(buf, count, datatype, status); if(fh->comm_->rank()==fh->comm_->size()-1){ @@ -161,17 +185,18 @@ namespace smpi{ fh->shared_mutex_->unlock(); } char c; - simgrid::smpi::Colls::bcast(&c, 1, MPI_BYTE, fh->comm_->size()-1, fh->comm_); + simgrid::smpi::colls::bcast(&c, 1, MPI_BYTE, fh->comm_->size() - 1, fh->comm_); return ret; } - int File::write(MPI_File fh, void *buf, int count, MPI_Datatype datatype, MPI_Status *status){ + int File::write(MPI_File fh, void* /*buf*/, int count, MPI_Datatype datatype, MPI_Status* status) + { //get position first as we may be doing non contiguous reads and it will probably be updated badly MPI_Offset position = fh->file_->tell(); MPI_Offset movesize = datatype->get_extent()*count; MPI_Offset writesize = datatype->size()*count; XBT_DEBUG("Position before write in MPI_File %s : %llu",fh->file_->get_path(),fh->file_->tell()); - MPI_Offset write = fh->file_->write(writesize); + MPI_Offset write = fh->file_->write(writesize, 1); XBT_VERB("Write in MPI_File %s, %lld bytes written, readsize %lld bytes, movesize %lld", fh->file_->get_path(), write, writesize, movesize); if(writesize!=movesize){ fh->file_->seek(position+movesize, SEEK_SET); @@ -181,16 +206,16 @@ namespace smpi{ return MPI_SUCCESS; } - int File::write_shared(MPI_File fh, void *buf, int count, MPI_Datatype datatype, MPI_Status *status){ + int File::write_shared(MPI_File fh, const void *buf, int count, MPI_Datatype datatype, MPI_Status *status){ fh->shared_mutex_->lock(); fh->seek(*(fh->shared_file_pointer_),MPI_SEEK_SET); - write(fh, buf, count, datatype, status); + write(fh, const_cast(buf), count, datatype, status); *(fh->shared_file_pointer_)=fh->file_->tell(); fh->shared_mutex_->unlock(); return MPI_SUCCESS; } - int File::write_ordered(MPI_File fh, void *buf, int count, MPI_Datatype datatype, MPI_Status *status){ + int File::write_ordered(MPI_File fh, const void *buf, int count, MPI_Datatype datatype, MPI_Status *status){ //0 needs to get the shared pointer value MPI_Offset val; if(fh->comm_->rank()==0){ @@ -199,16 +224,16 @@ namespace smpi{ val=count*datatype->size(); } MPI_Offset result; - simgrid::smpi::Colls::scan(&val, &result, 1, MPI_OFFSET, MPI_SUM, fh->comm_); + simgrid::smpi::colls::scan(&val, &result, 1, MPI_OFFSET, MPI_SUM, fh->comm_); fh->seek(result, MPI_SEEK_SET); - int ret = fh->op_all(buf, count, datatype, status); + int ret = fh->op_all(const_cast(buf), count, datatype, status); if(fh->comm_->rank()==fh->comm_->size()-1){ fh->shared_mutex_->lock(); *(fh->shared_file_pointer_)=fh->file_->tell(); fh->shared_mutex_->unlock(); } char c; - simgrid::smpi::Colls::bcast(&c, 1, MPI_BYTE, fh->comm_->size()-1, fh->comm_); + simgrid::smpi::colls::bcast(&c, 1, MPI_BYTE, fh->comm_->size() - 1, fh->comm_); return ret; } @@ -222,7 +247,7 @@ namespace smpi{ int File::sync(){ //no idea - return simgrid::smpi::Colls::barrier(comm_); + return simgrid::smpi::colls::barrier(comm_); } MPI_Info File::info(){ @@ -241,5 +266,15 @@ namespace smpi{ MPI_Comm File::comm(){ return comm_; } + + MPI_Errhandler File::errhandler(){ + return errhandler_; + } + + void File::set_errhandler(MPI_Errhandler errhandler){ + errhandler_=errhandler; + if(errhandler_!= MPI_ERRHANDLER_NULL) + errhandler->ref(); + } } }