mpich icon indicating copy to clipboard operation
mpich copied to clipboard

gpu/hip: Deprecated enum hipMemoryType memoryType

Open gigaio-staylor opened this issue 1 year ago • 2 comments

Looks like MPICH code needs to change from using memoryType to type.

Per ROCm 6.0 Changelog: Deprecated enum hipMemoryType memoryType; is removed from HIP struct hipPointerAttribute_t union.

Affected files appear to be: ./modules/yaksa/src/backend/hip/pup/yaksuri_hipi_get_ptr_attr.c ./src/pmi/mpl/src/gpu/mpl_gpu_hip.c ./src/mpi/romio/mpl/src/gpu/mpl_gpu_hip.c ./src/mpl/src/gpu/mpl_gpu_hip.c ./src/pm/hydra/modules/pmi/mpl/src/gpu/mpl_gpu_hip.c ./src/pm/hydra/modules/mpl/src/gpu/mpl_gpu_hip.c

gigaio-staylor avatar Dec 30 '23 01:12 gigaio-staylor

I have found this issue when compiling mpich 4.2.1 with ROCm 6.1.0. The suggested change has solved the compilation error.

arturo-ges avatar May 25 '24 18:05 arturo-ges

On further testing, it looks like the enum hipMemoryType values, and errors returned by hipPointerGetAttributes changed. hipPointerGetAttributes return success when an 'unregistered' buffer is passed in, and there's a new hipMemoryTypeUnregistered enum value.

ROCM-6 hipMemoryType: https://github.com/ROCm/HIP/blob/900a5e9828f55ae004de99314c5f992e6a0ebc9b/include/hip/hip_runtime_api.h#L245

Also, this breaks the logic of MPI_gpu_query_pointer_attr so it will need to be rewritten...

BKitor avatar Jun 25 '24 00:06 BKitor

Fixed in 8e58a6874438e86862350004117e3047b48962c5.

raffenet avatar Jul 17 '24 20:07 raffenet