On 13.02.2023 17:30, Xenia Ragiadakou wrote: > On 2/13/23 17:11, Jan Beulich wrote: >> On 13.02.2023 15:57, Xenia Ragiadakou wrote: >>> --- a/xen/arch/x86/cpu/Makefile >>> +++ b/xen/arch/x86/cpu/Makefile >>> @@ -10,4 +10,6 @@ obj-y += intel.o >>> obj-y += intel_cacheinfo.o >>> obj-y += mwait-idle.o >>> obj-y += shanghai.o >>> -obj-y += vpmu.o vpmu_amd.o vpmu_intel.o >>> +obj-y += vpmu.o >>> +obj-$(CONFIG_AMD_SVM) += vpmu_amd.o >>> +obj-$(CONFIG_INTEL_VMX) += vpmu_intel.o >> >> This code was moved from hvm/ to cpu/ for the specific reason of also >> being used by PV guests. (Sadly the comments at the top weren't >> corrected at that time.) > > Hmm, the init functions are prefixed with svm/vmx. > Does vpmu depend on svm/vmx support?
There are interactions, but I don't think there's a dependency. You may want to ask Boris (whom I have now added to Cc). Jan
