On Sat, Jan 04, 2025 at 01:43:56AM +0000, Edgecombe, Rick P wrote: > On Mon, 2024-12-23 at 17:25 +0100, Paolo Bonzini wrote: > > 22: missing review comment from v1 > > > > > + /* TDX only supports x2APIC, which requires an in-kernel local APIC. */ > > > + if (!vcpu->arch.apic) > > > + return -EINVAL; > > > > nit: Use kvm_apic_present() > > Oops, nice catch. Sorry this fell through. I made a patch for this earlier but missed it while rebasing to a later dev branch and never sent it. Below is a rebased version against the current KVM CoCo queue to fold in if still needed. Sounds like this might be already dealt with in Paolo's upcoming CoCo queue branch though. Regards, Tony 8< --------------------