From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26061 invoked by alias); 20 Nov 2008 08:13:20 -0000 Received: (qmail 26019 invoked by uid 22791); 20 Nov 2008 08:13:18 -0000 X-Spam-Check-By: sourceware.org Received: from ti-out-0910.google.com (HELO ti-out-0910.google.com) (209.85.142.189) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 20 Nov 2008 08:12:43 +0000 Received: by ti-out-0910.google.com with SMTP id d10so173136tib.12 for ; Thu, 20 Nov 2008 00:12:40 -0800 (PST) Received: by 10.110.63.6 with SMTP id l6mr2491004tia.50.1227168760323; Thu, 20 Nov 2008 00:12:40 -0800 (PST) Received: by 10.110.103.3 with HTTP; Thu, 20 Nov 2008 00:12:40 -0800 (PST) Message-ID: Date: Thu, 20 Nov 2008 15:19:00 -0000 From: teawater To: "Michael Snyder" Subject: Re: [RFA] Resubmit process record and replay, 7/10 Cc: "gdb-patches@sourceware.org" In-Reply-To: <4924C417.5070206@vmware.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <4924C417.5070206@vmware.com> 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: 2008-11/txt/msg00547.txt.bz2 Thanks Michael. On Thu, Nov 20, 2008 at 09:57, Michael Snyder wrote: > teawater wrote: >> >> This patch add code to make GNU/Linux native-dependent code can record >> execute log with itself. >> >> 2008-11-16 Hui Zhu >> >> Record execute log in linux-nat. >> >> * linux-nat.c (my_waitpid_record): New function. This function >> can resume and wait inferior and record execute log of it. >> (linux_nat_wait): If process record and replay target is used >> and this is not a step resume, call function >> "my_waitpid_record" instead function "my_waitpid". > > Hui, another name change suggestion: > Instead of "my_waitpid_record", how about "process_record_waitpid"? > > I don't think it needs to have "linux" in the name, since it is > a static function and can only be invoked from linux-nat.c. > OK. I will change it. Hui