Skip to content

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