Hi Boris,
On 4/20/22 3:00 PM, Borislav Petkov wrote:
On Tue, Apr 19, 2022 at 05:48:57AM -0700, Sathyanarayanan Kuppuswamy wrote:
Make sense. We can just go with CONFIG_INTEL_TDX_ATTESTATION.
Sounds like you didn't read my mail. Kai was questioning the need for a
Kconfig symbol at all. And me too.
If this thing is not going to be used by anything else besides TDX
guests, why does it need a Kconfig option at all?!
Sorry, it is a typo. I meant we can just compile it with
TDX guest config (CONFIG_INTEL_TDX_GUEST).
Boris, this is a simple platform driver which adds IOCTL interfaces to allow
user space to get TDREPORT and TDQuote support.
So, would prefer to leave in platform/x86 or move it to arch/x86/coco/tdx/ ?
See above.
I agree with dropping the new CONFIG option. But regarding driver
location, which is preferred (platform/x86/intel/tdx or arch/x86/coco/tdx/)?
--
Sathyanarayanan Kuppuswamy
Linux Kernel Developer