coll/han: ptrdiff_t max size bugfix

wenduwan · wenduwan · commit 80884d46eb9a · 2024-08-19T16:33:58.000-07:00
Fix buggy integer comparison of (ptrdiff_t) displ &lt; SIZE_MAX

Signed-off-by: Wenduo Wang &lt;wenduwan@amazon.com&gt;
diff --git a/ompi/mca/coll/han/coll_han_gatherv.c b/ompi/mca/coll/han/coll_han_gatherv.c
@@ -170,7 +170,7 @@ int mca_coll_han_gatherv_intra(const void *sbuf, size_t scount, struct ompi_data
         }
 
         for (up_peer = 0; up_peer < up_size; ++up_peer) {
-            up_displs[up_peer] = INT_MAX;
+            up_displs[up_peer] = PTRDIFF_MAX;
         }
 
         /* Calculate recv counts for the inter-node gatherv - no need to gather
diff --git a/ompi/mca/coll/han/coll_han_scatterv.c b/ompi/mca/coll/han/coll_han_scatterv.c
@@ -174,7 +174,7 @@ int mca_coll_han_scatterv_intra(const void *sbuf, ompi_count_array_t scounts, om
         }
 
         for (up_peer = 0; up_peer < up_size; ++up_peer) {
-            up_displs[up_peer] = SIZE_MAX;
+            up_displs[up_peer] = PTRDIFF_MAX;
         }
 
         /* Calculate send counts for the inter-node scatterv */

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ int mca_coll_han_gatherv_intra(const void *sbuf, size_t scount, struct ompi_data`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`for (up_peer = 0; up_peer < up_size; ++up_peer) {`
`173`		`- up_displs[up_peer] = INT_MAX;`
	`173`	`+ up_displs[up_peer] = PTRDIFF_MAX;`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`/* Calculate recv counts for the inter-node gatherv - no need to gather`
Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ int mca_coll_han_scatterv_intra(const void *sbuf, ompi_count_array_t scounts, om`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`for (up_peer = 0; up_peer < up_size; ++up_peer) {`
`177`		`- up_displs[up_peer] = SIZE_MAX;`
	`177`	`+ up_displs[up_peer] = PTRDIFF_MAX;`
`178`	`178`	`}`
`179`	`179`
`180`	`180`	`/* Calculate send counts for the inter-node scatterv */`