Re: [PATCH 6/6] kvm_stat: Add powerpc support

[Alexander Graf <agraf@suse.de>]

On 17.06.14 09:54, Michael Ellerman wrote:
> Add support for powerpc platforms. We use uname -m, which allows us to
> detect ppc, ppc64 and ppc64le/el.
>
> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>

Could you please add support for PR KVM tracepoints along the way? There 
we do know the exit reason for every single guest <-> host transition. I 
would like to move to a similar model with HV in the future, so we can 
hopefully just reuse this by then.


Alex

> ---
>   scripts/kvm/kvm_stat | 18 ++++++++++++++++--
>   1 file changed, 16 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/kvm/kvm_stat b/scripts/kvm/kvm_stat
> index 2e0f5ed..db867fe 100755
> --- a/scripts/kvm/kvm_stat
> +++ b/scripts/kvm/kvm_stat
> @@ -12,7 +12,7 @@
>   # the COPYING file in the top-level directory.
>   
>   import curses
> -import sys, os, time, optparse
> +import sys, os, time, optparse, ctypes
>   
>   class DebugfsProvider(object):
>       def __init__(self):
> @@ -199,7 +199,21 @@ def s390_init():
>           'sc_perf_evt_open' : 331,
>       })
>   
> +def ppc_init():
> +    globals().update({
> +        'sc_perf_evt_open' : 319,
> +        'ioctl_numbers' : {
> +            'SET_FILTER' : 0x80002406 | (ctypes.sizeof(ctypes.c_char_p) << 16),
> +            'ENABLE'     : 0x20002400,
> +            'DISABLE'    : 0x20002401,
> +        }
> +    })
> +
>   def detect_platform():
> +    if os.uname()[4].startswith('ppc'):
> +        ppc_init()
> +        return
> +
>       for line in file('/proc/cpuinfo').readlines():
>           if line.startswith('flags'):
>               for flag in line.split():
> @@ -220,7 +234,7 @@ def invert(d):
>   for f in filters:
>       filters[f] = (filters[f][0], invert(filters[f][1]))
>   
> -import ctypes, struct, array
> +import struct, array
>   
>   libc = ctypes.CDLL('libc.so.6')
>   syscall = libc.syscall