Comments on the assumption releasing locks after a crash

utsaslab · Apr 19, 2021 · fddae5d · fddae5d
1 parent 4c5a752
commit fddae5d
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/P-CLHT/include/clht_lb_res.h b/P-CLHT/include/clht_lb_res.h
@@ -159,6 +159,8 @@ typedef volatile uint8_t clht_lock_t;
 
 typedef struct ALIGNED(CACHE_LINE_SIZE) bucket_s
 {
+  // Although our current implementation does not provide post-crash mechanism,
+  // the locks should be released after a crash (Please refer to the function clht_lock_initialization())
   clht_lock_t lock;
   volatile uint32_t hops;
   clht_addr_t key[ENTRIES_PER_BUCKET];
@@ -181,6 +183,8 @@ typedef struct ALIGNED(CACHE_LINE_SIZE) clht
       struct clht_hashtable_s* ht_oldest;
       struct ht_ts* version_list;
       size_t version_min;
+      // Although our current implementation does not provide post-crash mechanism,
+      // the locks should be released after a crash (Please refer to the function clht_lock_initialization())
       volatile clht_lock_t resize_lock;
       volatile clht_lock_t gc_lock;
       volatile clht_lock_t status_lock;