Skip to content

Commit 7984d7d

Browse files
committed
opal/common/ucx: Remove unused debugging macro
Will be reintroduced later if needed and after adaptation to the OMPI infrastructure. Signed-off-by: Artem Polyakov <artpol84@gmail.com>
1 parent 43f16d8 commit 7984d7d

File tree

4 files changed

+1
-231
lines changed

4 files changed

+1
-231
lines changed

ompi/mca/osc/ucx/osc_ucx_component.c

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -552,33 +552,19 @@ int ompi_osc_ucx_free(struct ompi_win_t *win) {
552552
ompi_osc_ucx_module_t *module = (ompi_osc_ucx_module_t*) win->w_osc_module;
553553
int ret;
554554

555-
WPOOL_DBG_OUT(dbg_level, "start, mem = %p lock flag = %d\n",
556-
(void *)module->mem, (int)module->state.lock);
557-
558555
assert(module->lock_count == 0);
559556
assert(opal_list_is_empty(&module->pending_posts) == true);
560557
OBJ_DESTRUCT(&module->outstanding_locks);
561558
OBJ_DESTRUCT(&module->pending_posts);
562559

563560
opal_common_ucx_wpmem_flush(module->mem, OPAL_COMMON_UCX_SCOPE_WORKER, 0);
564561

565-
WPOOL_DBG_OUT(dbg_level, "after mem_flush, mem = %p lock flag = %d\n",
566-
(void *)module->mem, (int)module->state.lock);
567-
568-
/*
569-
while (module->state.lock != TARGET_LOCK_UNLOCKED) {
570-
ucp_worker_progress(mca_osc_ucx_component.wpool->recv_worker);
571-
}
572-
*/
573-
574562
ret = module->comm->c_coll->coll_barrier(module->comm,
575563
module->comm->c_coll->coll_barrier_module);
576564
if (ret != OMPI_SUCCESS) {
577565
return ret;
578566
}
579567

580-
WPOOL_DBG_OUT(dbg_level, "after barrier, mem = %p\n", (void *)module->mem);
581-
582568
free(module->addrs);
583569
free(module->state_addrs);
584570

ompi/mca/osc/ucx/osc_ucx_passive_target.c

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,6 @@ static inline int start_shared(ompi_osc_ucx_module_t *module, int target) {
3333
return ret;
3434
}
3535

36-
WPOOL_DBG_OUT(dbg_level, "start_shared: after fadd, result_value = %d",
37-
(int)result_value);
38-
3936
assert((int64_t)result_value >= 0);
4037
if (result_value >= TARGET_LOCK_EXCLUSIVE) {
4138
ret = opal_common_ucx_wpmem_post(module->state_mem,
@@ -222,8 +219,6 @@ int ompi_osc_ucx_unlock_all(struct ompi_win_t *win) {
222219
int comm_size = ompi_comm_size(module->comm);
223220
int ret = OMPI_SUCCESS;
224221

225-
WPOOL_DBG_OUT(dbg_level, "start, mem = %p\n", (void *)module->mem);
226-
227222
if (module->epoch_type.access != PASSIVE_ALL_EPOCH) {
228223
return OMPI_ERR_RMA_SYNC;
229224
}
@@ -235,9 +230,6 @@ int ompi_osc_ucx_unlock_all(struct ompi_win_t *win) {
235230
return ret;
236231
}
237232

238-
WPOOL_DBG_OUT(dbg_level, "done flushing: mem = %p\n",
239-
(void *)module->mem);
240-
241233
if (!module->lock_all_is_nocheck) {
242234
int i;
243235
for (i = 0; i < comm_size; i++) {
@@ -247,9 +239,6 @@ int ompi_osc_ucx_unlock_all(struct ompi_win_t *win) {
247239

248240
module->epoch_type.access = NONE_EPOCH;
249241

250-
WPOOL_DBG_OUT(dbg_level, "fini: mem = %p\n",
251-
(void *)module->mem);
252-
253242
return ret;
254243
}
255244

0 commit comments

Comments
 (0)