[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 7/13] Restore pc on watchpoint hits
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 7/13] Restore pc on watchpoint hits |
Date: |
Fri, 04 Jul 2008 23:15:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Paul Brook wrote:
> On Thursday 03 July 2008, Jan Kiszka wrote:
>> In order to provide accurate information about the triggering
>> instruction, this patch adds the required bits to restore the pc if the
>> access happened inside a TB. With the BP_STOP_BEFORE_ACCESS flag, the
>> watchpoint user can control if the debug trap should be issued on or
>> after the accessing instruction.
>
> I'm fairly sure this won't work. What happens when an interrupt occurs?
When in interrupt is raised before the watchpoint is re-triggered (after
restore), it might be handled first - hmm... Maybe this has to be
prevented to avoid inconsistencies (ie. no guest IRQ delivery when
watchpoints are pending). Will check again and update the patch.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 1/13] Return appropriate watch message to gdb, (continued)
- [Qemu-devel] [PATCH 1/13] Return appropriate watch message to gdb, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 11/13] Add debug exception hook, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 9/13] Improve debugging of SMP guests, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 8/13] Remove premature memop TB terminations, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 6/13] Switch self-modified code recompilation to next_cflags, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 13/13] x86: Debug register emulation, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 10/13] Introduce BP_WATCHPOINT_HIT flag, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 4/13] Respect length of watchpoints, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 7/13] Restore pc on watchpoint hits, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 3/13] Set mem_io_vaddr on io_read, Jan Kiszka, 2008/07/03
- [Qemu-devel] [PATCH 5/13] Introduce next_cflags, Jan Kiszka, 2008/07/03
- [Qemu-devel] Re: [PATCH 0/13] Enhance debugging support - 2nd take, Jan Kiszka, 2008/07/11