Refactor MPIPtr cconvert to support Julia 1.14

[nhz2]

Alternative to #935

CC @eschnett @giordano @vchuravy

[eschnett]

One of the main features of ccall and its automatic conversion to pointers is that it's GC safe, i.e. that it holds on to the respective Julia objects while a pointer to the object is in use. I didn't follow the logic that you're introducing, but since this kind of error (losing a GC root) is very difficult to diagnose, I want to recommend to add respective comments to the code, stating which part of the machinery is holding on to objects, which functions are called before objects are converted to pointers, and which are called on pointers.

[nhz2]

I've added some comments using Claude Code

[eschnett]

Thanks! The description sounds correct, and I like the ASCII art.

[nhz2]

Bump

[KristofferC]

Bump, would be nice to have this fixed (+ new release if possible).

[giordano]

I believe we'd need some help with the review, none of us is particularly familiar with what's the intended way to go about it. I definitely prefer this kind of approach over #935 because that felt too ad hoc, but the need for CConvWrapper here is also odd.

[lcw]

We need to support OpenAPI as well.

[nhz2]

We need to support OpenAPI as well.

What do you mean by this?

[lcw]

We need to support OpenAPI as well.

What do you mean by this?

I think we might need to change these lines as well

MPI.jl/ext/OneAPIExt.jl

Lines 7 to 21 in 61cc82e

	function Base.cconvert(::Type{MPIPtr}, A::oneAPI.oneArray{T}) where T
	A
	end
	function Base.unsafe_convert(::Type{MPIPtr}, X::oneAPI.oneArray{T}) where T
	reinterpret(MPIPtr, Base.unsafe_convert(oneAPI.ZePtr{T}, X))
	end
	# only need to define this for strided arrays: all others can be handled by generic machinery
	function Base.unsafe_convert(::Type{MPIPtr}, V::SubArray{T,N,P,I,true}) where {T,N,P<:oneAPI.oneArray,I}
	X = parent(V)
	pX = Base.unsafe_convert(oneAPI.ZePtr{T}, X)
	pV = pX + ((V.offset1 + V.stride1) - first(LinearIndices(X)))*sizeof(T)
	return reinterpret(MPIPtr, pV)
	end

[nhz2]

I've updated OneAPIExt.jl and added an intermediate pointer type trait function mpi_ptr_type.