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

Queue delay watermark #1920

Merged
merged 1 commit into from
Apr 11, 2024
Merged

Conversation

j-bos
Copy link
Contributor

@j-bos j-bos commented Nov 2, 2023

Introduce a watermark attribute, SAI_QUEUE_STAT_DELAY_WATERMARK, to report delay experienced for packets per-queue.

At minimum, this should account the longest time a packet was buffered in the queue, so that the reported value will increase during queue congestion.

Implementations are permitted to include additional processing latency in this measurement, depending on the details of the ASIC design and in what stages it generates rx and tx timestamps.

@j-bos j-bos marked this pull request as ready for review November 3, 2023 02:10
inc/saiqueue.h Outdated Show resolved Hide resolved
Copy link
Collaborator

@kcudnik kcudnik left a comment

Choose a reason for hiding this comment

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

approve only for code quality, please wait for others to review

@j-bos j-bos force-pushed the delay-watermark branch from a907e2a to 608b3c6 Compare March 7, 2024 17:50
@kcudnik
Copy link
Collaborator

kcudnik commented Mar 7, 2024

You need to squash commits since you changed the name of the enum

Signed-off-by: Jason Bos <[email protected]>
@j-bos j-bos force-pushed the delay-watermark branch from 608b3c6 to 8263163 Compare March 8, 2024 19:45
@j-bos
Copy link
Contributor Author

j-bos commented Mar 12, 2024

You need to squash commits since you changed the name of the enum

Done.

@rlhui rlhui merged commit e5064d5 into opencomputeproject:master Apr 11, 2024
3 checks passed
@j-bos j-bos deleted the delay-watermark branch June 13, 2024 23:26
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.

4 participants