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
