X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/347996b4a10c4e8579080692afa60e0afb88b60a..a3f6c5e832f9966433370562ef59fc4670038c94:/src/simix/smx_vm.cpp diff --git a/src/simix/smx_vm.cpp b/src/simix/smx_vm.cpp index fe322159a9..de015591a5 100644 --- a/src/simix/smx_vm.cpp +++ b/src/simix/smx_vm.cpp @@ -1,46 +1,21 @@ -/* Copyright (c) 2007-2015. The SimGrid Team. - * All rights reserved. */ +/* Copyright (c) 2007-2015. 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 "smx_private.h" -#include "xbt/sysdep.h" -#include "xbt/log.h" -#include "xbt/dict.h" #include "mc/mc.h" -#include "src/surf/host_interface.hpp" -#include "src/surf/virtual_machine.hpp" - -//If you need to log some stuffs, just uncomment these two lines and uses XBT_DEBUG for instance -XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_vm, simix, "Logging specific to SIMIX (vms)"); - -/* **** create a VM **** */ - -/** - * \brief Internal function to create a SIMIX host. - * \param name name of the host to create - * \param data some user data (may be NULL) - */ -sg_host_t SIMIX_vm_create(const char *name, sg_host_t ind_phys_host) -{ - /* Create surf associated resource */ - surf_vm_model_create(name, ind_phys_host); - sg_host_t host = sg_host_by_name(name); - SIMIX_host_create(host); - - /* We will be able to register the VM to its physical host, so that we can promptly - * retrieve the list VMs on the physical host. */ - - return host; -} +#include "simgrid/s4u/VirtualMachine.hpp" +#include "smx_private.h" +#include "src/plugins/vm/VirtualMachineImpl.hpp" +#include "src/surf/HostImpl.hpp" +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_vm, simix, "Logging specific to SIMIX Virtual Machines"); /* works for VMs and PMs */ static long host_get_ramsize(sg_host_t vm, int *overcommit) { s_vm_params_t params; - vm->extension()->getParams(¶ms); + static_cast(vm)->parameters(¶ms); if (overcommit) *overcommit = params.overcommit; @@ -51,11 +26,11 @@ static long host_get_ramsize(sg_host_t vm, int *overcommit) /* **** start a VM **** */ static int __can_be_started(sg_host_t vm) { - sg_host_t pm = surf_vm_get_pm(vm); + sg_host_t pm = static_cast(vm)->pimpl_vm_->getPm(); int pm_overcommit = 0; long pm_ramsize = host_get_ramsize(pm, &pm_overcommit); - long vm_ramsize = host_get_ramsize(vm, NULL); + long vm_ramsize = host_get_ramsize(vm, nullptr); if (!pm_ramsize) { /* We assume users do not want to care about ramsize. */ @@ -63,17 +38,17 @@ static int __can_be_started(sg_host_t vm) } if (pm_overcommit) { - XBT_INFO("%s allows memory overcommit.", sg_host_get_name(pm)); + XBT_VERB("%s allows memory overcommit.", sg_host_get_name(pm)); return 1; } long total_ramsize_of_vms = 0; - xbt_dynar_t dyn_vms = pm->extension()->getVms(); + xbt_dynar_t dyn_vms = pm->pimpl_->getVms(); { unsigned int cursor = 0; sg_host_t another_vm; xbt_dynar_foreach(dyn_vms, cursor, another_vm) { - long another_vm_ramsize = host_get_ramsize(vm, NULL); + long another_vm_ramsize = host_get_ramsize(vm, nullptr); total_ramsize_of_vms += another_vm_ramsize; } } @@ -89,291 +64,222 @@ static int __can_be_started(sg_host_t vm) return 1; } -void SIMIX_vm_start(sg_host_t ind_vm) +void SIMIX_vm_start(sg_host_t vm) { - if (__can_be_started(ind_vm)) - static_cast( - ind_vm->extension() - )->setState(SURF_VM_STATE_RUNNING); + if (__can_be_started(vm)) + static_cast(vm->pimpl_)->setState(SURF_VM_STATE_RUNNING); else - THROWF(vm_error, 0, "The VM %s cannot be started", sg_host_get_name(ind_vm)); + THROWF(vm_error, 0, "The VM %s cannot be started", vm->name().c_str()); } -e_surf_vm_state_t SIMIX_vm_get_state(sg_host_t ind_vm) +e_surf_vm_state_t SIMIX_vm_get_state(sg_host_t vm) { - return static_cast( - ind_vm->extension() - )->getState(); + return static_cast(vm->pimpl_)->getState(); } /** - * \brief Function to migrate a SIMIX VM host. + * @brief Function to migrate a SIMIX VM host. * - * \param host the vm host to migrate (a sg_host_t) + * @param host the vm host to migrate (a sg_host_t) */ -void SIMIX_vm_migrate(sg_host_t ind_vm, sg_host_t ind_dst_pm) +void SIMIX_vm_migrate(sg_host_t vm, sg_host_t dst_pm) { /* precopy migration makes the VM temporally paused */ - xbt_assert(SIMIX_vm_get_state(ind_vm) == SURF_VM_STATE_SUSPENDED); + xbt_assert(SIMIX_vm_get_state(vm) == SURF_VM_STATE_SUSPENDED); /* jump to vm_ws_xigrate(). this will update the vm location. */ - surf_vm_migrate(ind_vm, ind_dst_pm); + static_cast(vm)->pimpl_vm_->migrate(dst_pm); } /** - * \brief Encompassing simcall to prevent the removal of the src or the dst node at the end of a VM migration + * @brief Encompassing simcall to prevent the removal of the src or the dst node at the end of a VM migration * The simcall actually invokes the following calls: - * simcall_vm_set_affinity(vm, src_pm, 0); * simcall_vm_migrate(vm, dst_pm); * simcall_vm_resume(vm); * * It is called at the end of the migration_rx_fun function from msg/msg_vm.c * - * \param vm VM to migrate - * \param src_pm Source physical host - * \param dst_pmt Destination physical host + * @param vm VM to migrate + * @param src_pm Source physical host + * @param dst_pmt Destination physical host */ void SIMIX_vm_migratefrom_resumeto(sg_host_t vm, sg_host_t src_pm, sg_host_t dst_pm) { - /* deinstall the current affinity setting for the CPU */ - SIMIX_vm_set_affinity(vm, src_pm, 0); - /* Update the vm location */ SIMIX_vm_migrate(vm, dst_pm); /* Resume the VM */ - smx_process_t self = SIMIX_process_self(); - SIMIX_vm_resume(vm, self->simcall.issuer); + SIMIX_vm_resume(vm); } /** - * \brief Function to get the physical host of the given SIMIX VM host. - * - * \param host the vm host to get_phys_host (a sg_host_t) - */ -void *SIMIX_vm_get_pm(sg_host_t ind_vm) -{ - /* jump to vm_ws_get_pm(). this will return the vm name. */ - return surf_vm_get_pm(ind_vm); -} - -/** - * \brief Function to set the CPU bound of the given SIMIX VM host. - * - * \param host the vm host (a sg_host_t) - * \param bound bound (a double) - */ -void SIMIX_vm_set_bound(sg_host_t ind_vm, double bound) -{ - /* jump to vm_ws_set_vm_bound(). */ - surf_vm_set_bound(ind_vm, bound); -} - -/** - * \brief Function to set the CPU affinity of the given SIMIX VM host. - * - * \param host the vm host (a sg_host_t) - * \param host the pm host (a sg_host_t) - * \param mask affinity mask (a unsigned long) - */ -void SIMIX_vm_set_affinity(sg_host_t ind_vm, sg_host_t ind_pm, unsigned long mask) -{ - /* make sure this at the MSG layer. */ - xbt_assert(SIMIX_vm_get_pm(ind_vm) == ind_pm); - - /* jump to vm_ws_set_vm_affinity(). */ - surf_vm_set_affinity(ind_vm, ind_pm, mask); -} - - -/** - * \brief Function to suspend a SIMIX VM host. This function stops the execution of the + * @brief Function to suspend a SIMIX VM host. This function stops the execution of the * VM. All the processes on this VM will pause. The state of the VM is * preserved on memory. We can later resume it again. * - * \param host the vm host to suspend (a sg_host_t) + * @param vm the vm host to suspend (a sg_host_t) */ -void SIMIX_vm_suspend(sg_host_t ind_vm, smx_process_t issuer) +void SIMIX_vm_suspend(sg_host_t vm, smx_actor_t issuer) { - const char *name = sg_host_get_name(ind_vm); + if (SIMIX_vm_get_state(vm) != SURF_VM_STATE_RUNNING) + THROWF(vm_error, 0, "VM(%s) is not running", vm->name().c_str()); - if (SIMIX_vm_get_state(ind_vm) != SURF_VM_STATE_RUNNING) - THROWF(vm_error, 0, "VM(%s) is not running", name); - - XBT_DEBUG("suspend VM(%s), where %d processes exist", name, xbt_swag_size(sg_host_simix(ind_vm)->process_list)); + XBT_DEBUG("suspend VM(%s), where %d processes exist", vm->name().c_str(), xbt_swag_size(sg_host_simix(vm)->process_list)); /* jump to vm_ws_suspend. The state will be set. */ - surf_vm_suspend(ind_vm); + static_cast(vm)->pimpl_vm_->suspend(); - smx_process_t smx_process, smx_process_safe; - xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(ind_vm)->process_list) { - XBT_DEBUG("suspend %s", smx_process->name); + smx_actor_t smx_process, smx_process_safe; + xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(vm)->process_list) { + XBT_DEBUG("suspend %s", smx_process->name.c_str()); SIMIX_process_suspend(smx_process, issuer); } XBT_DEBUG("suspend all processes on the VM done done"); } -void simcall_HANDLER_vm_suspend(smx_simcall_t simcall, sg_host_t ind_vm) +void simcall_HANDLER_vm_suspend(smx_simcall_t simcall, sg_host_t vm) { - if (simcall->issuer->host == ind_vm) { - XBT_ERROR("cannot suspend the VM where I run"); - DIE_IMPOSSIBLE; - } + xbt_assert(simcall->issuer->host != vm, "cannot suspend the VM where I run"); - SIMIX_vm_suspend(ind_vm, simcall->issuer); + SIMIX_vm_suspend(vm, simcall->issuer); XBT_DEBUG("simcall_HANDLER_vm_suspend done"); } /** - * \brief Function to resume a SIMIX VM host. This function restart the execution of the + * @brief Function to resume a SIMIX VM host. This function restart the execution of the * VM. All the processes on this VM will run again. * - * \param host the vm host to resume (a sg_host_t) + * @param vm the vm host to resume (a sg_host_t) */ -void SIMIX_vm_resume(sg_host_t ind_vm, smx_process_t issuer) +void SIMIX_vm_resume(sg_host_t vm) { - const char *name = sg_host_get_name(ind_vm); + if (SIMIX_vm_get_state(vm) != SURF_VM_STATE_SUSPENDED) + THROWF(vm_error, 0, "VM(%s) was not suspended", vm->name().c_str()); - if (SIMIX_vm_get_state(ind_vm) != SURF_VM_STATE_SUSPENDED) - THROWF(vm_error, 0, "VM(%s) was not suspended", name); - - XBT_DEBUG("resume VM(%s), where %d processes exist", name, xbt_swag_size(sg_host_simix(ind_vm)->process_list)); + XBT_DEBUG("resume VM(%s), where %d processes exist", + vm->name().c_str(), xbt_swag_size(sg_host_simix(vm)->process_list)); /* jump to vm_ws_resume() */ - surf_vm_resume(ind_vm); + static_cast(vm)->pimpl_vm_->resume(); - smx_process_t smx_process, smx_process_safe; - xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(ind_vm)->process_list) { - XBT_DEBUG("resume %s", smx_process->name); - SIMIX_process_resume(smx_process, issuer); + smx_actor_t smx_process, smx_process_safe; + xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(vm)->process_list) { + XBT_DEBUG("resume %s", smx_process->name.c_str()); + SIMIX_process_resume(smx_process); } } -void simcall_HANDLER_vm_resume(smx_simcall_t simcall, sg_host_t ind_vm) -{ - SIMIX_vm_resume(ind_vm, simcall->issuer); -} - - /** - * \brief Function to save a SIMIX VM host. + * @brief Function to save a SIMIX VM host. * This function is the same as vm_suspend, but the state of the VM is saved to the disk, and not preserved on memory. * We can later restore it again. * - * \param host the vm host to save (a sg_host_t) + * @param vm the vm host to save (a sg_host_t) */ -void SIMIX_vm_save(sg_host_t ind_vm, smx_process_t issuer) +void SIMIX_vm_save(sg_host_t vm, smx_actor_t issuer) { - const char *name = sg_host_get_name(ind_vm); + const char *name = sg_host_get_name(vm); - if (SIMIX_vm_get_state(ind_vm) != SURF_VM_STATE_RUNNING) + if (SIMIX_vm_get_state(vm) != SURF_VM_STATE_RUNNING) THROWF(vm_error, 0, "VM(%s) is not running", name); - - XBT_DEBUG("save VM(%s), where %d processes exist", name, xbt_swag_size(sg_host_simix(ind_vm)->process_list)); + XBT_DEBUG("save VM(%s), where %d processes exist", name, xbt_swag_size(sg_host_simix(vm)->process_list)); /* jump to vm_ws_save() */ - surf_vm_save(ind_vm); + static_cast(vm)->pimpl_vm_->resume(); - smx_process_t smx_process, smx_process_safe; - xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(ind_vm)->process_list) { - XBT_DEBUG("suspend %s", smx_process->name); + smx_actor_t smx_process, smx_process_safe; + xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(vm)->process_list) { + XBT_DEBUG("suspend %s", smx_process->name.c_str()); SIMIX_process_suspend(smx_process, issuer); } } -void simcall_HANDLER_vm_save(smx_simcall_t simcall, sg_host_t ind_vm) +void simcall_HANDLER_vm_save(smx_simcall_t simcall, sg_host_t vm) { - SIMIX_vm_save(ind_vm, simcall->issuer); + SIMIX_vm_save(vm, simcall->issuer); } /** - * \brief Function to restore a SIMIX VM host. This function restart the execution of the + * @brief Function to restore a SIMIX VM host. This function restart the execution of the * VM. All the processes on this VM will run again. * - * \param host the vm host to restore (a sg_host_t) + * @param vm the vm host to restore (a sg_host_t) */ -void SIMIX_vm_restore(sg_host_t ind_vm, smx_process_t issuer) +void SIMIX_vm_restore(sg_host_t vm) { - const char *name = sg_host_get_name(ind_vm); + if (SIMIX_vm_get_state(vm) != SURF_VM_STATE_SAVED) + THROWF(vm_error, 0, "VM(%s) was not saved", vm->name().c_str()); - if (SIMIX_vm_get_state(ind_vm) != SURF_VM_STATE_SAVED) - THROWF(vm_error, 0, "VM(%s) was not saved", name); - - XBT_DEBUG("restore VM(%s), where %d processes exist", name, xbt_swag_size(sg_host_simix(ind_vm)->process_list)); + XBT_DEBUG("restore VM(%s), where %d processes exist", + vm->name().c_str(), xbt_swag_size(sg_host_simix(vm)->process_list)); /* jump to vm_ws_restore() */ - surf_vm_resume(ind_vm); + static_cast(vm)->pimpl_vm_->resume(); - smx_process_t smx_process, smx_process_safe; - xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(ind_vm)->process_list) { - XBT_DEBUG("resume %s", smx_process->name); - SIMIX_process_resume(smx_process, issuer); + smx_actor_t smx_process, smx_process_safe; + xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(vm)->process_list) { + XBT_DEBUG("resume %s", smx_process->name.c_str()); + SIMIX_process_resume(smx_process); } } -void simcall_HANDLER_vm_restore(smx_simcall_t simcall, sg_host_t ind_vm) -{ - SIMIX_vm_restore(ind_vm, simcall->issuer); -} - - /** - * \brief Function to shutdown a SIMIX VM host. This function powers off the + * @brief Function to shutdown a SIMIX VM host. This function powers off the * VM. All the processes on this VM will be killed. But, the state of the VM is * preserved on memory. We can later start it again. * - * \param host the vm host to shutdown (a sg_host_t) + * @param vm the VM to shutdown (a sg_host_t) */ -void SIMIX_vm_shutdown(sg_host_t ind_vm, smx_process_t issuer) +void SIMIX_vm_shutdown(sg_host_t vm, smx_actor_t issuer) { - const char *name = sg_host_get_name(ind_vm); - - if (SIMIX_vm_get_state(ind_vm) != SURF_VM_STATE_RUNNING) - THROWF(vm_error, 0, "VM(%s) is not running", name); + if (SIMIX_vm_get_state(vm) != SURF_VM_STATE_RUNNING) + THROWF(vm_error, 0, "VM(%s) is not running", vm->name().c_str()); - XBT_DEBUG("shutdown %s", name); - XBT_DEBUG("%d processes in the VM", xbt_swag_size(sg_host_simix(ind_vm)->process_list)); + XBT_DEBUG("shutdown VM %s, that contains %d processes", + vm->name().c_str(),xbt_swag_size(sg_host_simix(vm)->process_list)); - smx_process_t smx_process, smx_process_safe; - xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(ind_vm)->process_list) { - XBT_DEBUG("kill %s", smx_process->name); + smx_actor_t smx_process, smx_process_safe; + xbt_swag_foreach_safe(smx_process, smx_process_safe, sg_host_simix(vm)->process_list) { + XBT_DEBUG("kill %s", smx_process->name.c_str()); SIMIX_process_kill(smx_process, issuer); } /* FIXME: we may have to do something at the surf layer, e.g., vcpu action */ - static_cast( - ind_vm->extension() - )->setState(SURF_VM_STATE_CREATED); + static_cast(vm->pimpl_)->setState(SURF_VM_STATE_CREATED); } -void simcall_HANDLER_vm_shutdown(smx_simcall_t simcall, sg_host_t ind_vm) +void simcall_HANDLER_vm_shutdown(smx_simcall_t simcall, sg_host_t vm) { - SIMIX_vm_shutdown(ind_vm, simcall->issuer); + SIMIX_vm_shutdown(vm, simcall->issuer); } - /** - * \brief Function to destroy a SIMIX VM host. + * @brief Function to destroy a SIMIX VM host. * - * \param host the vm host to destroy (a sg_host_t) + * @param vm the vm host to destroy (a sg_host_t) */ -void SIMIX_vm_destroy(sg_host_t ind_vm) +void SIMIX_vm_destroy(sg_host_t vm) { /* this code basically performs a similar thing like SIMIX_host_destroy() */ + XBT_DEBUG("destroy %s", vm->name().c_str()); - const char *hostname = sg_host_get_name(ind_vm); - - XBT_DEBUG("destroy %s", hostname); + /* FIXME: this is really strange that everything fails if the next line is removed. + * This is as if we shared these data with the PM, which definitely should not be the case... + * + * We need to test that suspending a VM does not suspends the processes running on its PM, for example. + * Or we need to simplify this code enough to make it actually readable (but this sounds harder than testing) + */ + vm->extension_set(nullptr); - /* this will call the registered callback function, i.e., SIMIX_host_destroy(). */ - sg_host_simix_destroy(ind_vm); + /* Don't free these things twice: they are the ones of my physical host */ + vm->pimpl_cpu = nullptr; + vm->pimpl_netcard = nullptr; - /* jump to vm_ws_destroy(). The surf level resource will be freed. */ - surf_vm_destroy(ind_vm); + vm->destroy(); }