mpich icon indicating copy to clipboard operation
mpich copied to clipboard

Add win_allocate version for all RMA test

Open mpichbot opened this issue 9 years ago • 4 comments

Originally by monicasee on 2015-06-19 12:04:11 -0500


Some RMA tests (i.e., test/mpi/rma/reqops.c) only test win_create, we should add win_allocate version for all of them in order to test shm communication. test/mpi/rma/manyrma2.c is an example to support this.

mpichbot avatar Oct 14 '16 19:10 mpichbot

Looks like we can pass info hints "shmem" to MPI_Win_alloc. If that enables shared memory RMA, then that is a much easier option to add to the tests.

hzhou avatar Aug 30 '22 19:08 hzhou

Looking at the test mentioned in the example, I don't think the use of of shmem info hint was the intention. The point of using MPI_Win_allocate is that processes on the same node could allocate shared memory in the window, so the RMA between them is shared memory access.

But for this particular test reqops.c, it shouldn't make any difference because only rank 0 allocates the memory.

yfguo avatar Aug 30 '22 20:08 yfguo

Need to check what other tests beside reqops.c need this change.

yfguo avatar Aug 30 '22 20:08 yfguo

I am going to cherry-pick the RMA changes from #5725 and go from there.

yfguo avatar Aug 30 '22 20:08 yfguo