Re: [PATCH] x86/paravirt: Fix build PARAVIRT_XXL=y without XEN_PV

From: Sathyanarayanan Kuppuswamy
Date: Wed Nov 17 2021 - 14:14:01 EST




On 11/17/21 10:48 AM, Borislav Petkov wrote:
On Wed, Nov 17, 2021 at 10:46:30AM -0800, Sathyanarayanan Kuppuswamy wrote:
TDX has a requirement to use HLT paravirt calls (which is currently
listed under PARAVIRT_XXL). Once we submit a patch to move it
under CONFIG_PARAVIRT, we will drop this dependency.

You already have this patch in some set:

https://lore.kernel.org/r/20211009053747.1694419-2-sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx

So what's this churn for?

Previously, we have posted ~45+ patches for TDX guest support. But we have noticed that submitting too many TDX patches at the same time seems
to complicates the review process.

So to make the review process simpler, we had planned to resubmit only a minimum set required to boot the TDX guest to user space (which comes around 25 patches).

Since moving HLT PV calls under CONFIG_PARAVIRT is an optimization fix
we did not include it in the minimal set.

If you think otherwise, please let me know. We will drop the use of
PARAVIRT_XXL and just use the above-mentioned patch.




--
Sathyanarayanan Kuppuswamy
Linux Kernel Developer