From a81ded1880f0c7f5f4cdb2f2fd775226a15516d7 Mon Sep 17 00:00:00 2001 From: AShivangi <124935354+AShivangi@users.noreply.github.com> Date: Tue, 10 Dec 2024 09:07:47 -0800 Subject: [PATCH] CR-1219475 and CR-1220306 (#8645) Signed-off-by: AShivangi --- .../core/tools/common/tests/TestTemporalSharingOvd.cpp | 6 +++--- src/runtime_src/core/tools/xbutil2/OO_Performance.cpp | 8 +------- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/src/runtime_src/core/tools/common/tests/TestTemporalSharingOvd.cpp b/src/runtime_src/core/tools/common/tests/TestTemporalSharingOvd.cpp index 8abe04217cf..7450e63b496 100644 --- a/src/runtime_src/core/tools/common/tests/TestTemporalSharingOvd.cpp +++ b/src/runtime_src/core/tools/common/tests/TestTemporalSharingOvd.cpp @@ -120,11 +120,11 @@ TestTemporalSharingOvd::run(std::shared_ptr dev) { // End of Run 2 if(XBU::getVerbose()){ - XBValidateUtils::logger(ptree, "Details", boost::str(boost::format("Spatially shared contexts latency: '%.1f' ms") % (latencySpatial * 1000))); - XBValidateUtils::logger(ptree, "Details", boost::str(boost::format("Temporally shared contexts latency: '%.1f' ms") % (latencyTemporal * 1000))); + XBValidateUtils::logger(ptree, "Details", boost::str(boost::format("Spatially shared contexts latency: %.1f ms") % (latencySpatial * 1000))); + XBValidateUtils::logger(ptree, "Details", boost::str(boost::format("Temporally shared contexts latency: %.1f ms") % (latencyTemporal * 1000))); } auto overhead = (latencyTemporal - latencySpatial); - XBValidateUtils::logger(ptree, "Details", boost::str(boost::format("Overhead: '%.1f' ms") % (overhead * 1000))); + XBValidateUtils::logger(ptree, "Details", boost::str(boost::format("Overhead: %.1f ms") % (overhead * 1000))); // Set the test status to passed ptree.put("status", XBValidateUtils::test_token_passed); diff --git a/src/runtime_src/core/tools/xbutil2/OO_Performance.cpp b/src/runtime_src/core/tools/xbutil2/OO_Performance.cpp index c09f5b80f3a..93e6bc11cb9 100644 --- a/src/runtime_src/core/tools/xbutil2/OO_Performance.cpp +++ b/src/runtime_src/core/tools/xbutil2/OO_Performance.cpp @@ -27,7 +27,7 @@ OO_Performance::OO_Performance( const std::string &_longName, bool _isHidden ) ; m_optionsHidden.add_options() - ("mode", boost::program_options::value(&m_action)->required(), "Action to perform: default, powersaver, balanced, performance, turbo") + ("mode", boost::program_options::value(&m_action)->required(), "Action to perform: default, powersaver, balanced, performance") //to-do: add turbo mode back to help! ; m_positionalOptions. @@ -38,12 +38,6 @@ OO_Performance::OO_Performance( const std::string &_longName, bool _isHidden ) void OO_Performance::execute(const SubCmdOptions& _options) const { - XBUtilities::verbose("SubCommand option: Power Mode"); - - XBUtilities::verbose("Option(s):"); - for (auto & aString : _options) - XBUtilities::verbose(std::string(" ") + aString); - // Honor help option first if (std::find(_options.begin(), _options.end(), "--help") != _options.end()) { printHelp();