lkml.org 
[lkml]   [2010]   [Mar]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] Enhance perf to collect KVM guest os statistics from host side
    From
    Date
    On Tue, 2010-03-16 at 15:48 +0800, Zhang, Yanmin wrote:
    > On Tue, 2010-03-16 at 07:41 +0200, Avi Kivity wrote:
    > > On 03/16/2010 07:27 AM, Zhang, Yanmin wrote:
    > > > From: Zhang, Yanmin<yanmin_zhang@linux.intel.com>
    > > >
    > > > Based on the discussion in KVM community, I worked out the patch to support
    > > > perf to collect guest os statistics from host side. This patch is implemented
    > > > with Ingo, Peter and some other guys' kind help. Yang Sheng pointed out a
    > > > critical bug and provided good suggestions with other guys. I really appreciate
    > > > their kind help.
    > > >
    > > > The patch adds new subcommand kvm to perf.
    > > >
    > > > perf kvm top
    > > > perf kvm record
    > > > perf kvm report
    > > > perf kvm diff
    > > >
    > > > The new perf could profile guest os kernel except guest os user space, but it
    > > > could summarize guest os user space utilization per guest os.
    > > >
    > > > Below are some examples.
    > > > 1) perf kvm top
    > > > [root@lkp-ne01 norm]# perf kvm --host --guest --guestkallsyms=/home/ymzhang/guest/kallsyms
    > > > --guestmodules=/home/ymzhang/guest/modules top
    > > >
    > > >
    > >
    > Thanks for your kind comments.
    >
    > > Excellent, support for guest kernel != host kernel is critical (I can't
    > > remember the last time I ran same kernels).
    > >
    > > How would we support multiple guests with different kernels?
    > With the patch, 'perf kvm report --sort pid" could show
    > summary statistics for all guest os instances. Then, use
    > parameter --pid of 'perf kvm record' to collect single problematic instance data.
    Sorry. I found currently --pid isn't process but a thread (main thread).

    Ingo,

    Is it possible to support a new parameter or extend --inherit, so 'perf record' and
    'perf top' could collect data on all threads of a process when the process is running?

    If not, I need add a new ugly parameter which is similar to --pid to filter out process
    data in userspace.

    Yanmin




    \
     
     \ /
      Last update: 2010-03-16 10:31    [W:0.025 / U:92.292 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site