Ping

Ping

[Tom Tromey]

This patch has not been reviewed:

    http://sources.redhat.com/ml/gdb-patches/2002-04/msg00899.html

Tom

ping (was: [PATCH] Fix for QTro remote packet)

[David Taylor]

Just a ping -- any progress on reviewing this one line patch?

    From: David Taylor <dtaylor@emc.com>
    To: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
    Sender: "gdb-patches-owner@sourceware.org" <gdb-patches-owner@sourceware.org>
    Date: Wed, 1 May 2013 13:38:06 -0400
    Subject: [PATCH] Fix for QTro remote packet

    In function remote_trace_set_readonly_regions in gdb/remote.c, the local
    variable 'offset' does not account for "QTro" at the start of the packet
    with the result that if there are any read-only regions, the packet is
    sent -- but without the "QTro" -- causing the remote stub to report that
    the packet is unsupported.


    For gdb/ChangeLog:

    2013-05-01  David Taylor  <dtaylor@emc.com>

	    * remote.c (remote_trace_set_readonly_regions): Do not overwrite
	    "QTro" at start of packet.

    Here's a one line patch to gdb/remote.c of gdb 7.6:

    --- remote.c~	2013-04-12 12:04:50.000000000 -0400
    +++ remote.c	2013-05-01 12:45:10.000000000 -0400
    @@ -10652,6 +10652,7 @@
	 return;			/* No information to give.  */

       strcpy (target_buf, "QTro");
    +  offset = strlen (target_buf);
       for (s = exec_bfd->sections; s; s = s->next)
	 {
	   char tmp1[40], tmp2[40];

    This is small enough that no copyright assignment should be needed, but
    EMC has one on file.

    David
    --
    David Taylor
    dtaylor@emc.com

Re: ping (was: [PATCH] Fix for QTro remote packet)

[Pedro Alves]

On 05/09/2013 08:06 PM, David Taylor wrote:
> Just a ping -- any progress on reviewing this one line patch?

http://sourceware.org/ml/gdb-patches/2013-05/msg00038.html

-- 
Pedro Alves

Re: ping (was: [PATCH] Fix for QTro remote packet)

[David Taylor]

Pedro Alves <palves@redhat.com> wrote:

> On 05/09/2013 08:06 PM, David Taylor wrote:
> > Just a ping -- any progress on reviewing this one line patch?
> 
> http://sourceware.org/ml/gdb-patches/2013-05/msg00038.html
> 
> -- 
> Pedro Alves

My apologies; I didn't realize you were a maintainer; I should have
checked.

Will someone please commit this patch?

Thanks.

David
--
David Taylor
dtaylor@emc.com

Re: ping

[Pedro Alves]

On 05/09/2013 09:09 PM, David Taylor wrote:

> Will someone please commit this patch?

I got this.

It'd be good to put this into 7.6 as well.

I guess I'll be the lucky one who champions the
"put 7.6.1 release notes in the wiki" process.

I've filed PR15455 (http://sourceware.org/bugzilla/show_bug.cgi?id=15455).

I though it'd be best to come up with a test that helps makes sure
QTro doesn't regress, so I wrote one.  More after lunch.  :-)

-- 
Pedro Alves