On 10/30/24 12:00, Rick Edgecombe wrote:
> +u64 tdh_phymem_page_reclaim(u64 page, u64 *rcx, u64 *rdx, u64 *r8)
> +{
> + struct tdx_module_args args = {
> + .rcx = page,
> + };
> + u64 ret;
This isn't quite what I'm looking for in these wrappers.
For instance:
> + /*
> + * Additional error information:
> + *
> + * - RCX: page type
> + * - RDX: owner
> + * - R8: page size (4K, 2M or 1G)
> + */
> + *rcx = args.rcx;
> + *rdx = args.rdx;
> + *r8 = args.r8;
If this were, instead:
u64 tdh_phymem_page_reclaim(u64 page, u64 *type, u64 *owner, u64 *size)
{
...
*type = args.rcx;
*owner = args.rdx;
*size = args.r8;
Then you wouldn't need the comment in the first place. Then you could
also be thinking about adding _some_ kind of type safety to the
arguments. The 'size' or the 'type' could totally be enums.
There's really zero value in having wrappers like these. They don't
have any type safety or add any readability or make the seamcall easier
to use. There's almost no value in having these versus just exporting
seamcall_ret() itself.
