From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Mvetl-0007OZ-VZ for qemu-devel@nongnu.org; Wed, 07 Oct 2009 18:22:17 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Mveth-0007LH-4q for qemu-devel@nongnu.org; Wed, 07 Oct 2009 18:22:17 -0400 Received: from [199.232.76.173] (port=48409 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Mvetg-0007LC-Rg for qemu-devel@nongnu.org; Wed, 07 Oct 2009 18:22:12 -0400 Received: from mx1.redhat.com ([209.132.183.28]:22341) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Mvetg-0008W5-FB for qemu-devel@nongnu.org; Wed, 07 Oct 2009 18:22:12 -0400 Received: from int-mx04.intmail.prod.int.phx2.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.17]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id n97MMBiu002763 for ; Wed, 7 Oct 2009 18:22:11 -0400 From: Juan Quintela Date: Thu, 8 Oct 2009 00:21:48 +0200 Message-Id: Subject: [Qemu-devel] [PATCH 0/3] rtc port to VMState (and fix rtc-td hack) List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org now: - minor indentation cleanup - integrate rtc-td-hack properly in normal rtc state - port to VMState >>From the fix rtc-td-hack that didn't change from last sent. v2: Only call rtc_coalesced_timer_update() if rtc_td_hack is true. Patch was failing to load its own images. v1: There hasn't still released a version with the rtc_td hack. Do we want to release 0.11 with this hack or change to a more maintenable mode, i.e. just upgrade the version to 3 and save always the new values. We should also propably add rtd_hack to the state, and make sure that value is the same in origing and destination of th emigration. Another problem is what we do with kvm, they have already done releases with the "mc146818rtc-td" savevm section. Notice that rtc-td-hack should really, really be enabled by defaulet. It is needed to have windows guest running correctly under load. What do you think? What should we do? Later, Juan. PD. Once that I was looking here, this driver has several features that are not used anywhere else. - rtc_mm_init() is defined and exported but not used - rtc_init_sqw() is defined, but only called from rtc_init() with sqw_irq = NULL - sqw_irq is only assigned in rtc_init_sqw() and always to NULL value. Is there a reason why we have the functionality that is not used anywhere? Juan Quintela (3): mc145818rtc: fix saving of rtc-td hack properly upgrading the version number mc146818rtc: port rtc to vmstate mc146818rtc: fix indentation hw/mc146818rtc.c | 117 +++++++++++++++++------------------------------------ 1 files changed, 38 insertions(+), 79 deletions(-)