All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Hecht <dhecht@vmware.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH 1/2] cpu steal time accounting
Date: Thu, 23 Feb 2006 10:17:11 -0800	[thread overview]
Message-ID: <43FDFC27.5030205@vmware.com> (raw)
In-Reply-To: <259022e94a98f0f5df18a1fe678ccb89@cl.cam.ac.uk>

Keir Fraser wrote:
> 
> On 22 Feb 2006, at 23:58, Dan Hecht wrote:
> 
>> To solve this, it may be best to have the hypervisor interface expose 
>> per-vcpu stolen time directly, rather than vcpu_time.  Then the guest 
>> does not need to try to guess whether to charge (system_time - 
>> vcpu_time) against idle or steal.
> 
> Yes, the distinction between stolen and available time does makes sense 
> (although I'm not sure 'available' is a great name) 

The term "available" came from looking at it from the perspective of the 
vcpu, rather than the hypervisor.  To the vcpu, the time that it's 
running or halted is, in a sense, "available" to it (even though, (as an 
optimization) the hypervisor might use the pcpu to do something else 
when the vcpu is halted).  But, anytime the hypervisor forces the vcpu 
to wait involuntarily, the time is no longer "available" to it, but stolen.

Said another way, on native hardware, stolen time is zero.  All time is 
"available" to the OS.  Though it might choose to halt for some of this 
time, the time is still "available".

> otherwise you can't 
> account for wakeup latencies. account_steal_time() would need to be 
> modified in Linux, though, as we would not need its dodgy heuristic for 
> deciding whether to account to stolen time or iowait/idle.
> 

Exactly.  We slightly refactor the account_steal_time() interface to 
have an interface that bypasses the heuristic.

Dan

  reply	other threads:[~2006-02-23 18:17 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <43FCCB2C.5000408@vmware.com>
2006-02-22 23:58 ` [PATCH 1/2] cpu steal time accounting Dan Hecht
2006-02-23  8:40   ` Keir Fraser
2006-02-23 18:17     ` Dan Hecht [this message]
2006-02-23  8:48   ` Keir Fraser
2006-02-23 10:04     ` Keir Fraser
2006-02-23 19:43       ` Dan Hecht
2006-02-23 13:18     ` Rik van Riel
2006-02-21 14:06 Tian, Kevin
  -- strict thread matches above, loose matches on Subject: below --
2006-02-21  9:05 Tian, Kevin
2006-02-21 12:56 ` Rik van Riel
2006-02-21  0:51 Rik van Riel
2006-02-21 17:42 ` Keir Fraser
2006-02-21 19:32   ` Rik van Riel
2006-02-21 21:24     ` Diwaker Gupta
2006-02-22  9:00     ` Keir Fraser
2006-02-22 14:27       ` Rik van Riel
2006-02-22 17:08         ` Keir Fraser
2006-02-22 17:11           ` Rik van Riel
2006-02-22 17:45             ` Keir Fraser
2006-02-24 19:04               ` 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=43FDFC27.5030205@vmware.com \
    --to=dhecht@vmware.com \
    --cc=Keir.Fraser@cl.cam.ac.uk \
    --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.