Skip to content

Add queue flushed packets and bytes stats#2280

Open
msherif1234 wants to merge 1 commit into
opencomputeproject:masterfrom
msherif1234:queue_flush_stats
Open

Add queue flushed packets and bytes stats#2280
msherif1234 wants to merge 1 commit into
opencomputeproject:masterfrom
msherif1234:queue_flush_stats

Conversation

@msherif1234
Copy link
Copy Markdown

@msherif1234 msherif1234 commented Apr 27, 2026

Adding queue flushed packets and bytes statistics to SAI queue statistics for better traffic management diagnostics.

When PFCWD is enabled, an egress queue may remain stuck due to a PFC storm. In this condition, traffic scheduled to the affected queue will be redirected to a disabled or blackhole destination so they are drained and dropped in a controlled manner.

This change adds queue flush packet and byte counters to report such hardware-flushed traffic through SAI queue statistics.

SAI_QUEUE_STAT_FLUSHED_PACKETS
SAI_QUEUE_STAT_FLUSHED_BYTES

@kcudnik
Copy link
Copy Markdown
Collaborator

kcudnik commented May 5, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@kcudnik
Copy link
Copy Markdown
Collaborator

kcudnik commented May 5, 2026

since you changed enums names you need to squash and force push to this branch

Signed-off-by: Mohamed S. Mahmoud <mmahmoud2201@gmail.com>
@msherif1234 msherif1234 force-pushed the queue_flush_stats branch from cbee6e6 to 317d6b0 Compare May 6, 2026 11:48
@tjchadaga
Copy link
Copy Markdown
Collaborator

@msherif1234 - are you able to have a quick discussion of this PR in the SAI community meeting on 5/7?

@tjchadaga
Copy link
Copy Markdown
Collaborator

@msherif1234
Copy link
Copy Markdown
Author

@msherif1234 - are you able to have a quick discussion of this PR in the SAI community meeting on 5/7?

Hi @tjchadaga sure I will join today meeting

Thank you!!

@msherif1234 msherif1234 changed the title Add queue flush packets and bytes stats Add queue flushed packets and bytes stats May 7, 2026
@tjchadaga
Copy link
Copy Markdown
Collaborator

@msherif1234
Copy link
Copy Markdown
Author

@JaiOCP

@msherif1234
Copy link
Copy Markdown
Author

@mthatty

@tjchadaga tjchadaga added the reviewed PR is discussed in SAI Meeting label May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewed PR is discussed in SAI Meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants