Messages in this thread | | | From | "Huang, Kai" <> | Subject | Re: [PATCH v12 07/22] x86/virt/tdx: Add skeleton to enable TDX on demand | Date | Tue, 27 Jun 2023 10:37:02 +0000 |
| |
On Mon, 2023-06-26 at 14:21 -0700, Sathyanarayanan Kuppuswamy wrote: > > + /* All '0's are just unused parameters. */ > > I have noticed that you add the above comment whenever you call seamcall() > with > 0 as parameters. Is this a ask from the maintainer? If not, I think you can > skip > it. Just explaining the parameters in seamcall function definition is good > enough.
Yes I followed maintainer (I didn't bother to find the exact link this time, though). I think in this way we don't need to go to TDX module spec to check whether 0 has meaning in each SEAMCALL, especially in code review. I kinda agree having them in multiple places is a little bit noisy, but I don't have a better way.
| |