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 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
