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

Add multi-CPU Affinity option support for Linux (not for FreeBSD) #1778

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

Conversation

davidBar-On
Copy link
Contributor

Suggested support for multi-CPU Affinity option on Linux. The PR is only for Linux, since I don't have FreeBSD machine.

The format of a CPUs list is using / as a separator ("n1/n1/...") since , is already used to separate between the client and server CPUs lists.

The fix is backward compatible, passing the CPUs list to the server in a new "server_cpu_set" JSON item and passing the first CPU defined in the server's list in the legacy "server_affinity" item.

@davidBar-On davidBar-On changed the title Add mutli-CPU Affinity option support for Linux (not for FreeBSD) Add multi-CPU Affinity option support for Linux (not for FreeBSD) Jan 5, 2025
@junousi
Copy link

junousi commented Mar 6, 2025

[To improve SNR, question removed.]

@davidBar-On
Copy link
Contributor Author

Hello @junousi, it is usually better to handle such discussions in iperf3 "discussion" (see "Discussions" at the top line). However, it this case I think that it is better to handle this discussion in other forums, probably a forum about NUMA (I am not familiar with this topic).

One suggestion about iperf3 is to make sure that the different 8 threads per stream are running on different CPU. E.g. this can be done by running the iperf3 test for long enough time (using the -t option) and in parallel running htop command to see the load of each CPU.

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