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 v2] linux-gen: ml: fix size of result array and minor clean up #2121

Merged
merged 4 commits into from
Sep 27, 2024

Conversation

JereLeppanen
Copy link
Collaborator

No description provided.

@odpbuild odpbuild changed the title linux-gen: ml: fix size of result array and minor clean up [PATCH v1] linux-gen: ml: fix size of result array and minor clean up Sep 20, 2024
Increase size of the result array from ML_MAX_COMPL_ID to
ML_MAX_COMPL_ID + 1. According to capability, completion IDs range
from 0 to ML_MAX_COMPL_ID inclusive, so the array must be one bigger.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
Fix error log typos and grammar, and remove unnecessary "Error:"
prefixes.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
This makes the difference to linux-dpdk slightly smaller.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
Minor clean up of some comments and whitespace.

Signed-off-by: Jere Leppänen <jere.leppanen@nokia.com>
Reviewed-by: Tuomas Taipale <tuomas.taipale@nokia.com>
@odpbuild odpbuild changed the title [PATCH v1] linux-gen: ml: fix size of result array and minor clean up [PATCH v2] linux-gen: ml: fix size of result array and minor clean up Sep 27, 2024
@JereLeppanen
Copy link
Collaborator Author

v2:

  • Rebase.
  • Add review tags.

@MatiasElo MatiasElo merged commit 649450d into OpenDataPlane:master Sep 27, 2024
170 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.

3 participants