diff options
Diffstat (limited to 'gprofng/src')
-rw-r--r-- | gprofng/src/BaseMetricTreeNode.h | 1 | ||||
-rw-r--r-- | gprofng/src/Dbe.cc | 12 | ||||
-rw-r--r-- | gprofng/src/DbeSession.cc | 3 | ||||
-rw-r--r-- | gprofng/src/Disasm.cc | 1 | ||||
-rw-r--r-- | gprofng/src/Experiment.cc | 2 | ||||
-rw-r--r-- | gprofng/src/HashMap.h | 3 | ||||
-rw-r--r-- | gprofng/src/IOActivity.h | 1 | ||||
-rw-r--r-- | gprofng/src/collctrl.cc | 14 | ||||
-rw-r--r-- | gprofng/src/collctrl.h | 1 | ||||
-rw-r--r-- | gprofng/src/collector_module.h | 20 | ||||
-rw-r--r-- | gprofng/src/gp-display-src.cc | 8 | ||||
-rw-r--r-- | gprofng/src/gp-print.h | 2 | ||||
-rw-r--r-- | gprofng/src/hwc_intel_icelake.h | 6 | ||||
-rw-r--r-- | gprofng/src/util.cc | 10 |
14 files changed, 30 insertions, 54 deletions
diff --git a/gprofng/src/BaseMetricTreeNode.h b/gprofng/src/BaseMetricTreeNode.h index d73d244..7698f9c 100644 --- a/gprofng/src/BaseMetricTreeNode.h +++ b/gprofng/src/BaseMetricTreeNode.h @@ -85,7 +85,6 @@ private: BaseMetricTreeNode *root; // root of tree BaseMetricTreeNode *parent; // my parent - bool aggregation; // value is based on children's values char *name; // bm->get_cmd() for metrics, unique string otherwise char *uname; // user-visible text char *unit; // see UNIT_* defines diff --git a/gprofng/src/Dbe.cc b/gprofng/src/Dbe.cc index 95daa72..09b7f94 100644 --- a/gprofng/src/Dbe.cc +++ b/gprofng/src/Dbe.cc @@ -9591,14 +9591,12 @@ dbeGetTLDataRepVals (VMode view_mode, hrtime_t start_ts, hrtime_t delta, } if (sampleVals != NULL) { - Sample* sample = (Sample*) packets->getObjValue (PROP_SMPLOBJ, packetIdx); - if (!sample || !sample->get_usage ()) - sample = sample; - else + Sample *sample = (Sample*) packets->getObjValue (PROP_SMPLOBJ, packetIdx); + if (sample != NULL) { - PrUsage* prusage = sample->get_usage (); - Vector<long long> *mstateVals = prusage->getMstateValues (); - sampleVals->store (eventIdx, mstateVals); + PrUsage *prusage = sample->get_usage (); + if (prusage != NULL) + sampleVals->store (eventIdx, prusage->getMstateValues ()); } } } diff --git a/gprofng/src/DbeSession.cc b/gprofng/src/DbeSession.cc index a6808d8..3649357 100644 --- a/gprofng/src/DbeSession.cc +++ b/gprofng/src/DbeSession.cc @@ -1164,8 +1164,6 @@ DbeSession::open_experiment (Experiment *exp, char *path) closedir (exp_dir); exp_names->sort (dir_name_cmp); Experiment **t_exp_list = new Experiment *[exp_names->size ()]; - int nsubexps = 0; - for (int j = 0, jsz = exp_names->size (); j < jsz; j++) { t_exp_list[j] = NULL; @@ -1222,7 +1220,6 @@ DbeSession::open_experiment (Experiment *exp, char *path) dexp->open (dpath); append (dexp); t_exp_list[j] = dexp; - nsubexps++; dexp->set_clock (exp->clock); // DbeView add_experiment() is split into two parts diff --git a/gprofng/src/Disasm.cc b/gprofng/src/Disasm.cc index e41bf67..19f2174 100644 --- a/gprofng/src/Disasm.cc +++ b/gprofng/src/Disasm.cc @@ -49,7 +49,6 @@ struct DisContext }; static const int MAX_DISASM_STR = 2048; -static const int MAX_INSTR_SIZE = 8; Disasm::Disasm (char *fname) { diff --git a/gprofng/src/Experiment.cc b/gprofng/src/Experiment.cc index a1f78fc..627a755c 100644 --- a/gprofng/src/Experiment.cc +++ b/gprofng/src/Experiment.cc @@ -1937,8 +1937,6 @@ private: } Experiment *exp; - char *hostname; - hrtime_t time, tstamp; }; void diff --git a/gprofng/src/HashMap.h b/gprofng/src/HashMap.h index 918c0dc..c5fdd34 100644 --- a/gprofng/src/HashMap.h +++ b/gprofng/src/HashMap.h @@ -78,9 +78,8 @@ copy_key (uint64_t a) } template<> inline void -delete_key (uint64_t a) +delete_key (uint64_t) { - a = a; } template<> inline int diff --git a/gprofng/src/IOActivity.h b/gprofng/src/IOActivity.h index cf462cf..f3a22ad 100644 --- a/gprofng/src/IOActivity.h +++ b/gprofng/src/IOActivity.h @@ -78,7 +78,6 @@ private: Hist_data *hist_data_file_all; Hist_data *hist_data_vfd_all; Hist_data *hist_data_callstack_all; - Hist_data *hist_data_callstack; DbeView *dbev; }; diff --git a/gprofng/src/collctrl.cc b/gprofng/src/collctrl.cc index 95324fb..d341cb0 100644 --- a/gprofng/src/collctrl.cc +++ b/gprofng/src/collctrl.cc @@ -876,9 +876,7 @@ Coll_Ctrl::set_clkprof (const char *string, char** warn) double dval = strtod (string, &endchar); if (*endchar == 'm' || *endchar == 0) /* user specified milliseconds */ dval = dval * 1000.; - else if (*endchar == 'u') /* user specified microseconds */ - dval = dval; - else + else if (*endchar != 'u') return dbe_sprintf (GTXT ("Unrecognized clock-profiling interval `%s'\n"), string); nclkprof_timer = (int) (dval + 0.5); } @@ -2788,7 +2786,7 @@ Coll_Ctrl::get (char * control) } if (!strncmp (control, ipc_str_javaprof, len)) { - if ((java_mode == 0)) + if (java_mode == 0) return strdup (ipc_str_off); return strdup (ipc_str_on); } @@ -2804,7 +2802,7 @@ Coll_Ctrl::get (char * control) } if (!strncmp (control, ipc_str_sample_sig, len)) { - if ((sample_sig == 0)) + if (sample_sig == 0) return strdup (ipc_str_off); char *str_signal = find_signal_name (sample_sig); if (str_signal != NULL) @@ -2838,15 +2836,15 @@ Coll_Ctrl::get (char * control) } if (!strncmp (control, ipc_str_iotrace, len)) { - if ((iotrace_enabled == 0)) + if (iotrace_enabled == 0) return strdup (ipc_str_off); return strdup (ipc_str_on); } if (!strncmp (control, ipc_str_count, len)) { - if ((count_enabled == 0)) + if (count_enabled == 0) return strdup (ipc_str_off); - if ((count_enabled < 0)) + if (count_enabled < 0) return strdup ("on\nstatic"); return strdup (ipc_str_on); } diff --git a/gprofng/src/collctrl.h b/gprofng/src/collctrl.h index 788df7a..0a98f71 100644 --- a/gprofng/src/collctrl.h +++ b/gprofng/src/collctrl.h @@ -271,7 +271,6 @@ private: char *node_name; /* name of machine on which experiment is run */ long ncpus; /* number of online CPUs */ int cpu_clk_freq; /* chip clock (MHz.), as reported from processor_info */ - int cpc_cpuver; /* chip version, as reported from libcpc */ long sys_resolution; /* system clock resolution */ int sample_period; /* period for sampling, seconds */ int sample_default; /* if period for sampling set by default */ diff --git a/gprofng/src/collector_module.h b/gprofng/src/collector_module.h index bb48ead..ebcdbca 100644 --- a/gprofng/src/collector_module.h +++ b/gprofng/src/collector_module.h @@ -40,12 +40,12 @@ struct tm; * If you add any, please put it in the right place */ typedef struct CollectorUtilFuncs { - int (*access)(); + int (*access)(const char *, int); int (*atoi)(const char *nptr); void *(*calloc)(size_t nelem, size_t elsize); int (*clearenv)(void); int (*close)(int); - int (*closedir)(); + int (*closedir)(DIR *); int (*execv)(const char *path, char *const argv[]); void (*exit)(int status); int (*fclose)(FILE *stream); @@ -66,20 +66,20 @@ typedef struct CollectorUtilFuncs off_t (*lseek)(int fd, off_t offset, int whence); void *(*malloc)(size_t size); void *(*memset)(void *s1, int c, size_t n); - int (*mkdir)(); + int (*mkdir)(const char *, mode_t); time_t (*mktime)(struct tm *timeptr); void *(*mmap)(void *, size_t, int, int, int, off_t); - void *(*mmap64_)(); - int (*munmap)(); + void *(*mmap64_)(void *, size_t, int, int, int, off_t); + int (*munmap)(void *, size_t); int (*open)(const char *, int, ...); int (*open_bare)(const char *, int, ...); - DIR *(*opendir)(); + DIR *(*opendir)(const char *); int (*pclose)(FILE *stream); FILE *(*popen)(const char *command, const char *mode); int (*putenv)(char *string); - ssize_t (*pwrite)(); - ssize_t (*pwrite64_)(); - ssize_t (*read)(); + ssize_t (*pwrite)(int, const void *, size_t, off_t); + ssize_t (*pwrite64_)(int, const void *, size_t, off_t); + ssize_t (*read)(int, void *, size_t); int (*setenv)(const char *name, const char *value, int overwrite); int (*sigfillset)(sigset_t *set); int (*sigprocmask)(int how, const sigset_t *set, sigset_t *oldset); @@ -112,7 +112,7 @@ typedef struct CollectorUtilFuncs int (*unsetenv)(const char *name); int (*vsnprintf)(char *str, size_t size, const char *format, va_list ap); pid_t (*waitpid)(pid_t pid, int *stat_loc, int options); - ssize_t (*write)(); + ssize_t (*write)(int, void *, size_t); double (*atof)(); void *n_a; } CollectorUtilFuncs; diff --git a/gprofng/src/gp-display-src.cc b/gprofng/src/gp-display-src.cc index 200e608..24af375 100644 --- a/gprofng/src/gp-display-src.cc +++ b/gprofng/src/gp-display-src.cc @@ -75,14 +75,6 @@ private: bool v_opt; int multiple; char *str_compcom; - bool hex_visible; - int src_visible; - int vis_src; - int vis_dis; - int threshold_src; - int threshold_dis; - int threshold; - int vis_bits; }; static int diff --git a/gprofng/src/gp-print.h b/gprofng/src/gp-print.h index 1b748ea..1a8ad3b 100644 --- a/gprofng/src/gp-print.h +++ b/gprofng/src/gp-print.h @@ -19,7 +19,7 @@ MA 02110-1301, USA. */ #ifndef _GP_PRINT_H -#define _ER_PRINT_H +#define _GP_PRINT_H #include "Command.h" #include "DbeApplication.h" diff --git a/gprofng/src/hwc_intel_icelake.h b/gprofng/src/hwc_intel_icelake.h index 46f4ac5..4fe2e26 100644 --- a/gprofng/src/hwc_intel_icelake.h +++ b/gprofng/src/hwc_intel_icelake.h @@ -24,8 +24,10 @@ #define SH(val, n) (((unsigned long long) (val)) << n) #define I(nm, event, umask, edge, cmask, inv, \ offcore_rsp, ldlat, frontend, period, mtr) \ - INIT_HWC(nm, mtr, SH(event, 0) | SH(umask, 8) | SH(edge, 18) \ - | SH(cmask, 24) | SH(inv, 23), PERF_TYPE_RAW), \ + .use_perf_event_type = 1, .type = PERF_TYPE_RAW, \ + .name = (nm), .metric = (mtr), .reg_num = REGNO_ANY, \ + .config = SH(event, 0) | SH(umask, 8) | SH(edge, 18) | SH(cmask, 24) \ + | SH(inv, 23), \ .config1 = SH(offcore_rsp, 0) | SH(ldlat, 0) | SH(frontend, 0), \ .val = period diff --git a/gprofng/src/util.cc b/gprofng/src/util.cc index 201f708..228140b 100644 --- a/gprofng/src/util.cc +++ b/gprofng/src/util.cc @@ -741,17 +741,13 @@ get_relative_link (const char *path_from, const char *path_to) s2 = canonical_path (s2); long l = dbe_sstrlen (s1); // try to find common directories - int common_slashes = 0; int last_common_slash = -1; for (int i = 0; i < l; i++) { - if (s1[i] != s2[i]) break; - if (s1[i] == 0) break; + if (s1[i] != s2[i] || s1[i] == 0) + break; if (s1[i] == '/') - { - common_slashes++; - last_common_slash = i; - } + last_common_slash = i; } // find slashes in remaining path_to int slashes = 0; |