Skip to content

Add buffer pool watermark support in watermarkstat, counterpoll, and show #521

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jun 4, 2019

Conversation

wendani
Copy link
Contributor

@wendani wendani commented Apr 23, 2019

$ watermarkstat -t buffer_pool -p
Shared pool maximum occupancy:
Pool Bytes


egress_lossless_pool 570336
egress_lossy_pool 2054624
ingress_lossless_pool 1664

$ counterpoll watermark enable

$ counterpoll watermark interval 20000

Apr 23 17:49:44.686196
Apr 23 17:50:04.686427
Apr 23 17:50:24.686704

$ counterpoll watermark disable

$ counterpoll show
Type Interval (in ms) Status


QUEUE_STAT default (10000) enable
PORT_STAT default (1000) enable
QUEUE_WATERMARK_STAT 20000 disable
PG_WATERMARK_STAT 20000 disable
BUFFER_POOL_WATERMARK_STAT 20000 disable

- What I did

- How I did it

- How to verify it
tested on brcm dut
- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

-->

@wendani wendani requested review from qiluo-msft and lguohan April 23, 2019 17:55
@wendani
Copy link
Contributor Author

wendani commented Apr 23, 2019

@mykolaf for review

@wendani wendani requested a review from qiluo-msft May 31, 2019 21:47
@wendani
Copy link
Contributor Author

wendani commented Jun 4, 2019

retest this please

1 similar comment
@stcheng
Copy link
Contributor

stcheng commented Jun 4, 2019

retest this please

@wendani wendani merged commit b7fcb1f into sonic-net:master Jun 4, 2019
@wendani wendani changed the title Add buffer pool watermark support in watermarkstat and counterpoll Add buffer pool watermark support in watermarkstat, counterpoll, and show Jun 4, 2019
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.

5 participants