Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Janis Johnson <janisjo@codesourcery.com>
To: Mark Kettenis <mark.kettenis@xs4all.nl>
Cc: gdb-patches@sourceware.org
Subject: Re: [RFA] bailing out of tests with multiple errors or timeouts
Date: Thu, 05 May 2011 15:59:00 -0000	[thread overview]
Message-ID: <4DC2C957.2070307@codesourcery.com> (raw)
In-Reply-To: <201105041810.p44IAVlq008135@glazunov.sibelius.xs4all.nl>

On 05/04/2011 11:10 AM, Mark Kettenis wrote:
>> Date: Wed, 04 May 2011 10:46:50 -0700
>> From: Janis Johnson <janisjo@codesourcery.com>
> 
>> +    timeout {
>> +	# A timeout here often indicates more timeouts later in the test.
>> +	fail "$test (timeout)"
>> +	fail "Remainder of test"
>> +	return
>> +    }
> 
> Often, so not always.  Really the best way to get rid of these is to
> fix the underlying problem.

I agree, the best way to get rid of timeouts and ERRORs in gdb tests is
to fix gdb.  Unfortunately, that's not an easy task for a target whose
support is buggy or incomplete whether in gdb, simulator, hardware, or
operating system.  It's painful to test on such targets when a few tests
time out multiple times.

What I'm looking for is a way to bail out of tests that are known to
time out multiple times on some targets, or have a cascade of ERRORs.
That might involve changes to specific tests like the examples I posted;
it might involve setting a magic variable somewhere to bail out of any
test that times out or runs into an error; it might involve having a
list of tests that bail out after a timeout or error.  Ideally there
would be an easily-recognizable failure message for those tests to
indicate it's more than a normal failure; maybe UNRESOLVED.

I'm not familiar enough with the workings of the gdb test infrastructure
or the general conventions of the gdb testsuite to know what might be
appropriate, but presumably others have run into this problem and have
ideas.

Janis Johnson
CodeSourcery / Mentor Graphics


  reply	other threads:[~2011-05-05 15:59 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-04 17:46 Janis Johnson
2011-05-04 18:13 ` Mark Kettenis
2011-05-05 15:59   ` Janis Johnson [this message]
2011-05-05 18:57     ` Joel Brobecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4DC2C957.2070307@codesourcery.com \
    --to=janisjo@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=mark.kettenis@xs4all.nl \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox