-
-
Notifications
You must be signed in to change notification settings - Fork 474
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
[BUG] Excessive CPU usage #77
Comments
@Raywando Good morning, and thank you for the report! It looks like this is a target I won't be able to test against to reproduce (judging by your blocking of the domain), so I'd like to ask a few questions:
Looking at your screenshot, I see Edit
I'm not arguing that you saw 100% CPU usage. I also agree that limiting the number of recursive calls with a queue is likely the correct approach to resolving that particular situation. However, I do think you have two things going on at once, and would like to see what happens when we eliminate one of them. If you're willing and able, I'd love to hear what happens when you increase your open file limit and rescan. Thanks again! |
I modified the title of the issue to drive folks here that have the |
Hi @epi052, Thanks for the response. Here is the information you asked for And my machine is Linux - Ubuntu 18.04 Although I only got these errors when my CPU usage was %100 used, so I don't think there is anything else wrong. As for the domain, I can send it to you somehow privately if you want so you can test it yourself since it contains a lot of recursions (Maybe on Twitter. My handle: @Raywando). Thanks again for the efforts! |
I'd recommend upping your soft limit to something like I have been watching the CPU usage during testing and still agree that it could use some tuning. Try raising your soft limit and let me know how it goes. Thanks again! |
Removed |
When the tool created multiple recursive jobs, at one point, it has output a lot of errors and the CPU usage got to 100% at my machine and it was finally killed.
I have even tried to lower the threads count from 50 to 20 but as the recursive jobs increase I think it didn't really matter
The best way in my opinion to avoid this is to add an argument that takes a number to set the most jobs running at once, and add the new jobs to queue.
Other than that, your tool is awesome, thank you for the efforts!
The text was updated successfully, but these errors were encountered: