Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pml/cm,ob1: pack data from application buffer in successive MPI_Start calls #12797

Merged
merged 2 commits into from
Sep 20, 2024

Commits on Sep 5, 2024

  1. pml/cm: pack data from application buffer in successive MPI_Start calls

    This patch fixes a bug exposed by MTT MPI_Bsend_init_overtake_c, where
    - MPI_Bsend_init(buf, ..., req)
    - MPI_Start(req)
    - MPI_Wait(req, stat)
    - modify(buf)
    - MPI_Start(req) // this does not send the updated data
    - MPI_Wait(buf)
    
    This is because PML/CM uses a bounce buffer send_req.req_buff to pack
    data from  the application buffer in MPI_Bsend_int but not MPI_Start.
    
    This patch adds a step to pack data from the application buffer in each
    MPI_Start call.
    
    Signed-off-by: Wenduo Wang <wenduwan@amazon.com>
    wenduwan committed Sep 5, 2024
    Configuration menu
    Copy the full SHA
    3226c6c View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2024

  1. Fix buffered sends for OB1 as well.

    Signed-off-by: George Bosilca <gbosilca@nvidia.com>
    bosilca committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    429c7b7 View commit details
    Browse the repository at this point in the history