Messages in this thread | | | Date | Wed, 28 Sep 2022 11:28:19 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH] perf test: Fix test case 87 ("perf record tests") for hybrid systems |
| |
Em Tue, Sep 27, 2022 at 01:15:13PM +0800, zhengjun.xing@linux.intel.com escreveu: > From: Zhengjun Xing <zhengjun.xing@linux.intel.com> > > The test case 87 ("perf record tests") failed on hybrid systems,the event > "cpu/br_inst_retired.near_call/p" is only for non-hybrid system. Correct > the test event to support both non-hybrid and hybrid systems. > > Before: > # ./perf test 87 > 87: perf record tests : FAILED! > > After: > # ./perf test 87 > 87: perf record tests : Ok
Looks sane, applied.
Ian, consider providing your Acked-by,
- Arnaldo
> Fixes: 24f378e66021 ("perf test: Add basic perf record tests") > Signed-off-by: Zhengjun Xing <zhengjun.xing@linux.intel.com> > Reviewed-by: Kan Liang <kan.liang@linux.intel.com> > --- > tools/perf/tests/shell/record.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/perf/tests/shell/record.sh b/tools/perf/tests/shell/record.sh > index 00c7285ce1ac..301f95427159 100755 > --- a/tools/perf/tests/shell/record.sh > +++ b/tools/perf/tests/shell/record.sh > @@ -61,7 +61,7 @@ test_register_capture() { > echo "Register capture test [Skipped missing registers]" > return > fi > - if ! perf record -o - --intr-regs=di,r8,dx,cx -e cpu/br_inst_retired.near_call/p \ > + if ! perf record -o - --intr-regs=di,r8,dx,cx -e br_inst_retired.near_call:p \ > -c 1000 --per-thread true 2> /dev/null \ > | perf script -F ip,sym,iregs -i - 2> /dev/null \ > | egrep -q "DI:" > -- > 2.25.1
--
- Arnaldo
| |