Skip to content

Port storm control enhancement#2258

Open
rpmarvell wants to merge 1 commit intoopencomputeproject:masterfrom
rpmarvell:rpmarvell_port_storm_control
Open

Port storm control enhancement#2258
rpmarvell wants to merge 1 commit intoopencomputeproject:masterfrom
rpmarvell:rpmarvell_port_storm_control

Conversation

@rpmarvell
Copy link
Copy Markdown
Contributor

  • Port storm control enhancement to add more clarity to port storm control attributes.

rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Feb 24, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from e16afbd to 3d33d38 Compare February 24, 2026 20:18
@rpmarvell rpmarvell changed the title Port storm control enhancemnets Port storm control enhancement Feb 25, 2026
rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Feb 25, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 3d33d38 to b83f165 Compare February 25, 2026 06:36
rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Feb 25, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from b83f165 to 29ca9bc Compare February 25, 2026 06:43
rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Feb 25, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 29ca9bc to 6da2b5c Compare February 25, 2026 06:48
rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Feb 25, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 6da2b5c to 9d26d53 Compare February 25, 2026 16:04
@tjchadaga
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga tjchadaga added the reviewed PR is discussed in SAI Meeting label Mar 4, 2026
@tjchadaga
Copy link
Copy Markdown
Collaborator

tjchadaga commented Mar 4, 2026

@rpmarvell - please break this into multiple lines

WARNING: Long brief > 200 on SAI_PORT_ATTR_FLOOD_STORM_CONTROL_POLICER_ID:
 - Enable flood (unknown unicast or unknown multicast) storm control policer on port. Deprecated. Use SAI_PORT_ATTR_UNKNOWN_UNICAST_STORM_CONTROL_POLICER_ID and SAI_PORT_ATTR_UNKNOWN_MULTICAST_STORM_CONTROL_POLICER_ID.

rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Mar 13, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 9d26d53 to 1ae3d76 Compare March 13, 2026 05:48
@rpmarvell
Copy link
Copy Markdown
Contributor Author

@rpmarvell - please break this into multiple lines

WARNING: Long brief > 200 on SAI_PORT_ATTR_FLOOD_STORM_CONTROL_POLICER_ID:
 - Enable flood (unknown unicast or unknown multicast) storm control policer on port. Deprecated. Use SAI_PORT_ATTR_UNKNOWN_UNICAST_STORM_CONTROL_POLICER_ID and SAI_PORT_ATTR_UNKNOWN_MULTICAST_STORM_CONTROL_POLICER_ID.

@tjchadaga : I have addressed it.

rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Mar 13, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 1ae3d76 to 60f2635 Compare March 13, 2026 05:52
rpmarvell added a commit to rpmarvell/SAI that referenced this pull request Mar 13, 2026
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 60f2635 to 8405679 Compare March 13, 2026 06:17
@tjchadaga
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@tjchadaga
Copy link
Copy Markdown
Collaborator

@rpmarvell - could you please resolve the conflicts?

@rpmarvell
Copy link
Copy Markdown
Contributor Author

@rpmarvell - could you please resolve the conflicts?

@tjchadaga : I have resolved the conflicts, could you please merge this change ?

@tjchadaga
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 31eddfe to 510305e Compare April 8, 2026 11:25
Signed-off-by: rpmarvell <rperumal@marvell.com>
@rpmarvell rpmarvell force-pushed the rpmarvell_port_storm_control branch from 510305e to fcc4d46 Compare April 9, 2026 05:33
@rpmarvell
Copy link
Copy Markdown
Contributor Author

@tjchadaga : Had some more conflicts/issues , resolved them. could you please merge this!

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