Skip to content

Conversation

@vkjammala-arista
Copy link
Contributor

@vkjammala-arista vkjammala-arista commented Mar 19, 2025

This document provides information about the implementation of Port Forward Error Correction (FEC) Frame Loss Ratio (FLR) support in SONiC.

Fixes #2068

Code PRs for tracking

The code pull requests should merge in the order they are mentioned in the table.

Repo PR Title State
sonic-swss Add FEC frame loss ratio monitoring with predictive analytics GitHub issue/pull request detail
sonic-utilities Add FEC FLR statistics support to port counters GitHub issue/pull request detail
sonic-mgmt modify fec-stats test for fec_flr GitHub issue/pull request detail

This document provides information about the implementation of Port
Forward Error Correction (FEC) Frame Loss Ratio (FLR) support in SONiC.
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

Copy link
Contributor

@prgeor prgeor left a comment

Choose a reason for hiding this comment

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

@vkjammala-arista Thanks for the HLD. Added few comments.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@prgeor
Copy link
Contributor

prgeor commented Jul 17, 2025

@vkjammala-arista can we change the title to "Predicted Frame Loss Ratio Estimation" ?

@vkjammala-arista
Copy link
Contributor Author

@vkjammala-arista can we change the title to "Predicted Frame Loss Ratio Estimation" ?

We are introducing observed FEC FLR as well in this HLD, that's the reason i have put FEC FLR support in SONiC.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

1) Add images to help understand predicted FEC FLR better.
2) Display accuracy (R^2) information along with predicted FEC FLR
   value.
3) Add small note on using window from 16 to 20. Update FEC FLR formula
   considering interleaving factor to include MFC.
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

1) portstat CLI fields renaming, FEC_FLR --> FLR(O) and FEC_FLR_PREDICTED --> FLR(P)
2) Rename "fec-flr-interval-factor" to "flr-interval-factor"
3) CLI output explanation is updated to highlight that atleast 2 non-zero bin values is needed for predicted flr computation.
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@prgeor prgeor merged commit 1b7dff9 into sonic-net:master Sep 29, 2025
1 check 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.

port FEC FLR support in SONiC

4 participants