From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21467 invoked by alias); 12 Jun 2014 14:52:09 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 21448 invoked by uid 89); 12 Jun 2014 14:52:08 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 X-HELO: qmta12.emeryville.ca.mail.comcast.net Received: from qmta12.emeryville.ca.mail.comcast.net (HELO qmta12.emeryville.ca.mail.comcast.net) (76.96.27.227) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 12 Jun 2014 14:52:07 +0000 Received: from omta05.emeryville.ca.mail.comcast.net ([76.96.30.43]) by qmta12.emeryville.ca.mail.comcast.net with comcast id DSFk1o0050vp7WL01Ss6gD; Thu, 12 Jun 2014 14:52:06 +0000 Received: from redwood.eagercon.com ([24.7.16.38]) by omta05.emeryville.ca.mail.comcast.net with comcast id DSs51o00C0pGQcg8RSs5Us; Thu, 12 Jun 2014 14:52:05 +0000 Message-ID: <5399BE95.3010902@eagercon.com> Date: Thu, 12 Jun 2014 14:52:00 -0000 From: Michael Eager User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: Ajit Kumar Agarwal , "gdb-patches@sourceware.org" CC: Vinod Kathail , Vidhumouli Hunsigida , Nagaraju Mekala Subject: Re: [Patch, microblaze]: Fix for remote G Packet message too long error for baremetal. References: In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2014-06/txt/msg00505.txt.bz2 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 > > * 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