Re: [PATCHv5 01/30] x86/tdx: Detect running as a TDX guest in early boot
From: Borislav Petkov
Date: Fri Mar 04 2022 - 11:02:38 EST
On Fri, Mar 04, 2022 at 07:47:37AM -0800, Dave Hansen wrote:
> Pure reviewer paranoia. :)
>
> https://lore.kernel.org/all/YhN5edJQ+LkVc0us@grain/
This is one of those things where when you look at them months, years
from now, you'd go "WTF was that added for?". Because it clearly is
there to catch, well, something you'll catch anyway in testing. Because
if you fail detecting you're running as a TDX guest, you'll know pretty
early about it.
So if it is pure paranoia, you should drop it. Or if there's at least
some merit for it being there, then slap a comment above it why that
check is happening.
Thx.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette