Enable background OS threads in TaskScheduler on Android |
||||||
Issue descriptionCurrently all task scheduler threads run with NORMAL thread priority on Android. One of the reasons is because PlatformThread::CanIncreaseCurrentThreadPriority() returns false. But Android is unique because Chrome processes are never shut down - they are killed, either by Chrome or by Android itself. Given that, can we #ifndef ANDROID portion of GetDesiredThreadPriority() code that deals with shutdown (and checks for CanIncreaseCurrentThreadPriority)? Additionally, what was the reason for adding HandlesMultipleThreadPriorities() check - is it also related to shutdown?
,
Feb 12 2018
This is worth resolving now.
,
Apr 12 2018
,
May 3 2018
Need a runtime check to only to do this after the Android version supporting it (ref. http://b/29177606). Need to remove the requirement to be able to up priority on shutdown since Android doesn't shutdown.
,
Oct 3
,
Oct 18
lizeb@ mentioned that we should be careful here as there's a level of backgrounding on Android that puts you in a cgroup that implies a single thread (precise nice value can be found in Android source he mentioned) |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by dskiba@chromium.org
, Jan 18 2018