Re: [PATCH v7 00/10] TDX host: metadata reading tweaks, bug fix and info dump

From: Huang, Kai
Date: Mon Nov 11 2024 - 15:49:32 EST


On Mon, 2024-11-11 at 12:33 -0800, Dave Hansen wrote:
> On 11/11/24 02:39, Kai Huang wrote:
> > This series does necessary tweaks to TDX host "global metadata" reading
> > code to fix some immediate issues in the TDX module initialization code,
> > with intention to also provide a flexible code base to support sharing
> > global metadata to KVM (and other kernel components) for future needs.
>
> Could we please just limit this to the bug fix and the new TD metadata
> infrastructure? Let's not mix it all up with the debugging printk()s.

It also has a patch to fail module initialization when NO_MOD_BBP feature is not
support.

Just want to confirm, do you want to remove the code to:

- print CMRs;
- print TDX module versoin;

?

Then I will need to:

- remove "printing CMRs" in patch 7 ("x86/virt/tdx: Trim away tail null CMRs").
- remove patch 10 ("x86/virt/tdx: Print TDX module version").