Skip to content

Fix slice transfer for flows with alpha channel - #620

Merged
vt-tv merged 4 commits into
dmf-mxl:mainfrom
jonasohland:feature/fabrics/sliced-alpha
Jul 29, 2026
Merged

Fix slice transfer for flows with alpha channel#620
vt-tv merged 4 commits into
dmf-mxl:mainfrom
jonasohland:feature/fabrics/sliced-alpha

Conversation

@jonasohland

Copy link
Copy Markdown
Contributor

Fix the RMA based protocol (the only one we have) to correctly handle alpha channel when transferring slices.

Excluding the SHM provider from the tests for now, because the writer cannot handle multiple writes without completion queue reads in between. We intent to fix this later together with some other possible optimizations.

Fixes #592

@jonasohland jonasohland self-assigned this Jul 23, 2026
@jonasohland jonasohland added the backport/v1.1 This PR should be back ported to the release branch of version 1.1. label Jul 23, 2026

@mlefebvre1 mlefebvre1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just 2 questions.

Comment thread lib/fabrics/ofi/src/internal/ProtocolEgressRMA.cpp Outdated
Comment thread lib/fabrics/ofi/src/internal/ProtocolEgressRMA.cpp Outdated
@jonasohland
jonasohland requested a review from mlefebvre1 July 24, 2026 17:15

@mlefebvre1 mlefebvre1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread lib/tests/fabrics/test_sliced.cpp
Comment thread lib/fabrics/ofi/src/internal/DataLayout.cpp Outdated
Comment thread lib/fabrics/ofi/src/internal/DataLayout.cpp Outdated
Comment thread lib/fabrics/ofi/src/internal/ProtocolEgressRMA.cpp
Comment thread lib/fabrics/ofi/src/internal/ProtocolEgressRMA.cpp
@jonasohland
jonasohland requested a review from k0zmo July 29, 2026 17:25
jonasohland and others added 4 commits July 29, 2026 19:54
Co-authored-by: Jonas Ohland <jonas.ohland@gmail.com>
Signed-off-by: Jonas Ohland (Riedel) <jonas.ohland@riedel.net>
Co-authored-by: Jonas Ohland <jonas.ohland@gmail.com>
Signed-off-by: Jonas Ohland (Riedel) <jonas.ohland@riedel.net>
Co-authored-by: Jonas Ohland <jonas.ohland@gmail.com>
Signed-off-by: Jonas Ohland (Riedel) <jonas.ohland@riedel.net>
Co-authored-by: Jonas Ohland <jonas.ohland@gmail.com>
Signed-off-by: Jonas Ohland (Riedel) <jonas.ohland@riedel.net>
@jonasohland
jonasohland force-pushed the feature/fabrics/sliced-alpha branch from e16d6eb to f1431e2 Compare July 29, 2026 17:54

@vt-tv vt-tv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving on behalf of @k0zmo

@vt-tv
vt-tv merged commit 4a8a50e into dmf-mxl:main Jul 29, 2026
9 checks passed
@backport-mxl-pull-requst

Copy link
Copy Markdown

Successfully created backport PR for release/v1.1:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/v1.1 This PR should be back ported to the release branch of version 1.1.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Open Fabrics: partial-grain transfer support for sliced alpha

4 participants