Skip to content

Commit 3e98876

Browse files
authored
Merge pull request #5379 from hoopoepg/topic/osc-ucx-suppress-compilation-warnings
OSC/UCX: suppressed compilation warnings
2 parents 5793f1b + eb70109 commit 3e98876

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

ompi/mca/osc/ucx/osc_ucx_comm.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -356,7 +356,7 @@ static inline int get_dynamic_win_info(uint64_t remote_addr, ompi_osc_ucx_module
356356
temp_dynamic_wins = (ompi_osc_dynamic_win_info_t *)(temp_buf + sizeof(uint64_t));
357357
contain = ompi_osc_find_attached_region_position(temp_dynamic_wins, 0, win_count,
358358
remote_addr, 1, &insert);
359-
assert(contain >= 0 && contain < win_count);
359+
assert(contain >= 0 && (uint64_t)contain < win_count);
360360

361361
status = ucp_ep_rkey_unpack(ep, temp_dynamic_wins[contain].rkey_buffer,
362362
&((module->win_info_array[target]).rkey));

ompi/mca/osc/ucx/osc_ucx_component.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -725,7 +725,7 @@ int ompi_osc_ucx_win_attach(struct ompi_win_t *win, void *base, size_t len) {
725725
return ret;
726726
}
727727

728-
assert(insert_index >= 0 && insert_index < module->state.dynamic_win_count);
728+
assert(insert_index >= 0 && (uint64_t)insert_index < module->state.dynamic_win_count);
729729

730730
memmove((void *)&module->local_dynamic_win_info[insert_index+1],
731731
(void *)&module->local_dynamic_win_info[insert_index],
@@ -777,7 +777,7 @@ int ompi_osc_ucx_win_detach(struct ompi_win_t *win, const void *base) {
777777
contain = ompi_osc_find_attached_region_position((ompi_osc_dynamic_win_info_t *)module->state.dynamic_wins,
778778
0, (int)module->state.dynamic_win_count,
779779
(uint64_t)base, 1, &insert);
780-
assert(contain >= 0 && contain < module->state.dynamic_win_count);
780+
assert(contain >= 0 && (uint64_t)contain < module->state.dynamic_win_count);
781781

782782
/* if we can't find region - just exit */
783783
if (contain < 0) {

0 commit comments

Comments
 (0)