Messages in this thread | | | Subject | Re: [PATCH 1/2] perf: Fix POWER9 metric 'lsu_other_stall_cpi' | From | kajoljain <> | Date | Fri, 8 May 2020 12:31:32 +0530 |
| |
On 5/8/20 12:01 PM, kajoljain wrote: > > > On 5/7/20 9:58 PM, Paul A. Clarke wrote: >> From: "Paul A. Clarke" <pc@us.ibm.com> >> >> The metric definition is too long for the current value of EXPR_MAX_OTHER. >> Increase the value EXPR_MAX_OTHER sufficiently to allow >> 'lsu_other_stall_cpi' to build properly. >> >> Before: >> -- >> $ perf list | grep lsu_other >> lsu_other_stall_cpi >> # perf stat -a --metrics lsu_other_stall_cpi /bin/sleep 1 >> Cannot find metric or group `lsu_other_stall_cpi' >> >> Usage: perf stat [<options>] [<command>] >> >> -M, --metrics <metric/metric group list> >> monitor specified metrics or metric groups (separated by ,) >> -- >> >> After: >> -- >> # perf stat -a --metrics lsu_other_stall_cpi /bin/sleep 1 >> >> Performance counter stats for 'system wide': >> >> 438,086,889 pm_cmplu_stall_lsu # 1.74 lsu_other_stall_cpi >> [...] >> -- >> >> Signed-off-by: Paul A. Clarke <pc@us.ibm.com> >> --- >> tools/perf/util/expr.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tools/perf/util/expr.h b/tools/perf/util/expr.h >> index 87d627bb699b..a0991959cca4 100644 >> --- a/tools/perf/util/expr.h >> +++ b/tools/perf/util/expr.h >> @@ -2,7 +2,7 @@ >> #ifndef PARSE_CTX_H >> #define PARSE_CTX_H 1 >> >> -#define EXPR_MAX_OTHER 20 >> +#define EXPR_MAX_OTHER 28 > > Hi Paul, > Ian already add one patch to increase EXPR_MAX_OTHER value to 68 which is Acked-by: Jiri
Sorry its 64 not 68 and I missed earlier response from Arnaldo and Ian.
Thanks, Kajol Jain > > Link to the patch: https://lkml.org/lkml/2020/5/1/1023 > > Thanks, > Kajol Jain > >> #define MAX_PARSE_ID EXPR_MAX_OTHER >> >> struct expr_parse_id { >>
| |