From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14478 invoked by alias); 12 Jun 2009 16:47:33 -0000 Received: (qmail 14465 invoked by uid 22791); 12 Jun 2009 16:47:32 -0000 X-SWARE-Spam-Status: No, hits=-2.2 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from web36207.mail.mud.yahoo.com (HELO web36207.mail.mud.yahoo.com) (209.191.68.233) by sourceware.org (qpsmtpd/0.43rc1) with SMTP; Fri, 12 Jun 2009 16:47:26 +0000 Received: (qmail 38792 invoked by uid 60001); 12 Jun 2009 16:47:24 -0000 Message-ID: <679033.38690.qm@web36207.mail.mud.yahoo.com> Received: from [123.237.143.113] by web36207.mail.mud.yahoo.com via HTTP; Fri, 12 Jun 2009 09:47:24 PDT Date: Fri, 12 Jun 2009 16:47:00 -0000 From: paawan oza Subject: Re: Submition of i386.record.floating.point.patch To: Hui Zhu Cc: Mark Kettenis , gdb-patches@sourceware.org MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: quoted-printable 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-06/txt/msg00336.txt.bz2 Hi Hui, please find my comments below. >Even if you think i386 need this register, it should not be the part >of process record patch. Oza : insns like FFREE changes register like FTAG.=20 so we need to record it.=20 The patch concentrates on any insn which has potential to change any of the= FPU environment registers and stack. > You can send a patch to make i386 support this register > first. Oza: gdb already has support for i386 for those(fpu) registers.=20 command -> [info float/info all-registers]=20 give all floating point registers including whole x87 FPU environment. regards, Oza. --- On Fri, 6/12/09, Hui Zhu wrote: > From: Hui Zhu > Subject: Re: Submition of i386.record.floating.point.patch > To: "paawan oza" > Cc: "Mark Kettenis" , gdb-patches@sourceware.org > Date: Friday, June 12, 2009, 10:49 AM > Even if you think i386 need this > register, it should not be the part > of process record patch. > You can send a patch to make i386 support this register > first. >=20 > Hui >=20 > On Thu, Jun 11, 2009 at 22:59, paawan oza > wrote: > > > > yes. I know. > > I clarified that i386 also need FTAG, FSTATUS, FCTRL, > registers... > > I am also waiting for the clarification from Mark. > > Regards, > > Oza. > > > > --- On Thu, 6/11/09, Hui Zhu > wrote: > > > >> From: Hui Zhu > >> Subject: Re: Submition of > i386.record.floating.point.patch > >> To: "paawan oza" > >> Cc: "Mark Kettenis" , > gdb-patches@sourceware.org > >> Date: Thursday, June 11, 2009, 1:15 PM > >> I had post my idea about your patch > >> in before. > >> > >> Thanks, > >> Hui > >> > >> On Thu, Jun 11, 2009 at 15:37, paawan oza > >> wrote: > >> > > >> > Hi Hui, > >> > Have you/any-body reviewed floating point > patch ? > >> > I am waiting for the review comments, and > also the > >> clarification from Mark. > >> > please reply. > >> > Regards, > >> > Oza. > >> > > >> > --- On Fri, 6/5/09, Hui Zhu > >> wrote: > >> > > >> >> From: Hui Zhu > >> >> Subject: Re: Submition of > >> i386.record.floating.point.patch > >> >> To: "Mark Kettenis" > >> >> Cc: gdb-patches@sourceware.org, > >> paawan1982@yahoo.com > >> >> Date: Friday, June 5, 2009, 8:28 AM > >> >> That is great.=A0 Thanks Mark. > >> >> > >> >> Hui > >> >> > >> >> On Fri, Jun 5, 2009 at 10:56, Mark > Kettenis > >> >> wrote: > >> >> >> Date: Fri, 5 Jun 2009 10:50:25 > +0800 > >> >> >> From: Hui Zhu > >> >> >> > >> >> >> Could someone help us about > this? > >> >> > > >> >> > I should be able to, but I'm > currently > >> travelling. > >> >> =A0Should be back next week. > >> >> > > >> >> > >> > > >> > > >> > > >> > > >> > > > > > > > > >=20