From 5d101264ec7a765ebffa8d15e7b4d33c1c2bea3c Mon Sep 17 00:00:00 2001 From: Sudheer Shanka Date: Fri, 15 Jan 2021 03:13:50 -0800 Subject: [PATCH] Update Expedited jobs to use IMP_FG proc_state. Removing BIND_FOREGROUND_SERVICE while binding to the job service which would otherwise elevate it to BFGS proc_state. Bug: 177641226 Test: atest CtsJobSchedulerTestCases Test: atest CtsHostsideNetworkTests:HostsideRestrictBackgroundNetworkTests Change-Id: I51756d0c22e141b8d6f0bd91784da6fd87a69168 --- .../java/com/android/server/job/JobServiceContext.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java b/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java index a9ca7308b25f9..26b5abed745cd 100644 --- a/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java +++ b/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java @@ -260,11 +260,9 @@ public final class JobServiceContext implements ServiceConnection { try { final int bindFlags; if (job.shouldTreatAsExpeditedJob()) { - // Add BIND_FOREGROUND_SERVICE to make it BFGS. Without it, it'll be - // PROCESS_STATE_IMPORTANT_FOREGROUND. Unclear which is better here. // TODO(171305774): The job should run on the little cores. We'll probably need // another binding flag for that. - bindFlags = Context.BIND_AUTO_CREATE | Context.BIND_FOREGROUND_SERVICE; + bindFlags = Context.BIND_AUTO_CREATE; } else { bindFlags = Context.BIND_AUTO_CREATE | Context.BIND_NOT_FOREGROUND | Context.BIND_NOT_PERCEPTIBLE;