On 2/9/2016 1:19 PM, Jan Beulich wrote:
On 08.02.16 at 17:57, <cz...@bitdefender.com> wrote:
This patch merges almost identical functions hvm_event_int3
and hvm_event_single_step into a single function called
hvm_event_software_breakpoint.
Except that "software breakpoint" is rather questionable a name
here, considering that on x86 this is basically an alias for "int3".
If it was "breakpoint", one might argue (see the other responses
you've got) that breakpoint event resulting from debug register
settings might then be candidates to come here too.
Jan
Yeah..should I then:
* keep both functions and only rename hvm_event_int3 to
hvm_event_software_breakpoint
* separate the code that gets the GFN of the instruction pointer in a
static inline function
?
Corneliu.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
