Re: [PATCH 04/15] x86/virt/tdx: Enable the Extensions right after basic TDX Module init

[Dan Williams (nvidia)]

Xu Yilun wrote:
> The detailed initialization flow for TDX Module Extensions has been
> fully implemented. Enable the flow after basic TDX Module
> initialization.
> 
> Theoretically, the Extensions doesn't need to be enabled right after
> basic TDX initialization. It could be enabled right before the first
> Extension SEAMCALL is issued. That would save or postpone memory usage.
> But it isn't worth the complexity, the needs for the Extensions are vast
> but the savings are little for a typical TDX capable system (about
> 0.001% of memory). So the Linux decision is to just enable it along with
> the basic TDX.

No real point in rehashing the rationale for the "any available, all the
time" policy yet again especially when this directly conflicts with the
"relatively large amount" comment in the original cover letter.

Otherwise I agree with the proposed reordering of this initial series.

In general though, no big showstoppers for me in this first 4.