Skip to content
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

Added ibtool roce_bw #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added ibtool roce_bw #15

wants to merge 1 commit into from

Conversation

EhudB
Copy link

@EhudB EhudB commented Dec 3, 2016

This is a total hack for using the mellanos RoCE, this was done in less then 12 hours in the mellanox hackatop, with help of guys from mellanox.
A big note is i had no idea of rdma before starting working on it, if there seems to be a problem it's probably because of this.

I'm sure that the team developing this library could do better work on conventions then me.
This was tested in the hackatop and should work, i currently have no servers for testing it again.

Thanks for Eden, Alon, Yossi my team mates for not letting me give up and cheering me.
Thanks for Dotan and other mellanox workers for helping me a lot on fixing this test!

This is a total hack for using the mellanos RoCE, this was done in less then 12 hours with help of guys from mellanox.
A big note is i had no idea of rdma before starting working on it, if there seems to be a problem it's probably becuase of this.

Thanks for Eden, Alon, Yossi for not letting me give up.
Thanks for Dotan and other mellanox workers for helping me alot on fixing this test!
@jgunthorpe
Copy link
Owner

It is very interesting to see that this is even possible, but to go forward I think a little more finesse would be needed.

There is room in the API for different kinds of paths, rocee should probably have a ROCEE1Path and ROCEE2Path class - from that basis it should be easier to see how to do this without introducing a copy of the rdma_bw command.

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.

2 participants