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

Test on and support additional NICs #51

Open
thearossman opened this issue Sep 5, 2024 · 3 comments
Open

Test on and support additional NICs #51

thearossman opened this issue Sep 5, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@thearossman
Copy link
Collaborator

Original development was on Mellanox ConnectX-5. #40 adds support for Intel X710. We'd like to test Retina compilation/functionality on a few more common NICs.

@thearossman thearossman added the enhancement New feature or request label Sep 5, 2024
@thearossman thearossman changed the title [Issue tracking] Test on and support additional NICs Test on and support additional NICs Sep 5, 2024
@sippejw
Copy link
Contributor

sippejw commented Sep 6, 2024

I've recently shifted to a box using the Intel E810 and can confirm that #40/#54 also work on this NIC.

@sippejw
Copy link
Contributor

sippejw commented Sep 6, 2024

Removing the default flag for Mellanox ConnectX-5 in core/Cargo.toml would improve out of the box compilation. When the mlx5 dependencies aren't installed, this flag must be manually removed in order to get Retina to build. This was previously referenced in #1.

default = ["mlx5"]

@thearossman
Copy link
Collaborator Author

This is great!! @sippejw could we add the server configurations you tested this on to the INSTALL.md documentation? (What CPU/OS were you using?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants