Support additional perf counter providers (#1529)

* Extends infrastructure to allow specifying arbitrary number of
  perf counter providers.
* Add support for Kestrel and Npgsql counters.

Closes #1513

Author: roji

src/Benchmarks.ServerJob/ServerJob.cs

@@ -143,6 +143,12 @@ public ServerJob ClearServerCounters()
         // Other collection options
         public bool CollectStartup { get; set; }
         public bool CollectCounters { get; set; }
+
+        /// <summary>
+        /// The list of performance counter providers to be collected. Defaults to <c>System.Runtime</c>.
+        /// </summary>
+        public List<string> CounterProviders { get; set; } = new List<string>();
+
         public string BasePath { get; set; }
         public int ProcessId { get; set; }
         public Dictionary<string, string> EnvironmentVariables { get; set; } = new Dictionary<string, string>();

src/BenchmarksDriver2/README.md

@@ -70,6 +70,7 @@ Options:
   --[JOB].dotnetTraceProviders <profile|flags>                             An optional profile name or list of dotnet-trace providers can be passed. Default is 'cpu-sampling'. See https://github.com/dotnet/diagnostics/blob/master/documentation/dotnet-trace-instructions.md for details. e.g., Microsoft-DotNETCore-SampleProfiler, Microsoft-Windows-DotNETRuntime, gc-verbose.  Can be used multiple times to set multiple providers.
   --[JOB].options.traceOutput <filename>                                   The name of the trace file. Can be a file prefix (app will add *.DATE*.zip) , or a specific name and no DATE* will be added e.g., c:\traces\mytrace
   --[JOB].collectCounters <true|false>                                     Whether to collect dotnet counters.
+  --[JOB].counterProviders <provider>                                      The name of a performance counter provider from which to collect.
   --[JOB].collectStartup <true|false>                                      Whether to includes the startup phase in the traces, i.e after the application is launched and before it is marked as ready. For a web application it means before it is ready to accept requests.
 
   ## Environment

src/BenchmarksServer/Microsoft.Diagnostics.Tools.RuntimeClient/Eventing/SessionConfiguration.cs

@@ -25,7 +25,7 @@ public SessionConfiguration(uint circularBufferSizeMB, EventPipeSerializationFor
                 throw new ArgumentException("Unrecognized format");
             if (providers == null)
                 throw new ArgumentNullException(nameof(providers));
-            if (providers.Count() <= 0)
+            if (providers.Count <= 0)
                 throw new ArgumentException($"Specified providers collection is empty.");
 
             CircularBufferSizeInMB = circularBufferSizeMB;
@@ -37,7 +37,6 @@ public SessionConfiguration(uint circularBufferSizeMB, EventPipeSerializationFor
         public uint CircularBufferSizeInMB { get; }
         public EventPipeSerializationFormat Format { get; }
 
-
         public IReadOnlyCollection<Provider> Providers => _providers.AsReadOnly();
 
         private readonly List<Provider> _providers;