X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/b9a15f078a62e397f321dfb570254652785cc377..c2505da34e3d93fa429d9f047e56257378901ee4:/src/s4u/s4u_host.cpp diff --git a/src/s4u/s4u_host.cpp b/src/s4u/s4u_host.cpp index 9c3b420d3c..ed367e8d8f 100644 --- a/src/s4u/s4u_host.cpp +++ b/src/s4u/s4u_host.cpp @@ -14,14 +14,15 @@ #include "src/surf/HostImpl.hpp" #include "xbt/log.h" #include "src/msg/msg_private.h" -#include "src/simix/smx_process_private.h" +#include "src/simix/ActorImpl.hpp" #include "src/simix/smx_private.h" #include "src/surf/cpu_interface.hpp" #include "simgrid/s4u/host.hpp" #include "simgrid/s4u/storage.hpp" +std::unordered_map host_list; // FIXME: move it to Engine + int MSG_HOST_LEVEL = -1; -int SIMIX_HOST_LEVEL = -1; int USER_HOST_LEVEL = -1; namespace simgrid { @@ -39,48 +40,71 @@ simgrid::xbt::signal Host::onStateChange; Host::Host(const char* name) : name_(name) { + xbt_assert(sg_host_by_name(name) == nullptr, "Refusing to create a second host named '%s'.", name); + host_list[name_] = this; } -Host::~Host() { +Host::~Host() +{ + xbt_assert(currentlyDestroying_, "Please call h->destroy() instead of manually deleting it."); + + delete pimpl_; delete pimpl_cpu; delete pimpl_netcard; delete mounts; } -Host *Host::by_name(std::string name) { - Host* host = Host::by_name_or_null(name.c_str()); - // TODO, raise an exception instead? - if (host == nullptr) - xbt_die("No such host: %s", name.c_str()); - return host; +/** @brief Fire the required callbacks and destroy the object + * + * Don't delete directly an Host, call h->destroy() instead. + * + * This is cumbersome but this is the simplest solution to ensure that the + * onDestruction() callback receives a valid object (because of the destructor + * order in a class hierarchy). + */ +void Host::destroy() +{ + if (!currentlyDestroying_) { + currentlyDestroying_ = true; + onDestruction(*this); + host_list.erase(name_); + delete this; + } } -Host* Host::by_name_or_null(const char* name) + +Host* Host::by_name(std::string name) { - return (Host*) xbt_dict_get_or_null(host_list, name); + return host_list.at(name); // Will raise a std::out_of_range if the host does not exist } -Host* Host::by_name_or_create(const char* name) +Host* Host::by_name_or_null(const char* name) { - Host* host = by_name_or_null(name); - if (host == nullptr) { - host = new Host(name); - xbt_dict_set(host_list, name, host, nullptr); + try { + return host_list.at(name); + } catch (std::out_of_range& e) { + return nullptr; } - return host; } Host *Host::current(){ - smx_process_t smx_proc = SIMIX_process_self(); + smx_actor_t smx_proc = SIMIX_process_self(); if (smx_proc == nullptr) xbt_die("Cannot call Host::current() from the maestro context"); - return SIMIX_process_get_host(smx_proc); + return smx_proc->host; } void Host::turnOn() { - simgrid::simix::kernelImmediate(std::bind(SIMIX_host_on, this)); + if (isOff()) { + simgrid::simix::kernelImmediate([&]{ + this->extension()->turnOn(); + this->pimpl_cpu->turnOn(); + }); + } } void Host::turnOff() { - simgrid::simix::kernelImmediate(std::bind(SIMIX_host_off, this, SIMIX_process_self())); + if (isOn()) { + simgrid::simix::kernelImmediate(std::bind(SIMIX_host_off, this, SIMIX_process_self())); + } } bool Host::isOn() { @@ -111,29 +135,22 @@ boost::unordered_map const& Host::mountedStorages() { /** Get the properties assigned to a host */ xbt_dict_t Host::properties() { - return simgrid::simix::kernelImmediate([&] { - simgrid::surf::HostImpl* surf_host = this->extension(); - return surf_host->getProperties(); - }); + return simgrid::simix::kernelImmediate([&] { return this->pimpl_->getProperties(); }); } /** Retrieve the property value (or nullptr if not set) */ const char*Host::property(const char*key) { - simgrid::surf::HostImpl* surf_host = this->extension(); - return surf_host->getProperty(key); + return this->pimpl_->getProperty(key); } void Host::setProperty(const char*key, const char *value){ - simgrid::simix::kernelImmediate([&] { - simgrid::surf::HostImpl* surf_host = this->extension(); - surf_host->setProperty(key,value); - }); + simgrid::simix::kernelImmediate([&] { this->pimpl_->setProperty(key, value); }); } /** Get the processes attached to the host */ xbt_swag_t Host::processes() { return simgrid::simix::kernelImmediate([&]() { - return ((smx_host_priv_t)this->extension(SIMIX_HOST_LEVEL))->process_list; + return this->extension()->process_list; }); } @@ -158,8 +175,8 @@ double Host::speed() { return pimpl_cpu->getSpeed(1.0); } /** @brief Returns the number of core of the processor. */ -int Host::coresCount() { - return pimpl_cpu->getCore(); +int Host::coreCount() { + return pimpl_cpu->coreCount(); } /** @brief Set the pstate at which the host should run */ @@ -175,20 +192,6 @@ int Host::pstate() return pimpl_cpu->getPState(); } -void Host::parameters(vm_params_t params) -{ - simgrid::simix::kernelImmediate([&]() { - this->extension()->getParams(params); - }); -} - -void Host::setParameters(vm_params_t params) -{ - simgrid::simix::kernelImmediate([&]() { - this->extension()->setParams(params); - }); -} - /** * \ingroup simix_storage_management * \brief Returns the list of storages mounted on an host. @@ -196,9 +199,7 @@ void Host::setParameters(vm_params_t params) */ xbt_dict_t Host::mountedStoragesAsDict() { - return simgrid::simix::kernelImmediate([&] { - return this->extension()->getMountedStorageList(); - }); + return simgrid::simix::kernelImmediate([&] { return this->pimpl_->getMountedStorageList(); }); } /** @@ -208,9 +209,7 @@ xbt_dict_t Host::mountedStoragesAsDict() */ xbt_dynar_t Host::attachedStorages() { - return simgrid::simix::kernelImmediate([&] { - return this->extension()->getAttachedStorageList(); - }); + return simgrid::simix::kernelImmediate([&] { return this->pimpl_->getAttachedStorageList(); }); } } // namespace simgrid