X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/f88215c694b16ecbae4a3e230d437f47bff54917..4cf7a52375e761511d30a77a2977e5f3ac8a59f9:/src/kernel/activity/IoImpl.cpp diff --git a/src/kernel/activity/IoImpl.cpp b/src/kernel/activity/IoImpl.cpp index 3621e1c127..61c160b931 100644 --- a/src/kernel/activity/IoImpl.cpp +++ b/src/kernel/activity/IoImpl.cpp @@ -1,21 +1,23 @@ -/* Copyright (c) 2007-2021. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2007-2022. 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/kernel/routing/NetPoint.hpp" -#include "simgrid/s4u/Host.hpp" -#include "simgrid/s4u/Io.hpp" +#include "src/kernel/activity/IoImpl.hpp" +#include "src/kernel/actor/ActorImpl.hpp" #include "src/kernel/actor/SimcallObserver.hpp" +#include "src/kernel/context/Context.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/cpu_interface.hpp" -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_io, simix, "Logging specific to SIMIX (io)"); +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(ker_io, kernel, "Kernel io-related synchronization"); namespace simgrid { namespace kernel { @@ -24,12 +26,30 @@ namespace activity { IoImpl::IoImpl() { piface_ = new s4u::Io(this); + actor::ActorImpl* self = actor::ActorImpl::self(); + if (self) { + set_actor(self); + self->activities_.emplace_back(this); + } +} + +IoImpl& IoImpl::set_sharing_penalty(double sharing_penalty) +{ + sharing_penalty_ = sharing_penalty; + return *this; +} + +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_host(); - timeout_detector_ = host->pimpl_cpu->sleep(timeout); + timeout_detector_ = host->get_cpu()->sleep(timeout); timeout_detector_->set_activity(this); return *this; } @@ -54,10 +74,12 @@ IoImpl& IoImpl::set_disk(resource::DiskImpl* disk) IoImpl* IoImpl::start() { - state_ = State::RUNNING; + set_state(State::RUNNING); surf_action_ = disk_->get_host()->get_netpoint()->get_englobing_zone()->get_disk_model()->io_start(disk_, size_, type_); + surf_action_->set_sharing_penalty(sharing_penalty_); surf_action_->set_activity(this); + set_start_time(surf_action_->get_start_time()); XBT_DEBUG("Create IO synchro %p %s", this, get_cname()); @@ -69,18 +91,18 @@ void IoImpl::post() performed_ioops_ = surf_action_->get_cost(); if (surf_action_->get_state() == resource::Action::State::FAILED) { if (disk_ && not disk_->is_on()) - state_ = State::FAILED; + set_state(State::FAILED); else - state_ = State::CANCELED; + set_state(State::CANCELED); } else if (timeout_detector_ && timeout_detector_->get_state() == resource::Action::State::FINISHED) { if (surf_action_->get_remains() > 0.0) { surf_action_->set_state(resource::Action::State::FAILED); - state_ = State::TIMEOUT; + set_state(State::TIMEOUT); } else { - state_ = State::DONE; + set_state(State::DONE); } } else { - state_ = State::DONE; + set_state(State::DONE); } clean_action(); @@ -92,10 +114,29 @@ void IoImpl::post() /* Answer all simcalls associated with the synchro */ finish(); } +void IoImpl::set_exception(actor::ActorImpl* issuer) +{ + switch (get_state()) { + case State::FAILED: + issuer->context_->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::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", to_c_str(state_)); + XBT_DEBUG("IoImpl::finish() in state %s", get_state_str()); while (not simcalls_.empty()) { smx_simcall_t simcall = simcalls_.front(); simcalls_.pop_front(); @@ -106,73 +147,16 @@ void IoImpl::finish() if (simcall->call_ == simix::Simcall::NONE) // FIXME: maybe a better way to handle this case continue; // if process handling comm is killed - if (auto* observer = dynamic_cast(simcall->observer_)) { // simcall is a wait_any? - const auto& ios = observer->get_ios(); - - for (auto* io : ios) { - io->unregister_simcall(simcall); - - if (simcall->timeout_cb_) { - simcall->timeout_cb_->remove(); - simcall->timeout_cb_ = nullptr; - } - } - - if (not MC_is_active() && not MC_record_replay_is_active()) { - auto element = std::find(ios.begin(), ios.end(), this); - int rank = element != ios.end() ? static_cast(std::distance(ios.begin(), element)) : -1; - observer->set_result(rank); - } - } - switch (state_) { - 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: - simcall->issuer_->exception_ = std::make_exception_ptr(TimeoutException(XBT_THROW_POINT, "Timeouted")); - break; - default: - xbt_assert(state_ == State::DONE, "Internal error in IoImpl::finish(): unexpected synchro state %s", - to_c_str(state_)); - } + handle_activity_waitany(simcall); + + set_exception(simcall->issuer_); simcall->issuer_->waiting_synchro_ = nullptr; simcall->issuer_->simcall_answer(); } } -void IoImpl::wait_any_for(actor::ActorImpl* issuer, const std::vector& ios, double timeout) -{ - if (timeout < 0.0) { - issuer->simcall_.timeout_cb_ = nullptr; - } else { - issuer->simcall_.timeout_cb_ = simix::Timer::set(SIMIX_get_clock() + timeout, [issuer, &ios]() { - issuer->simcall_.timeout_cb_ = nullptr; - for (auto* io : ios) - io->unregister_simcall(&issuer->simcall_); - // default result (-1) is set in actor::IoWaitanySimcall - issuer->simcall_answer(); - }); - } - - for (auto* io : ios) { - /* associate this simcall to the the synchro */ - io->simcalls_.push_back(&issuer->simcall_); - - /* see if the synchro is already finished */ - if (io->state_ != State::WAITING && io->state_ != State::RUNNING) { - io->finish(); - break; - } - } -} - } // namespace activity } // namespace kernel } // namespace simgrid