Skip to content

add -I option for watch peak traffic, add --phy option for RDMA#11

Open
kirbyzhou wants to merge 5 commits into
rolandriegel:masterfrom
kirbyzhou:master
Open

add -I option for watch peak traffic, add --phy option for RDMA#11
kirbyzhou wants to merge 5 commits into
rolandriegel:masterfrom
kirbyzhou:master

Conversation

@kirbyzhou

Copy link
Copy Markdown

add -I option, show traffic stats of each interval instead of one second.
Which is useful for watch peak traffic.

add --phy option, show traffic stats of phy(nic) rx/tx bytes instead of rx/tx bytes.
Which is useful for RDMA card.

@llamafilm

llamafilm commented Mar 24, 2023

Copy link
Copy Markdown

Thank you for adding this! I was looking for a way to monitor RDMA traffic and this is working well. I'm using Mellanox NIC on Rocky 9.1 with inbox driver.
I hope cockpit adds support for this too.

@kirbyzhou

kirbyzhou commented Apr 7, 2023

Copy link
Copy Markdown
Author

Can you share your ib/net device name?
I want to verify some problem.

@llamafilm

llamafilm commented Apr 7, 2023

Copy link
Copy Markdown

Sure, it's called mlx5_0 according to ibv_devinfo. Is that what you meant? Or enp197s0f0np0 in ip.

@kirbyzhou

Copy link
Copy Markdown
Author

Sure, it's called mlx5_0 according to ibv_devinfo. Is that what you meant? Or enp197s0f0np0 in ip.

Thanks.
And can you try --list-props with your ib device to list all your property names?

@llamafilm

Copy link
Copy Markdown

Sorry, I don’t know what command you’re talking about.

@kirbyzhou

kirbyzhou commented Apr 12, 2023 via email

Copy link
Copy Markdown
Author

@llamafilm

Copy link
Copy Markdown

Ah ok here ya go. This is a dual-port Mellanox plus onboard NICs.
nload-props.txt

@Low-power

Copy link
Copy Markdown

Interesting to see this, because I have a similar implementation (for infiniband) that I did years ago in version 0.7.4.
I recently applied the changes into git and pushed it here in a separated branch for references:
https://github.com/Low-power/nload/tree/rivoreo-fork-infiniband

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.

3 participants