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

Changing worker count does not work consistently #428

Closed
jawz101 opened this Issue Jun 30, 2016 · 4 comments

Comments

2 participants
@jawz101

jawz101 commented Jun 30, 2016

v4.2.9

Nexus 5. Used to be able to use all 4 cores. Now it defaults to 2 still even if I set to 4. App restarts and still set to 2.

@d4rken

This comment has been minimized.

Owner

d4rken commented Jun 30, 2016

Works for me on a Nexus5.
Try killing SD Maid, possibly the restart is not working correctly.

If that doesn't help lets start with a debug log:
https://github.com/d4rken/sdmaid-public/wiki/Reporting-a-bug#debugrun-log

Note that the setting does not directly relate to how many cores SD Maid uses, just to how many workers may be active in parallel.

@jawz101

This comment has been minimized.

jawz101 commented Jul 1, 2016

I collected a debug log. Can I email this to you? I don't know if any of this is sensitive info

@jawz101

This comment has been minimized.

jawz101 commented Jul 1, 2016

I went ahead and emailed it. I'm on a Nexus 5 as well. Tried using both CyanogenMod 13 and Purity ROM(which is just a CM derivative

@d4rken

This comment has been minimized.

Owner

d4rken commented Jul 1, 2016

There might be an issue with the automatic relaunch. As i think it's unexpected behavior anyways I've replaced it with a message that requests the user to manually restart SD Maid.

(Didn't see any sensitive info in the log)
sdmaid_logfile_1467336315084.txt

@d4rken d4rken closed this Jul 1, 2016

@d4rken d4rken added bug and removed 0 needs info/repro labels Jul 1, 2016

@d4rken d4rken changed the title from 4.2.9 multithreading no longer adjustable to Changing worker count does not work consistently Jul 1, 2016

@d4rken d4rken added c: General UI&UX and removed c: Core labels Jul 17, 2016

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