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

can pcapp support soft-roce rdma packet header parse? #1569

Open
FlashBarryAllen opened this issue Sep 8, 2024 · 6 comments · May be fixed by #1603
Open

can pcapp support soft-roce rdma packet header parse? #1569

FlashBarryAllen opened this issue Sep 8, 2024 · 6 comments · May be fixed by #1603
Labels

Comments

@FlashBarryAllen
Copy link

Question

I found that our pcapp support 7 layer. However, rdma is now populate. And I wonder if we will develop rdma parse and edit in L4 or L5 layer.

Operating systems

Linux

@tigercosmos
Copy link
Collaborator

@FlashBarryAllen Seems that we don't have that implementation yet. It would be nice if you can help add the RMDA support in the library.

@seladb
Copy link
Owner

seladb commented Sep 8, 2024

@FlashBarryAllen are you referring to this protocol? https://en.wikipedia.org/wiki/RDMA_over_Converged_Ethernet

@FlashBarryAllen
Copy link
Author

@seladb yes, that's what I want.

@FlashBarryAllen
Copy link
Author

@tigercosmos I just found it on linux-kernel, and read the soft-roce. If I can, I will implement it by myself.

@FlashBarryAllen
Copy link
Author

FlashBarryAllen commented Sep 10, 2024

@tigercosmos I have try code the infinibandlayer myself. And I will complete the left once I have time.

https://github.com/FlashBarryAllen/PcapPlusPlus/commit/64cbd6197d6c71e223ac399c252be508b63ade73
https://github.com/seladb/PcapPlusPlus/commit/9f6016fbafb9b25374b45c64c2cfc21f0d942002

@tigercosmos
Copy link
Collaborator

@FlashBarryAllen I took a quick look. It looks nice but may need more discussion on the design. Feel free to open PR after you finish the code and do the cleanup.

@egecetin egecetin linked a pull request Nov 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants