On Mon, 2022-06-27 at 15:57 -0700, Dave Hansen wrote: > 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). OK. I'll write a patch to replace 0/1/2 magic numbers in try_accept_one(). -- Thanks, -Kai