From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3926 invoked by alias); 24 Nov 2013 23:48:57 -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 3916 invoked by uid 89); 24 Nov 2013 23:48:57 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=3.0 required=5.0 tests=AWL,BAYES_50,RDNS_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 X-HELO: mail-wg0-f42.google.com Received: from Unknown (HELO mail-wg0-f42.google.com) (74.125.82.42) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Sun, 24 Nov 2013 23:48:56 +0000 Received: by mail-wg0-f42.google.com with SMTP id a1so1242883wgh.5 for ; Sun, 24 Nov 2013 15:48:47 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=N/InXlvnr6Ha+L5l90LCb5gJAlIYevigf/4xQ2Cf2Lc=; b=fook2j20Vcztd2FfeJ0Pb4avnT5rc07TLejMRReWXEw8jthGBlTgZZPDYFYn4FbKPA fJh0zaL3qSOiv3f+Y0nSJ97GRzHxfWkltx2DLVmoso20QqSKrMFQeACEG8aP8j2E2uko BwufqpOGadDOX7ijJXg5Vs/LiQRv4MCyON5rdTF6mTDMjploQVKZeZAmVbbOuVTLwVkm ssH0IkhAIQyy/j/1u+HgHxiSNVDCi0kbtI6258uqIWBfakpdcBSd9fh+cH+TQlzEQ9L2 Q+osQ+wgrVKRkfOUefVfTGmd0RxUpVvH5BWOWxDNQQ2L8AhfSiYYMJ6hN1Dti31Rw6WI 4hdg== X-Gm-Message-State: ALoCoQnpfwsPvkaXRElD2VnaPR7D3OYfn0SYL1BK4tcApjG2a364xRCW7wPDC/XPUtcvbqCLWCji X-Received: by 10.194.121.133 with SMTP id lk5mr33304wjb.77.1385336927138; Sun, 24 Nov 2013 15:48:47 -0800 (PST) Received: from [192.168.1.2] ([182.185.192.168]) by mx.google.com with ESMTPSA id nb16sm41835184wic.0.2013.11.24.15.48.44 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sun, 24 Nov 2013 15:48:46 -0800 (PST) Message-ID: <52929058.1050202@linaro.org> Date: Sun, 24 Nov 2013 23:49:00 -0000 From: Omair Javaid User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.1 MIME-Version: 1.0 To: oza Pawandeep CC: Yao Qi , "gdb-patches@sourceware.org" , patches , Eli Zaretskii Subject: Re: [PATCH 0/2] GDB process record and reverse debugging improvements for arm*-linux* References: <52687B1A.2050003@codesourcery.com> <527C580B.5020100@linaro.org> <83k3gj5p9t.fsf@gnu.org> <52809426.40405@linaro.org> In-Reply-To: <52809426.40405@linaro.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2013-11/txt/msg00748.txt.bz2 On Mon 11 Nov 2013 01:24:06 PM PKT, Omair Javaid wrote: > On Fri 08 Nov 2013 01:18:45 PM PKT, oza Pawandeep wrote: >> Hi, >> >> The previous support for arm instruction and thumb instruction for >> record and reply was added by me. >> but there some reverse test case failing... >> I had issues with employer disclaimer, so could not proceed. >> I have got the disclaimer now and is up-to date with FSF. >> >> some of the queries to you regarding thumb2 support ? >> >> 1) did you bring in co processor and dsp instruction support for thumb2 ? >> >> 2) can you please run gdb-rever test suite and see if your code passes >> the same ? >> >> >> Regards, >> Oza. >> >> >> On Fri, Nov 8, 2013 at 12:55 PM, Eli Zaretskii wrote: >>> >>>> Date: Fri, 08 Nov 2013 08:18:35 +0500 >>>> From: Omair Javaid >>>> CC: gdb-patches@sourceware.org, Patch Tracking >>>> >>>> --- gdb/NEWS 11 Oct 2013 13:51:26 -0000 1.622 >>>> +++ gdb/NEWS 7 Nov 2013 23:58:26 -0000 >>>> @@ -3,6 +3,10 @@ >>> >>> This part is OK, but please make sure to have 2 spaces between >>> sentences >>> >>>> +* Improved support of process record-replay and reverse debugging on >>>> + arm*-linux* targets. Support for thumb32 and syscall instructions >>> ^^ >>> here. >>> >>> Thanks. >> > > Hello Oza, > > I really appreciate your work on arm process record and I am trying to > improve on the work you have done previously. I have sent 3 patches > upstream which include basic support for Thumb32 instructions and > system call instructions along with some fixes. > > There were around 1500 failures in gdb.reverse on arm that have been > significantly reduced. Here is a current status of gdb.reverse test > suite on arm targets: > > arm-native configuration: > # of expected passes 2253 > # of unexpected failures 71 > > arm-remote configuration: > # of expected passes 2142 > # of unexpected failures 122 > > I am currently working on patches to fix above unexpected failures and > also working on supporting co-processor and other unsupported > instruction types. > > -- > Omair. After incorporating all suggestions I am posting a final version of all three patches posted earlier. Looking for a go ahead for commit. == PATCH 0 == This patch adds a news entry about improved process record-replay on arm*-linux* targets. It also adds linux-record.o to gdb_target_obs in configure.tgt and enables gdb.reverse testsuite for arm*-linux* targets. gdb: 2013-11-08 Omair Javaid * NEWS: Add note on improved process record-replay on arm*-linux* targets. * configure.tgt: Add linux-record.o to gdb_target_obs. gdb/testsuite: 2013-10-24 Omair Javaid * lib/gdb.exp: Enable gdb.reverse testsuite for arm*-linux* targets. --- gdb/NEWS | 4 ++++ gdb/configure.tgt | 2 +- gdb/testsuite/lib/gdb.exp | 6 ++++-- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/gdb/NEWS b/gdb/NEWS index 10834df..bcf3e09 100644 --- a/gdb/NEWS +++ b/gdb/NEWS @@ -3,6 +3,10 @@ *** Changes since GDB 7.6 +* Improved support of process record-replay and reverse debugging on + arm*-linux* targets. Support for thumb32 and syscall instructions + recording has been added. + * GDB now supports Fission DWP file format version 2. http://gcc.gnu.org/wiki/DebugFission diff --git a/gdb/configure.tgt b/gdb/configure.tgt index ea0faf1..5410f72 100644 --- a/gdb/configure.tgt +++ b/gdb/configure.tgt @@ -89,7 +89,7 @@ arm*-wince-pe | arm*-*-mingw32ce*) arm*-*-linux*) # Target: ARM based machine running GNU/Linux gdb_target_obs="arm-tdep.o arm-linux-tdep.o glibc-tdep.o \ - solib-svr4.o symfile-mem.o linux-tdep.o" + solib-svr4.o symfile-mem.o linux-tdep.o linux-record.o" build_gdbserver=yes ;; arm*-*-netbsd* | arm*-*-knetbsd*-gnu) diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index bde4e48..047aefb 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -1774,7 +1774,8 @@ proc supports_process_record {} { return [target_info gdb,use_precord] } - if { [istarget "x86_64-*-linux*"] || [istarget "i\[34567\]86-*-linux*"] } { + if { [istarget "arm*-*-linux*"] || [istarget "x86_64-*-linux*"] + || [istarget "i\[34567\]86-*-linux*"] } { return 1 } @@ -1789,7 +1790,8 @@ proc supports_reverse {} { return [target_info gdb,can_reverse] } - if { [istarget "x86_64-*-linux*"] || [istarget "i\[34567\]86-*-linux*"] } { + if { [istarget "arm*-*-linux*"] || [istarget "x86_64-*-linux*"] + || [istarget "i\[34567\]86-*-linux*"] } { return 1 } --