summaryrefslogtreecommitdiffstats
path: root/arch/x86
diff options
context:
space:
mode:
authorSean Christopherson <seanjc@google.com>2023-10-27 11:21:44 -0700
committerPaolo Bonzini <pbonzini@redhat.com>2023-11-13 05:28:37 -0500
commitc0db19232c1ed6bd7fcb825c28b014c52732c19e (patch)
tree7710400327cd96868ea54c68cc44da30f36abeeb /arch/x86
parente97b39c5c4362dc1cbc37a563ddac313b96c84f3 (diff)
downloadlinux-stable-c0db19232c1ed6bd7fcb825c28b014c52732c19e.tar.gz
linux-stable-c0db19232c1ed6bd7fcb825c28b014c52732c19e.tar.bz2
linux-stable-c0db19232c1ed6bd7fcb825c28b014c52732c19e.zip
KVM: Assert that mmu_invalidate_in_progress *never* goes negative
Move the assertion on the in-progress invalidation count from the primary MMU's notifier path to KVM's common notification path, i.e. assert that the count doesn't go negative even when the invalidation is coming from KVM itself. Opportunistically convert the assertion to a KVM_BUG_ON(), i.e. kill only the affected VM, not the entire kernel. A corrupted count is fatal to the VM, e.g. the non-zero (negative) count will cause mmu_invalidate_retry() to block any and all attempts to install new mappings. But it's far from guaranteed that an end() without a start() is fatal or even problematic to anything other than the target VM, e.g. the underlying bug could simply be a duplicate call to end(). And it's much more likely that a missed invalidation, i.e. a potential use-after-free, would manifest as no notification whatsoever, not an end() without a start(). Signed-off-by: Sean Christopherson <seanjc@google.com> Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Fuad Tabba <tabba@google.com> Tested-by: Fuad Tabba <tabba@google.com> Message-Id: <20231027182217.3615211-3-seanjc@google.com> Reviewed-by: Kai Huang <kai.huang@intel.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch/x86')
0 files changed, 0 insertions, 0 deletions