From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1170 invoked by alias); 25 Jun 2014 14:09:43 -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 1151 invoked by uid 89); 25 Jun 2014 14:09:42 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Wed, 25 Jun 2014 14:09:40 +0000 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s5PE9XVA027853 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 25 Jun 2014 10:09:34 -0400 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id s5PE9VJk002041; Wed, 25 Jun 2014 10:09:31 -0400 Message-ID: <53AAD81A.9000707@redhat.com> Date: Wed, 25 Jun 2014 14:09:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Michael Eager , Ajit Kumar Agarwal CC: "gdb-patches@sourceware.org" , Michael Eager , Vinod Kathail , Vidhumouli Hunsigida , Nagaraju Mekala Subject: Re: [Patch, microblaze]: Fix for remote G Packet message too long error for baremetal. References: <53A023B1.5000105@redhat.com> <859f27cb-8c46-46c1-9625-7287c60f3ae9@BY2FFO11FD007.protection.gbl> <53A1ABF0.9080004@redhat.com> <74281fd5-518a-4d7f-977a-6fa1320f6db9@BY2FFO11FD016.protection.gbl> <53A1B61F.9080803@redhat.com> <736c2e0d-6ff1-40c3-8120-dc6f5d91e6b1@BL2FFO11FD052.protection.gbl> <53A8290A.1050701@redhat.com> <53A94147.4050700@redhat.com> <57ebe4b0-83eb-4208-9778-472ecf0048d4@BY2FFO11FD038.protection.gbl> <53A96993.5040804@redhat.com> <53A986EC.4040005@eagerm.com> In-Reply-To: <53A986EC.4040005@eagerm.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2014-06/txt/msg00880.txt.bz2 On 06/24/2014 03:10 PM, Michael Eager wrote: > On 06/24/14 05:05, Pedro Alves wrote: >> On 06/24/2014 11:28 AM, Ajit Kumar Agarwal wrote: >>>>> As I mentioned before, please don't forget to document the new target features in the manual. >>> >>> Would you mind in explaining which manual need to be changed for the new target. >> >> The GDB manual, gdb/doc/gdb.texinfo, describes all the standard XML target >> features. See the "Standard Target Features" node, and add a new subsection >> for MicroBlaze. > > microblaze-gdb connects to the target using a JTAG interface which talks > to a Xilinx-proprietary program called XMD. XMD implements the gdbserver > protocol. As far as I'm aware, XMD does not provide target feature > descriptions. It's expected that XMD does not provide target descriptions, because the support on the GDB side was missing. Ajit's patch teaches the microblaze port about a couple new standard features. All standard xml target features are defined by GDB, and should be documented. Nothing stops GDB from talking to servers other than XMD after all. After the patch is in, if future XMDs (or other servers) want to expose random microblaze registers to GDB, like, e.g., I/O control registers, it just needs to send in a target description that includes them, and GDB will present them (without GDB changes). Meanwhile, Ajit's patch keeps compatibility with current XMDs by adding a G packet size to target description mapping (a standard transitioning mechanism also used by ARM and MIPS). Hope that clarifies things. Thanks, -- Pedro Alves