Grafico dei commit

431 Commit

Autore SHA1 Messaggio Data
yugang-amd bbad1d8539 GDA docs style edits (#362)
* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/install.rst

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Update docs/sphinx/_toc.yml.in

Co-authored-by: yugang-amd <yugang.wang@amd.com>

* Apply suggestions from code review

Co-authored-by: yugang-amd <yugang.wang@amd.com>

---------

Co-authored-by: Yiltan <ytemucin@amd.com>
2025-12-10 17:03:58 -05:00
Aurelien Bouteiller 64460f0ec9 Bump version number for post-7.2 devel (#356) 2025-12-10 13:03:20 -05:00
Yiltan fddbe7b15d Add default context alltoall API (#350) 2025-12-10 11:43:15 -05:00
Aurelien Bouteiller 1a16b3bedc Reenable building test-only with external MPI (#352) 2025-12-10 11:40:29 -05:00
Aurelien Bouteiller ef5f2be215 Update version to 3.2.0 for 7.2.0 rocm release (#351) 2025-12-09 10:26:55 -05:00
Anatolii Rozanov d0c8380650 Add host API for *_on_stream operations (#340)
* Add functional test for barrier_all_on_stream

* Add rocshmem_barrier_all_on_stream support for GDA and RO backends

Implements rocshmem_barrier_all_on_stream operation for
GPU Direct Access and Reverse Offload backends.

Previously, rocshmem_barrier_all_on_stream was only supported for IPC backend.

* Add functional test for rocshmem_broadcastmem_on_stream

* Add host-side rocshmem_broadcastmem_on_stream API

Implement stream-based broadcast collective operation

- Add rocshmem_broadcastmem_on_stream host API and kernel implementation
- Add functional test TeamBroadcastmemOnStreamTester with multi-stream
  support and correctness verification
- Use per-workgroup contexts to avoid contention across parallel streams

API:
rocshmem_broadcastmem_on_stream(team, dest, source, nelems, pe_root, stream)

* Add functional test for rocshmem_getmem_on_stream

* Add host-side rocshmem_getmem_on_stream API

Implement stream-based point-to-point RMA get operation

- Add rocshmem_getmem_on_stream host API and kernel implementation
- Support for asynchronous getmem operations on HIP streams
- Add backend support for GDA, RO, and IPC contexts
- Use work-group collective getmem for efficient memory transfer

API:
rocshmem_getmem_on_stream(dest, source, nelems, pe, stream)

(AI Assist)

* Add host-side rocshmem_putmem_on_stream API

- Add rocshmem_putmem_on_stream for asynchronous remote writes
- Support for concurrent RMA operations on HIP streams
- Add backend support for GDA, RO, and IPC contexts
- Use work-group device collective operation

API:
rocshmem_putmem_on_stream(dest, source, bytes, pe, stream)

(AI Assist)

* Add functional test for rocshmem_putmem_on_stream

* Add host-side rocshmem_putmem_signal_on_stream API

Enables asynchronous putmem operations with signaling on HIP streams.

The implementation includes:
- Kernel wrapper rocshmem_putmem_signal_kernel
- Host interface putmem_signal_on_stream method
- Context layer support across all backends (IPC, GDA, RO)
- Public API

Function signature:
void rocshmem_putmem_signal_on_stream(void *dest, const void *source,
                                      size_t bytes, uint64_t *sig_addr,
                                      uint64_t signal, int sig_op,
                                      int pe, hipStream_t stream);

* Add functional test for rocshmem_putmem_signal_on_stream

* Add host-side rocshmem_signal_wait_until_on_stream API

Enables asynchronous signal wait operations on HIP streams.

The implementation includes:
- Kernel wrapper rocshmem_signal_wait_until_kernel
- Host interface signal_wait_until_on_stream method
- Context layer support across all backends (IPC, GDA, RO)
- Native uint64_t support in wait_until API (generated from P2P_SYNC.py)

Function signature:
void rocshmem_signal_wait_until_on_stream(uint64_t *sig_addr, int cmp,
                                          uint64_t cmp_value,
                                          hipStream_t stream);

(AI Assist)

* Add functional test for rocshmem_signal_wait_until_on_stream

* Add documentation for stream API functions

This commit adds API documentation for the following host-side
stream functions:

- rocshmem_barrier_all_on_stream (collective routines)
- rocshmem_broadcastmem_on_stream (collective routines)
- rocshmem_getmem_on_stream (RMA operations)
- rocshmem_putmem_on_stream (RMA operations)
- rocshmem_putmem_signal_on_stream (signaling operations)
- rocshmem_signal_wait_until_on_stream (point-to-point sync)

The documentation includes function signatures, parameter descriptions,
and detailed explanations of asynchronous behavior and stream handling.

(AI Assist)

* Rename "bytes" -> "nelems"

* Add "_TEST_" to the variables used in tests

* Remove incorrect hipStreamDefault usage

hipStreamDefault is not a default stream. This is a flag.

If stream == nullptr, then just pass it to kernel. It will launch the kernel on the default stream
2025-12-09 08:55:46 -06:00
Dimple Prajapati fbe57306b9 Add IBGDA backend flag to enable bitcode generation (#347)
* Change to enable ibgda bitcode compilation

* Apply suggestion from @abouteiller

---------

Co-authored-by: Aurelien Bouteiller <aurelien.bouteiller@amd.com>
2025-12-08 16:19:48 -08:00
Avinash Kethineedi 1acf454048 Refactor: modularize RMA and AMO WQE posting functions (#331)
* Refactor: modularize RMA and AMO WQE posting functions
  - Extract shared logic for SQ/CQ waiting, doorbell ringing, and WQE building
* Remove unused variables
* Update return buffer address calculation for atomics
2025-12-08 14:54:41 -06:00
Yiltan d5bcb3a201 Fix docs rendering issue (#349) 2025-12-08 15:54:06 -05:00
Yiltan ecd4c9f561 Remove unused fence policy (#348) 2025-12-08 14:06:53 -05:00
Aurelien Bouteiller c99bc21e10 Functional tests without MPI support (#343)
* Let functional tests build without external MPI

* Fix error conditions when using uuid startup with internal MPI

* Do not abort if libibverbs is not found but not using GDA

* Enabled RO functional test initialized with TEST_UUID

* Reduce load time for ro backend_can_run and prevent mpilib_dlclose
crashing

* Fix case TEST_UUID=1, ROCSHMEM_BACKEND='' (autoloading gda)
2025-12-08 11:46:16 -05:00
Yiltan baaf8091b5 [GDA/BNXT] Optimize Alltoall using put signal (#334)
* Modularize bnxt

* add post_wqe_amo_single

* add alltoall with putsignal impl

* make ringing the doorbell optional
2025-12-05 12:41:22 -05:00
Avinash Kethineedi f907ef91e4 IPC: insert __threadfence_system() after *wg RMA APIs to guarantee global memory visibility (#346) 2025-12-04 10:21:25 -06:00
Edgar Gabriel 224c969bef reenable gfx1100 (#328)
* reenable gfx1100

use the modified version of the flat_store_short assembly instruction as suggested by the compiler team (32bit input value instead of 16bit)

* add fix for gfx1201

add the same fix for gfx1201 that was introduced for gfx1100
2025-12-03 13:49:38 -06:00
Anatolii Rozanov 5577feb70d Add host API for alltoallmem_on_stream collective operation (#333)
* Add host-side rocshmem_alltoallmem_on_stream function

Function signature:
  rocshmem_alltoallmem_on_stream(rocshmem_team_t team, void *dest,
                                 const void *source, size_t size,
                                 hipStream_t stream)

- The function launches rocshmem_alltoallmem_kernel which calls
device-side alltoall<char> workgroup collective through default context.
- Uses dynamic block size determination via occupancy API.
- Implemented for all backends.

* Fix incorrect sync buffer size allocation for alltoall in GDA and IPC backends

When allocating memory for alltoall_pSync_pool in setup_teams() and
teams_init() functions, the code incorrectly used ROCSHMEM_BCAST_SYNC_SIZE
instead of ROCSHMEM_ALLTOALL_SYNC_SIZE.

* Add functional test for team_alltoallmem_on_stream

This commit adds a new functional test to verify the correctness of
the host-side rocshmem_team_alltoallmem_on_stream API.

* Add documentation for rocshmem_alltoallmem_on_stream

This commit adds API documentation for the host-side
rocshmem_alltoallmem_on_stream function in the collective routines
section. The documentation includes:
2025-12-03 08:40:24 -05:00
Yiltan 8b350a51fe Updated important missing enviroment variables (#344) 2025-12-02 11:40:30 -05:00
Aurelien Bouteiller 0f7da76018 MLX5 cards have a vendor-id that does not match the pci-vendor-id for (#342)
some reason.

Signed-off-by: Aurelien Bouteiller <abouteil@amd.com>
2025-12-02 11:32:37 -05:00
Kutovoi, Vadim 29000a5644 gda: add check for active interfaces when selecting the GDA backend (#327)
* gda: add check for active interfaces when selecting the GDA backend

* fix __func__ maco in rocshmem_ctx_pe_quiet

* gda: switch to more generic RDMA NIC term in has_active_ib_interface

* gda: add active MLX5 and Pensando vendor ID checks for backend selection
2025-12-01 15:49:25 -05:00
Adel Johar ba77bdd9a6 [Docs] Move environment variables to separate page (#341) 2025-12-01 14:25:27 -05:00
Yiltan 774159a08f Cleanup readme.md 2025-12-01 10:43:18 -05:00
Yiltan 5606fdafd6 Update docs for GDA (#337) 2025-12-01 09:38:11 -05:00
Yiltan d9e2890222 Add rocshmem_int64_p (#335) 2025-11-26 10:31:23 -05:00
Yiltan 7ab9823169 Alltoall bug fix for RCCL (#329) 2025-11-20 11:16:00 -05:00
Edgar Gabriel 2ae2033648 add relaxed_ordering option (#324)
* add relaxed_ordering option

add an environment variable that allows to control setting the
IBV_ACCESS_RELAXED_ORDERING flag when registering memory with the
ibv_reg_mr* functions.

* missed a spot
2025-11-20 08:20:25 -06:00
Yiltan 1347d5d628 [GDA] Alltoall optimization - single warp (#319)
* Remove testing of data types
As the collective is templated, we are just testing if sizeof(T) works

* Added single threaded varients

* Applied thread puts optimization to barrier

* Apply single threaded optimization to alltoall

* This optimization only works on bnxt, so place a switch to protect it

* Handle the edge case where the thread count is smaller than the number of PEs
2025-11-19 14:25:29 -05:00
Anatolii Rozanov 618bdef082 Fix typo in README.md (#326) 2025-11-19 08:36:01 -06:00
Yiltan 1fb0fc4bd0 Update change log for rocm 7.1.1 (#320) (#321)
(cherry picked from commit cb239930bc4167806269cbb94e26cf02837daa7a)
2025-11-14 13:51:03 -05:00
Edgar Gabriel ef3ba6cd45 disable gfx1100 temporarily (#322) 2025-11-14 10:46:19 -07:00
Yiltan 73786e203e only use rocm_install if we build the tools (#316) 2025-11-12 10:58:49 -05:00
Edgar Gabriel e1a7e20b1b replace MPI function call. (#317)
* replace MPI function call.

* add two missing defs for RO
2025-11-12 07:38:47 -06:00
Dana Robinson 65790c1b4f Fix typo in CONTRIBUTING.md (#315) 2025-11-09 12:58:19 -06:00
Yiltan 80f0a39866 Use dlopen for libnuma (#312) 2025-11-07 10:12:11 -05:00
Edgar Gabriel d185fe3555 initial commit for gfx12 support (#305) 2025-11-07 08:54:03 -06:00
Edgar Gabriel 4fc5541d78 disable memory tests (#310)
disable fine-grain and coarse-grain memory testst until a fix is
available in ROCm 7.1 and/or our CI image. Otherwise we might miss other
errors due to constant CI failures.
2025-11-07 08:04:31 -06:00
Allen Hubbe e2dcf99456 gda: fix getmem_nbi_wg source and dest (#311)
A copy paste mistake in a previous commit caused source and dest to
be reversed.  Correct the source and dest params.

Fixes: 6de67d5d7c

Signed-off-by: Allen Hubbe <allen.hubbe@amd.com>
2025-11-06 16:21:20 -06:00
Yiltan cd9b5ee806 Memset queues (#313) 2025-11-06 14:16:53 -05:00
Yiltan 110f9c8793 Added ibv_wrapper which opens library using dlopen (#309) 2025-11-05 16:12:44 -05:00
Allen Hubbe 6de67d5d7c gda ionic: use all threads in wave operations (#295)
Use all available threads for polling the cq to increase the maximum
message rate.  Even when posting a single wqe in the wave, use all
available theads for polling the cq to reserve space in the sq.

Changes were needed in the rocshmem abstraction to avoid disabling gpu
threads, like taking turns or using only the first thread in a wave or
wavefront.  To avoid breaking other gda implementations, reimplement
turn-based or single thread strategy in post_wqe_rma_turn and
post_wqe_rma_single.

Signed-off-by: Allen Hubbe <allen.hubbe@amd.com>
2025-11-05 11:01:14 -06:00
Aurelien Bouteiller b7a6d86c6b python venv madness round 2: use ensurepip if installed (#308)
When creating a python venv during the install_dependencies script, we try to use ensurepip if it is installed, as it deals better with cases where multiple venvs are active simultaneously. (as seen in CI buildbot)
2025-11-05 10:52:22 -05:00
Aurelien Bouteiller 8c175315f2 Add backend type query method, use it to disable 32bit amo testers on gda (#307)
* Add backend type query method, use it to disable 32bit amo testers on
gda

* The infrateam testers work
2025-11-05 10:24:07 -05:00
erieaton-amd 7b5765ec0e Fix rocshmem_ptr definition signature (#306)
Makes the signature of the definition match the declaration in rocshmem.hpp.

Signed-off-by: Eric Eaton <erieaton@amd.com>
2025-11-04 12:42:47 -05:00
Aurelien Bouteiller e155af8704 install_dependencies pip issues with ubuntu 24 (#302)
* The install_dependencies script would fail on ubuntu 24.04
they changed how pip works so we need to create a venv first now

* Fix install_dependencies for ubuntu 22

* Make sure we build in the builddir and install in the installdir
combine installdir for ucx and ompi when user-provided by INSTALL_DIR
retain prior behavior if not overridden to avoid breaking CI scripts
2025-10-31 16:34:36 -04:00
Yiltan 8dd2112ec8 Alltoall linear parallel Optimization (#303) 2025-10-31 10:26:44 -04:00
Yiltan 5f87bb061b [GDA] Implement internal_direct_barrier_wg (#299) 2025-10-31 10:26:24 -04:00
Allen Hubbe ed91c8cce2 functional_tests: n, nskip, nloop, nlarge options (#297)
To make the functional tests more useful for benchmarking, allow user to
specify the number of loops and related parameters via command options.

Signed-off-by: Allen Hubbe <allen.hubbe@amd.com>
2025-10-30 11:54:49 -04:00
Edgar Gabriel c0285ac0ce minor change to MPI detection logic (#294)
somehow the test whether we requested MPI support or not stopped
working, although no obvious code change can be located.

Make the if-statement more stringent by explicitely testing whether
USE_MPI_SUPPORT is "ON".
2025-10-28 12:54:26 -05:00
akolliasAMD 87d87cc881 renamed memcpy to memcpy_lane (#296) 2025-10-28 09:33:13 -06:00
Yiltan ff29d139cb [GDA] Improve error messages (#292) 2025-10-27 16:51:15 -04:00
Yiltan 6290db319c [GDA/BNXT] Implemented CQE Collapsing (#279) 2025-10-23 14:53:44 -04:00
Aurelien Bouteiller 054bc33dc4 Tests/syncall (#291)
* SyncAll test case would run Sync

* Despecialized name for argument reader

* Rename sync-test to team-sync-test as it uses teams

* Another stab at probing NUM_GPUS
2025-10-23 13:40:41 -04:00