Skip to content

Per Lane PRBS statuses and statistics#2204

Merged
tjchadaga merged 1 commit into
opencomputeproject:masterfrom
sdasgupta-marvell:sdasgupta_prbs
Oct 10, 2025
Merged

Per Lane PRBS statuses and statistics#2204
tjchadaga merged 1 commit into
opencomputeproject:masterfrom
sdasgupta-marvell:sdasgupta_prbs

Conversation

@sdasgupta-marvell

Copy link
Copy Markdown
Contributor

This proposal extends existing PRBS functionality to

  • Support per-lane statuses and statistics
  • Introduce Bit Error Rate (BER) as a new metric

@sdasgupta-marvell

Copy link
Copy Markdown
Contributor Author

Addressed comments raised during review (Extending Error Count stats to 16 lanes + leave holes for future scope)

@rlhui rlhui added the reviewed PR is discussed in SAI Meeting label Sep 15, 2025
@rck-innovium

Copy link
Copy Markdown
Contributor

@prgeor @j-bos Thank you for your comments during the SAI community review. The comments have been addressed. Please review.

@tjchadaga
tjchadaga requested a review from prgeor October 7, 2025 21:06
@tjchadaga

Copy link
Copy Markdown
Collaborator

@sdasgupta-marvell, @rck-innovium - please help resolve conflicts

@tjchadaga

tjchadaga commented Oct 7, 2025

Copy link
Copy Markdown
Collaborator

@j-bos, @prgeor, @JaiOCP - gentle reminder on this. Could you please help review this PR?

@prgeor

prgeor commented Oct 9, 2025

Copy link
Copy Markdown
Collaborator

@sdasgupta-marvell please resolve merge conflict.

Signed-off-by: Spandan Dasgupta <sdasgupta@marvell.com>
@tjchadaga

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga
tjchadaga merged commit 19f3224 into opencomputeproject:master Oct 10, 2025
3 checks passed
tjchadaga pushed a commit that referenced this pull request Oct 10, 2025
Signed-off-by: Spandan Dasgupta <sdasgupta@marvell.com>
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.

5 participants