[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/8] target-i386: Re-introduce optimal breakpoin
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 4/8] target-i386: Re-introduce optimal breakpoint removal |
Date: |
Wed, 16 Sep 2015 07:57:04 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 09/16/2015 01:57 AM, Paolo Bonzini wrote:
>
>
> On 15/09/2015 20:45, Richard Henderson wrote:
>> + /* Fold the global and local enable bits together into the
>> + global fields, then xor to show which registers have
>> + changed collective enable state. */
>> + int mod = ((old_dr7 | old_dr7 * 2) ^ (new_dr7 | new_dr7 * 2)) &
>> 0xff;
>
> The AND is not needed at all but, if you add it, you might as well use
> "& 0xaa" which is clearer. But even better, just do:
>
> target_ulong old_dr7 = env->dr[7];
> int mod = old_dr7 ^ new_dr7;
> ...
> if ((mod & ~0xff) == 0) {
>
>
> and test with
>
> if (mod & (3 << i * 2))
>
> inside the loop.
Nope. I wrote that the first time myself. We're interested in two different
things: (1) whether or not something changed outside enable bits, and (2)
whether the enable state changed.
Since (2) is a combination of both global and local enable bits, we must
combine them *and then xor* to see if the enable state actually changes. Just
using (mod & (3 << n)) will report "change" when local enable turns off, but
global enable remains on. Which is not what we want.
Perhaps that comment could stand to be expanded...
r~
- [Qemu-devel] [PATCH 0/8] target-i386: Implement debug extensions, Richard Henderson, 2015/09/15
- [Qemu-devel] [PATCH 5/8] target-i386: Move hw_*breakpoint_* functions, Richard Henderson, 2015/09/15
- [Qemu-devel] [PATCH 6/8] target-i386: Optimize setting dr[0-3], Richard Henderson, 2015/09/15
- [Qemu-devel] [PATCH 8/8] target-i386: Check CR4[DE] for processing DR4/DR5, Richard Henderson, 2015/09/15
- [Qemu-devel] [PATCH 7/8] target-i386: Handle I/O breakpoints, Richard Henderson, 2015/09/15
- [Qemu-devel] [PATCH 1/8] target-i386: Move breakpoint related functions to new file, Richard Henderson, 2015/09/15
- Re: [Qemu-devel] [PATCH 0/8] target-i386: Implement debug extensions, Paolo Bonzini, 2015/09/21