All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Dario Faggioli <dario.faggioli@citrix.com>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>,
	jbeulich@suse.com, Ian Campbell <ian.campbell@citrix.com>,
	george.dunlap@citrix.com, xen-devel@lists.xen.org
Subject: Re: [PATCH] perfc: count number of explicit vcpu yields
Date: Mon, 9 Nov 2015 11:03:41 -0500	[thread overview]
Message-ID: <20151109160341.GD8121@char.us.oracle.com> (raw)
In-Reply-To: <1447084040.2722.12.camel@citrix.com>

On Mon, Nov 09, 2015 at 04:47:20PM +0100, Dario Faggioli wrote:
> On Mon, 2015-11-09 at 15:23 +0000, Ian Campbell wrote:
> > On Mon, 2015-11-09 at 16:14 +0100, Dario Faggioli wrote:
> > >  
> > > > As far as xl accessibility --- doesn't xenperf already read them
> > > > out?
> > > > 
> > > Mmm... ISTR having tried without much luck, and having heard that
> > > it
> > > wasn't functional any longer, but maybe I'm confusing it with
> > > something
> > > else. I'll try again.
> > 
> > I happened to have an xterm with this in its scroll buffer from
> > earlier:
> > 
> > root@marilith-n0:~# xenperf 
> > invalid hypercalls                  T=         0           0         
> >  0          0          0
> > trap: wfi                           T=     85068       56068     
> >  11762       8461       8777
> > trap: wfe                           T=      2811         614       
> >  716        964        517
> > trap: cp15 32-bit access            T=         4           1         
> >  1          1          1
> > trap: cp15 64-bit access            T=         0           0         
> >  0          0          0
> > trap: cp14 32-bit access            T=         0           0         
> >  0          0          0
> > trap: cp14 64-bit access            T=         0           0         
> >  0          0          0
> > trap: cp14 dbg access               T=         0           0         
> >  0          0          0
> > trap: cp access                     T=         0           0         
> >  0          0          0
> > trap: 32-bit smc                    T=         0           0         
> >  0          0          0
> > trap: 32-bit hvc                    T=      1291         510       
> >  568        161         52
> > trap: guest instr abort             T=         0           0         
> >  0          0          0
> > trap: guest data abort              T=     16446        6885      
> >  4358       3431       1772
> > trap: condition failed              T=         0           0         
> >  0          0          0
> > [...]
> > 
> > So I think it works.
> > 
> Ok, thanks for the confirmation, and sorry for the noise then.
> 
> > You need to build the hypervisor binary with perfc=y (and optionally
> > perfc_arrays=y) or it doesn't do anything.
> > 
> Yeah, that I knew.

And you need to apply an patch this patch for x86:

http://lists.xen.org/archives/html/xen-devel/2015-11/msg00633.html
since the lock_profile also adds tons of data in the 'struct spinlock'
making the 'struct domain' bigger than 4K (to 68K!).
> 
> Thanks again and Regards,
> Dario
> -- 
> <<This happens because I choose it to happen!>> (Raistlin Majere)
> -----------------------------------------------------------------
> Dario Faggioli, Ph.D, http://about.me/dario.faggioli
> Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
> 



> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

      reply	other threads:[~2015-11-09 16:03 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-09 14:00 [PATCH] perfc: count number of explicit vcpu yields Ian Campbell
2015-11-09 14:09 ` Jan Beulich
2015-11-09 14:14 ` George Dunlap
2015-11-09 14:53 ` Dario Faggioli
2015-11-09 15:02   ` Boris Ostrovsky
2015-11-09 15:14     ` Dario Faggioli
2015-11-09 15:23       ` Ian Campbell
2015-11-09 15:47         ` Dario Faggioli
2015-11-09 16:03           ` 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=20151109160341.GD8121@char.us.oracle.com \
    --to=konrad.wilk@oracle.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=dario.faggioli@citrix.com \
    --cc=george.dunlap@citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=xen-devel@lists.xen.org \
    /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.