From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:49086) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SV2Us-0003PM-MI for qemu-devel@nongnu.org; Thu, 17 May 2012 11:20:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SV2Ul-0008Ox-MM for qemu-devel@nongnu.org; Thu, 17 May 2012 11:20:10 -0400 Received: from mail-pz0-f45.google.com ([209.85.210.45]:34655) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SV2Ul-0008IN-9u for qemu-devel@nongnu.org; Thu, 17 May 2012 11:20:03 -0400 Received: by dadv2 with SMTP id v2so3231707dad.4 for ; Thu, 17 May 2012 08:20:00 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <4FB51719.8060607@redhat.com> Date: Thu, 17 May 2012 17:19:53 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v6 0/7] RTC: New logic to emulate RTC List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Zhang, Yang Z" Cc: "'aliguori@us.ibm.com'" , "'qemu-devel@nongnu.org'" Il 17/05/2012 04:28, Zhang, Yang Z ha scritto: > Changes in v6: > Rebase to latest QEMU > Fix a bug that fail to pass tests/rtc-test: > In previous version, it uses host time as the base point to calculate guest RTC. It works when guest uses host based clock. But for vm and rt based clock, it's wrong. Because guest's clock may not synchronous with host. In current patch, it use rtc_clock as the reference point and successes to pass the testing. > Adapt paolo's suggestion to use more reasonable code for migration. Nice work! I've made my testcase even nastier and done the following changes: * simplified the code to remove the double update timer. The idea is that if you can latch the update-in-progress flag 244us before the timer expires in update_in_progress, instead of using a separate timer. * improved bisectability by squashing some patches and separating others. * changed the code to do all computation in nanoseconds * simplified the handling of the offset (and I think also made it more precise) * added another corner case: the SET bit does not stop interrupts, while divider reset does The result is at git://github.com/bonzini/qemu.git in branch rtc-intel. You're welcome to give it a further shake, but IMO it's good to go for 1.2! Paolo