commit ebde58121d34e30f57ab173bf425244ce0712d48

Author: Maynard Johnson <maynardj@us.ibm.com>

Date:   Wed Oct 9 13:12:21 2013 -0500

    Converge operf and ocount utility functions

    When the ocount tool was developed, a number of utility

    functions were needed that were very similar to operf utility

    functions, with just minor changes.  The decision was made at

    the time to copy these functions into ocount and change them

    as needed.  To avoid dual maintenance on very similar functions,

    we should converge the two tools to use one common set of utility

    functions. The main reason for not doing so in the first place

    was to make it easier to review ocount patches and not have to

    look at operf changes at the same time.

    Signed-off-by: Maynard Johnson <maynardj@us.ibm.com>

diff --git a/Makefile.am b/Makefile.am

index 293114b..2fe8d2f 100644

--- a/Makefile.am

+++ b/Makefile.am

@@ -19,9 +19,9 @@ SUBDIRS = \

 	events \

 	doc \

 	gui \

+	libpe_utils \

 	libperf_events \

 	pe_profiling \

-	libpe_utils \

 	pe_counting \

 	agents

 #### ATTENTION ####

diff --git a/libpe_utils/op_pe_utils.cpp b/libpe_utils/op_pe_utils.cpp

index dc9459e..b85d175 100644

--- a/libpe_utils/op_pe_utils.cpp

+++ b/libpe_utils/op_pe_utils.cpp

@@ -52,7 +52,9 @@ extern op_cpu cpu_type;

 using namespace std;

-static int _op_get_next_online_cpu(DIR * dir, struct dirent *entry)

+// Global functions

+

+int op_pe_utils::op_get_next_online_cpu(DIR * dir, struct dirent *entry)

 {

 #define OFFLINE 0x30

 	unsigned int cpu_num;

@@ -86,8 +88,6 @@ static int _op_get_next_online_cpu(DIR * dir, struct dirent *entry)

 		return cpu_num;

 }

-// Global functions

-

 int op_pe_utils::op_get_sys_value(const char * filename)

 {

 	char str[10];

@@ -148,7 +148,7 @@ int op_pe_utils::op_get_cpu_for_perf_events_cap(void)

 			goto error;

 		} else {

 			struct dirent *entry = NULL;

-			retval = _op_get_next_online_cpu(dir, entry);

+			retval = op_get_next_online_cpu(dir, entry);

 			closedir(dir);

 		}

 	} else {

@@ -310,40 +310,6 @@ int op_pe_utils::op_validate_app_name(char ** app, char ** save_appname)

 	out: return rc;

 }

-static int _get_next_online_cpu(DIR * dir, struct dirent *entry)

-{

-#define OFFLINE 0x30

-	unsigned int cpu_num;

-	char cpu_online_pathname[40];

-	int res;

-	FILE * online;

-	again:

-	do {

-		entry = readdir(dir);

-		if (!entry)

-			return -1;

-	} while (entry->d_type != DT_DIR);

-

-	res = sscanf(entry->d_name, "cpu%u", &cpu_num);

-	if (res <= 0)

-		goto again;

-

-	errno = 0;

-	snprintf(cpu_online_pathname, 40, "/sys/devices/system/cpu/cpu%u/online", cpu_num);

-	if ((online = fopen(cpu_online_pathname, "r")) == NULL) {

-		cerr << "Unable to open " << cpu_online_pathname << endl;

-		if (errno)

-			cerr << strerror(errno) << endl;

-		return -1;

-	}

-	res = fgetc(online);

-	fclose(online);

-	if (res == OFFLINE)

-		goto again;

-	else

-		return cpu_num;

-}

-

 set<int> op_pe_utils::op_get_available_cpus(int max_num_cpus)

 {

@@ -392,7 +358,7 @@ set<int> op_pe_utils::op_get_available_cpus(int max_num_cpus)

 		if (all_cpus_avail) {

 			available_cpus.insert(cpu);

 		} else {

-			real_cpu = _get_next_online_cpu(dir, entry);

+			real_cpu = op_get_next_online_cpu(dir, entry);

 			if (real_cpu < 0) {

 				err_msg = "Internal Error: Number of online cpus cannot be determined.";

 				rc = -1;

@@ -803,7 +769,8 @@ static bool convert_event_vals(vector<operf_event_t> * evt_vec)

-void op_pe_utils::op_process_events_list(vector<string> & passed_evts)

+void op_pe_utils::op_process_events_list(vector<string> & passed_evts,

+                                         bool do_profiling, bool do_callgraph)

 {

 	string cmd = OP_BINDIR;

@@ -812,7 +779,9 @@ void op_pe_utils::op_process_events_list(vector<string> & passed_evts)

 		     << OP_MAX_EVENTS << "." << endl;

 		exit(EXIT_FAILURE);

 	}

-	cmd += "/ophelp --check-events --ignore-count ";

+	cmd += "/ophelp --check-events ";

+	if (!do_profiling)

+		cmd += "--ignore-count ";

 	for (unsigned int i = 0; i <  passed_evts.size(); i++) {

 		FILE * fp;

 		string full_cmd = cmd;

@@ -825,6 +794,8 @@ void op_pe_utils::op_process_events_list(vector<string> & passed_evts)

 			event_spec = _handle_powerpc_event_spec(event_spec);

 #endif

+		if (do_callgraph)

+			full_cmd += " --callgraph=1 ";

 		full_cmd += event_spec;

 		fp = popen(full_cmd.c_str(), "r");

 		if (fp == NULL) {

@@ -836,14 +807,21 @@ void op_pe_utils::op_process_events_list(vector<string> & passed_evts)

 			pclose(fp);

 			cerr << "Error retrieving info for event "

 			     << event_spec << endl;

+			if (do_callgraph)

+				cerr << "Note: When doing callgraph profiling, the sample count must be"

+				     << endl << "15 times the minimum count value for the event."  << endl;

 			exit(EXIT_FAILURE);

 		}

 		pclose(fp);

 		char * event_str = op_xstrndup(event_spec.c_str(), event_spec.length());

 		operf_event_t event;

 		strncpy(event.name, strtok(event_str, ":"), OP_MAX_EVT_NAME_LEN - 1);

+		if (do_profiling)

+			event.count = atoi(strtok(NULL, ":"));

+		else

+			event.count = 0UL;

 		/* Event name is required in the event spec in order for

-		 * 'ophelp --check-events --ignore-count' to pass.  But since unit mask

+		 * 'ophelp --check-events' to pass.  But since unit mask

 		 *  and domain control bits are optional, we need to ensure the result of

 		 *  strtok is valid.

 		 */

@@ -854,7 +832,6 @@ void op_pe_utils::op_process_events_list(vector<string> & passed_evts)

 		int place =  _OP_UM;

 		char * endptr = NULL;

 		event.evt_um = 0UL;

-		event.count = 0UL;

 		event.no_kernel = 0;

 		event.no_user = 0;

 		event.throttled = false;

@@ -904,7 +881,7 @@ void op_pe_utils::op_process_events_list(vector<string> & passed_evts)

 #endif

 }

-void op_pe_utils::op_get_default_event(void)

+void op_pe_utils::op_get_default_event(bool do_callgraph)

 {

 	operf_event_t dft_evt;

 	struct op_default_event_descr descr;

@@ -918,7 +895,18 @@ void op_pe_utils::op_get_default_event(void)

 	}

 	memset(&dft_evt, 0, sizeof(dft_evt));

-	dft_evt.count = descr.count;

+	if (do_callgraph) {

+		struct op_event * _event;

+		op_events(cpu_type);

+		if ((_event = find_event_by_name(descr.name, 0, 0))) {

+			dft_evt.count = _event->min_count * CALLGRAPH_MIN_COUNT_SCALE;

+		} else {

+			cerr << "Error getting event info for " << descr.name << endl;

+			exit(EXIT_FAILURE);

+		}

+	} else {

+		dft_evt.count = descr.count;

+	}

 	dft_evt.evt_um = descr.um;

 	strncpy(dft_evt.name, descr.name, OP_MAX_EVT_NAME_LEN - 1);

 	_get_event_code(&dft_evt, cpu_type);

diff --git a/libpe_utils/op_pe_utils.h b/libpe_utils/op_pe_utils.h

index 400eed3..08b6fae 100644

--- a/libpe_utils/op_pe_utils.h

+++ b/libpe_utils/op_pe_utils.h

@@ -18,11 +18,13 @@

 #include <dirent.h>

 #include <vector>

+#include <set>

 #include "op_cpu_type.h"

 #define OP_APPNAME_LEN 1024

 #define OP_MAX_EVENTS 24

+#define CALLGRAPH_MIN_COUNT_SCALE 15

 /* A macro to be used for ppc64 architecture-specific code.  The '__powerpc__' macro

  * is defined for both ppc64 and ppc32 architectures, so we must further qualify by

@@ -38,8 +40,10 @@ extern int op_check_perf_events_cap(bool use_cpu_minus_one);

 extern int op_get_sys_value(const char * filename);

 extern int op_get_cpu_for_perf_events_cap(void);

 extern int op_validate_app_name(char ** app, char ** save_appname);

-extern void op_get_default_event(void);

-extern void op_process_events_list(std::vector<std::string> & passed_evts);

+extern void op_get_default_event(bool do_callgraph);

+extern void op_process_events_list(std::vector<std::string> & passed_evts,

+                                   bool do_profiling, bool do_callgraph);

+extern int op_get_next_online_cpu(DIR * dir, struct dirent *entry);

 extern std::set<int> op_get_available_cpus(int max_num_cpus);

 }

diff --git a/libperf_events/Makefile.am b/libperf_events/Makefile.am

index 7163610..cf5f434 100644

--- a/libperf_events/Makefile.am

+++ b/libperf_events/Makefile.am

@@ -7,6 +7,7 @@ AM_CPPFLAGS = \

 	-I ${top_srcdir}/libop \

 	-I ${top_srcdir}/libdb \

 	-I ${top_srcdir}/libperf_events \

+	-I ${top_srcdir}/libpe_utils \

 	@PERF_EVENT_FLAGS@ \

 	@OP_CPPFLAGS@

diff --git a/libperf_events/operf_counter.cpp b/libperf_events/operf_counter.cpp

index b4cceaa..319e859 100644

--- a/libperf_events/operf_counter.cpp

+++ b/libperf_events/operf_counter.cpp

@@ -31,6 +31,7 @@

 #include "operf_process_info.h"

 #include "op_libiberty.h"

 #include "operf_stats.h"

+#include "op_pe_utils.h"

 using namespace std;

@@ -645,7 +646,7 @@ void operf_record::setup()

 		} else if (all_cpus_avail) {

 			real_cpu = cpu;

 		} else {

-			real_cpu = op_get_next_online_cpu(dir, entry);

+			real_cpu = op_pe_utils::op_get_next_online_cpu(dir, entry);

 			if (real_cpu < 0) {

 				err_msg = "Internal Error: Number of online cpus cannot be determined.";

 				rc = -1;

diff --git a/libperf_events/operf_utils.cpp b/libperf_events/operf_utils.cpp

index 30e64d8..faed9a6 100644

--- a/libperf_events/operf_utils.cpp

+++ b/libperf_events/operf_utils.cpp

@@ -65,161 +65,6 @@ static list<event_t *> unresolved_events;

 static struct operf_transient trans;

 static bool sfile_init_done;

-/* Some architectures (e.g., ppc64) do not use the same event value (code) for oprofile

- * and for perf_events.  The operf-record process requires event values that perf_events

- * understands, but the operf-read process requires oprofile event values.  The purpose of

- * the following method is to map the operf-record event value to a value that

- * opreport can understand.

- */

-#if PPC64_ARCH

-extern op_cpu cpu_type;

-#define NIL_CODE ~0U

-

-#if HAVE_LIBPFM3

-static bool _get_codes_for_match(unsigned int pfm_idx, const char name[],

-                                 vector<operf_event_t> * evt_vec)

-{

-	unsigned int num_events = evt_vec->size();

-	int tmp_code, ret;

-	char evt_name[OP_MAX_EVT_NAME_LEN];

-	unsigned int events_converted = 0;

-	for (unsigned int i = 0; i < num_events; i++) {

-		operf_event_t event = (*evt_vec)[i];

-		if (event.evt_code != NIL_CODE) {

-			events_converted++;

-			continue;

-		}

-		memset(evt_name, 0, OP_MAX_EVT_NAME_LEN);

-		if (!strcmp(event.name, "CYCLES")) {

-			strcpy(evt_name ,"PM_CYC") ;

-		} else if (strstr(event.name, "_GRP")) {

-			string str = event.name;

-			strncpy(evt_name, event.name, str.rfind("_GRP"));

-		} else {

-			strncpy(evt_name, event.name, strlen(event.name));

-		}

-		if (strncmp(name, evt_name, OP_MAX_EVT_NAME_LEN))

-			continue;

-		ret = pfm_get_event_code(pfm_idx, &tmp_code);

-		if (ret != PFMLIB_SUCCESS) {

-			string evt_name_str = event.name;

-			string msg = "libpfm cannot find event code for " + evt_name_str +

-					"; cannot continue";

-			throw runtime_error(msg);

-		}

-		event.evt_code = tmp_code;

-		(*evt_vec)[i] = event;

-		events_converted++;

-		cverb << vrecord << "Successfully converted " << event.name << " to perf_event code "

-		      << hex << tmp_code << endl;

-	}

-	return (events_converted == num_events);

-}

-#else

-static bool _op_get_event_codes(vector<operf_event_t> * evt_vec)

-{

-	int ret, i;

-	unsigned int num_events = evt_vec->size();

-	char evt_name[OP_MAX_EVT_NAME_LEN];

-	unsigned int events_converted = 0;

-	uint64_t code[1];

-

-	typedef struct {

-		uint64_t    *codes;

-		char        **fstr;

-		size_t      size;

-		int         count;

-		int         idx;

-	} pfm_raw_pmu_encode_t;

-

-	pfm_raw_pmu_encode_t raw;

-	raw.codes = code;

-	raw.count = 1;

-	raw.fstr = NULL;

-

-	if (pfm_initialize() != PFM_SUCCESS)

-		throw runtime_error("Unable to initialize libpfm; cannot continue");

-

-	for (unsigned int i = 0; i < num_events; i++) {

-		operf_event_t event = (*evt_vec)[i];

-		if (event.evt_code != NIL_CODE) {

-			events_converted++;

-			continue;

-		}

-		memset(evt_name, 0, OP_MAX_EVT_NAME_LEN);

-		if (!strcmp(event.name, "CYCLES")) {

-			strcpy(evt_name ,"PM_CYC") ;

-		} else if (strstr(event.name, "_GRP")) {

-			string str = event.name;

-			strncpy(evt_name, event.name, str.rfind("_GRP"));

-		} else {

-			strncpy(evt_name, event.name, strlen(event.name));

-		}

-

-		memset(&raw, 0, sizeof(raw));

-		ret = pfm_get_os_event_encoding(evt_name, PFM_PLM3, PFM_OS_NONE, &raw;;

-		if (ret != PFM_SUCCESS) {

-			string evt_name_str = event.name;

-			string msg = "libpfm cannot find event code for " + evt_name_str +

-					"; cannot continue";

-			throw runtime_error(msg);

-		}

-

-		event.evt_code = raw.codes[0];

-		(*evt_vec)[i] = event;

-		events_converted++;

-		cverb << vrecord << "Successfully converted " << event.name << " to perf_event code "

-		      << hex << event.evt_code << endl;

-	}

-	return (events_converted == num_events);

-}

-#endif

-

-bool OP_perf_utils::op_convert_event_vals(vector<operf_event_t> * evt_vec)

-{

-	unsigned int i, count;

-	char name[256];

-	int ret;

-	for (unsigned int i = 0; i < evt_vec->size(); i++) {

-		operf_event_t event = (*evt_vec)[i];

-		if (cpu_type == CPU_PPC64_POWER7) {

-			if (!strncmp(event.name, "PM_RUN_CYC", strlen("PM_RUN_CYC"))) {

-				event.evt_code = 0x600f4;

-			} else if (!strncmp(event.name, "PM_RUN_INST_CMPL", strlen("PM_RUN_INST_CMPL"))) {

-				event.evt_code = 0x500fa;

-			} else {

-				event.evt_code = NIL_CODE;

-			}

-		} else {

-			event.evt_code = NIL_CODE;

-		}

-		(*evt_vec)[i] = event;

-	}

-

-#if HAVE_LIBPFM3

-	if (pfm_initialize() != PFMLIB_SUCCESS)

-		throw runtime_error("Unable to initialize libpfm; cannot continue");

-

-	ret = pfm_get_num_events(&count);

-	if (ret != PFMLIB_SUCCESS)

-		throw runtime_error("Unable to use libpfm to obtain event code; cannot continue");

-	for(i =0 ; i < count; i++)

-	{

-		ret = pfm_get_event_name(i, name, 256);

-		if (ret != PFMLIB_SUCCESS)

-			continue;

-		if (_get_codes_for_match(i, name, evt_vec))

-			break;

-	}

-	return (i != count);

-#else

-	return _op_get_event_codes(evt_vec);

-#endif

-}

-

-#endif // PPC64_ARCH

-

-

 static inline void update_trans_last(struct operf_transient * trans)

 {

 	trans->last = trans->current;

@@ -1465,38 +1310,3 @@ void OP_perf_utils::op_get_kernel_event_data(struct mmap_data *md, operf_record

 	md->prev = old;

 	pc->data_tail = old;

 }

-

-

-int OP_perf_utils::op_get_next_online_cpu(DIR * dir, struct dirent *entry)

-{

-#define OFFLINE 0x30

-	unsigned int cpu_num;

-	char cpu_online_pathname[40];

-	int res;

-	FILE * online;

-	again:

-	do {

-		entry = readdir(dir);

-		if (!entry)

-			return -1;

-	} while (entry->d_type != DT_DIR);

-

-	res = sscanf(entry->d_name, "cpu%u", &cpu_num);

-	if (res <= 0)

-		goto again;

-

-	errno = 0;

-	snprintf(cpu_online_pathname, 40, "/sys/devices/system/cpu/cpu%u/online", cpu_num);

-	if ((online = fopen(cpu_online_pathname, "r")) == NULL) {

-		cerr << "Unable to open " << cpu_online_pathname << endl;

-		if (errno)

-			cerr << strerror(errno) << endl;

-		return -1;

-	}

-	res = fgetc(online);

-	fclose(online);

-	if (res == OFFLINE)

-		goto again;

-	else

-		return cpu_num;

-}

diff --git a/libperf_events/operf_utils.h b/libperf_events/operf_utils.h

index 4c191fe..2a979e3 100644

--- a/libperf_events/operf_utils.h

+++ b/libperf_events/operf_utils.h

@@ -87,8 +87,6 @@ int op_write_output(int output, void *buf, size_t size);

 int op_write_event(event_t * event, u64 sample_type);

 int op_read_from_stream(std::ifstream & is, char * buf, std::streamsize sz);

 int op_mmap_trace_file(struct mmap_info & info, bool init);

-int op_get_next_online_cpu(DIR * dir, struct dirent *entry);

-bool op_convert_event_vals(std::vector<operf_event_t> * evt_vec);

 void op_reprocess_unresolved_events(u64 sample_type, bool print_progress);

 void op_release_resources(void);

 }

diff --git a/pe_counting/ocount.cpp b/pe_counting/ocount.cpp

index 5a85c3f..db847ea 100644

--- a/pe_counting/ocount.cpp

+++ b/pe_counting/ocount.cpp

@@ -720,9 +720,9 @@ static void process_args(int argc, char * const argv[])

 	if (ocount_options::evts.empty()) {

 		// Use default event

-		op_pe_utils::op_get_default_event();

+		op_pe_utils::op_get_default_event(false);

 	} else  {

-		op_pe_utils::op_process_events_list(ocount_options::evts);

+		op_pe_utils::op_process_events_list(ocount_options::evts, false, false);

 	}

 	cverb << vdebug << "Number of events passed is " << events.size() << endl;

 	return;

diff --git a/pe_profiling/Makefile.am b/pe_profiling/Makefile.am

index b27cbc7..8c232c4 100644

--- a/pe_profiling/Makefile.am

+++ b/pe_profiling/Makefile.am

@@ -6,6 +6,7 @@ AM_CPPFLAGS = \

 	-I ${top_srcdir}/libop \

 	-I ${top_srcdir}/libutil++ \

 	-I ${top_srcdir}/libperf_events \

+	-I ${top_srcdir}/libpe_utils \

 	@PERF_EVENT_FLAGS@ \

 	@OP_CPPFLAGS@

@@ -15,7 +16,8 @@ AM_CXXFLAGS = @OP_CXXFLAGS@

 AM_LDFLAGS = @OP_LDFLAGS@

 bin_PROGRAMS = operf

-operf_LDADD =	../libperf_events/libperf_events.a \

+operf_LDADD = ../libperf_events/libperf_events.a \

+	../libpe_utils/libpe_utils.a \

 	../libutil++/libutil++.a \

 	../libdb/libodb.a \

 	../libop/libop.a \

diff --git a/pe_profiling/operf.cpp b/pe_profiling/operf.cpp

index 3fec123..89e9c4b 100644

--- a/pe_profiling/operf.cpp

+++ b/pe_profiling/operf.cpp

@@ -35,6 +35,7 @@

 #include <getopt.h>

 #include <iostream>

 #include "operf_utils.h"

+#include "op_pe_utils.h"

 #include "op_libiberty.h"

 #include "string_manip.h"

 #include "cverb.h"

@@ -50,6 +51,7 @@

 #include "op_netburst.h"

 using namespace std;

+using namespace op_pe_utils;

 typedef enum END_CODE {

 	ALL_OK = 0,

@@ -73,11 +75,11 @@ uid_t my_uid;

 bool no_vmlinux;

 int kptr_restrict;

 char * start_time_human_readable;

+std::vector<operf_event_t> events;

+

 #define DEFAULT_OPERF_OUTFILE "operf.data"

-#define CALLGRAPH_MIN_COUNT_SCALE 15

-static char full_pathname[PATH_MAX];

 static char * app_name_SAVE = NULL;

 static char ** app_args = NULL;

 static 	pid_t jitconv_pid = -1;

@@ -88,7 +90,6 @@ static string samples_dir;

 static bool startApp;

 static string outputfile;

 static char start_time_str[32];

-static vector<operf_event_t> events;

 static bool jit_conversion_running;

 static void convert_sample_data(void);

 static int sample_data_pipe[2];

@@ -948,517 +949,6 @@ out:

 }

-static int find_app_file_in_dir(const struct dirent * d)

-{

-	if (!strcmp(d->d_name, app_name))

-		return 1;

-	else

-		return 0;

-}

-

-static int get_PATH_based_pathname(char * path_holder, size_t n)

-{

-	int retval = -1;

-

-	char * real_path = getenv("PATH");

-	char * path = (char *) xstrdup(real_path);

-	char * segment = strtok(path, ":");

-	while (segment) {

-		struct dirent ** namelist;

-		int rc = scandir(segment, &namelist, find_app_file_in_dir, NULL);

-		if (rc < 0) {

-			if (errno != ENOENT) {

-				cerr << strerror(errno) << endl;

-				cerr << app_name << " cannot be found in your PATH." << endl;

-				break;

-			}

-		} else if (rc == 1) {

-			size_t applen = strlen(app_name);

-			size_t dirlen = strlen(segment);

-

-			if (applen + dirlen + 2 > n) {

-				cerr << "Path segment " << segment

-				     << " prepended to the passed app name is too long"

-				     << endl;

-				retval = -1;

-				break;

-			}

-

-			if (!strcmp(segment, ".")) {

-				if (getcwd(path_holder, PATH_MAX) == NULL) {

-					retval = -1;

-					cerr << "getcwd [3] failed when processing <cur-dir>/" << app_name << " found via PATH. Aborting."

-							<< endl;

-					break;

-				}

-			} else {

-				strncpy(path_holder, segment, dirlen);

-			}

-			strcat(path_holder, "/");

-			strncat(path_holder, app_name, applen);

-			retval = 0;

-			free(namelist[0]);

-			free(namelist);

-

-			break;

-		}

-		segment = strtok(NULL, ":");

-	}

-	free(path);

-	return retval;

-}

-int validate_app_name(void)

-{

-	int rc = 0;

-	struct stat filestat;

-	size_t len = strlen(app_name);

-

-	if (len > (size_t) (OP_APPNAME_LEN - 1)) {

-		cerr << "app name longer than max allowed (" << OP_APPNAME_LEN

-		     << " chars)\n";

-		cerr << app_name << endl;

-		rc = -1;

-		goto out;

-	}

-

-	if (index(app_name, '/') == app_name) {

-		// Full pathname of app was specified, starting with "/".

-		strncpy(full_pathname, app_name, len);

-	} else if ((app_name[0] == '.') && (app_name[1] == '/')) {

-		// Passed app is in current directory; e.g., "./myApp"

-		if (getcwd(full_pathname, PATH_MAX) == NULL) {

-			rc = -1;

-			cerr << "getcwd [1] failed when trying to find app name " << app_name << ". Aborting."

-			     << endl;

-			goto out;

-		}

-		strcat(full_pathname, "/");

-		if ((strlen(full_pathname) + strlen(app_name + 2) + 1) > PATH_MAX) {

-			rc = -1;

-			cerr << "Length of current dir (" << full_pathname << ") and app name ("

-			     << (app_name + 2) << ") exceeds max allowed (" << PATH_MAX << "). Aborting."

-			     << endl;

-			goto out;

-		}

-		strcat(full_pathname, (app_name + 2));

-	} else if (index(app_name, '/')) {

-		// Passed app is in a subdirectory of cur dir; e.g., "test-stuff/myApp"

-		if (getcwd(full_pathname, PATH_MAX) == NULL) {

-			rc = -1;

-			cerr << "getcwd [2] failed when trying to find app name " << app_name << ". Aborting."

-			     << endl;

-			goto out;

-		}

-		strcat(full_pathname, "/");

-		strcat(full_pathname, app_name);

-	} else {

-		// Passed app name, at this point, MUST be found in PATH

-		rc = get_PATH_based_pathname(full_pathname, PATH_MAX);

-	}

-

-	if (rc) {

-		cerr << "Problem finding app name " << app_name << ". Aborting."

-		     << endl;

-		goto out;

-	}

-	app_name_SAVE = app_name;

-	app_name = full_pathname;

-	if (stat(app_name, &filestat)) {

-		char msg[OP_APPNAME_LEN + 50];

-		snprintf(msg, OP_APPNAME_LEN + 50, "Non-existent app name \"%s\"",

-		         app_name);

-		perror(msg);

-		rc = -1;

-	}

-

-	out: return rc;

-}

-

-static void _get_event_code(operf_event_t * event)

-{

-	FILE * fp;

-	char oprof_event_code[9];

-	string command;

-	u64 base_code, config;

-	char buf[20];

-	if ((snprintf(buf, 20, "%lu", event->count)) < 0) {

-		cerr << "Error parsing event count of " << event->count << endl;

-		exit(EXIT_FAILURE);

-	}

-

-	base_code = config = 0ULL;

-

-	command = OP_BINDIR;

-	command += "ophelp ";

-	command += event->name;

-

-	fp = popen(command.c_str(), "r");

-	if (fp == NULL) {

-		cerr << "Unable to execute ophelp to get info for event "

-		     << event->name << endl;