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

faq/tcp: add an entry related to 10GbE networks and faster #23

Closed
wants to merge 482 commits into from
Closed

faq/tcp: add an entry related to 10GbE networks and faster #23

wants to merge 482 commits into from

Conversation

ggouaillardet
Copy link
Contributor

  • set send and receive buffers size to zero
  • try multiple links

Signed-off-by: Gilles Gouaillardet gilles@rist.or.jp

Ralph Castain and others added 30 commits July 16, 2015 09:12
The HTTP server error log shows that some people are still going to
nightly/trunk.  So let's be nice and redirect them to nightly/master.
There have been several man pages fixes since v1.8.4 (which is the
version of man pages currently on the web site).  This commit updates
all the man pages to those that are contained in the v1.8.8 tarball.
Put the comment in a PHP block so that it is not sent back to the client.
Ralph Castain and others added 14 commits September 1, 2016 13:39
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
This list archive is empty.  Rename the directory so that it effectively
won't be found, and won't be used as a trigger to create the
"archives" link in the php.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
@ggouaillardet
Copy link
Contributor Author

@bwbarrett @bosilca can you please double check this ?

this is related to https://www.mail-archive.com/users@lists.open-mpi.org/msg31283.html

if there are some recommended settings for AWS, they can be added and/or linked too

@bgoglin
Copy link
Contributor

bgoglin commented Jul 10, 2017

I don't understand "Open MPI < v2.0.3 or > v2.1.2".

@ggouaillardet
Copy link
Contributor Author

oops, that is a pretty dumb typo i will fix right away

@bosilca
Copy link
Member

bosilca commented Jul 10, 2017

I would rather be more specific, and not tie this capability to a specific version of OMPI, but instead talk about the real reason why this settings are not necessary anymore. We should say that in modern version of the Linux kernel, setting the TCP send and receive buffers is a unnecessary operation that can lead to inefficiencies. We suggest users to check their TCP buffer sizes using ompi_info --param btl tcp -l 4 and make sure that btl_tcp_sndbuf and btl_tcp_rcvbuf are set to 0. Starting from Open MPI 2.0.3 this are the default settings.

@bwbarrett
Copy link
Member

I agree with George. Also, the btl_num_links option doesn't work correctly. We need to sort that out before we suggest it in a FAQ.

…lti links

- set send and receive buffers size to zero
- warn about the currently non working multiple links option

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
@ggouaillardet
Copy link
Contributor Author

i updated this PR, please double check the wording

@gpaulsen
Copy link
Member

@ggouaillardet, @bwbarrett @bosilca: Is this still relevant? Should we pull it in, or are we still waiting on some fixes?

@bwbarrett
Copy link
Member

We need to test, because I think multilink works in 4.0. Either way, I think adding the multilinks answer is a really bad look for Open MPI.

@bosilca
Copy link
Member

bosilca commented Sep 24, 2018

We need closure on this topic. I just added this topic to the developer's meeting agenda.

@bwbarrett bwbarrett force-pushed the master branch 2 times, most recently from 6a06abf to f8d32ff Compare October 7, 2018 04:44
@gpaulsen gpaulsen added the ⚠️ WIP-DNM! Work in Progress. Do NOT Merge! label Oct 16, 2018
@bwbarrett
Copy link
Member

Closing in favor of #111

@bwbarrett bwbarrett closed this Oct 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ WIP-DNM! Work in Progress. Do NOT Merge!
Projects
None yet
Development

Successfully merging this pull request may close these issues.