Use atomics to access objects' user_data.

argonne-lcf · Aug 24, 2023 · c12cc2a · c12cc2a
1 parent 03d5a7a
commit c12cc2a
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/cconfigspace.c b/src/cconfigspace.c
@@ -155,9 +155,7 @@ ccs_object_set_user_data(ccs_object_t object, void *user_data)
 {
 	CCS_CHECK_BASE_OBJ(object);
 	_ccs_object_internal_t *obj = (_ccs_object_internal_t *)object;
-	CCS_RWLOCK_WRLOCK(obj->lock);
-	obj->user_data = user_data;
-	CCS_RWLOCK_UNLOCK(obj->lock);
+	CCS_ATOMIC_STORE(obj->user_data, user_data);
 	return CCS_RESULT_SUCCESS;
 }
 
@@ -167,9 +165,7 @@ ccs_object_get_user_data(ccs_object_t object, void **user_data_ret)
 	CCS_CHECK_BASE_OBJ(object);
 	CCS_CHECK_PTR(user_data_ret);
 	_ccs_object_internal_t *obj = (_ccs_object_internal_t *)object;
-	CCS_RWLOCK_RDLOCK(obj->lock);
-	*user_data_ret = obj->user_data;
-	CCS_RWLOCK_UNLOCK(obj->lock);
+	*user_data_ret              = CCS_ATOMIC_LOAD(obj->user_data);
 	return CCS_RESULT_SUCCESS;
 }