On 4/16/2021 5:04 AM, Vitaly Kuznetsov wrote:
Vineeth Pillai <virem...@linux.microsoft.com> writes:


+#if IS_ENABLED(CONFIG_HYPERV)
+static void hv_init_vmcb(struct vmcb *vmcb)
+{
+       struct hv_enlightenments *hve = &vmcb->hv_enlightenments;
+
+       if (npt_enabled &&
+           ms_hyperv.nested_features & HV_X64_NESTED_ENLIGHTENED_TLB)
Nitpick: we can probably have a 'static inline' for

  "npt_enabled && ms_hyperv.nested_features & HV_X64_NESTED_ENLIGHTENED_TLB"

e.g. 'hv_svm_enlightened_tlbflush()'
Makes sense, will do.

Thanks,
Vineeth




Reply via email to