xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: "W. Michael Petullo" <mike@flyn.org>
Cc: xen-devel@lists.xensource.com, Anil Madhavapeddy <anil@recoil.org>
Subject: Re: Questions about OProfile
Date: Mon, 18 Apr 2011 16:10:58 -0400	[thread overview]
Message-ID: <20110418201058.GA12562@dumpdata.com> (raw)
In-Reply-To: <20110418183359.GA2124@imp.local>

On Mon, Apr 18, 2011 at 01:33:59PM -0500, W. Michael Petullo wrote:
> >>>> First, what is the status of Xenoprof? The latest patch is against 0.9.5,
> >>>> but OProfile 0.9.6 is about 16 months old. Is Xenoprof actively
> >>>> maintained?
> 
> >>> There is a crufty pv_ops patch that I forward ported to 2.6.32 in this tree:
> >>> 
> >>> https://github.com/avsm/linux-2.6.32-xen-oprofile
> 
> >> So, is it true that I need to 1) use this patch or 2) use a non-PV
> >> Ops Dom0? I thought 2.6.32 was already non-PV Ops.
>  
> > I don't understand the question. The above tree is a pv_ops kernel,
> > with additional patches applied so it can be a full Xen dom0 (including
> > netback, etc).  I'm only sporadically updating it as I need to, so you
> > probably just want the oprofile patch [1] if you want to apply it to
> > some other kernel.
> > 
> > https://github.com/avsm/linux-2.6.32-xen-oprofile/commit/4df4780e638d14249b7aa4005511218a9183c324
> 
> I ported Anil's patch to Linux 2.6.38. The result is available at:
> 
> 	http://www.flyn.org/patches/linux-2.6.38-xen-passive-oprofile/linux-2.6.38-xen-passive-oprofile.patch.gz
> 
> With this patch applied to Linux 2.6.38 (along with various orthagonal Xen
> patches), I can perform passive profiling on an unprivileged Xen domain.

Woot! Awesome. For fun I tried to stick on 2.6.39, but got this:

 patch -p1 --dry-run < ~/linux-2.6.38-xen-passive-oprofile.patch 
patching file arch/x86/include/asm/xen/hypercall.h
Hunk #1 succeeded at 447 (offset 7 lines).
patching file arch/x86/oprofile/Makefile
patching file arch/x86/oprofile/xenoprof.c
patching file arch/x86/xen/mmu.c
Hunk #1 succeeded at 2349 (offset 55 lines).
Hunk #2 succeeded at 2381 (offset 55 lines).
Hunk #3 succeeded at 2403 with fuzz 1 (offset 55 lines).
patching file drivers/oprofile/buffer_sync.c
patching file drivers/oprofile/cpu_buffer.c
Hunk #5 FAILED at 259.
Hunk #6 FAILED at 275.
Hunk #7 FAILED at 408.
Hunk #8 succeeded at 448 (offset 10 lines).
3 out of 8 hunks FAILED -- saving rejects to file drivers/oprofile/cpu_buffer.c.rej
patching file drivers/oprofile/cpu_buffer.h
patching file drivers/oprofile/event_buffer.h
patching file drivers/oprofile/oprof.c
patching file drivers/oprofile/oprof.h
patching file drivers/oprofile/oprofile_files.c
patching file drivers/xen/xenoprof/xenoprofile.c
patching file include/linux/oprofile.h
Hunk #4 succeeded at 130 (offset 7 lines).
patching file include/xen/interface/xen.h
patching file include/xen/interface/xenoprof.h
patching file include/xen/xenoprof.h
patching file include/xen/xen-ops.h


And looking at the code it would require some little help in cpu_buffer
depatment.

It definitly has a lot of #ifdef CONFIG_XEN in it.. you wouldn't be
by any chance signing up for also trying to upstream it?

      reply	other threads:[~2011-04-18 20:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-05 17:03 Questions about OProfile W. Michael Petullo
2011-04-05 17:49 ` Anil Madhavapeddy
2011-04-05 21:45   ` W. Michael Petullo
2011-04-05 21:53     ` Anil Madhavapeddy
2011-04-06 16:31       ` W. Michael Petullo
2011-04-11 14:22         ` Konrad Rzeszutek Wilk
2011-04-18 18:33       ` W. Michael Petullo
2011-04-18 20:10         ` Konrad Rzeszutek Wilk [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110418201058.GA12562@dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=anil@recoil.org \
    --cc=mike@flyn.org \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).