From 137cb973fb0eb203f47e50adce03baa4ff06a21b Mon Sep 17 00:00:00 2001 From: Nicholas Verne Date: Tue, 18 Sep 2018 11:04:33 +1000 Subject: Add VmStatus to StartVmResponse. When this status indicates the VM is already running, CrostiniManager can avoid waiting forever for a signal that Tremplin has started in the VM. Bug: crbug:884576 Change-Id: I7c4577a4fa5d0bcba2c158a046ad31e81236b421 Reviewed-on: https://chromium-review.googlesource.com/1226733 Commit-Ready: Nicholas Verne Tested-by: Nicholas Verne Reviewed-by: Jeffrey Kardatzke --- dbus/vm_concierge/service.proto | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/dbus/vm_concierge/service.proto b/dbus/vm_concierge/service.proto index 6d3aac1..c3dc590 100644 --- a/dbus/vm_concierge/service.proto +++ b/dbus/vm_concierge/service.proto @@ -103,6 +103,20 @@ message StartVmRequest { string owner_id = 7; } +enum VmStatus { + // Unknown status. + VM_STATUS_UNKNOWN = 0; + + // The VM is already running. + VM_STATUS_RUNNING = 1; + + // The VM is currently starting up. + VM_STATUS_STARTING = 2; + + // The VM failed to startup. + VM_STATUS_FAILURE = 3; +} + // Information sent back by vm_concierge in response to a StartVm dbus message. message StartVmResponse { // If true, the VM was started successfully. |vm_info| will have non-default @@ -114,6 +128,10 @@ message StartVmResponse { // Information about the VM that was started, if successful. VmInfo vm_info = 3; + + // Status of the VM. If VM_STATUS_RUNNING, it is not necessary to wait for a + // TremplinStartedSignal before starting a container in the VM. + VmStatus status = 4; } // Information that must be included with every StopVm dbus message. -- cgit v1.2.3