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

Fix NET_Sleep_Timeout (-pingboost 3) #391

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

In-line
Copy link
Collaborator

@In-line In-line commented Mar 7, 2017

  1. NET_Sleep_Timeout allows server to wakeup earlier, if some data arrived to the one of the used sockets. This PR allows to put limit on that. Sometimes server can accelerate 2, even 4 times and more. At some point this can be used to harm server (hosters usually limit CPU usage) without further benefit.
  2. I added some safety checks and corrected condition (Same as Fix tv_usec calculation in NET_Sleep_Timeout #403)​, which allows server calculate delay more accurately
  3. I removed some strange code that was responsible for the limit like in 1, but every (fps/100) + 1 frame.

@In-line
Copy link
Collaborator Author

In-line commented Mar 7, 2017

It'll be cool, if somebody with better English will write description for REAMDE.md :)

Here's a draft:
sv_rehlds_max_accelerated_frames // Max server acceleration in FPS for pingboost -3, negative value disables check. Default: -1

@theAsmodai
Copy link
Collaborator

theAsmodai commented Mar 7, 2017

I have a few other thoughts on this matter:
2a1f674

@In-line
Copy link
Collaborator Author

In-line commented Mar 7, 2017

@theAsmodai What is cons of my approach?

@hajimura
Copy link

Ping

@In-line
Copy link
Collaborator Author

In-line commented Dec 14, 2017 via email

@maxpain
Copy link

maxpain commented Aug 3, 2020

Any chance to merge this?

@UnrealKaraulov
Copy link

Any chance to merge this?

No any chance, this rehlds project very inactively very long time :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants