X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/8a16d3b0a913a37d1346f472c1ccf727fb8fed15..cf6de77237062e35cbc13438716835a1403a1444:/src/kernel/activity/IoImpl.cpp?ds=sidebyside diff --git a/src/kernel/activity/IoImpl.cpp b/src/kernel/activity/IoImpl.cpp index 966cae13da..8f18e6db44 100644 --- a/src/kernel/activity/IoImpl.cpp +++ b/src/kernel/activity/IoImpl.cpp @@ -1,28 +1,52 @@ -/* Copyright (c) 2007-2020. 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 "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" -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::kernel::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; +} -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) { - 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 +69,15 @@ IoImpl& IoImpl::set_disk(resource::DiskImpl* disk) return *this; } -IoImpl& IoImpl::set_storage(resource::StorageImpl* storage) -{ - storage_ = storage; - return *this; -} - IoImpl* IoImpl::start() { - state_ = State::RUNNING; - if (storage_) - surf_action_ = storage_->io_start(size_, type_); - else - surf_action_ = disk_->io_start(size_, type_); + set_state(State::RUNNING); + surf_action_ = disk_->io_start(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()); - IoImpl::on_start(*this); return this; } @@ -70,63 +86,71 @@ 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 (disk_ && not disk_->is_on()) + set_state(State::FAILED); else - state_ = State::CANCELED; - } else if (surf_action_->get_state() == resource::Action::State::FINISHED) { - state_ = State::DONE; + set_state(State::CANCELED); } else if (timeout_detector_ && timeout_detector_->get_state() == resource::Action::State::FINISHED) { - state_ = State::TIMEOUT; + if (surf_action_->get_remains() > 0.0) { + surf_action_->set_state(resource::Action::State::FAILED); + set_state(State::TIMEOUT); + } else { + set_state(State::DONE); + } + } else { + set_state(State::DONE); } + clean_action(); if (timeout_detector_) { timeout_detector_->unref(); 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::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; + /* 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); + + 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