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

Memory leaks #79

Open
aleksander opened this issue Nov 24, 2016 · 7 comments
Open

Memory leaks #79

aleksander opened this issue Nov 24, 2016 · 7 comments

Comments

@aleksander
Copy link

Arch Linux / AMDGPU-Pro 16.30

At start SilentArmy memory consumption is about 40 Mb.
After 20 hours of mining SilentArmy memory consumption is about 4 Gb.

Clang static analyzer found no problems.
I'll try to find any with valgrind.

@ddobreff
Copy link

Confirmed but only with eXtremal's optimizations.

@aleksander aleksander changed the title Possible memory leaks Memory leaks Nov 25, 2016
@ghost
Copy link

ghost commented Dec 4, 2016

Confirmed on Ubuntu 16.04.1 / AMDGPU-Pro 16.40.

@ghost ghost mentioned this issue Dec 5, 2016
@ghost
Copy link

ghost commented Dec 5, 2016

@aleksander, give this a try: #83

@aleksander
Copy link
Author

@jramos, thank you! testing it right now.

@ghost
Copy link

ghost commented Dec 5, 2016

I let it run overnight on my machine. There's still a slight memory leak somewhere, but it's much better for me now.

@RichAyotte
Copy link

Here's a quick script that whipped up that restarts the miner every hour.

#!/bin/bash

trap ctrl_c INT
function ctrl_c() {
	/usr/bin/pkill silentarmy
	exit
}

while :
do
	/usr/bin/pkill silentarmy
	sleep 5
	/path-to-/silentarmy -c stratum+tcp://your-pool.com:2323 -u t1xxxxxxxxxxxxxxxxxxxxxxxx/xxxxxx0/[email protected] -p "password" --use=0 & 
	sleep 3600
done

@mbevand
Copy link
Owner

mbevand commented Dec 19, 2016

I have committed jramos's pull request #83. I am currently in the middle of a major rewrite. I'll investigate the (other?) leaks later.

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 a pull request may close this issue.

4 participants