diff options
author | Sean Christopherson <[email protected]> | 2023-10-27 11:21:44 -0700 |
---|---|---|
committer | Paolo Bonzini <[email protected]> | 2023-11-13 05:28:37 -0500 |
commit | c0db19232c1ed6bd7fcb825c28b014c52732c19e (patch) | |
tree | 7710400327cd96868ea54c68cc44da30f36abeeb /tools/perf/scripts/python/stackcollapse.py | |
parent | e97b39c5c4362dc1cbc37a563ddac313b96c84f3 (diff) |
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 <[email protected]>
Reviewed-by: Paolo Bonzini <[email protected]>
Reviewed-by: Fuad Tabba <[email protected]>
Tested-by: Fuad Tabba <[email protected]>
Message-Id: <[email protected]>
Reviewed-by: Kai Huang <[email protected]>
Signed-off-by: Paolo Bonzini <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions