>>> On 15.02.17 at 12:13, <sergey.dya...@citrix.com> wrote: > On Wed, 2017-02-15 at 04:00 -0700, Jan Beulich wrote: >> Similarly, when __context_switch() is being bypassed also on the second >> (switch-in) path, VMCS ownership may have been lost and hence needs >> re-establishing. Since there's no existing hook to put this in, add a >> new one. > > BTW, and what about vmx_do_resume()? It does vmx_load_vmcs() in cases > when VMCS was cleared by someone.
Not sure I understand the question correctly - it would seem to me that this was a half hearted attempt to deal with the situation we now find needs more changes, i.e. I would assume that call could now be dropped from the function. Or if it was to stay, it should likely become a vmx_vmcs_reload() call (with the inner conditional dropped). Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel