Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
model-checker : new comparison for reached pairs (automaton state + values of proposi...
[simgrid.git] / src / mc / mc_liveness.c
index 874e7510fad0bd5414724877ae9a0feae27a29e4..32e8b3b3081901d653881a1ae1db88e57241b4a5 100644 (file)
@@ -19,36 +19,106 @@ mc_pair_t new_pair(mc_snapshot_t sn, mc_state_t sg, xbt_state_t st){
     
 }
 
-int reached(mc_pair_t pair){
+int snapshot_compare(mc_snapshot_t s1, mc_snapshot_t s2){
+  
+  if(s1->num_reg != s2->num_reg)
+    return 1;
+
+  int i;
+  for(i=0 ; i< s1->num_reg ; i++){
+
+    if(s1->regions[i]->size != s2->regions[i]->size)
+      return 1;
+
+    if(s1->regions[i]->start_addr != s2->regions[i]->start_addr)
+      return 1;
+
+    if(memcmp(s1->regions[i]->data, s2->regions[i]->data, s1->regions[i]->size) != 0)
+      return 1;
+    
+  }
+
+  return 0;
+
+}
+
+int reached(xbt_automaton_t a){
 
   if(xbt_dynar_is_empty(reached_pairs)){
-     return 0;
+    return 0;
   }else{
     MC_SET_RAW_MEM;
-    char hash[41] = "";
-    xbt_sha((const char *)&pair, hash);
-    if(xbt_dynar_member(reached_pairs, hash)){
-      MC_UNSET_RAW_MEM;
-      return 1;
-    }else{
-      MC_UNSET_RAW_MEM;
-      return 0;
+  
+    mc_pair_reached_t pair = NULL;
+    pair = xbt_new0(s_mc_pair_reached_t, 1);
+    pair->automaton_state = a->current_state;
+    pair->prop_ato = xbt_dynar_new(sizeof(int), NULL);
+    pair->system_state = xbt_new0(s_mc_snapshot_t, 1);
+    MC_take_snapshot(pair->system_state);
+    
+    /* Get values of propositional symbols */
+    unsigned int cursor = 0;
+    xbt_propositional_symbol_t ps = NULL;
+    xbt_dynar_foreach(a->propositional_symbols, cursor, ps){
+      int (*f)() = ps->function;
+      int res = (*f)();
+      xbt_dynar_push(pair->prop_ato, &res);
+    }
+    
+    cursor = 0;
+    mc_pair_reached_t pair_test;
+    
+    xbt_dynar_foreach(reached_pairs, cursor, pair_test){
+      if(automaton_state_compare(pair_test->automaton_state, pair->automaton_state) == 0){
+       //XBT_DEBUG("Same automaton state");
+       if(xbt_dynar_compare(pair_test->prop_ato, pair->prop_ato, propositional_symbols_compare_value) == 0){
+         //XBT_DEBUG("Same values of propositional symbols");
+         if(snapshot_compare(pair_test->system_state, pair->system_state) == 0){
+           //XBT_DEBUG("Same system state");
+           MC_UNSET_RAW_MEM;
+           return 1;
+         }
+       }
+      }
     }
+
+    MC_UNSET_RAW_MEM;
+    return 0;
+    
   }
 }
 
-void set_pair_reached(mc_pair_t pair){
+int set_pair_reached(xbt_automaton_t a){
 
-  char hash[41] = "";
-  xbt_sha((const char *)&pair, hash);  
-  XBT_DEBUG("Hash to pushed %s", hash);
-  if(strcmp(hash, "da39a3ee5e6b4b0d3255bfef95601890afd80709") == 0){
-    XBT_DEBUG("Error in hash, pair empty !");
-    sleep(4);
-  }
-  xbt_dynar_push(reached_pairs, &hash);
+  if(reached(a) == 0){
+
+    MC_SET_RAW_MEM;
+
+    mc_pair_reached_t pair = NULL;
+    pair = xbt_new0(s_mc_pair_reached_t, 1);
+    pair->automaton_state = a->current_state;
+    pair->prop_ato = xbt_dynar_new(sizeof(int), NULL);
+    pair->system_state = xbt_new0(s_mc_snapshot_t, 1);
+    MC_take_snapshot(pair->system_state);
     
+    /* Get values of propositional symbols */
+    unsigned int cursor = 0;
+    xbt_propositional_symbol_t ps = NULL;
+    xbt_dynar_foreach(a->propositional_symbols, cursor, ps){
+      int (*f)() = ps->function;
+      int res = (*f)();
+      xbt_dynar_push(pair->prop_ato, &res);
+    }
+     
+    xbt_dynar_push(reached_pairs, &pair); 
+   
+    MC_UNSET_RAW_MEM;
+
+    return 1;
+
+  }
+
+  return 0;
 }
 
 void MC_pair_delete(mc_pair_t pair){
@@ -165,7 +235,7 @@ void MC_vddfs_stateful_init(xbt_automaton_t a){
   }
 
   visited_pairs = xbt_dynar_new(sizeof(char *), NULL); 
-  reached_pairs = xbt_dynar_new(sizeof(char *), NULL); 
+  reached_pairs = xbt_dynar_new(sizeof(mc_pair_reached_t), NULL); 
 
   MC_take_snapshot(init_snapshot);
 
@@ -238,6 +308,7 @@ void MC_vddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
   XBT_DEBUG("********************* ( Depth = %d, search_cycle = %d )", xbt_fifo_size(mc_stack_liveness_stateful), search_cycle);
   XBT_DEBUG("Pair : graph=%p, automaton=%p(%s), %u interleave", current_pair->graph_state, current_pair->automaton_state, current_pair->automaton_state->id,MC_state_interleave_size(current_pair->graph_state));
   
+  a->current_state = current_pair->automaton_state;
 
   mc_stats_pair->visited_pairs++;
 
@@ -350,13 +421,13 @@ void MC_vddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
   
     xbt_dynar_foreach(successors, cursor, pair_succ){
 
-      XBT_DEBUG("Search reached pair %p : graph=%p, automaton=%p", pair_succ, pair_succ->graph_state, pair_succ->automaton_state);
+      /*XBT_DEBUG("Search reached pair %p : graph=%p, automaton=%p", pair_succ, pair_succ->graph_state, pair_succ->automaton_state);
       char hash[41];
       XBT_DEBUG("Const char pair %s", (const char *)&pair_succ);
       xbt_sha((const char *)&pair_succ, hash);
-      XBT_DEBUG("Hash pair_succ %s", hash);
+      XBT_DEBUG("Hash pair_succ %s", hash);*/
 
-      if((search_cycle == 1) && (reached(pair_succ) == 1)){
+      if((search_cycle == 1) && (reached(a) == 1)){
        XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*");
        XBT_INFO("|             ACCEPTANCE CYCLE            |");
        XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*");
@@ -372,10 +443,10 @@ void MC_vddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
       if(visited(pair_succ, search_cycle) == 0){
 
        //mc_stats_pair->executed_transitions++;
+       set_pair_visited(pair_succ, search_cycle);
+
        MC_SET_RAW_MEM;
        xbt_fifo_unshift(mc_stack_liveness_stateful, pair_succ);
-       set_pair_visited(pair_succ, search_cycle);
        MC_UNSET_RAW_MEM;
 
        MC_vddfs_stateful(a, search_cycle, 0);
@@ -384,8 +455,8 @@ void MC_vddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
 
        if((search_cycle == 0) && ((pair_succ->automaton_state->type == 1) || (pair_succ->automaton_state->type == 2))){
 
-         set_pair_reached(pair_succ);
          XBT_DEBUG("Acceptance pair : graph=%p, automaton=%p(%s)", pair_succ->graph_state, pair_succ->automaton_state, pair_succ->automaton_state->id);
+         int res = set_pair_reached(a);
          
          MC_SET_RAW_MEM;
          xbt_fifo_unshift(mc_stack_liveness_stateful, pair_succ);
@@ -393,9 +464,11 @@ void MC_vddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
          
          MC_vddfs_stateful(a, 1, 1);
 
-         //MC_SET_RAW_MEM;
-         xbt_dynar_pop(reached_pairs, NULL);
-         //MC_UNSET_RAW_MEM;
+         if(res){
+           MC_SET_RAW_MEM;
+           xbt_dynar_pop(reached_pairs, NULL);
+           MC_UNSET_RAW_MEM;
+         }
 
        }
        
@@ -453,7 +526,7 @@ void MC_ddfs_stateful_init(xbt_automaton_t a){
     }
   }
 
-  reached_pairs = xbt_dynar_new(sizeof(char *), NULL); 
+  reached_pairs = xbt_dynar_new(sizeof(mc_pair_reached_t), NULL); 
 
   MC_take_snapshot(init_snapshot);
 
@@ -507,24 +580,25 @@ void MC_ddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
     return;
 
   if(restore == 1){
-    MC_restore_snapshot(((mc_pair_t)xbt_fifo_get_item_content(xbt_fifo_get_first_item(mc_stack_liveness_stateful)))->system_state);
-    MC_UNSET_RAW_MEM;
-  }
-
-  /* Get current state */
-  current_pair = (mc_pair_t)xbt_fifo_get_item_content(xbt_fifo_get_first_item(mc_stack_liveness_stateful));
-
-  if(restore==1){
+    current_pair = (mc_pair_t)xbt_fifo_get_item_content(xbt_fifo_get_first_item(mc_stack_liveness_stateful));
+    MC_restore_snapshot(current_pair->system_state);
     xbt_swag_foreach(process, simix_global->process_list){
       if(MC_process_is_enabled(process)){
        MC_state_interleave_process(current_pair->graph_state, process);
       }
     }
+    MC_UNSET_RAW_MEM;
   }
 
+  /* Get current state */
+  current_pair = (mc_pair_t)xbt_fifo_get_item_content(xbt_fifo_get_first_item(mc_stack_liveness_stateful));
+
+
   XBT_DEBUG("********************* ( Depth = %d, search_cycle = %d )", xbt_fifo_size(mc_stack_liveness_stateful), search_cycle);
   XBT_DEBUG("Pair : graph=%p, automaton=%p(%s), %u interleave", current_pair->graph_state, current_pair->automaton_state, current_pair->automaton_state->id,MC_state_interleave_size(current_pair->graph_state));
 
+  a->current_state = current_pair->automaton_state;
+
   mc_stats_pair->visited_pairs++;
 
   int value;
@@ -559,7 +633,7 @@ void MC_ddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
     /* Debug information */
     if(XBT_LOG_ISENABLED(mc_liveness, xbt_log_priority_debug)){
       req_str = MC_request_to_string(req, value);
-      XBT_DEBUG("%u Execute: %s", search_cycle, req_str);
+      XBT_DEBUG("Execute: %s", req_str);
       xbt_free(req_str);
     }
 
@@ -638,7 +712,7 @@ void MC_ddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
 
       //XBT_DEBUG("Search visited pair : graph=%p, automaton=%p", pair_succ->graph_state, pair_succ->automaton_state);
 
-      if((search_cycle == 1) && (reached(pair_succ) == 1)){
+      if((search_cycle == 1) && (reached(a) == 1)){
        XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*");
        XBT_INFO("|             ACCEPTANCE CYCLE            |");
        XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*");
@@ -660,7 +734,7 @@ void MC_ddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
     
       if((search_cycle == 0) && ((pair_succ->automaton_state->type == 1) || (pair_succ->automaton_state->type == 2))){
 
-       set_pair_reached(pair_succ);
+       int res = set_pair_reached(a);
        XBT_DEBUG("Acceptance pair : graph=%p, automaton=%p(%s)", pair_succ->graph_state, pair_succ->automaton_state, pair_succ->automaton_state->id);
        
        MC_SET_RAW_MEM;
@@ -669,9 +743,11 @@ void MC_ddfs_stateful(xbt_automaton_t a, int search_cycle, int restore){
        
        MC_ddfs_stateful(a, 1, 1);
 
-       MC_SET_RAW_MEM;
-       xbt_dynar_pop(reached_pairs, NULL);
-       MC_UNSET_RAW_MEM;
+       if(res){
+         MC_SET_RAW_MEM;
+         xbt_dynar_pop(reached_pairs, NULL);
+         MC_UNSET_RAW_MEM;
+       }
       }
 
     }
@@ -713,43 +789,6 @@ mc_pair_stateless_t new_pair_stateless(mc_state_t sg, xbt_state_t st){
 }
 
 
-int reached_stateless(mc_pair_stateless_t pair){
-
-  if(xbt_dynar_is_empty(reached_pairs)){
-    return 0;
-  }else{
-    MC_SET_RAW_MEM;
-  
-    char hash[41] ="";
-    xbt_sha((const char*)&pair, hash);
-    XBT_DEBUG("Hash : %s", hash);
-    if(xbt_dynar_member(reached_pairs, hash)){
-      XBT_DEBUG("Pair already reached");
-      MC_UNSET_RAW_MEM;
-      return 1;
-    }else{
-      MC_UNSET_RAW_MEM;
-      return 0;
-    }
-  }
-}
-
-void set_pair_stateless_reached(mc_pair_stateless_t pair){
-
-    //MC_SET_RAW_MEM;
-    char hash[41] = "";
-    xbt_sha((const char*)&pair, hash);
-    XBT_DEBUG("Hash to pushed %s", hash);
-    if(strcmp(hash, "da39a3ee5e6b4b0d3255bfef95601890afd80709") == 0){
-      XBT_DEBUG("Error in hash, pair empty !");
-      sleep(4);
-    }
-      
-    xbt_dynar_push(reached_pairs, &hash); 
-    //MC_UNSET_RAW_MEM;
-
-}
-
 
 void MC_ddfs_stateless_init(xbt_automaton_t a){
 
@@ -771,7 +810,11 @@ void MC_ddfs_stateless_init(xbt_automaton_t a){
     }
   }
 
-  reached_pairs = xbt_dynar_new(sizeof(char *), NULL); 
+  reached_pairs = xbt_dynar_new(sizeof(mc_pair_reached_t), NULL); 
+
+  initial_snapshot = xbt_new0(s_mc_snapshot_t, 1);
+  MC_take_snapshot(initial_snapshot);
+
   MC_UNSET_RAW_MEM; 
 
   unsigned int cursor = 0;
@@ -783,10 +826,6 @@ void MC_ddfs_stateless_init(xbt_automaton_t a){
       MC_SET_RAW_MEM;
       mc_initial_pair = new_pair_stateless(initial_graph_state, state);
       xbt_fifo_unshift(mc_stack_liveness_stateless, mc_initial_pair);
-  
-      initial_snapshot = xbt_new0(s_mc_snapshot_t, 1);
-      MC_take_snapshot(initial_snapshot);
-      
       MC_UNSET_RAW_MEM;
       
       if(cursor == 0){
@@ -802,10 +841,6 @@ void MC_ddfs_stateless_init(xbt_automaton_t a){
        MC_SET_RAW_MEM;
        mc_initial_pair = new_pair_stateless(initial_graph_state, state);
        xbt_fifo_unshift(mc_stack_liveness_stateless, mc_initial_pair);
-       
-       initial_snapshot = xbt_new0(s_mc_snapshot_t, 1);
-       MC_take_snapshot(initial_snapshot);
-       
        MC_UNSET_RAW_MEM;
        
        if(cursor == 0){
@@ -842,6 +877,9 @@ void MC_ddfs_stateless(xbt_automaton_t a, int search_cycle, int replay){
 
   /* Get current state */
   current_pair = (mc_pair_stateless_t)xbt_fifo_get_item_content(xbt_fifo_get_first_item(mc_stack_liveness_stateless));
+
+  /* Update current state in automaton */
+  a->current_state = current_pair->automaton_state;
  
   XBT_DEBUG("********************* ( Depth = %d, search_cycle = %d )", xbt_fifo_size(mc_stack_liveness_stateless), search_cycle);
   XBT_DEBUG("Pair : graph=%p, automaton=%p(%s), %u interleave", current_pair->graph_state, current_pair->automaton_state, current_pair->automaton_state->id,MC_state_interleave_size(current_pair->graph_state));
@@ -889,8 +927,8 @@ void MC_ddfs_stateless(xbt_automaton_t a, int search_cycle, int replay){
 
 
     MC_SET_RAW_MEM;
-    /* Create the new expanded graph_state */
 
+    /* Create the new expanded graph_state */
     next_graph_state = MC_state_pair_new();
 
     /* Get enabled process and insert it in the interleave set of the next graph_state */
@@ -904,8 +942,7 @@ void MC_ddfs_stateless(xbt_automaton_t a, int search_cycle, int replay){
 
     MC_UNSET_RAW_MEM;
     
-    
-    
+
     cursor= 0;
     xbt_dynar_foreach(current_pair->automaton_state->out, cursor, transition_succ){
 
@@ -943,13 +980,11 @@ void MC_ddfs_stateless(xbt_automaton_t a, int search_cycle, int replay){
       MC_UNSET_RAW_MEM;
     }
 
-    cursor = 0;
-
-   
+    cursor = 0; 
 
     xbt_dynar_foreach(successors, cursor, pair_succ){
      
-      if((search_cycle == 1) && (reached_stateless(pair_succ) == 1)){
+      if((search_cycle == 1) && (reached(a) == 1)){
        XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*");
        XBT_INFO("|             ACCEPTANCE CYCLE            |");
        XBT_INFO("*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*-*");
@@ -969,8 +1004,8 @@ void MC_ddfs_stateless(xbt_automaton_t a, int search_cycle, int replay){
      
       if((search_cycle == 0) && ((pair_succ->automaton_state->type == 1) || (pair_succ->automaton_state->type == 2))){
 
-       XBT_DEBUG("Acceptance pair : graph=%p, automaton=%p(%s)", pair_succ->graph_state, pair_succ->automaton_state, pair_succ->automaton_state->id);
-       set_pair_stateless_reached(pair_succ);
+       XBT_DEBUG("Acceptance pair %p : graph=%p, automaton=%p(%s)", pair_succ, pair_succ->graph_state, pair_succ->automaton_state, pair_succ->automaton_state->id);
+       int res = set_pair_reached(a);
 
        MC_SET_RAW_MEM;
        xbt_fifo_unshift(mc_stack_liveness_stateless, pair_succ);
@@ -978,9 +1013,11 @@ void MC_ddfs_stateless(xbt_automaton_t a, int search_cycle, int replay){
       
        MC_ddfs_stateless(a, 1, 1);
        
-       MC_SET_RAW_MEM;
-       xbt_dynar_pop(reached_pairs, NULL);
-       MC_UNSET_RAW_MEM;
+       if(res){
+         MC_SET_RAW_MEM;
+         xbt_dynar_pop(reached_pairs, NULL);
+         MC_UNSET_RAW_MEM;
+       }
       }
     }