Re: [RFA] Submit process record and replay third time, 3/9

[teawater <teawater@gmail.com>]

So. Your question in prev mail is OK?

On Wed, Jan 21, 2009 at 03:47, Marc Khouzam <marc.khouzam@ericsson.com> wrote:
> Nothing like a public email to make one realize he is confused :-)
>
> I just realized that 'record' is only used to enable recording while
> 'stoprecord' should be used to turn it off.  That works for me.
>
> So all is well, sorry about that.
>
> Marc
>
>> -----Original Message-----
>> From: gdb-patches-owner@sourceware.org
>> [mailto:gdb-patches-owner@sourceware.org] On Behalf Of Marc Khouzam
>> Sent: Tuesday, January 20, 2009 1:31 PM
>> To: teawater; gdb-patches@sourceware.org
>> Subject: RE: [RFA] Submit process record and replay third time, 3/9
>>
>> Hi,
>>
>> again, thanks for the great work on Process Record and Replay.
>> I almost have the Eclipse DSF-GDB frontent for it.
>> However I ran into a small issue where a frontend is unable to turn
>> off Record.  I'm not sure if there should be an MI command
>> corresponding
>> to the 'record' command, but if I have to use 'record' directly to
>> turn *off* reverse debugging, I get a prompt
>>
>> "Process record target already running, do you want to delete the old
>> record log?(y or [n])"
>>
>> The prompt is not the problem.  The problem is that the
>> default is 'no'.
>> I think that when running in a frontend (at least in eclipse), the
>> default
>> choice automatically gets selected. Therefore, the frontend
>> cannot turn
>> off
>> recording.
>>
>> Could we make the default to be 'yes'?
>> I get the feeling there is a better way to handle this... I just don't
>> know how.
>>
>> Thanks
>>
>> Marc
>>
>>
>> > -----Original Message-----
>> > From: gdb-patches-owner@sourceware.org
>> > [mailto:gdb-patches-owner@sourceware.org] On Behalf Of teawater
>> > Sent: Monday, January 12, 2009 10:06 PM
>> > To: gdb-patches@sourceware.org
>> > Subject: Re: [RFA] Submit process record and replay third time, 3/9
>> >
>> > Update follow the cvs-head.
>> >
>> > On Thu, Jan 8, 2009 at 13:46, teawater <teawater@gmail.com> wrote:
>> > >
>> > > This patch add the process record and replay target.  This
>> > is the core
>> > > part of process record and replay.
>> > >
>> > > 2009-01-08  Hui Zhu  <teawater@gmail.com>
>> > >
>> > >        Process record and replay target.
>> > >
>> > >        * Makefile.in (record.c): New file.
>> > >        * record.c, record.h: New file.
>> > >
>> > >        2008-12-28  Michael Snyder  <msnyder@vmware.com>
>> > >        * Comments, spelling, white space clean-ups.
>> > >
>> > >        2008-12-26  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.h: Don't export record_not_record.
>> > >        * record.c (record_not_record): Rename to in_record_wait.
>> > >        (record_not_record_set): Rename to in_record_wait_set.
>> > >        (record_not_record_cleanup): Rename to
>> > in_record_wait_cleanup.
>> > >        (record_store_registers): Check in_record_wait flag.
>> > >        (record_xfer_partial): Ditto.
>> > >
>> > >        2008-10-07  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.h (record_exec_direction): Delete.
>> > >        (RECORD_IS_REPLAY): Consult infrun global
>> direction variable.
>> > >        * record.c: (record_wait_cleanups): Use infrun state
>> > variable.
>> > >        (record_wait): Ditto.
>> > >        (record_get_exec_direction,
>> > record_set_exec_direction): Remove.
>> > >        (record_can_execute_reverse): New target method.
>> > >
>> > >        2008-10-06  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.c (displaced_step_fixup): Remove.
>> > >        (record_message_cleanups): Remove displaced step handling.
>> > >        (record_message): Remove displaced step handling.
>> > >
>> > >        2008-10-06  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.c (record_regcache_raw_write_regnum): Remove.
>> > >
>> > >        2008-10-05  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.c, record.h: Rename execdir to exec_direction.
>> > >
>> > >        2008-10-02  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.c (record_open): Call target_can_async_p() instead
>> > >        of relying on a global variable.
>> > >        * record.h (record_linux_async_permitted): Delete.
>> > >
>> > >        2008-09-19  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.c (trivial): Fix two commas in a comment.
>> > >        * record.c (record_wait): On end of record log, return
>> > >        TARGET_WAITKIND_NO_HISTORY and let infrun decide
>> what to do.
>> > >
>> > >        2008-09-06  Michael Snyder  <msnyder@vmware.com>
>> > >        * record.c: Comment and message string cleanup.
>> > >        Add some function header comments.
>> > >
>> > >        2008-08-01  Michael Snyder  <msnyder@specifix.com>
>> > >        * record.c (_initialize_record): Clarify language in help
>> > >        strings.
>> > >        Fix up comment format (period must be followed by
>> > two spaces).
>> >
>>
>