On Wed, Aug 28, 2024 at 04:53:57PM +0300, Nikolay Borisov wrote:
On 28.08.24 г. 12:35 ч., Kirill A. Shutemov wrote:
Rename tdx_parse_tdinfo() to tdx_setup() and move setting NOTIFY_ENABLES there.
The function will be extended to adjust TD configuration.
<offtopic> Since this deals with renaming, I think it will make sense to rename tdx_early_init() to tdx_guest_init/tdx_guest_early_init as it becomes confusing as to which parts of the TDX pertain to the host and which to the guest. Right now we only have the guest portions under arch/x86/coco/tdx but when the kvm/vmx stuff land things will become somewhat messy.. </offtopic>
I don't see a problem with the current state. KVM side will land under arch/x86/virt/vmx/tdx, so the path will give it away.