From b543ae3a4ccc2387d4a33443731b5baf9d1dda8d Mon Sep 17 00:00:00 2001
From: Utz-Uwe Haus <uhaus@hpe.com>
Date: Tue, 7 Sep 2021 06:39:54 +0000
Subject: [PATCH] Fix (cosmetic) data race printing CDOID in REQUIRE
---
maestro/cdo.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/maestro/cdo.c b/maestro/cdo.c
index 105ec021..6416536a 100644
--- a/maestro/cdo.c
+++ b/maestro/cdo.c
@@ -1606,11 +1606,6 @@ mstro_cdo_require(mstro_cdo cdo)
return MSTRO_INVARG;
}
- WITH_CDO_ID_STR(lidstr, &cdo->id, {
- WITH_CDO_ID_STR(gidstr, &cdo->gid, {
- INFO("Requiring CDO `%s' (lid %s, gid %s) from pool\n",
- cdo->name, lidstr, gidstr);});});
-
/* some code duplication (structurally) with OFFER, but we keep it that way for now */
if(!mstro_cdo_state_check(cdo, MSTRO_CDO_STATE_SEALED)) {
if(mstro_cdo_state_check(cdo,
@@ -1632,6 +1627,7 @@ mstro_cdo_require(mstro_cdo cdo)
}
/* We have a sealed CDO. Good. */
+
WITH_CDO_ID_STR(id, &cdo->gid, {
status = mstro_pool__add(cdo, MSTRO_CDO_STATE_REQUIRED);
if(status!=MSTRO_OK) {
--
GitLab