-
Notifications
You must be signed in to change notification settings - Fork 135
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
[PATCH v5] api: tm: add node stats capability and query API #1325
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments to sync this PR with #1305.
4c538be
to
8566ac8
Compare
8566ac8
to
0bf45f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except for some minor style tweaks and missing copyright the API changes are looking OK to me.
0bf45f1
to
f663b54
Compare
Add node per level capability to provide statistics. Also add a stats query API for getting supported node statistics. Signed-off-by: Nithin Dabilpuram <[email protected]>
Add node stats API stub. Signed-off-by: Nithin Dabilpuram <[email protected]>
Add node stats test case similar to queue stats testcase. Signed-off-by: Nithin Dabilpuram <[email protected]>
f663b54
to
f1730f2
Compare
Hi @nithind1988, Are we still expecting changes to this PR or is it ready from your point of view? |
api: tm: add node stats capability and query API
Add capability to advertise capability of nodes at a given level
to provide statistics.
Also add a new API to query for packet sent count in green, yellow
and red color and packet drop count stats, along with corresponding
octet count statistics. TM node stats query will return only platform
supported statistics.
Signed-off-by: Nithin Dabilpuram [email protected]
linux-gen: tm: add node stats API stub
Add node stats API stub.
Signed-off-by: Nithin Dabilpuram [email protected]