From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9257 invoked by alias); 4 Jul 2013 13:24: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 9244 invoked by uid 89); 4 Jul 2013 13:24:42 -0000 X-Spam-SWARE-Status: No, score=-4.8 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL autolearn=ham version=3.3.1 Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Thu, 04 Jul 2013 13:24:41 +0000 Received: from svr-orw-fem-01.mgc.mentorg.com ([147.34.98.93]) by relay1.mentorg.com with esmtp id 1UujWZ-0005qL-0Y from Luis_Gustavo@mentor.com ; Thu, 04 Jul 2013 06:24:39 -0700 Received: from NA1-MAIL.mgc.mentorg.com ([147.34.98.181]) by svr-orw-fem-01.mgc.mentorg.com with Microsoft SMTPSVC(6.0.3790.4675); Thu, 4 Jul 2013 06:24:38 -0700 Received: from [172.30.14.191] ([172.30.14.191]) by NA1-MAIL.mgc.mentorg.com with Microsoft SMTPSVC(6.0.3790.3959); Thu, 4 Jul 2013 06:24:38 -0700 Message-ID: <51D57792.3070107@codesourcery.com> Date: Thu, 04 Jul 2013 13:24:00 -0000 From: Luis Machado Reply-To: lgustavo@codesourcery.com User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Ulrich Weigand CC: Pedro Alves , Tom Tromey , Stan Shebs , GDB Patches Subject: Re: [PATCH, testsuite] Don't run SREC, IHEX and TEKHEX tests for MIPS N64. References: <201307041322.r64DMeIf026838@d06av02.portsmouth.uk.ibm.com> In-Reply-To: <201307041322.r64DMeIf026838@d06av02.portsmouth.uk.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2013-07/txt/msg00172.txt.bz2 On 07/04/2013 10:22 AM, Ulrich Weigand wrote: > Luis Machado wrote: >> On 07/03/2013 12:05 PM, Pedro Alves wrote: >>> On 07/02/2013 07:50 PM, Luis Machado wrote: >>>> - >>>> -if {[istarget "spu*-*-*"]} then { >>>> - # The internal address format used for the combined Cell/B.E. >>>> - # debugger requires 64-bit. >>>> - set is64bitonly "yes" >>>> -} >>>> - >>> >>> I'm not sure this equates to sizeof pointer == 64-bit. >>> This bit may need to be retained. [Adding Ulrich]. >> >> Fair enough. Ulrich, let me know if the pointer check in the attached >> patch doesn't make sense for Cell BE. > > No, the pointer check is not enough. The problem is that on Cell/B.E., > GDB uses a (purely internal to GDB) address format to encode multiple > address spaces into a synthetic GDB address space. See the long > comment in spu-tdep.h for details. > > The "restore" command needs a full internal 64-bit address on Cell/B.E. > to uniquely identify the location where to restore memory. Thus using > a data format that can only remember 32-bit addresses don't work: > http://sourceware.org/ml/gdb-patches/2008-09/msg00139.html > > However, just testing the size of a pointer will return 32-bit on SPU, > because that's the ABI-defined pointer size (the 64 bits are purely > GDB internal). So I think Cell/B.E. / SPU will need to remain a > special case here for now. > > Bye, > Ulrich > I see. I'll put the spu exception back into the testcase then. Thanks, Luis