Use newly introduced TDCALL_0() instead of tdcall() to issue
MR_REPORT tdcall.

It cuts code bloat substantially:

Function                                     old     new   delta
tdx_mcall_get_report0                        229     111    -118

Signed-off-by: Kirill A. Shutemov <kirill.shute...@linux.intel.com>
---
 arch/x86/coco/tdx/tdx.c | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/arch/x86/coco/tdx/tdx.c b/arch/x86/coco/tdx/tdx.c
index 42436a43bb49..45be53d5eeb4 100644
--- a/arch/x86/coco/tdx/tdx.c
+++ b/arch/x86/coco/tdx/tdx.c
@@ -101,19 +101,15 @@ static inline u64 tdg_vm_wr(u64 field, u64 value, u64 
mask)
  */
 int tdx_mcall_get_report0(u8 *reportdata, u8 *tdreport)
 {
-       struct tdx_module_args args = {
-               .rcx = virt_to_phys(tdreport),
-               .rdx = virt_to_phys(reportdata),
-               .r8 = TDREPORT_SUBTYPE_0,
-       };
        u64 ret;
 
-       ret = __tdcall(TDG_MR_REPORT, &args);
-       if (ret) {
-               if (TDCALL_RETURN_CODE(ret) == TDCALL_INVALID_OPERAND)
-                       return -EINVAL;
+       ret = TDCALL_0(TDG_MR_REPORT, virt_to_phys(tdreport),
+                      virt_to_phys(reportdata), TDREPORT_SUBTYPE_0, 0);
+
+       if (TDCALL_RETURN_CODE(ret) == TDCALL_INVALID_OPERAND)
+               return -EINVAL;
+       else if (ret)
                return -EIO;
-       }
 
        return 0;
 }
-- 
2.43.0


Reply via email to