[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2] x86/HVM: Avoid cache flush operations during hvm_load



>>> On 02.06.15 at 14:47, <ross.lagerwall@xxxxxxxxxx> 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@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.