On Thu, Jun 25, 2020 at 01:17:22PM -0700, Fenghua Yu wrote:

> +static bool fixup_pasid_exception(void)
> +{
> +     if (!IS_ENABLED(CONFIG_INTEL_IOMMU_SVM))
> +             return false;
> +     if (!static_cpu_has(X86_FEATURE_ENQCMD))
> +             return false;

elsewhere you had another variation:

+       if (!IS_ENABLED(CONFIG_INTEL_IOMMU_SVM))
+               return;
+
+       if (!cpu_feature_enabled(X86_FEATURE_ENQCMD))
+               return;

Which is it, and why do we need the CONFIG thing when combined with the
enabled thing?

Reply via email to