Re: [PATCH 06/25] x86/virt/tdx: Export TDX KeyID information

From: Paolo Bonzini
Date: Tue Sep 10 2024 - 12:26:59 EST


On 8/30/24 23:18, Dave Hansen wrote:
I do like the idea of exporting a couple of helpers that are quite hard
to misuse instead of exporting the variable.

Yeah, that's nicer.

Paolo

Separate from Dave's question, I wonder if we should open code set_hkid_to_hpa()
inside tdh_phymem_page_wbinvd(). The signature could change to
tdh_phymem_page_wbinvd(hpa_t pa, u16 hkid). set_hkid_to_hpa() is very
lightweight, so I don't think doing it outside the loop is much gain. It makes
the code cleaner.
Yeah, do what's cleanest. This is all super cold code.