Fixed OSX CPU utilization by replacing broken kevent() with select() #165

Merged
merged 1 commit into from Nov 27, 2016

Conversation

Projects
None yet
2 participants
@micsell
Contributor

micsell commented Jul 9, 2015

Current kevent() implementation in SoftEther is broken. It always exits with -1.
This causes 100% CPU utilization during connected session.
Attached select() implementation of the same functionality.
Now the CPU usage is normal.
Please use (Contribution option (1))

@dnobori

This comment has been minimized.

Show comment
Hide comment
@dnobori

dnobori Sep 6, 2015

Member

Your great patch is much appreciated. We are considering to apply your patch into the SoftEther VPN main tree.

SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch

You have two options which are described on the above policy.
Could you please choose either option 1 or 2, and specify it clearly on the reply?

Member

dnobori commented Sep 6, 2015

Your great patch is much appreciated. We are considering to apply your patch into the SoftEther VPN main tree.

SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch

You have two options which are described on the above policy.
Could you please choose either option 1 or 2, and specify it clearly on the reply?

abhishekmunie added a commit to abhishekmunie/SoftEtherVPN that referenced this pull request Sep 11, 2016

@dnobori dnobori merged commit 697bff4 into SoftEtherVPN:master Nov 27, 2016

@dnobori dnobori referenced this pull request Nov 27, 2016

Closed

Is this project DEAD? #268

@dnobori

This comment has been minimized.

Show comment
Hide comment
@dnobori

dnobori Nov 27, 2016

Member

Thank you so much for your contribution to enrich the SoftEther VPN source code.

Your patch has been merged on the main source-tree of SoftEther VPN.

As a token of our gratitude, your GitHub username has been added on the AUTHORS.TXT file and on the header of the related source file.
Please see: https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT

Thanks again for your contribution.

Member

dnobori commented Nov 27, 2016

Thank you so much for your contribution to enrich the SoftEther VPN source code.

Your patch has been merged on the main source-tree of SoftEther VPN.

As a token of our gratitude, your GitHub username has been added on the AUTHORS.TXT file and on the header of the related source file.
Please see: https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT

Thanks again for your contribution.

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