From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17816 invoked by alias); 31 May 2009 15:44:47 -0000 Received: (qmail 17807 invoked by uid 22791); 31 May 2009 15:44:45 -0000 X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mail-pz0-f175.google.com (HELO mail-pz0-f175.google.com) (209.85.222.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 31 May 2009 15:44:40 +0000 Received: by pzk5 with SMTP id 5so7163120pzk.12 for ; Sun, 31 May 2009 08:44:38 -0700 (PDT) MIME-Version: 1.0 Received: by 10.110.37.3 with SMTP id k3mr127406tik.1.1243784677212; Sun, 31 May 2009 08:44:37 -0700 (PDT) In-Reply-To: <765425.28174.qm@web36202.mail.mud.yahoo.com> References: <765425.28174.qm@web36202.mail.mud.yahoo.com> Date: Sun, 31 May 2009 15:44:00 -0000 Message-ID: Subject: Re: submition of i386.record.floating.point.patch From: Hui Zhu To: paawan oza Cc: gdb-patches@sourceware.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit 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-05/txt/msg00659.txt.bz2 Hi Oza, Thanks for your works. Before I read your patch, I have some ideas with your RFA: 1. Please don't post compress package. I did it in before. But it's not friendly for review. 2. Post the readme and changelog to the Email body. If you like, post the patch to the email body too. A lof of people like it. 3. Maybe you can post some test example to test the patch. I suggest you re-send a rfa that post most thing to email body. And if you want, you can post the url to http://sourceware.org/gdb/wiki/ReversibleDebugging "Support i386 more better". Thanks, Hui On Sun, May 31, 2009 at 19:49, paawan oza wrote: > Hello, > > I have been working on reversible debugging-i386 floating point support for last couple of weeks. > I would like to submit the patch for review. > > please find the package attached. > package contains following. > > 1) Description > 2) ChangeLog (I am not sure it follows proper formatting) > 3) The Patch. > > please consider it for review and kindly send your comments. > > Regards, > Oza. > > > > > > > --- On Sun, 5/31/09, Hui Zhu wrote: > >> From: Hui Zhu >> Subject: Re: floating point support for i386 done, for record/replay >> To: "paawan oza" >> Cc: "gdb ml" >> Date: Sunday, May 31, 2009, 11:02 AM >> I think maybe you can post your patch >> and your example to gdb-patch to >> let people review it. >> >> Hui >> >> On Sun, May 31, 2009 at 01:46, paawan oza >> wrote: >> > >> > Hi Hui, >> > I have been going through GNU doc about how I can >> submit the patch (floating point record support). >> > the process looked bit complicated as the patch >> involves more than 15 lines of change. >> > I am not sure how to go ahead, because this is the >> very first time I am contributing. >> > please guide me with that. >> > I will be waiting for your reply. >> > Regards, >> > Oza. >> > >> > --- On Thu, 5/28/09, paawan oza >> wrote: >> > >> >> From: paawan oza >> >> Subject: i386, floating point support done for >> record/replay >> >> To: "Hui Zhu" >> >> Cc: "gdb ml" >> >> Date: Thursday, May 28, 2009, 1:25 PM >> >> >> >> Hi Hui, >> >> >> >> I have finished the implementation part for i386 >> floating >> >> point support >> >> for record/replay. >> >> I have tested with some small programs, and it >> gives >> >> results as expected. >> >> (As there are lots of floating point insns, not >> every insn >> >> is tested, but coding is simple) >> >> I am quiet excited about the patch and nervous at >> the same >> >> time, as it is my first contribution : ) >> >> I have just concentrated mainly on making the >> things work, >> >> rather than optimizing it. >> >> please feel free to suggest any design change. >> >> >> >> please let me know how I can go ahead. >> >> >> >> Regards, >> >> Oza. >> >> >> >> >> >> >> >> >> >> --- On Sun, 5/24/09, Hui Zhu >> >> wrote: >> >> >> >> > From: Hui Zhu >> >> > Subject: Re: floating point support i386 >> >> implementation partially done >> >> > To: "paawan oza" >> >> > Cc: "gdb ml" >> >> > Date: Sunday, May 24, 2009, 9:27 AM >> >> > On Sun, May 24, 2009 at 03:03, paawan >> >> > oza >> >> > wrote: >> >> > > >> >> > > Hi Hui, >> >> > > >> >> > > floating point support patch is over for >> the >> >> case >> >> > > where ModR/M Byte is Within 00H to BFH. >> >> > > >> >> > > there are some doubts. >> >> > > >> >> > > 1) the insns like, >> >> > > FRSTOR, FLDENV, FLDCW (which loads >> floating >> >> point >> >> > environment registers like status, control, >> tag etc.. >> >> > initializing part) >> >> > > I think we need to record those also, >> upon >> >> executing >> >> > such insns. >> >> > > what is your opinion? >> >> > If this reg can be access by gdb, I think you >> need >> >> deal >> >> > with them. >> >> > >> >> > > >> >> > > >> >> > > otherwise, patch is ready : ), >> >> > > of course I need to work on REG= support >> to >> >> finish the >> >> > rest of the record of FPU support. >> >> > > >> >> > Great! >> >> > >> >> > Thanks, >> >> > Hui >> >> > >> >> >> >> >> >> >> >> >> > >> > >> > >> > >> > > >