Stop any previously running collection before starting the current one

Author: wesleywiser

collector/src/rustc-fake.rs

@@ -83,7 +83,16 @@ fn main() {
             }
 
             "xperf-stat" | "xperf-stat-self-profile" => {
-                // Read the path to tracelog.exe from either an environment variable, falling back to assuming it's on the PATH.
+                // Read the path to xperf.exe and tracelog.exe from an environment variable, falling back to assuming it's on the PATH.
+                let xperf = std::env::var("XPERF").unwrap_or("xperf.exe".to_string());
+                let mut cmd = Command::new(&xperf);
+                assert!(cmd.output().is_ok(), "xperf.exe could not be started");
+
+                // go ahead and run xperf -stop counters in case there are leftover counters running from a failed prior attempt
+                let mut cmd = Command::new(&xperf);
+                cmd.args(&["-stop", "counters"]);
+                cmd.status().expect("failed to spawn xperf");
+
                 let tracelog = std::env::var("TRACELOG").unwrap_or("tracelog.exe".to_string());
                 let mut cmd = Command::new(tracelog);
                 assert!(cmd.output().is_ok(), "tracelog.exe could not be started");
@@ -115,10 +124,6 @@ fn main() {
                     dur.subsec_nanos()
                 );
 
-                let xperf = std::env::var("XPERF").unwrap_or("xperf.exe".to_string());
-                let mut cmd = Command::new(&xperf);
-                assert!(cmd.output().is_ok(), "xperf.exe could not be started");
-
                 let xperf = |args: &[&str]| {
                     let mut cmd = Command::new(&xperf);
                     cmd.args(args);