[PATCH 00/20] x86/tdx: Rewrite TDCALL wrappers

Kirill A. Shutemov
2024-05-17T14:19:55Z

Sean noticing that the TDCALL wrappers were generating a lot of awful
code.

TDCALL calls are centralized into a few megawrappers that take the
struct tdx_module_args as input. Most of the call sites only use a few
arguments, but they have to zero out unused fields in the structure to
avoid data leaks to the VMM. This leads to the compiler generating
inefficient code: dozens of instructions per call site to clear unused
fields of the structure.

This issue can be avoided by using more targeted wrappers.

After the rewrite code size is cut by ~3K:

add/remove: 7/15 grow/shrink: 1/17 up/down: 212/-3502 (-3290)

Please take a look. I would appreciate any feedback.

Kirill A. Shutemov (20):
x86/tdx: Introduce tdvmcall_trampoline()
x86/tdx: Add macros to generate TDVMCALL wrappers
x86/tdx: Convert port I/O handling to use new TDVMCALL macros
x86/tdx: Convert HLT handling to use new TDVMCALL_0()
x86/tdx: Convert MSR read handling to use new TDVMCALL_1()
x86/tdx: Convert MSR write handling to use new TDVMCALL_0()
x86/tdx: Convert CPUID handling to use new TDVMCALL_4()
x86/tdx: Convert MMIO handling to use new TDVMCALL macros
x86/tdx: Convert MAP_GPA hypercall to use new TDVMCALL macros
x86/tdx: Convert GET_QUOTE hypercall to use new TDVMCALL macros
x86/tdx: Rewrite tdx_panic() without __tdx_hypercall()
x86/tdx: Rewrite tdx_kvm_hypercall() without __tdx_hypercall()
x86/tdx: Rewrite hv_tdx_hypercall() without __tdx_hypercall()
x86/tdx: Add macros to generate TDCALL wrappers
x86/tdx: Convert PAGE_ACCEPT tdcall to use new TDCALL_0() macro
x86/tdx: Convert VP_INFO tdcall to use new TDCALL_5() macro
x86/tdx: Convert VM_RD/VM_WR tdcalls to use new TDCALL macros
x86/tdx: Convert VP_VEINFO_GET tdcall to use new TDCALL_5() macro
x86/tdx: Convert MR_REPORT tdcall to use new TDCALL_0() macro
x86/tdx: Remove old TDCALL wrappers

arch/x86/boot/compressed/tdx.c | 32 +---
arch/x86/coco/tdx/tdcall.S | 145 ++++++++++-----
arch/x86/coco/tdx/tdx-shared.c | 26 +--
arch/x86/coco/tdx/tdx.c | 298 ++++++++----------------------
arch/x86/hyperv/ivm.c | 33 +---
arch/x86/include/asm/shared/tdx.h | 159 +++++++++++-----
arch/x86/include/asm/tdx.h | 2 +
arch/x86/virt/vmx/tdx/tdxcall.S | 29 +--
tools/objtool/noreturns.h | 2 +-
9 files changed, 322 insertions(+), 404 deletions(-)

--
2.43.0