From: Martin Quinson <martin.quinson@ens-rennes.fr>
Date: Sun, 12 Nov 2023 14:01:06 +0000 (+0100)
Subject: Merge branch 'master' of framagit.org:simgrid/simgrid
X-Git-Tag: v3.35~77
X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/a0d47b6eaf07218a602ed6606da925fbd3629909

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

a0d47b6eaf07218a602ed6606da925fbd3629909
diff --cc src/mc/explo/odpor/ReversibleRaceCalculator.cpp
index d35c167d67,5fcb30e2c3..575d8e048e
--- a/src/mc/explo/odpor/ReversibleRaceCalculator.cpp
+++ b/src/mc/explo/odpor/ReversibleRaceCalculator.cpp
@@@ -153,16 -150,17 +153,17 @@@ bool ReversibleRaceCalculator::is_race_
    return true;
  }
  
 -bool ReversibleRaceCalculator::is_race_reversible_MutexWait(const Execution& E, Execution::EventHandle e1,
 -                                                            const Transition* /*e2*/)
 +bool ReversibleRaceCalculator::is_race_reversible_MutexWait(const Execution& E, const Transition* /*other_transition*/,
 +                                                            const Transition* /*t2*/)
  {
-   // TODO: for now we over approximate the reversibility
- 
-   return true;
+   // Only an Unlock can be dependent with a Wait
+   // and in this case, the Unlock enbaled the wait
+   // Not reversibled
+   return false;
  }
  
 -bool ReversibleRaceCalculator::is_race_reversible_SemAsyncLock(const Execution&, Execution::EventHandle /*e1*/,
 -                                                               const Transition* /*e2*/)
 +bool ReversibleRaceCalculator::is_race_reversible_SemAsyncLock(const Execution&, const Transition* /*other_transition*/,
 +                                                               const Transition* /*t2*/)
  {
    // SemAsyncLock is always enabled
    return true;