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.

Sign up for GitHub

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

Jump to bottom

[PATCH v6] api: cls: add pmr/cos create/destroy multi functions #1912

Merged
merged 10 commits into from
Oct 20, 2023
Merged

[PATCH v6] api: cls: add pmr/cos create/destroy multi functions #1912

merged 10 commits into from
Oct 20, 2023

Conversation

MatiasElo
Copy link
Collaborator

@MatiasElo MatiasElo commented Sep 20, 2023 •
edited
Loading

V2:

  • Changed odp_cls_cos_create_multi() parameter const odp_cls_cos_param_t *param[] to const odp_cls_cos_param_t param[]
  • Changed odp_cls_pmr_create_multi() parameter const odp_pmr_create_opt_t *opt[] to const odp_pmr_create_opt_t opt[]

V3:

  • Added odp_cos_destroy_multi() and odp_cls_pmr_destroy_multi() APIs

V4:

  • Added implementation and validation tests

@odpbuild odpbuild changed the title api: cls: add pmr/cos create multi functions [PATCH v1] api: cls: add pmr/cos create multi functions Sep 20, 2023
@odpbuild odpbuild changed the title [PATCH v1] api: cls: add pmr/cos create multi functions [PATCH v2] api: cls: add pmr/cos create multi functions Sep 22, 2023
@odpbuild odpbuild changed the title [PATCH v2] api: cls: add pmr/cos create multi functions [PATCH v3] api: cls: add pmr/cos create multi functions Sep 27, 2023
@MatiasElo MatiasElo changed the title [PATCH v3] api: cls: add pmr/cos create multi functions [PATCH v3] api: cls: add pmr/cos create/destroy multi functions Sep 27, 2023
@odpbuild odpbuild changed the title [PATCH v3] api: cls: add pmr/cos create/destroy multi functions [PATCH v4] api: cls: add pmr/cos create/destroy multi functions Sep 29, 2023
@MatiasElo MatiasElo added the API next API change for next release label Oct 18, 2023
@odpbuild odpbuild changed the title [PATCH v4] api: cls: add pmr/cos create/destroy multi functions [PATCH v5] api: cls: add pmr/cos create/destroy multi functions Oct 19, 2023
@MatiasElo MatiasElo added API Ready API change ready to be merged (with the next API version increment) and removed API next API change for next release labels Oct 19, 2023
@MatiasElo
api: cls: add odp_cls_pmr_create_multi() function
0ec5913 
Add new odp_cls_pmr_create_multi() function for creating multiple packet
matching rules with a single call. Depending on the implementation, using
the multi variant can decrease per PMR rule create latency.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Kiran Kumar K <kirankumark@marvell.com>
@MatiasElo
linux-gen: cls: implement odp_cls_pmr_create_multi() function
a391357 
Implement new odp_cls_pmr_create_multi() function for creating multiple
packet matching rules with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@MatiasElo
api: cls: add odp_cls_cos_create_multi() function
579e7d5 
Add new odp_cls_cos_create_multi() function for creating multiple class-of-
services with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Kiran Kumar K <kirankumark@marvell.com>
@MatiasElo
linux-gen: cls: implement odp_cls_cos_create_multi() function
3c94dbc 
Implement new odp_cls_cos_create_multi() function for creating multiple
class-of-services with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@MatiasElo
api: cls: add odp_cos_destroy_multi() function
ffb863c 
Add new odp_cos_destroy_multi() function for destroying multiple CoSes
with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Kiran Kumar K <kirankumark@marvell.com>
@MatiasElo
linux-gen: cls: implement odp_cos_destroy_multi() function
aada9bc 
Implement new odp_cos_destroy_multi() function for destroying multiple
CoSes with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@MatiasElo
api: cls: add odp_cls_pmr_destroy_multi() function
ded6ef7 
Add new odp_cls_pmr_destroy_multi() function for destroying multiple PMRs
with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
Reviewed-by: Kiran Kumar K <kirankumark@marvell.com>
@MatiasElo
linux-gen: cls: implement odp_cls_pmr_destroy_multi() function
edee7a0 
Implement new odp_cls_pmr_destroy_multi() function for destroying multiple
PMRs with a single call.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@MatiasElo
validation: cls: add tests for create/destroy cos multi variants
928ab1d 
Add validation tests for new odp_cls_cos_create_multi() and
odp_cos_destroy_multi() APIs.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@MatiasElo
validation: cls: add tests for create/destroy pmr multi variants
b7f65b7 
Add validation tests for new odp_cls_pmr_create_multi() and
odp_cls_pmr_destroy_multi() APIs.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com>
@odpbuild odpbuild changed the title [PATCH v5] api: cls: add pmr/cos create/destroy multi functions [PATCH v6] api: cls: add pmr/cos create/destroy multi functions Oct 20, 2023
@MatiasElo MatiasElo enabled auto-merge (rebase) October 20, 2023 17:21
@MatiasElo MatiasElo merged commit 8afd898 into OpenDataPlane:master Oct 20, 2023
174 checks passed
@MatiasElo MatiasElo deleted the dev/api-cls-functions branch October 23, 2023 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kkirankk kkirankk kkirankk approved these changes

@psavol psavol psavol approved these changes

Assignees
No one assigned
Labels
API Ready API change ready to be merged (with the next API version increment)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MatiasElo @psavol @kkirankk