This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Regression for attach from stdin [Re: [pushed] Re: [PATCH v6 0/2] enable target-async by default]
- From: Mark Wielaard <mjw at redhat dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: Pedro Alves <palves at redhat dot com>, gdb-patches at sourceware dot org
- Date: Wed, 02 Jul 2014 10:59:34 +0200
- Subject: Re: Regression for attach from stdin [Re: [pushed] Re: [PATCH v6 0/2] enable target-async by default]
- Authentication-results: sourceware.org; auth=none
- References: <1400878753-24688-1-git-send-email-palves at redhat dot com> <538739A2 dot 2050105 at redhat dot com> <20140701162830 dot GA25877 at host2 dot jankratochvil dot net>
On Tue, 2014-07-01 at 18:28 +0200, Jan Kratochvil wrote:
> On Thu, 29 May 2014 15:44:02 +0200, Pedro Alves wrote:
> > I went ahead and pushed this all in, with Eli's comments addressed.
>
> 329ea57934a9d4b250a0b417af1ec47bc2d0ceb6 is the first bad commit
> commit 329ea57934a9d4b250a0b417af1ec47bc2d0ceb6
> Author: Pedro Alves <palves@redhat.com>
> Date: Thu May 29 19:58:57 2014 +0100
> enable target async by default; separate MI and target notions of async
>
> This affects GCC testsuite where guality.exp does not start at all with:
> gdb: took too long to attach
Thanks for tracking this down Jan. Should this patch be reverted or
should gcc guality.h be patched to work around the new async default
somehow?
Currently I am using the following patch to guality.h in gcc which seems
to work around it, but I don't really understand why:
diff --git a/gcc/testsuite/gcc.dg/guality/guality.h
b/gcc/testsuite/gcc.dg/guality/guality.h
index 8b657f2..d83e270 100644
--- a/gcc/testsuite/gcc.dg/guality/guality.h
+++ b/gcc/testsuite/gcc.dg/guality/guality.h
@@ -243,6 +243,7 @@ main (int argc, char *argv[])
|| fprintf (guality_gdb_input, "\
set height 0\n\
attach %i\n\
+print guality_attached\n\
set guality_attached = 1\n\
b %i\n\
continue\n\
Thanks,
Mark