From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4143 invoked by alias); 3 May 2013 11:38:10 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 4134 invoked by uid 89); 3 May 2013 11:38:10 -0000 X-Spam-SWARE-Status: No, score=-4.7 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL autolearn=ham version=3.3.1 Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Fri, 03 May 2013 11:38:09 +0000 Received: from svr-orw-fem-01.mgc.mentorg.com ([147.34.98.93]) by relay1.mentorg.com with esmtp id 1UYEJT-0005jr-3z from Hafiz_Abid@mentor.com ; Fri, 03 May 2013 04:38:07 -0700 Received: from SVR-IES-FEM-01.mgc.mentorg.com ([137.202.0.104]) by svr-orw-fem-01.mgc.mentorg.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675); Fri, 3 May 2013 04:38:06 -0700 Received: from abidh-ubunto1104 (137.202.0.76) by SVR-IES-FEM-01.mgc.mentorg.com (137.202.0.104) with Microsoft SMTP Server (TLS) id 14.2.247.3; Fri, 3 May 2013 12:38:04 +0100 Date: Fri, 03 May 2013 11:38:00 -0000 From: "Abid, Hafiz" Subject: Re: [patch] Change trace buffer size(v5) To: Pedro Alves CC: Yao Qi , , , In-Reply-To: <5182A2F8.2050900@redhat.com> (from palves@redhat.com on Thu May 2 18:31:36 2013) Message-ID: <1367581084.5163.1@abidh-ubunto1104> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; delsp=Yes; format=Flowed Content-Disposition: inline Content-Transfer-Encoding: quoted-printable X-SW-Source: 2013-05/txt/msg00050.txt.bz2 On 02/05/13 18:31:36, Pedro Alves wrote: > Hi! >=20 > Unfortunately, I just noticed that this broke "tstart" if a > trace run is already running: >=20 > (gdb) tstart > A trace is running already. Start a new run? (y or n) y > Sending packet: $QTinit#59...Packet received: OK > Sending packet: $QTDP:3:0000000000410296:E:0:0#49...Packet received:=20=20 > OK > Sending packet: $QTDPsrc:3:410296:at:0:4:6d61696e#8a...Packet=20=20 > received: OK > Sending packet:=20=20 > $QTDV:1:0000000000000000:1:74726163655f74696d657374616d70#4f...Packet=20= =20 > received: OK > Sending packet:=20=20 > $:0000000000400200,000000000040021c:000000000040021c,000000000040023c:000= 000000040023c,0000000000400260:0000000000400260,00000000004002c0:0000000000= 4002c0,0000000000400ff8:0000000000400ff8,000000000040149e:000000000040149e,= 00000000004015b8:00000000004015b8,0000000000401628:0000000000401628,0000000= 000401688:0000000000401688,0000000000402210:0000000000402210,00000000004022= 1e:0000000000402220,00000000004029e0:00000000004029e0,000000000043a230:0000= 00000043a230,000000000043a239:000000000043a240,000000000044cafd:00000000004= 4cb00,000000000044e834:000000000044e838,00000000004560a4#ab...Packet=20=20 > received: > Sending packet: $QTDisconnected:0#e2...Packet received: OK > Sending packet: $QTBuffer:circular:0#f8...Packet received: OK > Sending packet: $QTBuffer:size:-1#8c...Packet received: E01 > Target returns error code '01'. > (gdb) >=20 > Perhaps gdbserver's QTinit handling should be calling > stop_tracing? >=20 > -- > Pedro Alves >=20 >=20 Hi Pedro, Thanks for letting me know. There is a testcase in=20=20 status-stop.exp(test_tstart_tstart) to check this case but it is not=20=20 working as intended. I will work on a patch. Regards, Abid >From gdb-patches-return-101045-listarch-gdb-patches=sources.redhat.com@sourceware.org Fri May 03 12:57:41 2013 Return-Path: Delivered-To: listarch-gdb-patches@sources.redhat.com Received: (qmail 13035 invoked by alias); 3 May 2013 12:57:41 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 13000 invoked by uid 89); 3 May 2013 12:57:37 -0000 X-Spam-SWARE-Status: No, score=-1.1 required=5.0 tests=BAYES_00,FREEMAIL_FROM,SPF_NEUTRAL autolearn=no version=3.3.1 Received: from slaygeal.uusia.org (HELO ip.uusia.org) (62.109.10.102) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Fri, 03 May 2013 12:57:34 +0000 Received: from [2a02:2560:6d4:26ca::1] (helo=waterlily.siamics.net) by ip.uusia.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.72) (envelope-from ) id 1UYFYI-0008Pd-5g; Fri, 03 May 2013 12:57:30 +0000 Received: from violet.siamics.net ([2001:470:1f13:1eb::1:1d]) by waterlily.siamics.net with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.72) (envelope-from ) id 1UYFYB-0007jE-Iu; Fri, 03 May 2013 19:57:23 +0700 Received: from localhost ([::1] helo=violet.siamics.net) by violet.siamics.net with esmtp (Exim 4.72) (envelope-from ) id 1UYFYA-000491-Mm; Fri, 03 May 2013 19:57:22 +0700 From: Ivan Shmakov To: bug-hurd@gnu.org, gdb-patches@sourceware.org Cc: Ivan Shmakov Subject: Re: [patch] for mig check in GDB's configure References: <8738u4sc19.fsf@kepler.schwinge.homeip.net> Date: Fri, 03 May 2013 12:57:00 -0000 In-Reply-To: (=?utf-8?B?IumZhuWysyIncw==?= message of "Fri, 3 May 2013 18:43:49 +0800") Message-ID: <87txmkxlu6.fsf@violet.siamics.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-SW-Source: 2013-05/txt/msg00051.txt.bz2 Content-length: 2572 >>>>> =E9=99=86=E5=B2=B3 writes: [=E2=80=A6] A few minor points. > From 13d3edd1f6dbbc20b2801cea1fc367bf9042f977 Mon Sep 17 00:00:00 2001 > From: hacklu > Date: Fri, 3 May 2013 18:27:08 +0800 > Subject: [PATCH] Patch check mig on GNU Hurd > 2013-05-3 hacklu There should be two spaces between the name and email, too. And the date should be 2013-05-03, as per ISO 8601. (One may want to check the respective Wikipedia article here.) As in, e. g.: $ date -uI=20 2013-05-03 $=20 Also, I'd urge you to use the =E2=80=9Creal=E2=80=9D full name -- the same= as you'd use when filing a GSoC application, or copyright assignment papers (as required by FSF), etc. > * configure.ac : uncorrectly check for mig on GUN Hurd =E2=80=A2 There should be no space before =E2=80=9C:=E2=80=9D; =E2=80=A2 the sentence should begin with a capital letter, and end with a period; =E2=80=A2 s/GUN/GNU/. Besides, I'm having trouble understanding the intended meaning of this sentence. Shouldn't it be, say, as follows instead? * configure.ac: Ensure MIG is available when building for GNU Hurd. [=E2=80=A6] > --- a/gdb/ChangeLog > +++ b/gdb/ChangeLog > @@ -1,3 +1,7 @@ > +2013-05-3 hacklu > + > + * configure.ac : uncorrectly check for mig on GUN Hurd > + * configure: Regenerate. > 2013-04-30 Samuel Thibault The same applies here. Also, there should be an empty line between the ChangeLog entries. [=E2=80=A6] > --- a/gdb/configure.ac > +++ b/gdb/configure.ac > @@ -488,6 +488,15 @@ AC_CHECK_TOOL(WINDRES, windres) > # Needed for GNU/Hurd. > AC_CHECK_TOOL(MIG, mig) > +case "${host}" in Note that =E2=80=98{}=E2=80=99 are superfluous here; case "$host" will do = the same, but is a tiny bit shorter. > + *-linux*|*-k*bsd-gnu*) > + ;; > + i[?]86-*-gnu*) > + if test "$MIG" =3D "" ; then > + AC_MSG_ERROR([MIG not found but required for $host]) > + fi > + ;; I guess that both cases should use the same indentation level =E2=80=94 either 4 spaces or 8 (or a TAB.) Check the rest of the GDB .ac code for the currently preferred style. Also, it's possible to avoid an indentation in the first case altogether, like: + *-linux*|*-k*bsd-gnu*) ;; Again, look at the rest of the GDB code and use a matching style. > +esac [=E2=80=A6] --=20 FSF associate member #7257