All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoffer Dall <cdall@linaro.org>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: "Radim Krčmář" <rkrcmar@redhat.com>,
	"Christoffer Dall" <christoffer.dall@linaro.org>,
	kvm@vger.kernel.org, "Marc Zyngier" <marc.zyngier@arm.com>,
	"Rik van Riel" <riel@redhat.com>
Subject: Re: vtime accounting
Date: Wed, 15 Mar 2017 09:30:48 +0100	[thread overview]
Message-ID: <20170315083048.GI1277@cbox> (raw)
In-Reply-To: <91c3d503-952a-d376-b5d3-df64bc2b6332@redhat.com>

On Wed, Mar 15, 2017 at 09:05:28AM +0100, Paolo Bonzini wrote:
> 
> 
> On 14/03/2017 19:41, Christoffer Dall wrote:
> > So would it be worth considering factoring out vtime accounting from
> > guest_enter/exit, such that we could do the vtime accounting from vcpu
> > load/put and mark the RCU extended quiescent state in the run loop?
> > 
> > Disclaimer: I haven't completely convinced myself that vtime accounting
> > from load/put works as it should.
> 
> Me neither, but if it works, it's worth at least benchmarking it to make
> an informed decision.
> 

I'll try to prototype it and see how it looks.

> > For example, when servicing a VM from
> > KVM, should we really be accounting this as kernel time, or as guest
> > time?
> 
> Servicing the request should be fast enough that it doesn't matter.  For
> the most common x86 vmexits, 50% of the time is spent executing the x86
> world switch microcode, and that is already being accounted as guest
> time already.  On microbenchmarks of course it would make a difference:
> 
> $ time ./x86/run x86/vmexit.flat -append cpuid -smp 4
> [...]
> real	0m1.327s
> user	0m1.877s        << this is really guest time
> sys	0m1.844s        << this is spent mostly in KVM_RUN
> 
> 
Good point, and if it's for statistics, this light change in behavior
can hardly constitute any form of user breakage.

Thanks,
-Christoffer

  reply	other threads:[~2017-03-15  8:31 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-08 10:57 vtime accounting Christoffer Dall
2017-03-09  8:16 ` Paolo Bonzini
2017-03-13 17:28 ` Radim Krčmář
2017-03-14  8:26   ` Christoffer Dall
2017-03-14  8:55     ` Marc Zyngier
2017-03-14 11:12       ` Christoffer Dall
2017-03-14 11:46         ` Marc Zyngier
2017-03-14 16:58     ` Radim Krčmář
2017-03-14 17:09       ` Paolo Bonzini
2017-03-14 18:41         ` Christoffer Dall
2017-03-14 19:32           ` Radim Krčmář
2017-03-14 20:01             ` Christoffer Dall
2017-03-14 21:52               ` Radim Krčmář
2017-03-15  8:09                 ` Paolo Bonzini
2017-03-15  8:05           ` Paolo Bonzini
2017-03-15  8:30             ` Christoffer Dall [this message]
2017-03-14 18:39       ` Christoffer Dall
2017-03-14 20:27         ` Radim Krčmář
2017-03-14 21:53           ` Radim Krčmář
2017-03-15  8:43           ` Christoffer Dall
2017-03-15 15:57             ` Radim Krčmář
2017-03-15 16:48               ` Christoffer Dall
2017-03-15 17:09                 ` Radim Krčmář
2017-03-24 15:04             ` Rik van Riel
2017-03-27 12:29               ` Wanpeng Li
2017-03-24 14:55     ` Rik van Riel

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=20170315083048.GI1277@cbox \
    --to=cdall@linaro.org \
    --cc=christoffer.dall@linaro.org \
    --cc=kvm@vger.kernel.org \
    --cc=marc.zyngier@arm.com \
    --cc=pbonzini@redhat.com \
    --cc=riel@redhat.com \
    --cc=rkrcmar@redhat.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.