qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] scripts/kvm/kvm_stat: define exit_reasons field


From: Laurent Vivier
Subject: Re: [Qemu-devel] [PATCH] scripts/kvm/kvm_stat: define exit_reasons field in ArchPPC
Date: Fri, 20 May 2016 10:28:48 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0


On 20/05/2016 09:07, Laurent Vivier wrote:
> Otherwise kvm_stat fails with the following error:
> 
>     AttributeError: 'ArchPPC' object has no attribute 'exit_reasons'
> 
> This has been introduced by:
> 
>     068294a scripts/kvm/kvm_stat: Group arch specific data
> 
> Set exit_reasons to 'None' (as it is done for ArchS390) for ArchPPC.
> 
> Test this with "kvm_stat -d" as tracepoints provider (-t) is
> also broken on PPC (looks like a kernel bug).

For this part, I've found the ioctl number used to reset the event queue
is not the good one on PPC (thanks to systemtap), I'm going to send
another patch to fix that too.

Laurent
> 
> Signed-off-by: Laurent Vivier <address@hidden>
> ---
>  scripts/kvm/kvm_stat | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/kvm/kvm_stat b/scripts/kvm/kvm_stat
> index 769d884..78ffe83 100755
> --- a/scripts/kvm/kvm_stat
> +++ b/scripts/kvm/kvm_stat
> @@ -261,6 +261,7 @@ class ArchPPC(Arch):
>          # numbers depend on the wordsize.
>          char_ptr_size = ctypes.sizeof(ctypes.c_char_p)
>          self.ioctl_numbers['SET_FILTER'] = 0x80002406 | char_ptr_size << 16
> +        self.exit_reasons = None
>  
>  class ArchA64(Arch):
>      def __init__(self):
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]