diff options
| author | Stefan Raspl <[email protected]> | 2020-12-08 22:08:29 +0100 |
|---|---|---|
| committer | Paolo Bonzini <[email protected]> | 2020-12-11 19:18:51 -0500 |
| commit | 111d0bda8eeb4b54e0c63897b071effbf9fd9251 (patch) | |
| tree | 362d8f8cfa4c8fd237355b8cd2403765a8b7a88a | |
| parent | 34c0f6f2695a2db81e09a3ab7bdb2853f45d4d3d (diff) | |
tools/kvm_stat: Exempt time-based counters
The new counters halt_poll_success_ns and halt_poll_fail_ns do not count
events. Instead they provide a time, and mess up our statistics. Therefore,
we should exclude them.
Removal is currently implemented with an exempt list. If more counters like
these appear, we can think about a more general rule like excluding all
fields name "*_ns", in case that's a standing convention.
Signed-off-by: Stefan Raspl <[email protected]>
Tested-and-reported-by: Christian Borntraeger <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Paolo Bonzini <[email protected]>
| -rwxr-xr-x | tools/kvm/kvm_stat/kvm_stat | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/kvm/kvm_stat/kvm_stat b/tools/kvm/kvm_stat/kvm_stat index d199a3694be8..b0bf56c5f120 100755 --- a/tools/kvm/kvm_stat/kvm_stat +++ b/tools/kvm/kvm_stat/kvm_stat @@ -742,7 +742,11 @@ class DebugfsProvider(Provider): The fields are all available KVM debugfs files """ - return self.walkdir(PATH_DEBUGFS_KVM)[2] + exempt_list = ['halt_poll_fail_ns', 'halt_poll_success_ns'] + fields = [field for field in self.walkdir(PATH_DEBUGFS_KVM)[2] + if field not in exempt_list] + + return fields def update_fields(self, fields_filter): """Refresh fields, applying fields_filter""" |