commit 57c0a893cd5cb987633d278e09f941f7899a81b0
author Andrew Geissler <geissonator@yahoo.com> Wed May 03 14:27:33 2023 -0600
committer Andrew Geissler <geissonator@yahoo.com> Wed May 10 13:19:12 2023 +0000
tree 32dd2abce4d11e54d48e42b1f3dce84b7c431ad7
parent 9e5cd389c3d59c0e23d321a1f42a4353e354d556

hypervisor: remove OSStart from state association

Turns out the hypervisor never utilized the OSStart BootProgress.
Remove it from our mapping to hypervisor state.

Our hostboot BIOS team is actually looking to utilize this
BootProgress to indicate when they start the hypervisor. This would just
be a standard BootProgress that PSM needs no special handling of.

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I99a7110de47943a98a35f0e8bf2516a27bb7e6fa

hypervisor_state_manager.cpp

diff --git a/hypervisor_state_manager.cpp b/hypervisor_state_manager.cpp
index d158818..da4d37f 100644
--- a/hypervisor_state_manager.cpp
+++ b/hypervisor_state_manager.cpp
@@ -69,11 +69,6 @@
         currentHostState(server::Host::HostState::Standby);
     }
     else if (bootProgress == "xyz.openbmc_project.State.Boot.Progress."
-                             "ProgressStages.OSStart")
-    {
-        currentHostState(server::Host::HostState::TransitioningToRunning);
-    }
-    else if (bootProgress == "xyz.openbmc_project.State.Boot.Progress."
                              "ProgressStages.OSRunning")
     {
         currentHostState(server::Host::HostState::Running);

test/hypervisor_state.cpp

diff --git a/test/hypervisor_state.cpp b/test/hypervisor_state.cpp
index 9136c83..b423f2e 100644
--- a/test/hypervisor_state.cpp
+++ b/test/hypervisor_state.cpp
@@ -27,12 +27,6 @@
     EXPECT_EQ(hypObj.currentHostState(), server::Host::HostState::Standby);
 
     bootProgress = "xyz.openbmc_project.State.Boot.Progress."
-                   "ProgressStages.OSStart";
-    hypObj.updateCurrentHostState(bootProgress);
-    EXPECT_EQ(hypObj.currentHostState(),
-              server::Host::HostState::TransitioningToRunning);
-
-    bootProgress = "xyz.openbmc_project.State.Boot.Progress."
                    "ProgressStages.OSRunning";
     hypObj.updateCurrentHostState(bootProgress);
     EXPECT_EQ(hypObj.currentHostState(), server::Host::HostState::Running);