openucx / ucc

Unified Collective Communication Library
https://openucx.github.io/ucc/
BSD 3-Clause "New" or "Revised" License
177 stars 85 forks source link

UTIL: fix memory barrier for aarch64 #939

Closed Sergei-Lebedev closed 3 months ago

Sergei-Lebedev commented 3 months ago

What

Fixes memory barrier in TL SHM.

Why ?

dmb guarantees only ordering, with dsb we correctly wait for store to finish before signaling

janjust commented 3 months ago

@bureddy Hey Deva, can you take a quick look - super small patch, but fixes a critical issue on GH systems.