X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/e1a5484101efbff9c8d982a0c4650f7953ee7e2f..a8c60905c3beb971d32d750d24409b60c3f882c4:/src/kernel/activity/IoImpl.cpp diff --git a/src/kernel/activity/IoImpl.cpp b/src/kernel/activity/IoImpl.cpp index 527aa19a2a..61c160b931 100644 --- a/src/kernel/activity/IoImpl.cpp +++ b/src/kernel/activity/IoImpl.cpp @@ -1,19 +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/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 { @@ -22,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; } @@ -52,9 +74,12 @@ IoImpl& IoImpl::set_disk(resource::DiskImpl* disk) IoImpl* IoImpl::start() { - state_ = State::RUNNING; - surf_action_ = disk_->io_start(size_, type_); + 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()); @@ -66,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(); @@ -89,29 +114,43 @@ 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()) { - const s_smx_simcall* simcall = simcalls_.front(); + smx_simcall_t simcall = simcalls_.front(); simcalls_.pop_front(); - 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_)); - } + + /* 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_ == simix::Simcall::NONE) // FIXME: maybe a better way to handle this case + continue; // if process handling comm is killed + + handle_activity_waitany(simcall); + + set_exception(simcall->issuer_); simcall->issuer_->waiting_synchro_ = nullptr; simcall->issuer_->simcall_answer();