Set efficiency mode by default

Set the lowest priority and EcoQoS to SimpleCom process.
Please specify --disable-efficiency-mode if you face performance issue.

See following URL for more details of efficiency mode:
https://devblogs.microsoft.com/performance-diagnostics/reduce-process-interference-with-task-manager-efficiency-mode/

Author: YaSuenag

README.md

@@ -43,6 +43,7 @@ Console app for serial connection.
 | `--log-file [logfile]` | <none> | Log serial communication to file |
 | `--stdin-logging` | false | Enable stdin logging<br><br>⚠️Possible to be logged each chars duplicately due to echo back from the console when this option is set, and also secrets (e.g. passphrase) typed into the console will be logged even if it is not shown on the console. |
 | `--batch` | false | Perform in batch mode<br><br>⚠️You have to set serial port in command line arguments, and you cannot set with `--show-dialog`, `--tty-resizer`, `--auto-reconnect`, `--log-file`. |
+| `--disable-efficiency-mode` | false | Disable [Efficiency Mode](https://devblogs.microsoft.com/performance-diagnostics/reduce-process-interference-with-task-manager-efficiency-mode/). Specify this option if you have performance issue in SimpleCom. This option cannot be set on setup dialog. |
 | `--help` | - | Show help message |
 
 # How to build

SimpleCom/SerialSetup.cpp

@@ -150,6 +150,7 @@ SimpleCom::SerialSetup::SerialSetup() :
 	_options[_T("--log-file")] = new CommandlineOption<LPTSTR>(_T("[logfile]"), _T("Log serial communication to file"), nullptr);
 	_options[_T("--stdin-logging")] = new CommandlineOption<bool>(_T(""), _T("Enable stdin logging"), false);
 	_options[_T("--batch")] = new CommandlineOption<bool>(_T(""), _T("Perform in batch mode"), false);
+	_options[_T("--disable-efficiency-mode")] = new CommandlineOption<bool>(_T(""), _T("Disable efficiency mode"), false);
 	_options[_T("--help")] = new CommandlineHelpOption(&_options);
 }
 

SimpleCom/SerialSetup.h

@@ -211,6 +211,14 @@ namespace SimpleCom {
 			return static_cast<CommandlineOption<bool>*>(_options[_T("--batch")])->get();
 		}
 
+		inline void DisableEfficiencyMode(bool disable) {
+			static_cast<CommandlineOption<bool>*>(_options[_T("--disable-efficiency-mode")])->set(disable);
+		}
+
+		inline bool IsEfficiencyMode() {
+			return !static_cast<CommandlineOption<bool>*>(_options[_T("--disable-efficiency-mode")])->get();
+		}
+
 		inline SerialDeviceScanner& GetDeviceScanner() {
 			return _scanner;
 		}

SimpleCom/SimpleCom.cpp

@@ -90,15 +90,31 @@ static int DoBatchMode(TString& device, DCB* dcb) {
 	return 0;
 }
 
+// https://devblogs.microsoft.com/performance-diagnostics/reduce-process-interference-with-task-manager-efficiency-mode/
+static void SetEfficiencyMode() {
+	HANDLE hProc = GetCurrentProcess();
+
+	// a) reduces process base priority to low
+	SetPriorityClass(hProc, IDLE_PRIORITY_CLASS);
+
+	// b) sets QoS mode to EcoQoS.
+	PROCESS_POWER_THROTTLING_STATE PowerThrottling = {
+		.Version = PROCESS_POWER_THROTTLING_CURRENT_VERSION,
+		.ControlMask = PROCESS_POWER_THROTTLING_EXECUTION_SPEED,
+		.StateMask = PROCESS_POWER_THROTTLING_EXECUTION_SPEED
+	};
+	CALL_WINAPI_WITH_DEBUGLOG(SetProcessInformation(hProc, ProcessPowerThrottling, &PowerThrottling, sizeof(PowerThrottling)), TRUE, __FILE__, __LINE__);
+}
+
 int _tmain(int argc, LPCTSTR argv[])
 {
 	DCB dcb;
 	TString device;
 	HWND parent_hwnd = GetParentWindow();
 	SimpleCom::SerialSetup setup;
 
-	// Serial port configuration
 	try {
+		// Serial port configuration
 		if (argc > 1) {
 			// command line mode
 			setup.ParseArguments(argc, argv);
@@ -108,6 +124,10 @@ int _tmain(int argc, LPCTSTR argv[])
 			setup.SetShowDialog(true);
 		}
 
+		if (setup.IsEfficiencyMode()) {
+			SetEfficiencyMode();
+		}
+
 		if (setup.GetWaitDevicePeriod() > 0) {
 			setup.GetDeviceScanner().SetTargetPort(setup.GetPort());
 			setup.GetDeviceScanner().WaitSerialDevices(parent_hwnd, setup.GetWaitDevicePeriod());

SimpleCom/app.manifest

@@ -3,16 +3,12 @@
   <assemblyIdentity version="1.2.6.0" name="SimpleCom.exe" />
   <compatibility xmlns="urn:schemas-microsoft-com:compatibility.v1">
     <application>
-      <!-- Windows 10 -->
+      <!-- Windows 10, 11 -->
       <supportedOS Id="{8e0f7a12-bfb3-4fe8-b9a5-48fd50a15a9a}" maxversiontested="10.0.19042.868" />
       <!-- Windows 8.1 -->
       <supportedOS Id="{1f676c76-80e1-4239-95bb-83d0f6d0da78}" />
       <!-- Windows 8 -->
       <supportedOS Id="{4a2f28e3-53b9-4441-ba9c-d69d4a4a6e38}" />
-      <!-- Windows 7 -->
-      <supportedOS Id="{35138b9a-5d96-4fbd-8e2d-a2440225f93a}" />
-      <!-- Windows Vista -->
-      <supportedOS Id="{e2011457-1546-43c5-a5fe-008deee3d3f0}" />
     </application>
   </compatibility>
   <trustInfo xmlns="urn:schemas-microsoft-com:asm.v2">
@@ -27,4 +23,4 @@
       <assemblyIdentity type="win32" name="Microsoft.Windows.Common-Controls" version="6.0.0.0" processorArchitecture="*" publicKeyToken="6595b64144ccf1df" language="*" />
     </dependentAssembly>
   </dependency>
-</assembly>
+</assembly>

SimpleComTest/SerialSetupTest.cpp

@@ -230,6 +230,7 @@ namespace SimpleComTest
 			Assert::IsNull(setup.GetLogFile());
 			Assert::AreEqual(false, setup.IsEnableStdinLogging());
 			Assert::AreEqual(false, setup.IsBatchMode());
+			Assert::AreEqual(true, setup.IsEfficiencyMode());
 		}
 
 		TEST_METHOD(ArgParserTest)
@@ -250,6 +251,7 @@ namespace SimpleComTest
 				_T("--auto-reconnect-timeout"), _T("20"),
 				_T("--log-file"), _T(R"(A:\test.log)"),
 				_T("--stdin-logging"),
+				_T("--disable-efficiency-mode"),
 				_T("COM100")
 			};
 
@@ -269,6 +271,7 @@ namespace SimpleComTest
 			Assert::AreEqual(20, setup.GetAutoReconnectTimeoutInSec());
 			Assert::AreEqual(_T(R"(A:\test.log)"), setup.GetLogFile());
 			Assert::AreEqual(true, setup.IsEnableStdinLogging());
+			Assert::AreEqual(false, setup.IsEfficiencyMode());
 			Assert::AreEqual(_T("COM100"), setup.GetPort().c_str());
 		}