From 3f3fa5618bea2e2f5ba436506272ca013951144f Mon Sep 17 00:00:00 2001 From: Deepak Kodihalli Date: Wed, 25 Oct 2017 04:18:49 -0500 Subject: Check only CPU presence before reconstructing OCCs OCC d-bus objects are typically created in response to the CPU inventory being created. At this point of time we just know the CPU is present. It may not be functional. Of course if the CPU is not functional, then the host may or may not choose to set the OCC active, but the OCC object must have been created in response to a present CPU. This commit employs the same logic for reconstructing OCC objects when the OCC app restarts. Previously an OCC object would be created only if corresponding CPU is present and functional. Instead, check only for presence. Change-Id: I4f85a644237b5b5aa7dc3e52d266375b786a8c0a Signed-off-by: Deepak Kodihalli --- occ_finder.cpp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/occ_finder.cpp b/occ_finder.cpp index b931452..0c02c35 100644 --- a/occ_finder.cpp +++ b/occ_finder.cpp @@ -114,12 +114,7 @@ std::vector get(sdbusplus::bus::bus& bus) "Present", true)); - match.emplace_back(std::make_tuple( - "xyz.openbmc_project.State.Decorator.OperationalStatus", - "Functional", - true)); - - // Select only if the CPU is marked 'Present' and 'Functional' + // Select only if the CPU is marked 'Present'. // Local variable to make it readable auto path = entry->first; auto service = entry->second.begin()->first; -- cgit v1.2.1