X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/1bd3b1e9e02dd5662a771c1b382cfa1d1d9a2e08..540f367f67ed84b6e6513f8a4f19a06ac3acbc2a:/src/surf/surf_c_bindings.cpp diff --git a/src/surf/surf_c_bindings.cpp b/src/surf/surf_c_bindings.cpp index ea95db4dbb..16ead873b4 100644 --- a/src/surf/surf_c_bindings.cpp +++ b/src/surf/surf_c_bindings.cpp @@ -41,9 +41,8 @@ void surf_presolve(void) while ((next_event_date = future_evt_set->next_date()) != -1.0) { if (next_event_date > NOW) break; - while ((event = future_evt_set->pop_leq(next_event_date, - &value, - (void **) &resource))) { + + while ((event = future_evt_set->pop_leq(next_event_date, &value, &resource))) { if (value >= 0){ resource->updateState(event, value, NOW); } @@ -77,32 +76,31 @@ double surf_solve(double max_date) XBT_DEBUG("Looking for next event in physical models"); double next_event_phy = surf_host_model->shareResources(NOW); if ((surf_min < 0.0 || next_event_phy < surf_min) && next_event_phy >= 0.0) { - surf_min = next_event_phy; + surf_min = next_event_phy; } if (surf_vm_model != NULL) { - XBT_DEBUG("Looking for next event in virtual models"); - double next_event_virt = surf_vm_model->shareResources(NOW); - if ((surf_min < 0.0 || next_event_virt < surf_min) && next_event_virt >= 0.0) - surf_min = next_event_virt; + XBT_DEBUG("Looking for next event in virtual models"); + double next_event_virt = surf_vm_model->shareResources(NOW); + if ((surf_min < 0.0 || next_event_virt < surf_min) && next_event_virt >= 0.0) + surf_min = next_event_virt; } - XBT_DEBUG("Min for resources (remember that NS3 don't update that value) : %f", surf_min); + XBT_DEBUG("Min for resources (remember that NS3 don't update that value): %f", surf_min); XBT_DEBUG("Looking for next trace event"); - do { - XBT_DEBUG("Next TRACE event : %f", next_event_date); - + while (1) { // Handle next occurring events until none remains next_event_date = future_evt_set->next_date(); + XBT_DEBUG("Next TRACE event: %f", next_event_date); if(! surf_network_model->shareResourcesIsIdempotent()){ // NS3, I see you - if(next_event_date!=-1.0 && surf_min!=-1.0) { + if (next_event_date!=-1.0 && surf_min!=-1.0) { surf_min = MIN(next_event_date - NOW, surf_min); } else{ surf_min = MAX(next_event_date - NOW, surf_min); } - XBT_DEBUG("Run for network at most %f", surf_min); + XBT_DEBUG("Run the NS3 network at most %fs", surf_min); // run until min or next flow model_next_action_end = surf_network_model->shareResources(surf_min); @@ -116,41 +114,39 @@ double surf_solve(double max_date) break; } - if ((surf_min == -1.0) || (next_event_date > NOW + surf_min)) break; + if ((surf_min == -1.0) || (next_event_date > NOW + surf_min)) + break; // next event occurs after the next resource change, bail out XBT_DEBUG("Updating models (min = %g, NOW = %g, next_event_date = %g)", surf_min, NOW, next_event_date); - while ((event = future_evt_set->pop_leq(next_event_date, - &value, - (void **) &resource))) { + + while ((event = future_evt_set->pop_leq(next_event_date, &value, &resource))) { if (resource->isUsed() || xbt_dict_get_or_null(watched_hosts_lib, resource->getName())) { surf_min = next_event_date - NOW; - XBT_DEBUG - ("This event will modify model state. Next event set to %f", - surf_min); + XBT_DEBUG("This event will modify model state. Next event set to %f", surf_min); } - /* update state of model_obj according to new value. Does not touch lmm. + /* update state of the corresponding resource to the new value. Does not touch lmm. It will be modified if needed when updating actions */ - XBT_DEBUG("Calling update_resource_state for resource %s with min %f", - resource->getName(), surf_min); + XBT_DEBUG("Calling update_resource_state for resource %s with min %f", resource->getName(), surf_min); resource->updateState(event, value, next_event_date); } - } while (1); + } /* FIXME: Moved this test to here to avoid stopping simulation if there are actions running on cpus and all cpus are with availability = 0. * This may cause an infinite loop if one cpu has a trace with periodicity = 0 and the other a trace with periodicity > 0. * The options are: all traces with same periodicity(0 or >0) or we need to change the way how the events are managed */ if (surf_min == -1.0) { - XBT_DEBUG("No next event at all. Bail out now."); + XBT_DEBUG("No next event at all. Bail out now."); return -1.0; } XBT_DEBUG("Duration set to %f", surf_min); + // Bump the time: jump into the future NOW = NOW + surf_min; - /* FIXME: model_list or model_list_invoke? revisit here later */ - /* sequential version */ + + // Inform the models of the date change xbt_dynar_foreach(all_existing_models, iter, model) { - model->updateActionsState(NOW, surf_min); + model->updateActionsState(NOW, surf_min); } TRACE_paje_dump_buffer (0); @@ -158,18 +154,13 @@ double surf_solve(double max_date) return surf_min; } -void routing_get_route_and_latency(sg_netcard_t src, sg_netcard_t dst, - xbt_dynar_t * route, double *latency){ - routing_platf->getRouteAndLatency(src, dst, route, latency); -} - /********* * MODEL * *********/ surf_action_t surf_model_extract_done_action_set(surf_model_t model){ if (model->getDoneActionSet()->empty()) - return NULL; + return NULL; surf_action_t res = &model->getDoneActionSet()->front(); model->getDoneActionSet()->pop_front(); return res; @@ -177,7 +168,7 @@ surf_action_t surf_model_extract_done_action_set(surf_model_t model){ surf_action_t surf_model_extract_failed_action_set(surf_model_t model){ if (model->getFailedActionSet()->empty()) - return NULL; + return NULL; surf_action_t res = &model->getFailedActionSet()->front(); model->getFailedActionSet()->pop_front(); return res; @@ -207,7 +198,7 @@ const char *surf_resource_name(surf_cpp_resource_t resource){ } surf_action_t surf_host_sleep(sg_host_t host, double duration){ - return host->pimpl_cpu->sleep(duration); + return host->pimpl_cpu->sleep(duration); }