X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/ba9a4cfeba4eb00e84cd17603fc9654e81445655..783b33766eccfc78e3b205d896323169ec42e7c1:/src/simix/smx_vm.cpp diff --git a/src/simix/smx_vm.cpp b/src/simix/smx_vm.cpp index 1bf7c7aa43..ba05d17e6a 100644 --- a/src/simix/smx_vm.cpp +++ b/src/simix/smx_vm.cpp @@ -1,5 +1,4 @@ -/* 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. */ @@ -9,15 +8,14 @@ #include "src/surf/virtual_machine.hpp" #include "src/surf/HostImpl.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) + * @brief Internal function to create a SIMIX host. + * @param name name of the host to create + * @param data some user data (may be nullptr) */ sg_host_t SIMIX_vm_create(const char *name, sg_host_t ind_phys_host) { @@ -52,7 +50,7 @@ static int __can_be_started(sg_host_t vm) 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. */ @@ -60,7 +58,7 @@ 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; } @@ -70,7 +68,7 @@ static int __can_be_started(sg_host_t vm) 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; } } @@ -105,9 +103,9 @@ e_surf_vm_state_t SIMIX_vm_get_state(sg_host_t ind_vm) } /** - * \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) { @@ -119,258 +117,227 @@ void SIMIX_vm_migrate(sg_host_t ind_vm, sg_host_t ind_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(); + smx_actor_t self = SIMIX_process_self(); SIMIX_vm_resume(vm, self->simcall.issuer); } /** - * \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. + * @brief Function to get the physical host of the given SIMIX VM host. * - * \param host the vm host (a sg_host_t) - * \param bound bound (a double) + * @param host the vm host to get_phys_host (a sg_host_t) */ -void SIMIX_vm_set_bound(sg_host_t ind_vm, double bound) +void *SIMIX_vm_get_pm(sg_host_t host) { - /* jump to vm_ws_set_vm_bound(). */ - surf_vm_set_bound(ind_vm, bound); + return surf_vm_get_pm(host); } /** - * \brief Function to set the CPU affinity of the given SIMIX VM host. + * @brief Function to set the CPU bound 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) + * @param host the vm host (a sg_host_t) + * @param bound bound (a double) */ -void SIMIX_vm_set_affinity(sg_host_t ind_vm, sg_host_t ind_pm, unsigned long mask) +void SIMIX_vm_set_bound(sg_host_t vm, double bound) { - /* 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); + surf_vm_set_bound(vm, bound); } - /** - * \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); + surf_vm_suspend(vm); - 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, smx_actor_t issuer) { - 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); + surf_vm_resume(vm); - 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); + 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, issuer); } } -void simcall_HANDLER_vm_resume(smx_simcall_t simcall, sg_host_t ind_vm) +void simcall_HANDLER_vm_resume(smx_simcall_t simcall, sg_host_t vm) { - SIMIX_vm_resume(ind_vm, simcall->issuer); + SIMIX_vm_resume(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); + surf_vm_save(vm); - 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, smx_actor_t issuer) { - const char *name = sg_host_get_name(ind_vm); - - if (SIMIX_vm_get_state(ind_vm) != SURF_VM_STATE_SAVED) - THROWF(vm_error, 0, "VM(%s) was not saved", name); + if (SIMIX_vm_get_state(vm) != SURF_VM_STATE_SAVED) + THROWF(vm_error, 0, "VM(%s) was not saved", vm->name().c_str()); - 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); + surf_vm_resume(vm); - 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); + 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, issuer); } } -void simcall_HANDLER_vm_restore(smx_simcall_t simcall, sg_host_t ind_vm) +void simcall_HANDLER_vm_restore(smx_simcall_t simcall, sg_host_t vm) { - SIMIX_vm_restore(ind_vm, simcall->issuer); + SIMIX_vm_restore(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(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("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() + vm->extension() )->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() */ - - const char *hostname = sg_host_get_name(ind_vm); - - XBT_DEBUG("destroy %s", hostname); - - /* this will call the registered callback function, i.e., SIMIX_host_destroy(). */ - sg_host_simix_destroy(ind_vm); - - /* jump to vm_ws_destroy(). The surf level resource will be freed. */ - surf_vm_destroy(ind_vm); + XBT_DEBUG("destroy %s", vm->name().c_str()); + + /* 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); + + /* Don't free these things twice: they are the ones of my physical host */ + vm->pimpl_cpu = nullptr; + vm->pimpl_netcard = nullptr; }