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

Redis status and statistics database connection broken would cause 99% cpu usage #12

Closed
GoogleCodeExporter opened this issue Jul 17, 2015 · 5 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Start a redis-server in localhost, as turn-server's user-statsdb:
./redis-server &

2. Start turn-server with statsdb enabled:
./turnserver -p 10001 -L 10.10.10.15 -a -f -v --no-cli -r localhost -O 
port=6379 

3. Kill or restart the redis-server started in "step 1":
kill

4. run top command, watch the cpu usage of turn-server:
top

What is the expected output? What do you see instead?
Expected output:
Turn server runs normally, and cpu usage is low.

Real output: 
The cpu usage is over 99% per thread:

  PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND                                                                                                                         
30877 xxxxxxxx  20   0  696m  18m 2424 R 99.4  0.5   0:22.79 turnserver         


30876 xxxxxxxx  20   0  696m  18m 2424 R 99.1  0.5   0:22.80 turnserver         


30878 xxxxxxxx  20   0  696m  18m 2424 R 99.1  0.5   0:22.72 turnserver 


What version of the product are you using? On what operating system?
Version: 
Version Coturn-4.4.5.1 'Ardee West'

Operating system:
Linux tmpserver 3.2.0-23-generic #36-Ubuntu SMP Tue Apr 10 20:39:51 UTC 2012 
x86_64 x86_64 x86_64 GNU/Linux

Please provide any additional information below.


Original issue reported on code.google.com by huxinb...@gmail.com on 4 May 2015 at 1:43

@GoogleCodeExporter
Copy link
Author

[deleted comment]

@GoogleCodeExporter
Copy link
Author

A possible patch for this problem.

Original comment by huxinb...@gmail.com on 4 May 2015 at 2:07

Attachments:

@GoogleCodeExporter
Copy link
Author

Original comment by mom040...@gmail.com on 27 May 2015 at 6:09

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

The fix will be available in 4.4.5.2. Thanks for the patch.

Original comment by mom040...@gmail.com on 28 May 2015 at 6:28

@GoogleCodeExporter
Copy link
Author

fixed in 4.4.5.2

Original comment by mom040...@gmail.com on 7 Jun 2015 at 7:52

  • Changed state: Fixed

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

No branches or pull requests

1 participant