#define __STDC_FORMAT_MACROS
#include <cinttypes>
-#include <boost/unordered_set.hpp>
+
+#include <utility>
+#include <unordered_set>
#include <xbt/sysdep.h>
-#include "internal_config.h"
+#include "src/internal_config.h"
#include "mc_object_info.h"
#include "mc_safety.h"
#include "mc_liveness.h"
#include "mc_smx.h"
#include "mc_dwarf.hpp"
-#include "mc/Frame.hpp"
-#include "mc/ObjectInformation.hpp"
-#include "mc/Variable.hpp"
+#include "src/mc/Frame.hpp"
+#include "src/mc/ObjectInformation.hpp"
+#include "src/mc/Variable.hpp"
#ifdef HAVE_SMPI
-#include "smpi/private.h"
+#include "src/smpi/private.h"
#endif
#include "xbt/mmalloc.h"
-#include "xbt/mmalloc/mmprivate.h"
+#include "src/xbt/mmalloc/mmprivate.h"
-#include <xbt/probes.h>
+#include "src/xbt/probes.h"
using simgrid::mc::remote;
-typedef struct s_pointers_pair {
- void *p1;
- void *p2;
- bool operator==(s_pointers_pair const& x) const {
- return this->p1 == x.p1 && this->p2 == x.p2;
- }
- bool operator<(s_pointers_pair const& x) const {
- return this->p1 < x.p1 || (this->p1 == x.p1 && this->p2 < x.p2);
- }
-} s_pointers_pair_t, *pointers_pair_t;
-
-namespace boost {
- template<>
- struct hash<s_pointers_pair> {
- typedef uintptr_t result_type;
- result_type operator()(s_pointers_pair const& x) const {
- return (result_type) x.p1 ^
- ((result_type) x.p2 << 8 | (result_type) x.p2 >> (8*sizeof(uintptr_t) - 8));
- }
- };
-}
-
-struct mc_compare_state {
- boost::unordered_set<s_pointers_pair> compared_pointers;
-};
-
extern "C" {
XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_compare, xbt,
"Logging specific to mc_compare in mc");
-/************************** Free functions ****************************/
-/********************************************************************/
+}
+
+namespace simgrid {
+namespace mc {
-static void stack_region_free(stack_region_t s)
-{
- if (s) {
- xbt_free(s);
+/** A hash which works with more stuff
+ *
+ * It can hash pairs: the standard hash currently doesn't include this.
+ */
+template<class X> struct hash : public std::hash<X> {};
+
+template<class X, class Y>
+struct hash<std::pair<X,Y>> {
+ std::size_t operator()(std::pair<X,Y>const& x) const
+ {
+ struct hash<X> h1;
+ struct hash<X> h2;
+ return h1(x.first) ^ h2(x.second);
}
-}
+};
-static void stack_region_free_voidp(void *s)
-{
- stack_region_free((stack_region_t) * (void **) s);
-}
+struct ComparisonState {
+ std::unordered_set<std::pair<void*, void*>, hash<std::pair<void*, void*>>> compared_pointers;
+};
-static void pointers_pair_free(pointers_pair_t p)
-{
- xbt_free(p);
}
-
-static void pointers_pair_free_voidp(void *p)
-{
- pointers_pair_free((pointers_pair_t) * (void **) p);
}
+using simgrid::mc::ComparisonState;
+
+extern "C" {
+
/************************** Snapshot comparison *******************************/
/******************************************************************************/
-/** \brief Try to add a pair a compared pointers to the set of compared pointers
- *
- * \result !=0 if the pointers were added (they were not in the set),
- * 0 otherwise (they were already in the set)
- */
-static int add_compared_pointers(mc_compare_state& state, void *p1, void *p2)
-{
- s_pointers_pair_t new_pair;
- new_pair.p1 = p1;
- new_pair.p2 = p2;
- return state.compared_pointers.insert(new_pair).second ? 1 : 0;
-}
-
-static int compare_areas_with_type(struct mc_compare_state& state,
+static int compare_areas_with_type(ComparisonState& state,
int process_index,
void* real_area1, mc_snapshot_t snapshot1, mc_mem_region_t region1,
void* real_area2, mc_snapshot_t snapshot2, mc_mem_region_t region2,
return 0;
if (addr_pointed1 == NULL || addr_pointed2 == NULL)
return 1;
- if (!add_compared_pointers(state, addr_pointed1, addr_pointed2))
+ if (!state.compared_pointers.insert(
+ std::make_pair(addr_pointed1, addr_pointed2)).second)
return 0;
pointer_level++;
case DW_TAG_structure_type:
case DW_TAG_class_type:
for(simgrid::mc::Member& member : type->members) {
- void *member1 =
- mc_member_resolve(real_area1, type, &member, snapshot1, process_index);
- void *member2 =
- mc_member_resolve(real_area2, type, &member, snapshot2, process_index);
+ void *member1 = simgrid::dwarf::resolve_member(
+ real_area1, type, &member, snapshot1, process_index);
+ void *member2 = simgrid::dwarf::resolve_member(
+ real_area2, type, &member, snapshot2, process_index);
mc_mem_region_t subregion1 = mc_get_region_hinted(member1, snapshot1, process_index, region1);
mc_mem_region_t subregion2 = mc_get_region_hinted(member2, snapshot2, process_index, region2);
res =
#endif
xbt_assert(r2->storage_type() != simgrid::mc::StorageType::Privatized);
- struct mc_compare_state state;
+ ComparisonState state;
std::vector<simgrid::mc::Variable>& variables = object_info->global_variables;
mc_snapshot_stack_t stack1,
mc_snapshot_stack_t stack2)
{
- struct mc_compare_state state;
+ ComparisonState state;
if (stack1->local_variables.size() != stack2->local_variables.size()) {
XBT_VERB("Different number of local variables");
int errors = 0;
int res_init;
- xbt_os_timer_t global_timer = xbt_os_timer_new();
- xbt_os_timer_t timer = xbt_os_timer_new();
-
- xbt_os_walltimer_start(global_timer);
-
-#ifdef MC_DEBUG
- xbt_os_walltimer_start(timer);
-#endif
-
int hash_result = 0;
if (_sg_mc_hash) {
hash_result = (s1->hash != s2->hash);
size_used2 = s2->stack_sizes[i];
if (size_used1 != size_used2) {
#ifdef MC_DEBUG
- if (is_diff == 0) {
- xbt_os_walltimer_stop(timer);
- mc_comp_times->stacks_sizes_comparison_time =
- xbt_os_timer_elapsed(timer);
- }
XBT_DEBUG("(%d - %d) Different size used in stacks : %zu - %zu", num1,
num2, size_used1, size_used2);
errors++;
num2, size_used1, size_used2);
#endif
XBT_TRACE3(mc, state_diff, num1, num2, "Different stack size");
-
- xbt_os_walltimer_stop(timer);
- xbt_os_timer_free(timer);
- xbt_os_walltimer_stop(global_timer);
- mc_snapshot_comparison_time = xbt_os_timer_elapsed(global_timer);
- xbt_os_timer_free(global_timer);
-
return 1;
#endif
}
i++;
}
-#ifdef MC_DEBUG
- if (is_diff == 0)
- xbt_os_walltimer_stop(timer);
- xbt_os_walltimer_start(timer);
-#endif
-
/* Init heap information used in heap comparison algorithm */
xbt_mheap_t heap1 = (xbt_mheap_t)s1->read_bytes(
alloca(sizeof(struct mdesc)), sizeof(struct mdesc),
XBT_VERB("(%d - %d) Different heap information", num1, num2);
#endif
- xbt_os_walltimer_stop(global_timer);
- mc_snapshot_comparison_time = xbt_os_timer_elapsed(global_timer);
- xbt_os_timer_free(global_timer);
-
return 1;
#endif
}
-#ifdef MC_DEBUG
- xbt_os_walltimer_start(timer);
-#endif
/* Stacks comparison */
unsigned cursor = 0;
if (diff_local > 0) {
XBT_TRACE3(mc, state_diff, num1, num2, "Different local variables");
#ifdef MC_DEBUG
- if (is_diff == 0) {
- xbt_os_walltimer_stop(timer);
- mc_comp_times->stacks_comparison_time = xbt_os_timer_elapsed(timer);
- }
XBT_DEBUG("(%d - %d) Different local variables between stacks %d", num1,
num2, cursor + 1);
errors++;
#endif
reset_heap_information();
- xbt_os_walltimer_stop(timer);
- xbt_os_timer_free(timer);
- xbt_os_walltimer_stop(global_timer);
- mc_snapshot_comparison_time = xbt_os_timer_elapsed(global_timer);
- xbt_os_timer_free(global_timer);
return 1;
#endif
// TODO, raise a difference instead?
xbt_assert(regions_count == s2->snapshot_regions.size());
- mc_comp_times->global_variables_comparison_time = 0;
-
for (size_t k = 0; k != regions_count; ++k) {
mc_mem_region_t region1 = s1->snapshot_regions[k].get();
mc_mem_region_t region2 = s2->snapshot_regions[k].get();
std::string const& name = region1->object_info()->file_name;
-#ifdef MC_DEBUG
- if (is_diff == 0)
- xbt_os_walltimer_stop(timer);
- xbt_os_walltimer_start(timer);
-#endif
-
/* Compare global variables */
is_diff =
compare_global_variables(region1->object_info( ), simgrid::mc::AddressSpace::Normal,
if (is_diff != 0) {
XBT_TRACE3(mc, state_diff, num1, num2, "Different global variables");
#ifdef MC_DEBUG
- xbt_os_walltimer_stop(timer);
- mc_comp_times->global_variables_comparison_time
- += xbt_os_timer_elapsed(timer);
XBT_DEBUG("(%d - %d) Different global variables in %s",
num1, num2, name.c_str());
errors++;
num1, num2, name.c_str());
#endif
- reset_heap_information();
- xbt_os_walltimer_stop(timer);
- xbt_os_timer_free(timer);
- xbt_os_walltimer_stop(global_timer);
- mc_snapshot_comparison_time = xbt_os_timer_elapsed(global_timer);
- xbt_os_timer_free(global_timer);
-
return 1;
#endif
}
}
-#ifdef MC_DEBUG
- xbt_os_walltimer_start(timer);
-#endif
-
/* Compare heap */
if (mmalloc_compare_heap(s1, s2) > 0) {
XBT_TRACE3(mc, state_diff, num1, num2, "Different heap");
#ifdef MC_DEBUG
- xbt_os_walltimer_stop(timer);
- mc_comp_times->heap_comparison_time = xbt_os_timer_elapsed(timer);
XBT_DEBUG("(%d - %d) Different heap (mmalloc_compare)", num1, num2);
errors++;
#else
XBT_VERB("(%d - %d) Different heap (mmalloc_compare)", num1, num2);
#endif
- reset_heap_information();
- xbt_os_walltimer_stop(timer);
- xbt_os_timer_free(timer);
- xbt_os_walltimer_stop(global_timer);
- mc_snapshot_comparison_time = xbt_os_timer_elapsed(global_timer);
- xbt_os_timer_free(global_timer);
-
return 1;
-#endif
- } else {
-#ifdef MC_DEBUG
- xbt_os_walltimer_stop(timer);
#endif
}
reset_heap_information();
- xbt_os_walltimer_stop(timer);
- xbt_os_timer_free(timer);
-
-#ifdef MC_VERBOSE
- xbt_os_walltimer_stop(global_timer);
- mc_snapshot_comparison_time = xbt_os_timer_elapsed(global_timer);
-#endif
-
- xbt_os_timer_free(global_timer);
-
-#ifdef MC_DEBUG
- print_comparison_times();
-#endif
-
#ifdef MC_VERBOSE
if (errors || hash_result)
XBT_VERB("(%d - %d) Difference found", num1, num2);
}
-/***************************** Statistics *****************************/
-/*******************************************************************/
-
-void print_comparison_times()
-{
- XBT_DEBUG("*** Comparison times ***");
- XBT_DEBUG("- Nb processes : %f", mc_comp_times->nb_processes_comparison_time);
- XBT_DEBUG("- Nb bytes used : %f", mc_comp_times->bytes_used_comparison_time);
- XBT_DEBUG("- Stacks sizes : %f", mc_comp_times->stacks_sizes_comparison_time);
- XBT_DEBUG("- GLobal variables : %f", mc_comp_times->global_variables_comparison_time);
- XBT_DEBUG("- Heap : %f", mc_comp_times->heap_comparison_time);
- XBT_DEBUG("- Stacks : %f", mc_comp_times->stacks_comparison_time);
-}
-
}
