Re: [PATCH] KVM: Create debugfs statistics for each VM

[Paolo Bonzini <pbonzini@redhat.com>]

On 02/02/2016 15:14, Christian Borntraeger wrote:
> FWIW, the newly created subfolders,
> 
> require QEMU commit 6590045e5dd2fb0b1d7cdc047ae0c52fd4bb5276
>     scripts/kvm/kvm_stat: Replaced os.listdir with os.walk
> 
> Otherwise you might get errors like 
> 
> Traceback (most recent call last):
>   File "scripts/kvm/kvm_stat", line 640, in <module>
>     curses.wrapper(tui, stats)
>   File "/usr/lib64/python2.7/curses/wrapper.py", line 43, in wrapper
>     return func(stdscr, *args, **kwds)
>   File "scripts/kvm/kvm_stat", line 547, in tui
>     refresh(sleeptime)
>   File "scripts/kvm/kvm_stat", line 523, in refresh
>     s = stats.get()
>   File "scripts/kvm/kvm_stat", line 483, in get
>     new = d.read()
>   File "scripts/kvm/kvm_stat", line 37, in read
>     return dict([(key, val(key)) for key in self._fields])
>   File "scripts/kvm/kvm_stat", line 36, in val
>     return int(file(self.base + '/' + key).read())
> IOError: [Errno 21] Is a directory: '/sys/kernel/debug/kvm/13123-31'
> 
> when using the "-d" option.
> 
> 
> Paolo, I still think that this a valuable addon for debugging. Any guidance?

Yeah, I agree...  Do we want to add a module parameter for this, and/or
a kernel configuration that for now defaults to N?

An alternative is to move kvm_stat from QEMU to tools/.

Paolo