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
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
Collaborator

/azp run

@azure-pipelines
Copy link

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
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
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
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
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).
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

3 participants