On 05/24/2011 12:57 PM, Roedel, Joerg wrote:
On Tue, May 24, 2011 at 05:28:38AM -0400, Nadav Har'El wrote: > On Tue, May 24, 2011, Roedel, Joerg wrote about "Re: [PATCH 08/31] nVMX: Fix local_vcpus_link handling": > > Probably we should make the local_vcpu_list from vmx generic, use it > > from svm and fix it this way. > > The point is, local_vcpu_list is now gone, replaced by a loaded_vmcss_on_cpu, > and vcpu->cpu is not set to -1 for any vcpu when a CPU is offlined - also in > VMX... loaded_vmcss_on_cpu sound similar, probably this can be generalized.
It's not the same: there is a main:1 relationship between vmcss and vcpus (like vmcbs and vcpus).
However, it may be that the general case for svm also needs to treat individual vmcbs differently.
Is this code already upstream or is this changed with your nVMX patch-set?
Not upstream yet (however generalization, if needed, will be done after it's upstream).
-- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
