X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/b5aa376000b34d535f13674f807370d4c96a4441..5ed37babb2fa9097abe82df299c0aa259ed84d5a:/src/kernel/activity/IoImpl.cpp diff --git a/src/kernel/activity/IoImpl.cpp b/src/kernel/activity/IoImpl.cpp index 17df987f99..21b55c90ed 100644 --- a/src/kernel/activity/IoImpl.cpp +++ b/src/kernel/activity/IoImpl.cpp @@ -1,28 +1,53 @@ -/* Copyright (c) 2007-2020. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2007-2023. 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. */ -#include "src/kernel/activity/IoImpl.hpp" + +#include +#include +#include +#include + #include "mc/mc.h" -#include "simgrid/Exception.hpp" -#include "simgrid/kernel/resource/Action.hpp" -#include "simgrid/s4u/Host.hpp" +#include "src/kernel/activity/IoImpl.hpp" +#include "src/kernel/actor/ActorImpl.hpp" +#include "src/kernel/actor/SimcallObserver.hpp" +#include "src/kernel/resource/CpuImpl.hpp" #include "src/kernel/resource/DiskImpl.hpp" #include "src/mc/mc_replay.hpp" -#include "src/simix/smx_private.hpp" -#include "src/surf/StorageImpl.hpp" -#include "src/surf/cpu_interface.hpp" +#include "src/surf/HostImpl.hpp" + +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(ker_io, kernel, "Kernel io-related synchronization"); + +namespace simgrid::kernel::activity { + +IoImpl::IoImpl() +{ + piface_ = new s4u::Io(this); + actor::ActorImpl* self = actor::ActorImpl::self(); + if (self) { + set_actor(self); + self->activities_.insert(this); + } +} -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_io, simix, "Logging specific to SIMIX (io)"); +IoImpl& IoImpl::set_sharing_penalty(double sharing_penalty) +{ + sharing_penalty_ = sharing_penalty; + return *this; +} -namespace simgrid { -namespace kernel { -namespace activity { +IoImpl& IoImpl::update_sharing_penalty(double sharing_penalty) +{ + sharing_penalty_ = sharing_penalty; + surf_action_->set_sharing_penalty(sharing_penalty); + return *this; +} IoImpl& IoImpl::set_timeout(double timeout) { - const s4u::Host* host = get_disk() ? get_disk()->get_host() : s4u::Host::by_name(get_storage()->get_host()); - timeout_detector_ = host->pimpl_cpu->sleep(timeout); + const s4u::Host* host = get_disk()->get_host(); + timeout_detector_ = host->get_cpu()->sleep(timeout); timeout_detector_->set_activity(this); return *this; } @@ -45,23 +70,41 @@ IoImpl& IoImpl::set_disk(resource::DiskImpl* disk) return *this; } -IoImpl& IoImpl::set_storage(resource::StorageImpl* storage) +IoImpl& IoImpl::set_host(s4u::Host* host) +{ + host_ = host; + return *this; +} + +IoImpl& IoImpl::set_dst_disk(resource::DiskImpl* disk) { - storage_ = storage; + dst_disk_ = disk; + return *this; +} + +IoImpl& IoImpl::set_dst_host(s4u::Host* host) +{ + dst_host_ = host; return *this; } IoImpl* IoImpl::start() { - state_ = State::RUNNING; - if (storage_) - surf_action_ = storage_->io_start(size_, type_); - else + set_state(State::RUNNING); + if (dst_host_ == nullptr) { + XBT_DEBUG("Starting regular I/O"); surf_action_ = disk_->io_start(size_, type_); + surf_action_->set_sharing_penalty(sharing_penalty_); + } else { + XBT_DEBUG("Starting streaming I/O"); + auto host_model = dst_host_->get_netpoint()->get_englobing_zone()->get_host_model(); + surf_action_ = host_model->io_stream(host_, disk_, dst_host_, dst_disk_, size_); + } + surf_action_->set_activity(this); + set_start_time(surf_action_->get_start_time()); XBT_DEBUG("Create IO synchro %p %s", this, get_cname()); - IoImpl::on_start(*this); return this; } @@ -70,14 +113,21 @@ void IoImpl::post() { performed_ioops_ = surf_action_->get_cost(); if (surf_action_->get_state() == resource::Action::State::FAILED) { - if ((storage_ && not storage_->is_on()) || (disk_ && not disk_->is_on())) - state_ = State::FAILED; + if (host_ && dst_host_) { // this is an I/O stream + if (not host_->is_on()) + set_state(State::SRC_HOST_FAILURE); + else if (not dst_host_->is_on()) + set_state(State::DST_HOST_FAILURE); + } else if ((disk_ && not disk_->is_on()) || (dst_disk_ && not dst_disk_->is_on())) + set_state(State::FAILED); else - state_ = State::CANCELED; - } else if (surf_action_->get_state() == resource::Action::State::FINISHED) { - state_ = State::DONE; - } else if (timeout_detector_ && timeout_detector_->get_state() == resource::Action::State::FINISHED) { - state_ = State::TIMEOUT; + set_state(State::CANCELED); + } else if (timeout_detector_ && timeout_detector_->get_state() == resource::Action::State::FINISHED && + surf_action_->get_remains() > 0.0) { + surf_action_->set_state(resource::Action::State::FAILED); + set_state(State::TIMEOUT); + } else { + set_state(State::DONE); } clean_action(); @@ -86,48 +136,62 @@ void IoImpl::post() timeout_detector_ = nullptr; } - on_completion(*this); - /* Answer all simcalls associated with the synchro */ finish(); } +void IoImpl::set_exception(actor::ActorImpl* issuer) +{ + switch (get_state()) { + case State::FAILED: + issuer->set_wannadie(); + static_cast(get_iface())->complete(s4u::Activity::State::FAILED); + issuer->exception_ = std::make_exception_ptr(StorageFailureException(XBT_THROW_POINT, "Storage failed")); + break; + case State::CANCELED: + issuer->exception_ = std::make_exception_ptr(CancelException(XBT_THROW_POINT, "I/O Canceled")); + break; + case State::SRC_HOST_FAILURE: + case State::DST_HOST_FAILURE: + issuer->set_wannadie(); + static_cast(get_iface())->complete(s4u::Activity::State::FAILED); + issuer->exception_ = std::make_exception_ptr(StorageFailureException(XBT_THROW_POINT, "Host failed")); + break; + case State::TIMEOUT: + issuer->exception_ = std::make_exception_ptr(TimeoutException(XBT_THROW_POINT, "Timeouted")); + break; + default: + xbt_assert(get_state() == State::DONE, "Internal error in IoImpl::finish(): unexpected synchro state %s", + get_state_str()); + } +} void IoImpl::finish() { + XBT_DEBUG("IoImpl::finish() in state %s", get_state_str()); while (not simcalls_.empty()) { - const s_smx_simcall* simcall = simcalls_.front(); + actor::Simcall* simcall = simcalls_.front(); simcalls_.pop_front(); - switch (state_) { - case State::DONE: - /* do nothing, synchro done */ - break; - case State::FAILED: - simcall->issuer_->context_->set_wannadie(); - simcall->issuer_->exception_ = - std::make_exception_ptr(StorageFailureException(XBT_THROW_POINT, "Storage failed")); - break; - case State::CANCELED: - simcall->issuer_->exception_ = std::make_exception_ptr(CancelException(XBT_THROW_POINT, "I/O Canceled")); - break; - case State::TIMEOUT: - XBT_DEBUG("IoImpl::finish(): execution timeouted"); - simcall->issuer_->exception_ = std::make_exception_ptr(simgrid::TimeoutException(XBT_THROW_POINT, "Timeouted")); - break; - default: - xbt_die("Internal error in IoImpl::finish(): unexpected synchro state %d", static_cast(state_)); - } - simcall->issuer_->waiting_synchro = nullptr; - simcall->issuer_->simcall_answer(); + /* If a waitany simcall is waiting for this synchro to finish, then remove it from the other synchros in the waitany + * list. Afterwards, get the position of the actual synchro in the waitany list and return it as the result of the + * simcall */ + + if (simcall->call_ == actor::Simcall::Type::NONE) // FIXME: maybe a better way to handle this case + continue; // if process handling comm is killed + + handle_activity_waitany(simcall); + + if (not simcall->issuer_->get_host()->is_on()) { + simcall->issuer_->set_wannadie(); + } else { + // Do not answer to dying actors + if (not simcall->issuer_->wannadie()) { + set_exception(simcall->issuer_); + simcall->issuer_->waiting_synchro_ = nullptr; + simcall->issuer_->simcall_answer(); + } + } } } -/************* - * Callbacks * - *************/ -xbt::signal IoImpl::on_start; -xbt::signal IoImpl::on_completion; - -} // namespace activity -} // namespace kernel -} // namespace simgrid +} // namespace simgrid::kernel::activity