Re: [Xen-devel] [PATCH] ptwr_emulate adjustments

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 16/1/07 12:40 pm, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> - really only do the invalid PAE PTE fixup on write to the high part

Okay. We could theoretically get failures the other way round too I should
think but clearly the existing code doesn't handle that properly (since
it'll lose the PAGE_PRESENT forever) so it can't be something we've seen in
practise. This clarification and extra check makes sense.

> - dump guest execution state when emulation fails

We can't put anything more than XENLOG_G_WARNING on a path that a guest can
exercise at will. One option is to make show_execution_state (and hence all
underlying state-dump functions) take the dump log level as a parameter.

> - use the passed in register pointer rather than re-calculating
>   guest_cpu_user_regs()

The assertion is broken and anyway really a bit overkill imo. Using regs
rather than guest_cpu_user_regs() makes sense.

 -- Keir

> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel