From: mlaurent <mathieu.laurent@ens-rennes.fr>
Date: Thu, 9 Nov 2023 09:34:39 +0000 (+0100)
Subject: Merge branch 'master' of https://framagit.org/simgrid/simgrid
X-Git-Tag: v3.35~83
X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/9fc682676a33abb6a4cc527c09a34db3c33e5711

Merge branch 'master' of https://framagit.org/simgrid/simgrid
---

9fc682676a33abb6a4cc527c09a34db3c33e5711
diff --cc src/mc/explo/DFSExplorer.cpp
index ee49e31d9b,a7c9dcf021..8d85325ef7
--- a/src/mc/explo/DFSExplorer.cpp
+++ b/src/mc/explo/DFSExplorer.cpp
@@@ -136,29 -138,19 +138,23 @@@ void DFSExplorer::run(
      const aid_t next = reduction_mode_ == ReductionMode::odpor ? state->next_odpor_transition()
                                                                 : std::get<0>(state->next_transition_guided());
  
-     if (next < 0) {
-       // If there is no more transition in the current state ), backtrace
-       XBT_VERB("%lu actors remain, but none of them need to be interleaved (depth %zu).", state->get_actor_count(),
-                stack_.size() + 1);
 +
-       if (state->get_actor_count() == 0) {
-         get_remote_app().finalize_app();
-         XBT_VERB("Execution came to an end at %s (state: %ld, depth: %zu)", get_record_trace().to_string().c_str(),
-                  state->get_num(), stack_.size());
+     if (next < 0 || not state->is_actor_enabled(next)) {
+       if (next >= 0) { // Actor is not enabled, then
+         XBT_INFO("Reduction %s wants to execute a disabled transition %s. If it's ODPOR, ReversibleRace is suboptimal.",
+                  to_c_str(reduction_mode_),
+                  state->get_actors_list().at(next).get_transition()->to_string(true).c_str());
+         if (reduction_mode_ == ReductionMode::odpor) {
+           XBT_INFO("Current trace:");
+           for (auto elm : get_textual_trace())
+             XBT_ERROR("%s", elm.c_str());
++	  // Remove the disabled transition from the wakeup tree so that ODPOR doesn't try it again
++	  state->remove_subtree_at_aid(next);
++	  state->add_sleep_set(state->get_actors_list().at(next).get_transition());
+         }
        }
- 
-       this->backtrack();
-       continue;
-     }
-     if (not state->is_actor_enabled(next)) {
-       // if ODPOR picked an actor that is not enabled -- ReversibleRace is an overapproximation
-       xbt_assert(reduction_mode_ == ReductionMode::odpor,
-                  "Only ODPOR should be fool enough to try to execute a disabled transition");
-       XBT_VERB("Preventing ODPOR exploration from executing a disabled transition. The reversibility of the race "
-                "must have been overapproximated");
- 
-       state->remove_subtree_at_aid(next);
-       state->add_sleep_set(state->get_actors_list().at(next).get_transition());
+       // If there is no more transition in the current state (or if ODPOR picked an actor that is not enabled --
+       // ReversibleRace is an overapproximation), backtrace
        XBT_VERB("%lu actors remain, but none of them need to be interleaved (depth %zu).", state->get_actor_count(),
                 stack_.size() + 1);