Re: [RFA/Prec] Add signal support for process record

[Hui Zhu <teawater at gmail dot com>]

On Tue, Sep 1, 2009 at 01:41, Michael Snyder<msnyder@vmware.com> wrote:
> Hui Zhu wrote:
>>
>> Hi guys,
>>
>> After discussion with Michael, ?we decide to deal with this issue step by
>> step.
>> This is the fix bug patch. ?It will fix the after get sig, inferior
>> get a lot of sig. ?Please help me with it.
>
> This takes care of the immediate problem, which is that the
> signals get delivered over and over during the record mode.
>
> Please check this in.

Thanks.  Checked in.

Hui

>
>
>> 2009-08-31 ?Hui Zhu ?<teawater@gmail.com>
>>
>> ? ? ? ?* record.c (record_resume_siggnal): Deleted.
>> ? ? ? ?(record_resume): Ditto.
>> ? ? ? ?(record_wait): Change record_resume_siggnal to
>> ? ? ? ?TARGET_SIGNAL_0.
>>
>> ---
>> ?record.c | ? ?4 +---
>> ?1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> --- a/record.c
>> +++ b/record.c
>> @@ -514,7 +514,6 @@ record_close (int quitting)
>> ?}
>>
>> ?static int record_resume_step = 0;
>> -static enum target_signal record_resume_siggnal;
>> ?static int record_resume_error;
>>
>> ?static void
>> @@ -522,7 +521,6 @@ record_resume (struct target_ops *ops, p
>> ? ? ? ? ? ? ? ?enum target_signal siggnal)
>> ?{
>> ? record_resume_step = step;
>> - ?record_resume_siggnal = siggnal;
>>
>> ? if (!RECORD_IS_REPLAY)
>> ? ? {
>> @@ -638,7 +636,7 @@ record_wait (struct target_ops *ops,
>> ? ? ? ? ? ? ? ? ? ? ? ?}
>> ? ? ? ? ? ? ? ? ? ? ?record_beneath_to_resume
>> (record_beneath_to_resume_ops,
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ptid, 1,
>> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? record_resume_siggnal);
>> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? TARGET_SIGNAL_0);
>> ? ? ? ? ? ? ? ? ? ? ?continue;
>> ? ? ? ? ? ? ? ? ? ?}
>> ? ? ? ? ? ? ? ?}
>
>