CR-1219475 and CR-1220306 (#8645)

Signed-off-by: AShivangi <[email protected]>
Xilinx · Dec 10, 2024 · a81ded1 · a81ded1
1 parent 42e5fb6
commit a81ded1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git 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<xrt_core::device> 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
@@ -27,7 +27,7 @@ OO_Performance::OO_Performance( const std::string &_longName, bool _isHidden )
   ;
 
   m_optionsHidden.add_options()
-    ("mode", boost::program_options::value<decltype(m_action)>(&m_action)->required(), "Action to perform: default, powersaver, balanced, performance, turbo")
+    ("mode", boost::program_options::value<decltype(m_action)>(&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();