Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Pierre Muller" <muller@ics.u-strasbg.fr>
To: "'Pedro Alves'" <pedro_alves@portugalmail.pt>
Cc: "'Joel Brobecker'" <brobecker@adacore.com>,
		<gdb-patches@sourceware.org>
Subject: RE: [RFA] ARI fix: Remove asprintf in breakpoint.c
Date: Tue, 09 Oct 2007 16:08:00 -0000	[thread overview]
Message-ID: <003401c80a8c$04f3b9c0$0edb2d40$@u-strasbg.fr> (raw)
In-Reply-To: <003101c80a82$5520fa70$ff62ef50$@u-strasbg.fr>

  With my change in gdb.base/sigstep.c
(and similar changes in gdb.base/sigXXX.c files)
and my change of gdb.exp to avoid interruption in
gdb.fortran/array-element.exp
I was able to complete a Cygwin testsuite in 45 minutes.
  A lot better than previously.

I did not use Pedro's patch to fix the
i386_skip_prologue, thus the number of failures
are still huge.

  Results are:
                === gdb Summary ===

# of expected passes            10186
# of unexpected failures        509
# of expected failures          59
# of known failures             30
# of unresolved testcases       38
# of untested testcases         8
# of unsupported tests          16
/usr/local/src/cvs/build/gdb/testsuite/../../gdb/gdb version
6.7.50-20071009-cvs -nx

gdb.pascal directory gives me just
a XFAIL for "Breakpoint 2" due to gpc problems with line information
and a KFAIL for "ptyp 'a single string'"

 Thus I should be able to test my patches locally now...

Pierre Muller
Pasca language maintainer.


> -----Original Message-----
> From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] On Behalf Of Pierre Muller
> Sent: Tuesday, October 09, 2007 4:40 PM
> To: 'Pedro Alves'
> Cc: 'Joel Brobecker'; gdb-patches@sourceware.org
> Subject: RE: [RFA] ARI fix: Remove asprintf in breakpoint.c
> 
>  The main problem was
> gdb.base/sigstep.exp
> that took about 2 hours and 20 min to complete alone :(
>   The main reason is that this is just a huge list of timeouts
> related to the fact that cygin only accepts
> ITIMER_REAL as first arg of setitimer.
>   I will send a patch to fix this, by checking the return
> value of setitimer.
> 
> Pierre
> 
> > -----Original Message-----
> > From: Pedro Alves [mailto:alves.ped@gmail.com] On Behalf Of Pedro
> Alves
> > Sent: Monday, October 08, 2007 9:34 PM
> > To: Pierre Muller
> > Cc: 'Joel Brobecker'; gdb-patches@sourceware.org
> > Subject: Re: [RFA] ARI fix: Remove asprintf in breakpoint.c
> >
> > Pierre Muller wrote:
> > > No, sorry, but the testsuite on cygwin is really a pain,
> > > it has some signal problems that makes the testsuite
> > > horribly slow, with tons of timeouts...
> > > and no I have no easy linux access.
> > >
> >
> > I know what you mean ...
> >
> > I use these patches to be able to test on Cygwin less painfully:
> >
> >   [PATCH] i386_skip_prologue
> >   http://sources.redhat.com/ml/gdb-patches/2006-12/msg00140.html
> >
> >   [[PATCH] Line info in the presence of __main, and
> > NOTE_INSN_FUNCTION_BEG after
> > scheduling:]
> >   http://gcc.gnu.org/ml/gcc-patches/2006-12/msg00633.html
> >
> > The gcc patch is really outdated, and doesn't apply to
> > current gcc svn, so I still have a gcc build from 20061123
> > to test gdb.  Far from ideal, but otherwise, many testcases timeout
> > and go untested.
> >
> > I keep wanting to fix this correctly, but I never get around to it.
> >  From the last discussion on the subject, the idea was to teach the
> > i386 (and any other arch that cares) prologue reader about
> > __main as a first step.  I don't think that's hard.
> >
> > Cheers,
> > Pedro Alves
> >
> >
> 
> 




      reply	other threads:[~2007-10-09 15:49 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-08 12:21 Pierre Muller
2007-10-08 13:42 ` Joel Brobecker
2007-10-08 13:51   ` Pierre Muller
2007-10-08 15:02     ` Joel Brobecker
2007-10-08 15:24       ` Pierre Muller
2007-10-08 16:09         ` Daniel Jacobowitz
2007-10-08 16:25           ` Joel Brobecker
2007-10-08 17:18             ` Daniel Jacobowitz
2007-10-08 19:38     ` Pedro Alves
2007-10-09 14:46       ` Pierre Muller
2007-10-09 16:08         ` Pierre Muller [this message]

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='003401c80a8c$04f3b9c0$0edb2d40$@u-strasbg.fr' \
    --to=muller@ics.u-strasbg.fr \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=pedro_alves@portugalmail.pt \
    /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