Skip to content

Conversation

@prasunsinha1
Copy link
Contributor

In this map, the key is the PG index and the status value (clear-on-read)
for each PG is from {0, 1}, where 0 indicates no drops were observed and 1
indicates packet drops.

Signed-off-by: Prasun Sinha prasunsinha@google.com

In this map, the key is the PG index and the status value (clear-on-read)
for each PG is from {0, 1}, where 0 indicates no drops were observed and 1
indicates packet drops.

Signed-off-by: Prasun Sinha <prasunsinha@google.com>
@prasunsinha1 prasunsinha1 marked this pull request as ready for review December 11, 2025 19:46
@tjchadaga
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@JaiOCP JaiOCP left a comment

Choose a reason for hiding this comment

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

LGTM

@rlhui rlhui added the reviewed PR is discussed in SAI Meeting label Dec 18, 2025
@rlhui
Copy link
Collaborator

rlhui commented Dec 18, 2025

@rajkumar38 please also sign-off as discussed, thanks.

@rlhui rlhui merged commit 563300a into opencomputeproject:master Dec 18, 2025
3 checks passed
JaiOCP added a commit to JaiOCP/SAI that referenced this pull request Dec 19, 2025
Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG Port and Switch Attributes

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG ACL Attributes

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG TAM Attributes

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG Time Interval Unit

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG Document

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG Document

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

CSIG Document

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

Fix: "INVAILD" typo in enum '_sai_dash_flow_entry_bulk_get_session_filter_key_t' (opencomputeproject#2224)

This PR fixes a typo in the enum definition for dash_flow_entry_bulk_get_session_filter_key_t where the member INVAILD is corrected to INVALID.

Changes:

saitypesextensions.h: Renamed enum member INVAILD → INVALID.
Following will be pushed to DASH's repo once these SAI repo changes are merged.

dash_headers.p4: Renamed enum member INVAILD → INVALID.

Non UEC Member Agreement Form (opencomputeproject#2226)

Signed-off-by: JaiOCP <jai.kumar@broadcom.com>

Seamless BFD (S-BFD) (opencomputeproject#2220)

Signed-off-by: Jason Bos <jbos@cisco.com>

Replicate VXLAN UDP sport security to switch level (opencomputeproject#2195)

- Add SAI_SWITCH_TUNNEL_ATTR_VXLAN_UDP_SPORT_SECURITY attribute to drop tunnel packets with UDP source port outside allowed range
- Add documentation for UDP source port validation on VXLAN packet ingress
- Update both saiswitch.h and saitunnel.h with security clarifications

Signed-off-by: Marian Pritsak <marianp@mellanox.com>

SRv6 SID Marking (opencomputeproject#2222)

* SID Marking APIs

Signed-off-by: Jason Bos <jbos@cisco.com>

[DASH] Add flow bulk get session event data struct (opencomputeproject#2175)

* [DASH] Add flow bulk get session event data struct

Add flow bulk get event callback and supporting
structures to provide ability to query flows
in asynchronous manner.

Introducing SAI_PORT_ATTR_PORT_PG_PKT_DROP_STATUS (opencomputeproject#2234)

In this map, the key is the PG index and the status value (clear-on-read)
for each PG is from {0, 1}, where 0 indicates no drops were observed and 1
indicates packet drops.

Signed-off-by: Prasun Sinha <prasunsinha@google.com>

Add flow entry to the bulk session event data (opencomputeproject#2237)

Add the ability to query the flows with the
bulk get session event data callback.

The commit contains a fix for bad ordering of
classes generated by Thrift compiler for the nested
structures used in this callback.

Signed-off-by: Marian Pritsak <marianp@mellanox.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