SAI icon indicating copy to clipboard operation
SAI copied to clipboard

Add new enum to support 8 lanes breakout mode

Open rajkumar38 opened this issue 9 months ago • 3 comments

Add new breakout mode enum to "sai_port_breakout_mode_type_t",

/** 8 lanes breakout Mode */ SAI_PORT_BREAKOUT_MODE_TYPE_8_LANE = 3,

rajkumar38 avatar May 02 '24 05:05 rajkumar38

@kcudnik Can you suggest how to fix this WARNING ? I'm excluding "SAI_PORT_BREAKOUT_MODE_TYPE_MAX " in ancestry.pl.

WARNING: Both enums have the same value SAI_PORT_BREAKOUT_MODE_TYPE_8_LANE and SAI_PORT_BREAKOUT_MODE_TYPE_MAX = 0x00000003

rajkumar38 avatar May 02 '24 06:05 rajkumar38

You need to add exception for max value in checking script

kcudnik avatar May 02 '24 11:05 kcudnik

You need to add exception for max value in checking script

Pls review.

rajkumar38 avatar May 02 '24 15:05 rajkumar38

Discussed in SAI meeting on 5/9, @prgeor , @JaiOCP - please help review/sign off

tjchadaga avatar May 09 '24 15:05 tjchadaga

@JaiOCP, Regarding your question on standards. There are different IEEE 802.3(https://en.wikipedia.org/wiki/Terabit_Ethernet) standards for defining Terabit Ethernet with different speeds. This captures the max lanes and speed per lane but doesn't explicitly talk about break-out. Its up-to vendor capability.

rajkumar38 avatar May 10 '24 05:05 rajkumar38

/azp run

kcudnik avatar May 27 '24 11:05 kcudnik

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar May 27 '24 11:05 azure-pipelines[bot]