From: Paolo Bonzini <pbonzini@redhat.com>
To: "Zhang, Yang Z" <yang.z.zhang@intel.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
Jan Kiszka <jan.kiszka@web.de>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"aliguori@us.ibm.com" <aliguori@us.ibm.com>,
Marcelo Tosatti <mtosatti@redhat.com>
Subject: Re: [PATCH v2 0/4] RTC: New logic to emulate RTC
Date: Mon, 20 Feb 2012 08:41:19 +0100 [thread overview]
Message-ID: <4F41F91F.7060206@redhat.com> (raw)
In-Reply-To: <A9667DDFB95DB7438FA9D7D576C3D87E071FC8@SHSMSX101.ccr.corp.intel.com>
On 02/20/2012 01:24 AM, Zhang, Yang Z wrote:
> Changes in v2:
> Add UIP check logic.
> Add logic that next second tick will occur in exactly 500ms later after setting the clock
>
> Current RTC emulation uses periodic timer(2 timers per second) to update RTC clock. And it will stop CPU staying at deep C-state for long period. Our experience shows the Pkg C6 residency reduced 6% when running 64 idle guest.
> The following patch stop the two periodic timer and only updating RTC clock when guest try to read it.
>
> ---
> Yang Zhang (4):
> RTC: Update the RTC clock only when reading it
> RTC: Add RTC update-ended interrupt support
> RTC: Add alarm support
> RTC: Add UIP(update in progress) check logic
>
> hw/mc146818rtc.c | 447 +++++++++++++++++++++++++++++++++++++-----------------
> 1 files changed, 308 insertions(+), 139 deletions(-)
>
> best regards
> yang
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
Thanks, this looks much better! I'll run it through some tests.
We also should try to keep migration working from older versions using
the load_old callback.
Paolo
WARNING: multiple messages have this Message-ID (diff)
From: Paolo Bonzini <pbonzini@redhat.com>
To: "Zhang, Yang Z" <yang.z.zhang@intel.com>
Cc: "aliguori@us.ibm.com" <aliguori@us.ibm.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
Jan Kiszka <jan.kiszka@web.de>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>
Subject: Re: [Qemu-devel] [PATCH v2 0/4] RTC: New logic to emulate RTC
Date: Mon, 20 Feb 2012 08:41:19 +0100 [thread overview]
Message-ID: <4F41F91F.7060206@redhat.com> (raw)
In-Reply-To: <A9667DDFB95DB7438FA9D7D576C3D87E071FC8@SHSMSX101.ccr.corp.intel.com>
On 02/20/2012 01:24 AM, Zhang, Yang Z wrote:
> Changes in v2:
> Add UIP check logic.
> Add logic that next second tick will occur in exactly 500ms later after setting the clock
>
> Current RTC emulation uses periodic timer(2 timers per second) to update RTC clock. And it will stop CPU staying at deep C-state for long period. Our experience shows the Pkg C6 residency reduced 6% when running 64 idle guest.
> The following patch stop the two periodic timer and only updating RTC clock when guest try to read it.
>
> ---
> Yang Zhang (4):
> RTC: Update the RTC clock only when reading it
> RTC: Add RTC update-ended interrupt support
> RTC: Add alarm support
> RTC: Add UIP(update in progress) check logic
>
> hw/mc146818rtc.c | 447 +++++++++++++++++++++++++++++++++++++-----------------
> 1 files changed, 308 insertions(+), 139 deletions(-)
>
> best regards
> yang
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
Thanks, this looks much better! I'll run it through some tests.
We also should try to keep migration working from older versions using
the load_old callback.
Paolo
next prev parent reply other threads:[~2012-02-20 7:41 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-20 0:24 [PATCH v2 0/4] RTC: New logic to emulate RTC Zhang, Yang Z
2012-02-20 0:24 ` [Qemu-devel] " Zhang, Yang Z
2012-02-20 7:41 ` Paolo Bonzini [this message]
2012-02-20 7:41 ` Paolo Bonzini
2012-02-21 0:00 ` Zhang, Yang Z
2012-02-21 0:00 ` [Qemu-devel] " Zhang, Yang Z
2012-02-22 11:19 ` Paolo Bonzini
2012-02-22 11:19 ` [Qemu-devel] " Paolo Bonzini
2012-02-23 1:49 ` Zhang, Yang Z
2012-02-23 1:49 ` [Qemu-devel] " Zhang, Yang Z
2012-02-23 6:51 ` Paolo Bonzini
2012-02-23 6:51 ` [Qemu-devel] " Paolo Bonzini
2012-02-23 7:43 ` Paolo Bonzini
2012-02-23 7:43 ` [Qemu-devel] " Paolo Bonzini
2012-02-24 0:55 ` Zhang, Yang Z
2012-02-24 0:55 ` [Qemu-devel] " Zhang, Yang Z
2012-02-24 6:56 ` Paolo Bonzini
2012-02-24 6:56 ` [Qemu-devel] " Paolo Bonzini
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=4F41F91F.7060206@redhat.com \
--to=pbonzini@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=jan.kiszka@web.de \
--cc=kvm@vger.kernel.org \
--cc=mtosatti@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=yang.z.zhang@intel.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.