Some bugfixes (patches 1&2), simplifications (patch 4), plus rework that lays the basis for introducing a (hopefully) neater drilldown mode in patch 7. Note that in absence of any knowledge of the interdependencies of the events in debugfs on other platforms like x86, this new 'functionality' was only added for S390 for now. Adding other platforms would be easy enough, though. Furthermore note that the data passing between Providers and Stats is pretty much at its limit now - if we'd ever wanted to put in more attributes along with the event data passed from Providers to Stats, we should probably switch to passing more involved data structures with indicator flags, etc. Not sure if performance is a concern, but from a coding/maintaining point of view that's something we should consider in that case. Changes in v2: * Patch [1]: - Switched to single underscore prefixing for "private" methods. * Patch [4]: - Renamed methods to "tracepoint_is_child" and "debugfs_is_child" from "trc_is_child" et al. - Also, moved "child_events" and "pid" from DebugfsProvider and TracepointProvider up to "Provider". --- Stefan Raspl (6): tools/kvm_stat: mark private methods as such tools/kvm_stat: eliminate extra guest/pid selection dialog tools/kvm_stat: cache compiled regular expression tools/kvm_stat: separate drilldown and fields filtering tools/kvm_stat: group child events indented after parent tools/kvm_stat: print 'Total' line for multiple events only tools/kvm/kvm_stat/kvm_stat | 435 ++++++++++++++++++++++------------------ tools/kvm/kvm_stat/kvm_stat.txt | 4 +- 2 files changed, 246 insertions(+), 193 deletions(-) -- 2.13.5