From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9399 invoked by alias); 30 Mar 2009 15:44:25 -0000 Received: (qmail 9389 invoked by uid 22791); 30 Mar 2009 15:44:23 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_39,SPF_PASS X-Spam-Check-By: sourceware.org Received: from imr2.ericy.com (HELO imr2.ericy.com) (198.24.6.3) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 30 Mar 2009 15:44:15 +0000 Received: from eusrcmw750.eamcs.ericsson.se (eusrcmw750.exu.ericsson.se [138.85.77.50]) by imr2.ericy.com (8.13.1/8.13.1) with ESMTP id n2UFhtxH010282; Mon, 30 Mar 2009 10:43:56 -0500 Received: from ecamlmw720.eamcs.ericsson.se ([142.133.1.72]) by eusrcmw750.eamcs.ericsson.se with Microsoft SMTPSVC(6.0.3790.1830); Mon, 30 Mar 2009 10:43:56 -0500 Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Subject: RE: [RFA] Submit process record and replay fourth time, 0/8 Date: Mon, 30 Mar 2009 15:52:00 -0000 Message-ID: <6D19CA8D71C89C43A057926FE0D4ADAA072C03F5@ecamlmw720.eamcs.ericsson.se> In-Reply-To: References: <6D19CA8D71C89C43A057926FE0D4ADAA072C016B@ecamlmw720.eamcs.ericsson.se> <6D19CA8D71C89C43A057926FE0D4ADAA072C03A4@ecamlmw720.eamcs.ericsson.se> From: "Marc Khouzam" To: "Hui Zhu" Cc: "Pedro Alves" , "Michael Snyder" , "Thiago Jung Bauermann" , "Eli Zaretskii" , "Mark Kettenis" , X-IsSubscribed: yes 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 X-SW-Source: 2009-03/txt/msg00674.txt.bz2 > Could you send me the bugs that you got? If it just can reproduce in > Eclipse, I will try it. >=20 > About these bugs, I think we need deal with them one by one and step > by step like before. :) I was suggesting using Eclipse, because it takes me time to isolate a bug and reproduce it in a smaller, command-line example. But when I have a little time, I will send whatever I find Marc >=20 > Thanks, > Hui >=20 > On Mon, Mar 30, 2009 at 23:28, Marc Khouzam=20 > wrote: > > I see. =A0I've only tried reverse with your patches so I don't know > > if the bugs I see also show up when using vmware, simics or=20 > gdbreplay... > > But if those bugs are already in HEAD, it would be good if we could > > get them fixed before 7.0. > > > > If anyone wants to try reverse with Eclipse to do more=20 > thourough testing, > > I'll help. > > > > Marc > > > > > >> -----Original Message----- > >> From: gdb-patches-owner@sourceware.org > >> [mailto:gdb-patches-owner@sourceware.org] On Behalf Of Hui Zhu > >> Sent: Monday, March 30, 2009 11:17 AM > >> To: Marc Khouzam > >> Cc: Pedro Alves; Michael Snyder; Thiago Jung Bauermann; Eli > >> Zaretskii; Mark Kettenis; gdb-patches@sourceware.org > >> Subject: Re: [RFA] Submit process record and replay fourth=20 > time, 0/8 > >> > >> Hi Marc, > >> > >> I think most of issue that you got is about the reverse=20 > debug control > >> part. =A0This is not in the p record patches. =A0The reverse=20 > debug include > >> reverse debug control part already in cvs-head. > >> For example, the 3 bugs that you sent to maillist are all=20 > for reverse > >> debug control part. But not for p record. > >> > >> Process record just a target that support reverse debug=20 > function. =A0It > >> depend on reverse debug function. > >> Actually, process record target was designed and developed together > >> with reverse debug (Michael and me). > >> > >> And I think this is a good reason for let process record in now. > >> Reverse debug function had already checked in cvs-head a=20 > lot of month. > >> =A0It will have a lot of customers after 7.0 release. =A0I=20 > think it need a > >> big and clear test. > >> Now, remote target support reverse debug in cvs-head. =A0The=20 > gdb stubs > >> support it are vmware, simics and gdbreplay. > >> So, just process record can use reverse debug in host=20 > directly. =A0And > >> there is a testsuite for reverse debug and process record called > >> gdb.twreverse (This a temp name) in branch reverse-20081226-branch. > >> It can be very easy to porting to cvs-head. =A0And we can=20 > add more test > >> (include for MI and eclipse) to increase the testsuit, p record and > >> reverse debug. > >> > >> So, I think let process record in can help move forward. =A0:) > >> > >> Thanks, > >> Hui > >> > >> > >> > >> On Mon, Mar 30, 2009 at 21:50, Marc Khouzam > >> wrote: > >> > Hi Hui, > >> > > >> > as you know, I am a big fan of your RecordAndReplay enhancements. > >> > In fact, I just demoed it at EclipseCon to show how we can > >> do Reverse > >> > Debugging in the CDT. =A0People were very very interested! > >> > > >> > However, I think there are still a few bugs. =A0These bugs=20 > may be hard > >> > to notice when working from the command line, but when > >> working in Eclipse > >> > they can be seen easily. =A0Those bugs are mostly unexpected > >> behavior such > >> > as jumping too far backwards. =A0They also happen more easily > >> when using > >> > a program that has a bit of complexity. =A0For example,=20 > the recursive > >> > bug that I found last week. > >> > > >> > Do you have access to eclipse? =A0We can work together to=20 > have you try > >> > the reverse debugging that I added to CDT, so that you=20 > can work with > >> > your patches more intensely and test them even better. > >> > > >> > I would really like to see your patches in GDB 7.0, so=20 > maybe using > >> > Eclipse to test it can help move forward. =A0What do you think? > >> > > >> > Marc > >> > > >> > > >> >> -----Original Message----- > >> >> From: Hui Zhu [mailto:teawater@gmail.com] > >> >> Sent: Monday, March 30, 2009 1:34 AM > >> >> To: Pedro Alves; Marc Khouzam; Michael Snyder; Thiago Jung > >> >> Bauermann; Eli Zaretskii; Mark Kettenis > >> >> Cc: gdb-patches@sourceware.org > >> >> Subject: Re: [RFA] Submit process record and replay fourth > >> time, 0/8 > >> >> > >> >> Hi guys, > >> >> > >> >> I am not sure your are reviewing the p record patches or done. > >> >> > >> >> Do you think all of them are ok to in? =A0:) > >> >> > >> >> > >> >> Thanks, > >> >> Hui > >> >> > >> >> > >> >> On Sat, Mar 21, 2009 at 23:58, Hui Zhu=20 > wrote: > >> >> > Hi guys, > >> >> > > >> >> > After third time submit, there are a lot of change with > >> >> process record > >> >> > and replay. =A0So I submit fourth time to make it clear=20 > to review. > >> >> > > >> >> > For this time, Most of changes were updated follow cvs head > >> >> and a lot > >> >> > of format fixes. =A0Patch for target.c was removed. =A0 > Record.c was > >> >> > updated a lot of parts according to the ideas of Pedro (Much > >> >> > appreciated). > >> >> > Thanks for help of everybody in the maillist. > >> >> > > >> >> > Process record and replay make gdb can record inferior > >> >> execute log and > >> >> > replay (include reverse debug). > >> >> > Now, it support I386-Linux single-thread inferior=20 > native debug. > >> >> > > >> >> > I've divided this patch into eight sections, for ease=20 > of review. > >> >> > They group as: > >> >> > 1) Architecture support layer (gdbarch.sh, gdbarch.c,=20 > gbarch.h). > >> >> > 2) New stratum of strata in target layer (target.h). > >> >> > 3) Process record and replay target (record.c, record.h, > >> >> Makefile.in). > >> >> > 4) Process record and replay for Linux (linux-record.c, > >> >> > linux-record.h, Makefile.in, configure.tgt). > >> >> > 5) Event handling (infrun.c). > >> >> > 6) Intel 386 target-dependent stuff (i386-tdep.c,=20 > i386-tdep.h). > >> >> > 7) Target-dependent code for GNU/Linux i386=20 > (i386-linux-tdep.c). > >> >> > 8) User interface and documentation. > >> >> > > >> >> > For the NEWS: > >> >> > * Process record and replay > >> >> > > >> >> > =A0In a architecture environment that supports=20 > ``process record and > >> >> > =A0replay'', ``process record and replay'' target can > >> record a log of > >> >> > =A0the process execution, and replay it with both forward > >> and reverse > >> >> > =A0execute commands. > >> >> > > >> >> > These patches be tested with testsuite gdb.twreverse in branch > >> >> > reverse-20081226-branch. > >> >> > > >> >> > Attachment is the compressed patches package to make get > >> >> all patches easy. > >> >> > > >> >> > Thanks, > >> >> > Hui > >> >> > > >> >> > >> > > >> > > >=20