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 v3] abi: dma: increase default transfer ID size #1942

Merged
merged 2 commits into from
Nov 1, 2023

Conversation

TuomasTaipale
Copy link
Collaborator

@TuomasTaipale TuomasTaipale commented Oct 31, 2023

Increase default odp_dma_transfer_id_t size to 64 bit from 32 bit. This enables different implementations to potentially utilize pointers as the transfer ID.

v2:

  • Reverted 64 bit transfer ID usage in linux-gen

v3:

  • Added reviewed-by tags

@odpbuild odpbuild changed the title abi: dma: increase default transfer ID size [PATCH v1] abi: dma: increase default transfer ID size Oct 31, 2023
@odpbuild odpbuild changed the title [PATCH v1] abi: dma: increase default transfer ID size [PATCH v2] abi: dma: increase default transfer ID size Nov 1, 2023
Increase default `odp_dma_transfer_id_t` size to 64 bit from 32 bit.
This enables different implementations to potentially utilize pointers
as the transfer ID.

Signed-off-by: Tuomas Taipale <tuomas.taipale@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Utilize `_odp_pri()` to get rid of repetitive `(uint64_t)(uintptr_t)`
casts.

Signed-off-by: Tuomas Taipale <tuomas.taipale@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
@odpbuild odpbuild changed the title [PATCH v2] abi: dma: increase default transfer ID size [PATCH v3] abi: dma: increase default transfer ID size Nov 1, 2023
@MatiasElo MatiasElo enabled auto-merge (rebase) November 1, 2023 14:12
@MatiasElo MatiasElo merged commit bbf24c4 into OpenDataPlane:master Nov 1, 2023
174 checks passed
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