Message ID | 20220916104904.99798-1-atrajeev@linux.vnet.ibm.com (mailing list archive) |
---|---|
State | Handled Elsewhere |
Headers | show |
Series | [V2] tools/perf/tests: Fix perf probe error log check in skip_if_no_debuginfo | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/github-powerpc_perf | success | Successfully ran 10 jobs. |
snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 6 jobs. |
snowpatch_ozlabs/github-powerpc_kernel_qemu | success | Successfully ran 23 jobs. |
On 9/16/22 4:19 PM, Athira Rajeev wrote: > The perf probe related tests like probe_vfs_getname.sh which > is in "tools/perf/tests/shell" directory have dependency on > debuginfo information in the kernel. Currently debuginfo > check is handled by skip_if_no_debuginfo function in the > file "lib/probe_vfs_getname.sh". skip_if_no_debuginfo function > looks for this specific error log from perf probe to skip > the testcase: > > <<>> > Failed to find the path for the kernel|Debuginfo-analysis is > not supported > <>> > > But in some case, like this one in powerpc, while running this > test, observed error logs is: > > <<>> > The /lib/modules/<version>/build/vmlinux file has no debug information. > Rebuild with CONFIG_DEBUG_INFO=y, or install an appropriate debuginfo > package. > Error: Failed to add events. > <<>> > > Update the skip_if_no_debuginfo function to include the above > error, to skip the test in these scenarios too. > > Reported-by: Disha Goel<disgoel@linux.vnet.ibm.com> > Signed-off-by: Athira Rajeev<atrajeev@linux.vnet.ibm.com> Tested-by: Disha Goel<disgoel@linux.vnet.ibm.com> > --- > changelog: > v1 -> v2: > Corrected formatting of spaces in error log. > With spaces in v1 of the patch, the egrep search was > considering spaces also. > > tools/perf/tests/shell/lib/probe_vfs_getname.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/perf/tests/shell/lib/probe_vfs_getname.sh b/tools/perf/tests/shell/lib/probe_vfs_getname.sh > index 5b17d916c555..b616d42bd19d 100644 > --- a/tools/perf/tests/shell/lib/probe_vfs_getname.sh > +++ b/tools/perf/tests/shell/lib/probe_vfs_getname.sh > @@ -19,6 +19,6 @@ add_probe_vfs_getname() { > } > > skip_if_no_debuginfo() { > - add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)" && return 2 > + add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)|(file has no debug information)" && return 2 > return 1 > }
On 9/16/22 16:19, Athira Rajeev wrote: > The perf probe related tests like probe_vfs_getname.sh which > is in "tools/perf/tests/shell" directory have dependency on > debuginfo information in the kernel. Currently debuginfo > check is handled by skip_if_no_debuginfo function in the > file "lib/probe_vfs_getname.sh". skip_if_no_debuginfo function > looks for this specific error log from perf probe to skip > the testcase: > > <<>> > Failed to find the path for the kernel|Debuginfo-analysis is > not supported > <>> > > But in some case, like this one in powerpc, while running this > test, observed error logs is: > > <<>> > The /lib/modules/<version>/build/vmlinux file has no debug information. > Rebuild with CONFIG_DEBUG_INFO=y, or install an appropriate debuginfo > package. > Error: Failed to add events. > <<>> > > Update the skip_if_no_debuginfo function to include the above > error, to skip the test in these scenarios too. Patch looks good to me. Reviewed-By: Kajol Jain <kjain@linux.ibm.com> Thanks, Kajol Jain > > Reported-by: Disha Goel <disgoel@linux.vnet.ibm.com> > Signed-off-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com> > --- > changelog: > v1 -> v2: > Corrected formatting of spaces in error log. > With spaces in v1 of the patch, the egrep search was > considering spaces also. > > tools/perf/tests/shell/lib/probe_vfs_getname.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/perf/tests/shell/lib/probe_vfs_getname.sh b/tools/perf/tests/shell/lib/probe_vfs_getname.sh > index 5b17d916c555..b616d42bd19d 100644 > --- a/tools/perf/tests/shell/lib/probe_vfs_getname.sh > +++ b/tools/perf/tests/shell/lib/probe_vfs_getname.sh > @@ -19,6 +19,6 @@ add_probe_vfs_getname() { > } > > skip_if_no_debuginfo() { > - add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)" && return 2 > + add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)|(file has no debug information)" && return 2 > return 1 > }
Em Fri, Sep 16, 2022 at 06:35:41PM +0530, kajoljain escreveu: > > > On 9/16/22 16:19, Athira Rajeev wrote: > > The perf probe related tests like probe_vfs_getname.sh which > > is in "tools/perf/tests/shell" directory have dependency on > > debuginfo information in the kernel. Currently debuginfo > > check is handled by skip_if_no_debuginfo function in the > > file "lib/probe_vfs_getname.sh". skip_if_no_debuginfo function > > looks for this specific error log from perf probe to skip > > the testcase: > > > > <<>> > > Failed to find the path for the kernel|Debuginfo-analysis is > > not supported > > <>> > > > > But in some case, like this one in powerpc, while running this > > test, observed error logs is: > > > > <<>> > > The /lib/modules/<version>/build/vmlinux file has no debug information. > > Rebuild with CONFIG_DEBUG_INFO=y, or install an appropriate debuginfo > > package. > > Error: Failed to add events. > > <<>> > > > > Update the skip_if_no_debuginfo function to include the above > > error, to skip the test in these scenarios too. > > Patch looks good to me. > > Reviewed-By: Kajol Jain <kjain@linux.ibm.com> Thanks, applied. - Arnaldo > Thanks, > Kajol Jain > > > > > Reported-by: Disha Goel <disgoel@linux.vnet.ibm.com> > > Signed-off-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com> > > --- > > changelog: > > v1 -> v2: > > Corrected formatting of spaces in error log. > > With spaces in v1 of the patch, the egrep search was > > considering spaces also. > > > > tools/perf/tests/shell/lib/probe_vfs_getname.sh | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tools/perf/tests/shell/lib/probe_vfs_getname.sh b/tools/perf/tests/shell/lib/probe_vfs_getname.sh > > index 5b17d916c555..b616d42bd19d 100644 > > --- a/tools/perf/tests/shell/lib/probe_vfs_getname.sh > > +++ b/tools/perf/tests/shell/lib/probe_vfs_getname.sh > > @@ -19,6 +19,6 @@ add_probe_vfs_getname() { > > } > > > > skip_if_no_debuginfo() { > > - add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)" && return 2 > > + add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)|(file has no debug information)" && return 2 > > return 1 > > }
diff --git a/tools/perf/tests/shell/lib/probe_vfs_getname.sh b/tools/perf/tests/shell/lib/probe_vfs_getname.sh index 5b17d916c555..b616d42bd19d 100644 --- a/tools/perf/tests/shell/lib/probe_vfs_getname.sh +++ b/tools/perf/tests/shell/lib/probe_vfs_getname.sh @@ -19,6 +19,6 @@ add_probe_vfs_getname() { } skip_if_no_debuginfo() { - add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)" && return 2 + add_probe_vfs_getname -v 2>&1 | egrep -q "^(Failed to find the path for the kernel|Debuginfo-analysis is not supported)|(file has no debug information)" && return 2 return 1 }
The perf probe related tests like probe_vfs_getname.sh which is in "tools/perf/tests/shell" directory have dependency on debuginfo information in the kernel. Currently debuginfo check is handled by skip_if_no_debuginfo function in the file "lib/probe_vfs_getname.sh". skip_if_no_debuginfo function looks for this specific error log from perf probe to skip the testcase: <<>> Failed to find the path for the kernel|Debuginfo-analysis is not supported <>> But in some case, like this one in powerpc, while running this test, observed error logs is: <<>> The /lib/modules/<version>/build/vmlinux file has no debug information. Rebuild with CONFIG_DEBUG_INFO=y, or install an appropriate debuginfo package. Error: Failed to add events. <<>> Update the skip_if_no_debuginfo function to include the above error, to skip the test in these scenarios too. Reported-by: Disha Goel <disgoel@linux.vnet.ibm.com> Signed-off-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com> --- changelog: v1 -> v2: Corrected formatting of spaces in error log. With spaces in v1 of the patch, the egrep search was considering spaces also. tools/perf/tests/shell/lib/probe_vfs_getname.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)