Re: [PATCH v3 03/26] x86/virt/tdx: Move low level SEAMCALL helpers out of <asm/tdx.h>
From: Dave Hansen
Date: Wed Jan 28 2026 - 11:42:16 EST
On 1/23/26 06:55, Chao Gao wrote:
> +++ b/arch/x86/virt/vmx/tdx/seamcall.h
Moving the code to a local header is a good thing. The more private
these things are, the better.
I _do_ like when I see these things have a label in the filename like:
internal.h
or even:
seamcall_internal.h
That really catches your eye. It would also be ideal to have a small
blurb at the top of the file to say what its scope is, just to explain
what folks should be adding to it or not.
If you get a chance to add those, all the better. But either way:
Acked-by: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>