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
next prev parent 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.