X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/c31bbbb83963c7db2b5970f5c81cbcdd7bc1c5d5..e63c67321577f0816604fe730cd0bc856970454d:/src/plugins/file_system/s4u_FileSystem.cpp diff --git a/src/plugins/file_system/s4u_FileSystem.cpp b/src/plugins/file_system/s4u_FileSystem.cpp index ea1aaf9777..628c813a7c 100644 --- a/src/plugins/file_system/s4u_FileSystem.cpp +++ b/src/plugins/file_system/s4u_FileSystem.cpp @@ -1,4 +1,4 @@ -/* Copyright (c) 2015-2019. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2015-2020. The SimGrid Team. All rights reserved. */ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ @@ -9,25 +9,29 @@ #include "src/surf/HostImpl.hpp" #include "src/surf/xml/platf_private.hpp" #include "xbt/config.hpp" +#include "xbt/parse_units.hpp" #include #include #include #include #include +#include #include XBT_LOG_NEW_DEFAULT_SUBCATEGORY(s4u_file, s4u, "S4U files"); int sg_storage_max_file_descriptors = 1024; -/** @defgroup plugin_filesystem - @beginrst - - @endrst +/** @defgroup plugin_filesystem Plugin FileSystem + * + * This adds the notion of Files on top of the storage notion that provided by the core of SimGrid. + * Activate this plugin at will. */ - namespace simgrid { + +template class xbt::Extendable; + namespace s4u { simgrid::xbt::Extension FileSystemDiskExt::EXTENSION_ID; simgrid::xbt::Extension FileSystemStorageExt::EXTENSION_ID; @@ -91,52 +95,56 @@ File::File(const std::string& fullpath, void* userdata) : File(fullpath, Host::c File::File(const std::string& fullpath, sg_host_t host, void* userdata) : fullpath_(fullpath) { - this->set_data(userdata); - // this cannot fail because we get a xbt_die if the mountpoint does not exist - if (not host->get_mounted_storages().empty()) { - local_storage_ = find_local_storage_on(host); - } - if (not host->get_disks().empty()) { - local_disk_ = find_local_disk_on(host); - } + kernel::actor::simcall([this, &host, userdata] { + this->set_data(userdata); + // this cannot fail because we get a xbt_die if the mountpoint does not exist + if (not host->get_mounted_storages().empty()) { + local_storage_ = find_local_storage_on(host); + } + if (not host->get_disks().empty()) { + local_disk_ = find_local_disk_on(host); + } - // assign a file descriptor id to the newly opened File - FileDescriptorHostExt* ext = host->extension(); - if (ext->file_descriptor_table == nullptr) { - ext->file_descriptor_table.reset(new std::vector(sg_storage_max_file_descriptors)); - std::iota(ext->file_descriptor_table->rbegin(), ext->file_descriptor_table->rend(), 0); // Fill with ..., 1, 0. - } - xbt_assert(not ext->file_descriptor_table->empty(), "Too much files are opened! Some have to be closed."); - desc_id = ext->file_descriptor_table->back(); - ext->file_descriptor_table->pop_back(); + // assign a file descriptor id to the newly opened File + FileDescriptorHostExt* ext = host->extension(); + if (ext->file_descriptor_table == nullptr) { + ext->file_descriptor_table = std::make_unique>(sg_storage_max_file_descriptors); + std::iota(ext->file_descriptor_table->rbegin(), ext->file_descriptor_table->rend(), 0); // Fill with ..., 1, 0. + } + xbt_assert(not ext->file_descriptor_table->empty(), "Too much files are opened! Some have to be closed."); + desc_id = ext->file_descriptor_table->back(); + ext->file_descriptor_table->pop_back(); - XBT_DEBUG("\tOpen file '%s'", path_.c_str()); - std::map* content = nullptr; - if (local_storage_) - content = local_storage_->extension()->get_content(); + XBT_DEBUG("\tOpen file '%s'", path_.c_str()); + std::map* content = nullptr; + if (local_storage_) + content = local_storage_->extension()->get_content(); - if (local_disk_) - content = local_disk_->extension()->get_content(); + if (local_disk_) + content = local_disk_->extension()->get_content(); - // if file does not exist create an empty file - if (content) { - auto sz = content->find(path_); - if (sz != content->end()) { - size_ = sz->second; - } else { - size_ = 0; - content->insert({path_, size_}); - XBT_DEBUG("File '%s' was not found, file created.", path_.c_str()); + // if file does not exist create an empty file + if (content) { + auto sz = content->find(path_); + if (sz != content->end()) { + size_ = sz->second; + } else { + size_ = 0; + content->insert({path_, size_}); + XBT_DEBUG("File '%s' was not found, file created.", path_.c_str()); + } } - } + }); } File::~File() { - Host::current()->extension()->file_descriptor_table->push_back(desc_id); + std::vector* desc_table = + Host::current()->extension()->file_descriptor_table.get(); + kernel::actor::simcall([this, desc_table] { desc_table->push_back(this->desc_id); }); } -void File::dump() +void File::dump() const { if (local_storage_) XBT_INFO("File Descriptor information:\n" @@ -186,7 +194,7 @@ sg_size_t File::read(sg_size_t size) if (host && host->get_name() != Host::current()->get_name() && read_size > 0) { /* the file is hosted on a remote host, initiate a communication between src and dest hosts for data transfer */ XBT_DEBUG("File is on %s remote host, initiate data transfer of %llu bytes.", host->get_cname(), read_size); - host->send_to(Host::current(), read_size); + host->sendto(Host::current(), read_size); } return read_size; @@ -207,7 +215,7 @@ sg_size_t File::write_on_disk(sg_size_t size, bool write_inside) if (host && host->get_name() != Host::current()->get_name()) { /* the file is hosted on a remote host, initiate a communication between src and dest hosts for data transfer */ XBT_DEBUG("File is on %s remote host, initiate data transfer of %llu bytes.", host->get_cname(), size); - Host::current()->send_to(host, size); + Host::current()->sendto(host, size); } XBT_DEBUG("WRITE %s on disk '%s'. size '%llu/%llu' '%llu:%llu'", get_path(), local_disk_->get_cname(), size, size_, sg_disk_get_size_used(local_disk_), sg_disk_get_size(local_disk_)); @@ -227,10 +235,12 @@ sg_size_t File::write_on_disk(sg_size_t size, bool write_inside) if (current_position_ > size_) size_ = current_position_; } - std::map* content = local_disk_->extension()->get_content(); + kernel::actor::simcall([this] { + std::map* content = local_disk_->extension()->get_content(); - content->erase(path_); - content->insert({path_, size_}); + content->erase(path_); + content->insert({path_, size_}); + }); return write_size; } @@ -244,7 +254,7 @@ sg_size_t File::write_on_storage(sg_size_t size, bool write_inside) if (host && host->get_name() != Host::current()->get_name()) { /* the file is hosted on a remote host, initiate a communication between src and dest hosts for data transfer */ XBT_DEBUG("File is on %s remote host, initiate data transfer of %llu bytes.", host->get_cname(), size); - Host::current()->send_to(host, size); + Host::current()->sendto(host, size); } XBT_DEBUG("WRITE %s on disk '%s'. size '%llu/%llu' '%llu:%llu'", get_path(), local_storage_->get_cname(), size, size_, @@ -265,10 +275,12 @@ sg_size_t File::write_on_storage(sg_size_t size, bool write_inside) if (current_position_ > size_) size_ = current_position_; } - std::map* content = local_storage_->extension()->get_content(); + kernel::actor::simcall([this] { + std::map* content = local_storage_->extension()->get_content(); - content->erase(path_); - content->insert({path_, size_}); + content->erase(path_); + content->insert({path_, size_}); + }); return write_size; } @@ -286,7 +298,7 @@ sg_size_t File::write(sg_size_t size, bool write_inside) return 0; } -sg_size_t File::size() +sg_size_t File::size() const { return size_; } @@ -313,12 +325,12 @@ void File::seek(sg_offset_t offset, int origin) } } -sg_size_t File::tell() +sg_size_t File::tell() const { return current_position_; } -void File::move(const std::string& fullpath) +void File::move(const std::string& fullpath) const { /* Check if the new full path is on the same mount point */ if (fullpath.compare(0, mount_point_.length(), mount_point_) == 0) { @@ -344,7 +356,7 @@ void File::move(const std::string& fullpath) } } -int File::unlink() +int File::unlink() const { /* Check if the file is on local storage */ std::map* content = nullptr; @@ -405,7 +417,7 @@ int File::remote_copy(sg_host_t host, const char* fullpath) size_t longest_prefix_length = 0; if (local_storage_) { /* Find the host that owns the storage where the file has to be copied */ - Storage* storage_dest = nullptr; + const Storage* storage_dest = nullptr; for (auto const& elm : host->get_mounted_storages()) { std::string mount_point = std::string(fullpath).substr(0, elm.first.size()); @@ -447,11 +459,11 @@ int File::remote_copy(sg_host_t host, const char* fullpath) if (src_host) { XBT_DEBUG("Initiate data transfer of %llu bytes between %s and %s.", read_size, src_host->get_cname(), dst_host->get_cname()); - src_host->send_to(dst_host, read_size); + src_host->sendto(dst_host, read_size); } /* Create file on remote host, write it and close it */ - File* fd = new File(fullpath, dst_host, nullptr); + auto* fd = new File(fullpath, dst_host, nullptr); if (local_storage_) { sg_size_t write_size = fd->local_storage_->write(read_size); fd->local_storage_->extension()->incr_used_size(write_size); @@ -473,8 +485,9 @@ int File::remote_move(sg_host_t host, const char* fullpath) FileSystemDiskExt::FileSystemDiskExt(const Disk* ptr) { const char* size_str = ptr->get_property("size"); + std::string dummyfile; if (size_str) - size_ = surf_parse_get_size(size_str, "disk size", ptr->get_name()); + size_ = surf_parse_get_size(dummyfile, -1, size_str, "disk size", ptr->get_name()); const char* current_mount_str = ptr->get_property("mount"); if (current_mount_str) @@ -497,9 +510,11 @@ std::map* FileSystemDiskExt::parse_content(const std::st if (filename.empty()) return nullptr; - std::map* parse_content = new std::map(); + auto* parse_content = new std::map(); std::ifstream* fs = surf_ifsopen(filename); + xbt_assert(not fs->fail(), "Cannot open file '%s' (path=%s)", filename.c_str(), + (boost::join(surf_path, ":")).c_str()); std::string line; std::vector tokens; @@ -524,9 +539,11 @@ std::map* FileSystemStorageExt::parse_content(const std: if (filename.empty()) return nullptr; - std::map* parse_content = new std::map(); + auto* parse_content = new std::map(); std::ifstream* fs = surf_ifsopen(filename); + xbt_assert(not fs->fail(), "Cannot open file '%s' (path=%s)", filename.c_str(), + (boost::join(surf_path, ":")).c_str()); std::string line; std::vector tokens; @@ -545,6 +562,26 @@ std::map* FileSystemStorageExt::parse_content(const std: delete fs; return parse_content; } + +void FileSystemStorageExt::decr_used_size(sg_size_t size) +{ + simgrid::kernel::actor::simcall([this, size] { used_size_ -= size; }); +} + +void FileSystemStorageExt::incr_used_size(sg_size_t size) +{ + simgrid::kernel::actor::simcall([this, size] { used_size_ += size; }); +} + +void FileSystemDiskExt::decr_used_size(sg_size_t size) +{ + simgrid::kernel::actor::simcall([this, size] { used_size_ -= size; }); +} + +void FileSystemDiskExt::incr_used_size(sg_size_t size) +{ + simgrid::kernel::actor::simcall([this, size] { used_size_ += size; }); +} } } @@ -666,7 +703,7 @@ void sg_file_close(const_sg_file_t fd) /** Retrieves the path to the file * @ingroup plugin_filesystem */ -const char* sg_file_get_name(sg_file_t fd) +const char* sg_file_get_name(const_sg_file_t fd) { xbt_assert((fd != nullptr), "Invalid file descriptor"); return fd->get_path(); @@ -675,12 +712,12 @@ const char* sg_file_get_name(sg_file_t fd) /** Retrieves the size of the file * @ingroup plugin_filesystem */ -sg_size_t sg_file_get_size(sg_file_t fd) +sg_size_t sg_file_get_size(const_sg_file_t fd) { return fd->size(); } -void sg_file_dump(sg_file_t fd) +void sg_file_dump(const_sg_file_t fd) { fd->dump(); } @@ -716,12 +753,12 @@ void sg_file_seek(sg_file_t fd, sg_offset_t offset, int origin) fd->seek(offset, origin); } -sg_size_t sg_file_tell(sg_file_t fd) +sg_size_t sg_file_tell(const_sg_file_t fd) { return fd->tell(); } -void sg_file_move(sg_file_t fd, const char* fullpath) +void sg_file_move(const_sg_file_t fd, const char* fullpath) { fd->move(fullpath); } @@ -803,8 +840,7 @@ xbt_dict_t sg_storage_get_content(const_sg_storage_t storage) xbt_dict_t content_as_dict = xbt_dict_new_homogeneous(::operator delete); for (auto const& entry : *content) { - sg_size_t* psize = new sg_size_t; - *psize = entry.second; + auto* psize = new sg_size_t(entry.second); xbt_dict_set(content_as_dict, entry.first.c_str(), psize); } return content_as_dict;