All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.