On 12/11/2024 2:25 AM, Paolo Bonzini wrote:
Applied to kvm-coco-queue, thanks. For now I used v1 of "TDX vCPU enter/exit" as it was posted, but I will check out the review comments later. Paolo
Hi Paolo, The the following two fixup patches to v1 of "TDX vCPU enter/exit" related to the later sections. One is https://github.com/intel/tdx/commit/22b7001fbb58771bf133a64e1b22fb9e47d8a11f , make tdx_vcpu_enter_exit() noinstr based on the discussion: https://lore.kernel.org/kvm/Z0SVf8bqGej_-7Sj@xxxxxxxxxx/ The other is https://github.com/intel/tdx/commit/13828e0b586eed6618ccdef9e4f58b09358564d2 , move the check of VCPU_TD_STATE_INITIALIZED from tdx_vcpu_run() to tdx_vcpu_pre_run() based on the discussion: https://lore.kernel.org/kvm/837bbbc7-e7f3-4362-a745-310fe369f43d@xxxxxxxxx/ So the check for VCPU_TD_STATE_INITIALIZED in tdx_handle_exit() is dropped in "TDX hypercalls may exit to userspace"
