Re: [PATCH 1/2] KVM test: Make the profiler could be configurated

2010-03-31 Thread Jason Wang
Michael Goldish wrote: - Jason Wang jasow...@redhat.com wrote: The patch let the profilers could be specified through configuration file. kvm_stat was kept as the default profiler. Looks good. Some minor style comments: Thanks for the comment, would re-send the patch. -- To

Re: [Autotest] [PATCH 1/2] KVM test: Make the profiler could be configurated

2010-03-31 Thread Lucas Meneghel Rodrigues
On Tue, Mar 30, 2010 at 2:02 PM, Michael Goldish mgold...@redhat.com wrote: - Jason Wang jasow...@redhat.com wrote: The patch let the profilers could be specified through configuration file. kvm_stat was kept as the default profiler. Looks good.  Some minor style comments:

[PATCH 1/2] KVM test: Make the profiler could be configurated

2010-03-30 Thread Jason Wang
The patch let the profilers could be specified through configuration file. kvm_stat was kept as the default profiler. Signed-off-by: Jason Wang jasow...@redhat.com --- client/tests/kvm/kvm_utils.py | 23 ++- client/tests/kvm/tests_base.cfg.sample |2 +- 2 files

Re: [PATCH 1/2] KVM test: Make the profiler could be configurated

2010-03-30 Thread Michael Goldish
- Jason Wang jasow...@redhat.com wrote: The patch let the profilers could be specified through configuration file. kvm_stat was kept as the default profiler. Looks good. Some minor style comments: Signed-off-by: Jason Wang jasow...@redhat.com --- client/tests/kvm/kvm_utils.py