This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [Patch, microblaze]: Add support of microblaze software single stepping
- From: Michael Eager <eager at eagercon dot com>
- To: Ajit Kumar Agarwal <ajit dot kumar dot agarwal at xilinx dot com>, Joel Brobecker <brobecker at adacore dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, Pedro Alves <palves at redhat dot com>, Vinod Kathail <vinodk at xilinx dot com>, Vidhumouli Hunsigida <vidhum at xilinx dot com>, Nagaraju Mekala <nmekala at xilinx dot com>
- Date: Fri, 11 Jul 2014 12:57:49 -0700
- Subject: Re: [Patch, microblaze]: Add support of microblaze software single stepping
- Authentication-results: sourceware.org; auth=none
- References: <a0a86472-2b0f-4a69-bd7d-ddea295e0f5e at BN1BFFO11FD011 dot protection dot gbl> <20140707145634 dot GF6038 at adacore dot com> <35bccb81-cae6-4581-b5fd-16dea7171d28 at BN1AFFO11FD012 dot protection dot gbl> <20140711135114 dot GA4888 at adacore dot com> <ab8da1f0-cade-46f8-97e0-888662ff368e at BY2FFO11FD002 dot protection dot gbl>
On 07/11/14 12:34, Ajit Kumar Agarwal wrote:
Please also explain how this patch was tested.
The changes were tested with the application for barematel with remote debug with XMD debugger( internal to Xilinx)which connects to the target and opens the gdbserver connection. Single stepping command were used for next_pc in straight line code, with imm instruction and the branch with delay Slot.
Your terminology may be a bit confusing. Gdbserver
is a program (part of GDB) which runs on a platform (e.g.
Linux), and which implements the GDB Remote Serial Protocol.
XMD does not run gdbserver. It has an independent implementation
of the GDB Remote Serial Protocol. It would be better to
say that XMD is a remote stub.
Please provide a disassembly of the test case and
a log of the commands to gdb and gdb's output, showing
stepping over imm and delay slot instructions.
Is there a test case which shows single stepping failing
without the patch?
Even better would be to update the gdb regression
test suite with the test case.
--
Michael Eager eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306 650-325-8077