fix SentinelPtr conversion error (#509)

Author: simonbyrne

src/MPI.jl

@@ -67,13 +67,13 @@ function __init__()
     end
 
     __init__deps()
-    
+
     # disable UCX memory cache, since it doesn't work correctly
     # https://github.com/openucx/ucx/issues/5061
     if !haskey(ENV, "UCX_MEMTYPE_CACHE")
         ENV["UCX_MEMTYPE_CACHE"] = "no"
     end
-    
+
     # Julia multithreading uses SIGSEGV to sync thread
     # https://docs.julialang.org/en/v1/devdocs/debuggingtips/#Dealing-with-signals-1
     # By default, UCX will error if this occurs (issue #337)

src/nonblocking.jl

@@ -59,6 +59,8 @@ if !@isdefined(Status)
     @assert sizeof(Status) == MPI_Status_size
 end
 
+Base.cconvert(::Type{Ptr{Status}}, x::SentinelPtr) = x
+Base.unsafe_convert(::Type{Ptr{Status}}, x::SentinelPtr) = reinterpret(Ptr{Status}, x)
 
 """
     MPI.Status

test/test_sendrecv.jl

@@ -67,26 +67,25 @@ rreq = MPI.Irecv!(recv_mesg, src,  src+32, comm)
 sreq = MPI.Isend(send_mesg, dst, rank+32, comm)
 
 req_arr = [sreq,rreq]
-(inds, stats) = MPI.Waitsome!(req_arr)
+inds = MPI.Waitsome(req_arr)
 for i in inds
-    (done, status) = MPI.Test!( req_arr[i] )
-    @test done
+    @test MPI.Test(req_arr[i])
 end
 
 rreq = MPI.Irecv!(recv_mesg, src,  src+32, comm)
 MPI.Cancel!(rreq)
-MPI.Wait!(rreq)
+MPI.Wait(rreq)
 @test rreq.buffer == nothing
 
 GC.gc()
 
 # ---------------------
 # MPI_Sendrecv function
 # ---------------------
-# 
+#
 # send datatype
 # ---------------------
-# We test this function by executing a left shift of the leftmost element in a 1D 
+# We test this function by executing a left shift of the leftmost element in a 1D
 # cartesian topology with periodic boundary conditions.
 #
 # Assume we have two processors, the data will look like this