chore: skip setting process priority on non-windows os (dotnet#2924)

filzrev · web-flow · commit ab8de310b3ce · 2025-12-16T08:11:31.000-05:00
diff --git a/src/BenchmarkDotNet/Extensions/ProcessExtensions.cs b/src/BenchmarkDotNet/Extensions/ProcessExtensions.cs
@@ -27,6 +27,10 @@ public static class ProcessExtensions
 
         public static void EnsureHighPriority(this Process process, ILogger logger)
         {
+            // Process.set_PriorityClass requires root on Unix
+            if (!OsDetector.IsWindows())
+                return;
+
             try
             {
                 process.PriorityClass = ProcessPriorityClass.High;
@@ -63,6 +67,10 @@ public static bool TrySetPriority(
             if (logger == null)
                 throw new ArgumentNullException(nameof(logger));
 
+            // Process.set_PriorityClass requires root on Unix
+            if (!OsDetector.IsWindows())
+                return false;
+
             try
             {
                 process.PriorityClass = priority;

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,10 @@ public static class ProcessExtensions`
`27`	`27`
`28`	`28`	`public static void EnsureHighPriority(this Process process, ILogger logger)`
`29`	`29`	`{`
	`30`	`+ // Process.set_PriorityClass requires root on Unix`
	`31`	`+ if (!OsDetector.IsWindows())`
	`32`	`+ return;`
	`33`	`+`
`30`	`34`	`try`
`31`	`35`	`{`
`32`	`36`	`process.PriorityClass = ProcessPriorityClass.High;`
`@@ -63,6 +67,10 @@ public static bool TrySetPriority(`
`63`	`67`	`if (logger == null)`
`64`	`68`	`throw new ArgumentNullException(nameof(logger));`
`65`	`69`
	`70`	`+ // Process.set_PriorityClass requires root on Unix`
	`71`	`+ if (!OsDetector.IsWindows())`
	`72`	`+ return false;`
	`73`	`+`
`66`	`74`	`try`
`67`	`75`	`{`
`68`	`76`	`process.PriorityClass = priority;`