From: teawater <teawater@gmail.com>
To: "Michael Snyder" <msnyder@vmware.com>
Cc: "Daniel Jacobowitz" <drow@false.org>,
"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [reverse RFA] no singlestep-over-BP in reverse
Date: Wed, 17 Sep 2008 15:32:00 -0000 [thread overview]
Message-ID: <daef60380809170831p283405f3ybcad8165fa046261@mail.gmail.com> (raw)
In-Reply-To: <48CFFE21.8030709@vmware.com>
Agree with you.
And I think Maybe need add a test for it.
I think is:
1. There is a statement that set the value of a variable.
For example:
a = 1;
2. Before this statement, the value of this variable is not same with new value.
3. Set a breakpoint on this statement, And check the value of this
variable when forward execute and reverse execute program break by
this breakpoint. If the value is the new value, fail. If is the old
value, pass.
Thanks,
Hui
On Wed, Sep 17, 2008 at 02:42, Michael Snyder <msnyder@vmware.com> wrote:
> teawater wrote:
>>
>> I think maybe some reverse target (in the future?) need it.
>> Someone already know it already deal the breakpoint. But the others
>> will not know. Maybe I will change inside record to second type.
>>
>>
>> If this single step affect some target, how about let target choice it
>> with itself?
>
> At the moment, my opinion is that gdb needs to be able
> to expect consistent behavior from the target(s).
>
> And I believe that consistent behavior / semantics should be:
>
> If you tell me that you are stopped at instruction 1000,
> regardless of whether you were going forward or backward
> when you got there, then I will expect that if I tell you
> to execute forward, you will execute the instruction at
> 1000.
>
> Therefore the machine state at this point should reflect
> the state BEFORE any side effects of the forward execution
> of that instruction.
>
> This is how it is for forward execution, and it only makes
> sense to expect the same for reverse execution. Any variables
> that are changed by the instruction should have their pre-
> execution values.
>
>
next prev parent reply other threads:[~2008-09-17 15:32 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-15 18:33 Michael Snyder
2008-09-15 18:43 ` Daniel Jacobowitz
2008-09-15 19:07 ` Michael Snyder
2008-09-15 21:15 ` Daniel Jacobowitz
2008-09-15 23:09 ` Michael Snyder
2008-09-16 0:09 ` Michael Snyder
2008-09-16 4:13 ` teawater
[not found] ` <daef60380809152110u663350abx76b283d519c5a09d@mail.gmail.com>
2008-09-16 18:40 ` Michael Snyder
2008-09-16 4:22 ` teawater
2008-09-16 15:04 ` teawater
2008-09-16 15:14 ` teawater
2008-09-16 15:22 ` Daniel Jacobowitz
2008-09-16 15:34 ` teawater
2008-09-16 15:35 ` teawater
2008-09-16 18:50 ` Michael Snyder
2008-09-16 18:45 ` Michael Snyder
2008-09-16 20:11 ` Joel Brobecker
2008-09-17 0:56 ` Michael Snyder
2008-09-17 15:44 ` teawater
2008-09-17 18:18 ` Michael Snyder
2008-09-17 15:32 ` teawater [this message]
2008-09-17 18:16 ` Michael Snyder
2008-09-18 6:39 ` teawater
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=daef60380809170831p283405f3ybcad8165fa046261@mail.gmail.com \
--to=teawater@gmail.com \
--cc=drow@false.org \
--cc=gdb-patches@sourceware.org \
--cc=msnyder@vmware.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox