#433 closed defect (fixed)

mips gdb stub ack handling improvement

Reported by: heard Owned by: Joel Sherrill
Priority: normal Milestone: 2
Component: tool Version: 4.6
Severity: major Keywords:
Cc: bugs@… Blocked By:
Blocking:

Description

The purpose of the
patch is to make the stub ignore all characters except '+' (ack) or
'-' (nak) when it has transmitted a packet and expects an
acknowledgment. Without the patch, putpacket() will retransmit each
time it receives anything other than '+' when expecting an ack, and
that, bitter experience has found, frequently results in the stub
failing to sync up when initially attaching to gdb.

I think that rtems users will notice significant improvement in the
behaviour of the mips stub if this patch is incorporated in the
rtems distribution; certainly I did in my stand-aline environment.
But I should hasten to add that it is only an improvement and not
a 100% cure for synchronization problems because '+' characters
can (and do) during as message data, and it's still possible for
these to be mistaken as acks. I have no idea how to fix that
short of redesigning the gdb-remote protocol.

Release:
RTEMS-4.6 and trunk

Attachments (1)

mips-stub.c.patch (1.1 KB) - added by heard on Dec 3, 2006 at 1:31:13 PM.
mips-stub.c.patch

Download all attachments as: .zip

Change History (2)

comment:1 Changed on Jul 17, 2003 at 9:00:57 PM by Joel Sherrill

Status: assignedclosed

State-Changed-From-To: open->closed
State-Changed-Why: Patch applied to 4.6 release branch and main CVS trunk.

Thank you Michael. Receiving an unsolicited patch from
you made my day and let me know that the old style spirit
of the Net is still alive. :)

Changed on Dec 3, 2006 at 1:31:13 PM by heard

Attachment: mips-stub.c.patch added

mips-stub.c.patch

Note: See TracTickets for help on using tickets.