Skip to content

ICMP Hardware offload session stats support#2354

Merged
prsunny merged 1 commit into
sonic-net:masterfrom
manamand2020:icmp-stats
Jun 1, 2026
Merged

ICMP Hardware offload session stats support#2354
prsunny merged 1 commit into
sonic-net:masterfrom
manamand2020:icmp-stats

Conversation

@manamand2020
Copy link
Copy Markdown
Contributor

@manamand2020 manamand2020 commented May 28, 2026

Added sections on supporting stats for ICMP hardware offload sessions. Following CLI support added using flexcounter:

$ sudo counterpoll icmp enable/disable
$ sudo counterpoll icmp interval 5000
$ show icmp stats
$ sonic-clear icmp counters

Support both traditional SAI stats API based and selective counters models. Selective counter is preferred over traditional SAI stats API.

Related PRs and specs:

Repo PR title State
sonic-swss Added support for ICMP-echo stats GitHub issue/pull request detail
sonic-swss-common Added ICMP-echo stats for flexcounter support GitHub issue/pull request detail
sonic-sairedis Changes for supporting ICMP echo stats GitHub issue/pull request detail
sonic-utilities Support for ICMP-echo stats GitHub issue/pull request detail

Signed-off-by: Manas Kumar Mandal <manamand@cisco.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

@prsunny prsunny merged commit 106bb28 into sonic-net:master Jun 1, 2026
2 checks passed
a114j0y pushed a commit to a114j0y/SONiC that referenced this pull request Jun 5, 2026
Added sections on supporting stats for ICMP hardware offload sessions. Following CLI support added using flexcounter:

$ sudo counterpoll icmp enable/disable
$ sudo counterpoll icmp interval 5000
$ show icmp stats
$ sonic-clear icmp counters

Support both traditional SAI stats API based and selective counters models. Selective counter is preferred over traditional SAI stats API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants