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}"
> }
>
>
