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").