>>> On 02.06.15 at 14:47, <ross.lagerw...@citrix.com> wrote:
> +void arch_hvm_load_post(struct domain *d)
> +{
> +    /* Re-enable cache flushes and flush the cache. */
> +    this_cpu(memory_type_changed_ignore) = 0;
> +    memory_type_changed(d);
> +}

Does this really need to be done unconditionally? I.e. couldn't this be
a tristate, with memory_type_changed() e.g. flipping its sign when it
gets actually called?

Also, are we certain that memory_type_changed() will never be
called from asynchronous code (i.e. this doesn't introduce a latent,
hard to debug issue)?

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to