This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [Patch, microblaze]: Add slr and shr regs
- From: Ajit Kumar Agarwal <ajit dot kumar dot agarwal at xilinx dot com>
- To: Michael Eager <eager at eagerm dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Cc: Vinod Kathail <vinodk at xilinx dot com>, Vidhumouli Hunsigida <vidhum at xilinx dot com>, Nagaraju Mekala <nmekala at xilinx dot com>
- Date: Mon, 9 Jun 2014 19:31:31 +0000
- Subject: RE: [Patch, microblaze]: Add slr and shr regs
- Authentication-results: sourceware.org; auth=none
- Authentication-results: spf=pass (sender IP is 149.199.60.83) smtp dot mailfrom=ajit dot kumar dot agarwal at xilinx dot com;
- References: <c6fe468a-5e02-430a-b34b-73de6bfd2b97 at BY2FFO11FD021 dot protection dot gbl> <537EFA08 dot 1060309 at eagercon dot com> <f0c2c22a-d3d6-45d1-92ba-660dd364fd89 at BL2FFO11FD051 dot protection dot gbl> <537FCEDA dot 9030504 at eagercon dot com> <2e5c185d-329c-46cf-930c-8cc2288891aa at BN1BFFO11FD019 dot protection dot gbl> <538431FB dot 2070904 at eagercon dot com> <865132b2-a593-4147-a7c6-cee25c1ed0fd at BN1AFFO11FD052 dot protection dot gbl> <53909299 dot 9010105 at eagerm dot com> <dab8980c-8551-4ebe-856e-4d8b91d207fe at BN1AFFO11FD059 dot protection dot gbl> <5395FCE6 dot 80702 at eagerm dot com>
-----Original Message-----
From: Michael Eager [mailto:eager@eagerm.com]
Sent: Monday, June 09, 2014 11:59 PM
To: Ajit Kumar Agarwal; gdb-patches@sourceware.org
Cc: Vinod Kathail; Vidhumouli Hunsigida; Nagaraju Mekala
Subject: Re: [Patch, microblaze]: Add slr and shr regs
On 06/09/14 10:26, Ajit Kumar Agarwal wrote:
>
> Sorry for the late response as I was on Vacation. Please find my response inlined below.
It's difficult to figure out which are your responses and which is text quoted from previous email.
AJA:
> SLR/SHR was added in MicroBlaze v8.10.a, EDK 13.1. XMD's gdbserver stub releasesed before this design throws an error mentioning gdb is not supported for the version below v8.10.a, EDK 13.1.
I don't know what this means. XMD released with EDK 9.1i doesn't generate any such error message and works with mb-gdb built from FSF sources.
Ajit: The necessity of the XMD to generate such error message came when we have shr and shl support in Microblaze v8.10.a, EDK 13.1 to have downward compatibility. The XMD you are using is quite old.
There are several ways to make gdb to adapt to different target architectures.
Pedro described the preferred solution, but there are others.
Ajit: We would surely like to implement as suggested by Pedro.
MJE:
>>> I do not have any way to test microblaze-linux. Make sure that your patch works with both -target=microblaze-linux and -target=microblaze-xilinx-gnu, and with both EDK 9.1i and >>14.2i, at a minimum. Please provide documentation which show that this patch works using gdbserver on a Linux target, as well as using XMD.
AJA:
> Same as the Explanation above. The native Linux support patch is not yet submitted and we will submitting the patches after this patch. XMD testing is done and it works fine.
This isn't responsive to my request.
The only way that your patch makes sense is if you build gdbserver for a microblaze-linux target. If this is not what you are doing, you need to explain (as previously
requested) how you are building and testing the patch.
Ajit: I might have misunderstood the question. We build the binutils which comes with gdb with target as microblaze-xilinx-elf. We do the following,
1. Invoke the XMD with gdbserver on.
2. XMD connects to the target.
3. Invoke mb-gdb with the executables to debug.
4.Invoke command " tar remote localhost:1234".
5. Invoke the command "load"
6. Set the breakpoints and do the normal debugging.
Kindly be more explicit regarding this question.
MJE:
> When you resubmit the patch, please make sure that you check for whitespace errors. Make sure that you address all of the questions and comments made here and by other reviewers.
Please respond to all comments.
Ajit: Sure I will addressed all the comments.
--
Michael Eager eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306 650-325-8077