mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Christian Borntraeger <borntraeger@de.ibm.com>
To: kvm-devel@lists.sourceforge.net
Cc: Avi Kivity <avi@qumranet.com>,
	Laurent Vivier <Laurent.Vivier@bull.net>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	cotte@de.ibm.com
Subject: Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting
Date: Mon, 13 Aug 2007 16:22:27 +0200	[thread overview]
Message-ID: <200708131622.27533.borntraeger@de.ibm.com> (raw)
In-Reply-To: <46C06638.6010108@qumranet.com>

Am Montag, 13. August 2007 schrieb Avi Kivity:

> Laurent's patch gives the best of both worlds: on old 'top', you get 
> guest time accounted as user time, while on new 'top' it is accounted 
> separately.  This is done by reporting user time as the sum of the real 
> user time and guest time.  A newer 'top' can subtract guest time from 
> user time to get the correct statistic.

Yes that looks promising. If I recall correctly we had some strange top 
behaviours when we introduced the steal time. Old top added the steal time to 
idle. We should check that.

> 
> 
> > My implementation uses a similar mechanism like hard and softirq. So I 
have an 
> > sie_enter an sie_exit and a task_is_in_sie function - like irq_enter and 
> > irq_exit. The main difference is based on the fact, that s390 has precise 
> > accouting for irq, steal, user and system time, and therefore my patch is 
> > based on architecture specifc code using CONFIG_VIRT_CPU_ACCOUNT. 
> >   
> 
> Okay, so the code should be under that config option, and kvm should 
> select it.

No hurry..that was specific to our implementation, not KVM :-)
Besided that, Ingo changed the accouting with his CFS scheduler, and I still 
have to figure out how CONFIG_VIRT_CPU_ACCOUNTING can be properly integrated 
in CFS.

Christian

  reply	other threads:[~2007-08-13 16:02 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-10 15:58 Laurent Vivier
2007-08-13  8:01 ` [kvm-devel] " Avi Kivity
2007-08-13  8:13   ` Laurent Vivier
2007-08-13  8:38     ` Avi Kivity
2007-08-13 13:08       ` Laurent Vivier
2007-08-13 13:22         ` Avi Kivity
2007-08-13 14:15     ` Christian Borntraeger
2007-08-13 14:19       ` Avi Kivity
2007-08-13 14:26         ` Christian Borntraeger
2007-08-13 14:37           ` Avi Kivity
2007-08-13 20:40             ` Heiko Carstens
2007-08-19  9:32               ` List stripping out cc's (was: Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting) Avi Kivity
2007-08-19 19:53                 ` List stripping out cc's Jeff Garzik
2007-08-19 20:10                   ` Avi Kivity
2007-08-13 14:05 ` [kvm-devel] [PATCH 0/2][KVM] guest time accounting Christian Borntraeger
2007-08-13 14:10   ` Avi Kivity
2007-08-13 14:22     ` Christian Borntraeger [this message]
2007-08-13 14:22   ` Laurent Vivier
2007-08-13 14:30     ` Avi Kivity
2007-08-13 14:41       ` Laurent Vivier
2007-08-13 15:22         ` Christian Borntraeger
2007-08-13 15:36           ` Laurent Vivier

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=200708131622.27533.borntraeger@de.ibm.com \
    --to=borntraeger@de.ibm.com \
    --cc=Laurent.Vivier@bull.net \
    --cc=avi@qumranet.com \
    --cc=cotte@de.ibm.com \
    --cc=kvm-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®