Re: [PATCH 10/17] KVM: TDX: Move set_external_spte_present() assert into TDX code
From: Huang, Kai
Date: Tue Mar 31 2026 - 06:43:27 EST
On Fri, 2026-03-27 at 13:14 -0700, Rick Edgecombe wrote:
> Move the MMU lockdep assert in set_external_spte_present() into the TDX
> specific op because the assert is TDX specific in intention.
>
> The TDP MMU has many lockdep asserts for various scenarios, and in fact
> the callchains that are used for TDX already have a lockdep assert which
> cover the case in set_external_spte_present().
>
cover -> covers
> However, these asserts are
> for management of the TDP root owned by KVM. In the
> set_external_spte_present() assert case, it is helping with a scheme to
> avoid contention in the TDX module during zap operations. That is very
> TDX specific.
>
> One option would be to just remove the assert in
> set_external_spte_present() and rely on the other ones in the TDP MMU. But
> that assert is for an a different intention, and too far away from the
"an a" -> a
> SEAMCALL that needs it.
>
> So move just move it to TDX code.
So just move it ...