From c95158840a7914d558a93b044c5ab0eeb0ea9337 Mon Sep 17 00:00:00 2001
From: William Cohen <wcohen@redhat.com>
Date: Tue, 9 Aug 2016 22:25:52 -0400
Subject: [PATCH] Only start the application if the perf events setup was
 successful

The code was starting the application before the performance events
were setup.  In some cases the the setup of the perf events may fail
and the code needs to verify that the performance events have been
successfully set up before starting the application.  Changed the
order of those steps to allow a check of the perf event setup before
launching the application.

Signed-off-by: William Cohen <wcohen@redhat.com>
---
 pe_counting/ocount.cpp | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/pe_counting/ocount.cpp b/pe_counting/ocount.cpp
index 4d9c104..7717717 100644
--- a/pe_counting/ocount.cpp
+++ b/pe_counting/ocount.cpp
@@ -257,16 +257,6 @@ bool start_counting(void)
 		proc_list = ocount_options::processes;
 	}
 
-	if (startApp) {
-		// Tell app_PID to start the app
-		cverb << vdebug << "telling child to start app" << endl;
-		if (write(start_app_pipe[1], &startup, sizeof(startup)) < 0) {
-			perror("Internal error on start_app_pipe");
-			return -1;
-		}
-		app_started = true;
-	}
-
 	orecord = new ocount_record(runmode, events, ocount_options::display_interval ? true : false);
 	bool ret;
 	switch (runmode) {
@@ -300,6 +290,16 @@ bool start_counting(void)
 		ret = false;
 	}
 
+	if (startApp && ret != false) {
+		// Tell app_PID to start the app
+		cverb << vdebug << "telling child to start app" << endl;
+		if (write(start_app_pipe[1], &startup, sizeof(startup)) < 0) {
+			perror("Internal error on start_app_pipe");
+			return false;
+		}
+		app_started = true;
+	}
+
 	return ret;
 }
 
-- 
2.7.4