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] api: cls: clarify back pressure configuration #2144

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JannePeltonen
Copy link
Collaborator

Description of the back pressure configuration in classifier is somewhat ambiguous as it leaves many details out. Clarify the presumed intent of the API by specifying how exactly the back pressure threshold for any given pktio + pool/queue + pfc_level -tuple is derived from the configuration.

@odpbuild odpbuild changed the title api: cls: clarify back pressure configuration [PATCH v1] api: cls: clarify back pressure configuration Nov 12, 2024
@MatiasElo MatiasElo added the API next API change for next release label Nov 19, 2024
@MatiasElo MatiasElo added this to the v1.46.0.0 release milestone Nov 22, 2024
include/odp/api/spec/classification.h Outdated Show resolved Hide resolved
include/odp/api/spec/classification.h Outdated Show resolved Hide resolved
include/odp/api/spec/classification.h Outdated Show resolved Hide resolved
@odpbuild odpbuild changed the title [PATCH v1] api: cls: clarify back pressure configuration [PATCH v2] api: cls: clarify back pressure configuration Dec 16, 2024
Description of the back pressure configuration in classifier is somewhat
ambiguous as it leaves many details out. Clarify the presumed intent of
the API by specifying how exactly the back pressure threshold for any
given pktio + pool/queue + pfc_level -tuple is derived from the
configuration.

Signed-off-by: Janne Peltonen <[email protected]>
Reviewed-by: Matias Elo <[email protected]>
Reviewed-by: Petri Savolainen <[email protected]>
@odpbuild odpbuild changed the title [PATCH v2] api: cls: clarify back pressure configuration [PATCH v3] api: cls: clarify back pressure configuration Dec 16, 2024
@JannePeltonen
Copy link
Collaborator Author

v3: added review tags

@MatiasElo MatiasElo removed this from the v1.46.0.0 release milestone Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API next API change for next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants