X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/af6d6b158ab3c99f190cb670b1918bbadaa9e250..8b8f538a38613e833acaf7366f7dcf84526f23a2:/src/mc/VisitedState.cpp diff --git a/src/mc/VisitedState.cpp b/src/mc/VisitedState.cpp index 9c9270f85f..a52b8fcddf 100644 --- a/src/mc/VisitedState.cpp +++ b/src/mc/VisitedState.cpp @@ -1,10 +1,11 @@ -/* Copyright (c) 2011-2022. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2011-2023. 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/mc/VisitedState.hpp" #include "src/mc/explo/Exploration.hpp" +#include "src/mc/mc_config.hpp" #include "src/mc/mc_private.hpp" #include @@ -17,11 +18,13 @@ XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_VisitedState, mc, "Logging specific to state namespace simgrid::mc { /** @brief Save the current state */ -VisitedState::VisitedState(unsigned long state_number, unsigned int actor_count) - : actor_count_(actor_count), num(state_number) +VisitedState::VisitedState(unsigned long state_number, unsigned int actor_count, RemoteApp& remote_app) + : heap_bytes_used_(remote_app.get_remote_process_memory().get_remote_heap_bytes()) + , actor_count_(actor_count) + , num_(state_number) { - this->heap_bytes_used = mc_model_checker->get_remote_process().get_remote_heap_bytes(); - this->system_state = std::make_shared(state_number); + this->system_state_ = std::make_shared(state_number, remote_app.get_page_store(), + remote_app.get_remote_process_memory()); } void VisitedStates::prune() @@ -30,7 +33,7 @@ void VisitedStates::prune() XBT_DEBUG("Try to remove visited state (maximum number of stored states reached)"); auto min_element = boost::range::min_element( states_, [](const std::unique_ptr& a, - const std::unique_ptr& b) { return a->num < b->num; }); + const std::unique_ptr& b) { return a->num_ < b->num_; }); xbt_assert(min_element != states_.end()); // and drop it: states_.erase(min_element); @@ -40,44 +43,46 @@ void VisitedStates::prune() /** @brief Checks whether a given state has already been visited by the algorithm. */ std::unique_ptr -VisitedStates::addVisitedState(unsigned long state_number, simgrid::mc::State* graph_state, bool compare_snapshots) +VisitedStates::addVisitedState(unsigned long state_number, simgrid::mc::State* graph_state, RemoteApp& remote_app) { - auto new_state = std::make_unique(state_number, graph_state->get_actor_count()); - graph_state->set_system_state(new_state->system_state); - XBT_DEBUG("Snapshot %p of visited state %ld (exploration stack state %ld)", new_state->system_state.get(), - new_state->num, graph_state->get_num()); - - auto [range_begin, range_end] = - boost::range::equal_range(states_, new_state.get(), compare_pair_by_actor_count_and_used_heap()); - - if (compare_snapshots) - for (auto i = range_begin; i != range_end; ++i) { - auto& visited_state = *i; - if (*visited_state->system_state.get() == *new_state->system_state.get()) { - // The state has been visited: - - std::unique_ptr old_state = - std::move(visited_state); - - if (old_state->original_num == -1) // I'm the copy of an original process - new_state->original_num = old_state->num; - else // I'm the copy of a copy - new_state->original_num = old_state->original_num; - - XBT_DEBUG("State %ld already visited ! (equal to state %ld (state %ld in dot_output))", new_state->num, - old_state->num, new_state->original_num); - - /* Replace the old state with the new one (with a bigger num) - (when the max number of visited states is reached, the oldest - one is removed according to its number (= with the min number) */ - XBT_DEBUG("Replace visited state %ld with the new visited state %ld", old_state->num, new_state->num); - - visited_state = std::move(new_state); - return old_state; - } + auto new_state = + std::make_unique(state_number, graph_state->get_actor_count(), remote_app); + + graph_state->set_system_state(new_state->system_state_); + XBT_DEBUG("Snapshot %p of visited state %ld (exploration stack state %ld)", new_state->system_state_.get(), + new_state->num_, graph_state->get_num()); + + auto [range_begin, range_end] = boost::range::equal_range(states_, new_state.get(), [](auto const& a, auto const& b) { + return std::make_pair(a->actor_count_, a->heap_bytes_used_) < std::make_pair(b->actor_count_, b->heap_bytes_used_); + }); + + for (auto i = range_begin; i != range_end; ++i) { + auto& visited_state = *i; + if (visited_state->system_state_->equals_to(*new_state->system_state_.get(), + remote_app.get_remote_process_memory())) { + // The state has been visited: + + std::unique_ptr old_state = std::move(visited_state); + + if (old_state->original_num_ == -1) // I'm the copy of an original process + new_state->original_num_ = old_state->num_; + else // I'm the copy of a copy + new_state->original_num_ = old_state->original_num_; + + XBT_DEBUG("State %ld already visited ! (equal to state %ld (state %ld in dot_output))", new_state->num_, + old_state->num_, new_state->original_num_); + + /* Replace the old state with the new one (with a bigger num) + (when the max number of visited states is reached, the oldest + one is removed according to its number (= with the min number) */ + XBT_DEBUG("Replace visited state %ld with the new visited state %ld", old_state->num_, new_state->num_); + + visited_state = std::move(new_state); + return old_state; } + } - XBT_DEBUG("Insert new visited state %ld (total : %lu)", new_state->num, (unsigned long)states_.size()); + XBT_DEBUG("Insert new visited state %ld (total : %lu)", new_state->num_, (unsigned long)states_.size()); states_.insert(range_begin, std::move(new_state)); this->prune(); return nullptr;