commit 646e001bb5e232f38a79cc5ac3d9b4ed9c74ac65
author Jooyung Han <jooyung@google.com> Mon Aug 21 17:34:37 2023 +0900
committer Jooyung Han <jooyung@google.com> Mon Aug 21 18:37:23 2023 +0900
tree 1c78c05e69b310de609e3407efd8b2377c7dc867
parent d1e04d2123694480426703ed97399c0102109fd4

init: clean up DelayService()

ServiceList's services_update_finished flag was overlapped with the
global flag: is_default_mount_namespace_ready. Now DelayService() relies
on the is_default_mount_namespace_ready flag.

Add a service description with 'updatable' flag and invoke 'start
<name>' in 'on init' block (which comes before APEX activation).
See the log for "Cannot start an updatable service".

Bug: 293535323
Test: see the comment
Change-Id: I9341ba1a95d9b3b7c6081b530850d61f105f0a56

init/builtins.cpp

diff --git a/init/builtins.cpp b/init/builtins.cpp
index a5b762c..7715424 100644
--- a/init/builtins.cpp
+++ b/init/builtins.cpp
@@ -1304,8 +1304,7 @@
     }
 
     if (!bootstrap) {
-        // Now start delayed services
-        ServiceList::GetInstance().MarkServicesUpdate();
+        ServiceList::GetInstance().StartDelayedServices();
     }
     return {};
 }