This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [Patch, microblaze]: Fix for remote G Packet message too long error for baremetal.
- From: Michael Eager <eager at eagercon dot com>
- To: Ajit Kumar Agarwal <ajit dot kumar dot agarwal at xilinx 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: Thu, 12 Jun 2014 07:52:05 -0700
- Subject: Re: [Patch, microblaze]: Fix for remote G Packet message too long error for baremetal.
- Authentication-results: sourceware.org; auth=none
- References: <fe2ea02f-37b9-4ebe-acd4-6d1192c26358 at BN1AFFO11FD059 dot protection dot gbl>
On 06/12/14 01:56, Ajit Kumar Agarwal wrote:
Based on the feedbacks, all changes have been incorporated.
Thanks!
This patch addresses some of the issues raised, such as eliminating
gdbserver changes which were unrelated. There were a number of other
issues raised previously. I'll let you know if there were any which
still need to be addressed.
May I know who will committing this patch to the upstream ?
When approved, as MicroBlaze maintainer, I will apply the patch.
[Patch, microblaze]: Fix for remote G Packet message too long error for baremetal.
Prior to version MicroBlaze v8.10.a,EDK 13.1, XMD's gdbserver stub returned 57
registers in response to GDB's G request. Starting with version MicroBlaze
v8.10.a, EDK 13.1, XMD added the slr and shr register, for a count of 59
registers.This patch adds these registers to the expected G response. This
fixes the above problem for baremetal and also supports the backward compatibility.
ChangeLog:
2014-06-12 Ajit Agarwal <ajitkum@xilinx.com>
* microblaze-tdep.c (microblaze_register_names): Add
the rshr and rslr register names.
(microblaze_gdbarch_init): Use of tdesc_has_registers.
Use of tdesc_find_feature. Use of tdesc_data_alloc.
Use of tdesc_numbered_register. Use of
microblaze_register_g_packet_guesses. Use of
tdesc_use_registers. Use of set_gdbarch_register_type.
(microblaze_register_g_packet_guesses): New.
* microblaze-tdep.h (microblaze_reg_num): Add
field MICROBLAZE_SLR_REGNUM and MICROBLAZE_SHR_REGNUM
MICROBLAZE_NUM_REGS.
(microblaze_frame_cache): Use of MICROBLAZE_NUM_REGS.
* features/microblaze-cpu.xml: New file.
* features/microblaze-linux.c: New file.
* features/microblaze-linux.xml: New file.
* regformats/reg-microblaze.dat: New file.
* features/Makefile (microblaze-linux): Add microblaze-linux
and microblaze-expedite.
Signed-off-by:Ajit Agarwal ajitkum@xilinx.com
Thanks & Regards
Ajit
--
Michael Eager eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306 650-325-8077