commit 855dee1102bf6edcd913b646495a6a095201264b
author Jiyong Park <jiyong@google.com> Mon Mar 17 13:01:29 2025 +0900
committer Jiyong Park <jiyong@google.com> Mon Mar 17 13:02:38 2025 +0900
tree e160ff81847f0e9259a5fc451f0f9927cf5ccb35
parent 0dc26b7d95bd33301d1186e89ee9722911d0be4e

Run start and stop in a worker thread

Bug: 401835074
Test: run, rerun, and resize
Change-Id: I71e63b96d537038faacdd120f75e928647ae06bd

android/TerminalApp/java/com/android/virtualization/terminal/VmLauncherService.kt

diff --git a/android/TerminalApp/java/com/android/virtualization/terminal/VmLauncherService.kt b/android/TerminalApp/java/com/android/virtualization/terminal/VmLauncherService.kt
index 7ec1115..052916f 100644
--- a/android/TerminalApp/java/com/android/virtualization/terminal/VmLauncherService.kt
+++ b/android/TerminalApp/java/com/android/virtualization/terminal/VmLauncherService.kt
@@ -39,6 +39,7 @@
 import android.system.virtualmachine.VirtualMachineException
 import android.util.Log
 import android.widget.Toast
+import androidx.annotation.WorkerThread
 import com.android.system.virtualmachine.flags.Flags
 import com.android.virtualization.terminal.MainActivity.Companion.TAG
 import com.android.virtualization.terminal.Runner.Companion.create
@@ -116,9 +117,11 @@
                 // done.
                 val diskSize = intent.getLongExtra(EXTRA_DISK_SIZE, image.getSize())
 
-                doStart(notification, displayInfo, diskSize, resultReceiver)
+                executorService.submit({
+                    doStart(notification, displayInfo, diskSize, resultReceiver)
+                })
             }
-            ACTION_SHUTDOWN_VM -> doShutdown(resultReceiver)
+            ACTION_SHUTDOWN_VM -> executorService.submit({ doShutdown(resultReceiver) })
             else -> {
                 Log.e(TAG, "Unknown command " + intent.action)
                 stopSelf()
@@ -128,6 +131,7 @@
         return START_NOT_STICKY
     }
 
+    @WorkerThread
     private fun doStart(
         notification: Notification,
         displayInfo: DisplayInfo,
@@ -384,6 +388,7 @@
         }
     }
 
+    @WorkerThread
     private fun doShutdown(resultReceiver: ResultReceiver) {
         if (debianService != null && debianService!!.shutdownDebian()) {
             // During shutdown, change the notification content to indicate that it's closing