Skip to content

[action] [PR:14743] Fix sai_thrift_read_port_counters for DNX#14867

Merged
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/14743
Oct 8, 2024
Merged

[action] [PR:14743] Fix sai_thrift_read_port_counters for DNX#14867
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/14743

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

SAI thrift call sai_thrift_read_port_counters() reads a collection of counters together, including SAI_PORT_STAT_IN_DROPPED_PKTS. Starting from BRCM SAI 11.x, SAI_PORT_STAT_IN_DROPPED_PKTS is not supported and retrieving the count may fail SAI call sonic-net/sonic-buildimage#19998. In order to retrieve other counters correctly, read SAI_PORT_STAT_IN_DROPPED_PKTS in a separate SAI call.

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

…_PKTS starting from SAI 11.x. (sonic-net#14743)

SAI thrift call sai_thrift_read_port_counters() reads a collection of counters together, including SAI_PORT_STAT_IN_DROPPED_PKTS. Starting from BRCM SAI 11.x, SAI_PORT_STAT_IN_DROPPED_PKTS is not supported and retrieving the count may fail SAI call sonic-net/sonic-buildimage#19998. In order to retrieve other counters correctly, read SAI_PORT_STAT_IN_DROPPED_PKTS in a separate SAI call.
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #14743

@mssonicbld mssonicbld merged commit 2f40332 into sonic-net:202405 Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants