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

[PATCH v9] api: dma: support source free and destination allocation features #2179

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

VamsiKrishnaA99
Copy link

No description provided.

@odpbuild odpbuild changed the title [PATCH v5] api: dma: support source free and destination allocation features [PATCH v6] api: dma: support source free and destination allocation features Jan 27, 2025
Copy link
Collaborator

@TuomasTaipale TuomasTaipale left a comment

Choose a reason for hiding this comment

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

Still a few comments from internal review, we're getting there.

For better trackability, you can rebase-amend the commits with the fixes locally on your branch and then force-push to your fork-remote-branch. The changes then appear in this same pull request and GitHub can track the diff between versions (GitHub automatically updates the "[PATCH vX]" prefix when new stuff is pushed to the PR-branch).

include/odp/api/spec/dma_types.h Outdated Show resolved Hide resolved
include/odp/api/spec/dma_types.h Outdated Show resolved Hide resolved
include/odp/api/spec/dma_types.h Outdated Show resolved Hide resolved
@VamsiKrishnaA99
Copy link
Author

Sure Tuomas, thanks for reviewing it.

@odpbuild odpbuild changed the title [PATCH v6] api: dma: support source free and destination allocation features [PATCH v7] api: dma: support source free and destination allocation features Feb 2, 2025
@VamsiKrishnaA99
Copy link
Author

Addressed V6 review comments.

@TuomasTaipale
Copy link
Collaborator

#2171 and #2176 for reference.

@odpbuild odpbuild changed the title [PATCH v7] api: dma: support source free and destination allocation features [PATCH v8] api: dma: support source free and destination allocation features Feb 4, 2025
Vamsi Attunuru added 2 commits February 4, 2025 10:55
This specification change details the necessary fields for
advertising the source segment freeing offload for data
format #ODP_DMA_FORMAT_PACKET. It also defines the required
fields to enable this feature.

Application can free all the source segments using
odp_dma_transfer_t:seg_free field. The 'unique_src_segs'
field can be used to indicate if the source packet is
reused across multiple segments to avoid any double-free
issues.

Additionally, application can use dma transfer options field
odp_dma_transfer_t:single_pool to free all source segments
to a single pool, provided all segments belong to a same pool.

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>
Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
This specification change outlines the necessary fields to advertise
the destination segment allocation offload feature. It also defines
the required fields in odp_dma_transfer_param_t and odp_dma_seg_t to
leverage this feature.

When the application opts for destination packet allocation, it will
receive the allocated packet details in odp_dma_result_t::dst_pkt
array after the DMA completion.

Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>
Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
@odpbuild odpbuild changed the title [PATCH v8] api: dma: support source free and destination allocation features [PATCH v9] api: dma: support source free and destination allocation features Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants