Re: [PATCH v5 15/22] x86/virt/tdx: Allocate and set up PAMTs for TDMRs

From: Dave Hansen
Date: Mon Jun 27 2022 - 18:58:11 EST


On 6/27/22 15:50, Kai Huang wrote:
>> Are Kirill's magic 0/1/2 numbers the same as
>>
>> TDX_PG_4K,
>> TDX_PG_2M,
>> TDX_PG_1G,
>>
>> ?
> Yes they are the same. Kirill uses 0/1/2 as input of TDX_ACCEPT_PAGE TDCALL.
> Here I only need them to distinguish different page sizes.
>
> Do you mean we should put TDX_PG_4K/2M/1G definition to asm/tdx.h, and
> try_accept_one() should use them instead of magic 0/1/2?

I honestly don't care how you do it as long as the magic numbers go away
(within reason).