[8.3 backport][gdb/testsuite] Fix regexp in skip_opencl_tests

Tom de Vries tdevries@suse.de
Thu Sep 12 18:58:00 GMT 2019


On 29-04-19 16:54, Tom de Vries wrote:
> Hi,
> 
> When running gdb-caching-proc.exp, if skip_opencl_tests fails like this:
> ...
> (gdb) run
> Starting program: \
>   build/gdb/testsuite/outputs/gdb.base/gdb-caching-proc/opencltest13530.x
> CHK_ERR (clGetPlatformIDs (1, &platform, NULL), -1001)
> src/gdb/testsuite/lib/opencl_hostapp.c:73 error: Unknown
> [Inferior 1 (process 13600) exited with code 01]
> (gdb)
> skip_opencl_tests: OpenCL support not detected
> ...
> then this regexp in skip_opencl_tests fails to match:
> ...
>         -re ".*$inferior_exited_re code.*${gdb_prompt} $" {
> ...
> so instead we hit the default clause after a 30 seconds timeout.  With the
> iteration count set at 10, we end up taking 6 minutes to run this test-case.
> 
> Fix this by adding the missing "with" in the regexp, bring back the runtime to
> half a minute.
> 
> Tested on x86_64-linux.
> 
> Committed to trunk.
> 

Hi,

ok to backport to 8.3 branch?

This applies cleanly, and reduces testing time with ~5 minutes (with a
total testing time of ~30 minutes) on my laptop.

Thanks,
- Tom

> 
> [gdb/testsuite] Fix regexp in skip_opencl_tests
> 
> gdb/testsuite/ChangeLog:
> 
> 2019-04-29  Tom de Vries  <tdevries@suse.de>
> 
> 	* lib/opencl.exp (skip_opencl_tests): Add missing "with" in regexp.
> 
> ---
>  gdb/testsuite/lib/opencl.exp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gdb/testsuite/lib/opencl.exp b/gdb/testsuite/lib/opencl.exp
> index 4d353de246..33eb437bc8 100644
> --- a/gdb/testsuite/lib/opencl.exp
> +++ b/gdb/testsuite/lib/opencl.exp
> @@ -58,7 +58,7 @@ gdb_caching_proc skip_opencl_tests {
>              verbose -log "\n$me: OpenCL support detected"
>              set result 0
>          }
> -        -re ".*$inferior_exited_re code.*${gdb_prompt} $" {
> +        -re ".*$inferior_exited_re with code.*${gdb_prompt} $" {
>              verbose -log "\n$me: OpenCL support not detected"
>              set result 1
>          }
> 



More information about the Gdb-patches mailing list