Messages in this thread | | | Subject | Re: [PATCH 2/2] perf buildid-cache: Add test for 16-byte build-id | From | Nicholas Fraser <> | Date | Tue, 16 Feb 2021 16:04:40 -0500 |
| |
Sorry, I just realized I forgot to run checkpatch.pl on these; they're missing the --signoff. I'll re-send.
On 2021-02-16 3:38 p.m., Nicholas Fraser wrote: > tests/shell/buildid.sh added an ELF executable with an MD5 build-id to > the perf debug cache but did not check whether the object was printed > by a subsequent call to "perf buildid-cache -l". It was being omitted > from the list. > > A previous commit fixed the bug that left it out of the list. This adds > a test for it. > --- > tools/perf/tests/shell/buildid.sh | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/tools/perf/tests/shell/buildid.sh b/tools/perf/tests/shell/buildid.sh > index 4861a20edee2..de02a23b7c7b 100755 > --- a/tools/perf/tests/shell/buildid.sh > +++ b/tools/perf/tests/shell/buildid.sh > @@ -50,6 +50,12 @@ check() > exit 1 > fi > > + ${perf} buildid-cache -l|grep $id > + if [ $? -ne 0 ]; then > + echo "failed: ${id} is not reported by \"perf buildid-cache -l\"" > + exit 1 > + fi > + > echo "OK for ${1}" > } > >
| |