From patchwork Wed Jan 21 03:51:05 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Zhang, Xiantao" X-Patchwork-Id: 3386 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n0L3kbs3014987 for ; Tue, 20 Jan 2009 19:46:37 -0800 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756257AbZAUDvK (ORCPT ); Tue, 20 Jan 2009 22:51:10 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756782AbZAUDvK (ORCPT ); Tue, 20 Jan 2009 22:51:10 -0500 Received: from mga14.intel.com ([143.182.124.37]:39084 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756257AbZAUDvJ convert rfc822-to-8bit (ORCPT ); Tue, 20 Jan 2009 22:51:09 -0500 Received: from azsmga001.ch.intel.com ([10.2.17.19]) by azsmga102.ch.intel.com with ESMTP; 20 Jan 2009 19:51:08 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.37,298,1231142400"; d="scan'208";a="102150978" Received: from unknown (HELO azsmsx601.amr.corp.intel.com) ([10.2.121.193]) by azsmga001.ch.intel.com with ESMTP; 20 Jan 2009 19:51:08 -0800 Received: from pdsmsx601.ccr.corp.intel.com (172.16.12.94) by azsmsx601.amr.corp.intel.com (10.2.121.193) with Microsoft SMTP Server (TLS) id 8.1.311.2; Tue, 20 Jan 2009 20:51:08 -0700 Received: from pdsmsx503.ccr.corp.intel.com ([172.16.12.95]) by pdsmsx601.ccr.corp.intel.com ([172.16.12.94]) with mapi; Wed, 21 Jan 2009 11:51:06 +0800 From: "Zhang, Xiantao" To: "Zhang, Xiantao" , Anthony Liguori CC: "qemu-devel@nongnu.org" , "kvm@vger.kernel.org" , Avi Kivity Date: Wed, 21 Jan 2009 11:51:05 +0800 Subject: RE: [PATCH] Always return latest pmsts instead of the old one. Thread-Topic: [PATCH] Always return latest pmsts instead of the old one. Thread-Index: Acl3XBBbW7k4FvQDTAeQ1zuLiLteCgC48K2gAE7hprA= Message-ID: <706158FABBBA044BAD4FE898A02E4BC2223BCEA8@pdsmsx503.ccr.corp.intel.com> References: <1231830545-1788-1-git-send-email-amit.shah@redhat.com> <706158FABBBA044BAD4FE898A02E4BC21C9A0745@pdsmsx503.ccr.corp.intel.com> <496FB0E1.3050306@us.ibm.com> <706158FABBBA044BAD4FE898A02E4BC21F768F0D@pdsmsx503.ccr.corp.intel.com> In-Reply-To: <706158FABBBA044BAD4FE898A02E4BC21F768F0D@pdsmsx503.ccr.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US MIME-Version: 1.0 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org Any comments ? If no, please help to check-in this patch. It blocks our work for a long time. Thanks! Xiantao -----Original Message----- From: kvm-owner@vger.kernel.org [mailto:kvm-owner@vger.kernel.org] On Behalf Of Zhang, Xiantao Sent: Monday, January 19, 2009 10:13 PM To: Anthony Liguori Cc: qemu-devel@nongnu.org; kvm@vger.kernel.org; Avi Kivity Subject: RE: [PATCH] Always return latest pmsts instead of the old one. Attached the new patch, please check. Xiantao Subject: Always return latest pmsts instead of the old one. It may lead to the issue when booting windows guests with acpi=1 if return the old pmsts. Signed-off-by: Xiantao Zhang Modified Paths: -------------- trunk/hw/acpi.c Modified: trunk/hw/acpi.c Anthony Liguori wrote: > Zhang, Xiantao wrote: >> For get_pmsts of acpi, it should always return latest value instead >> of the old one. Xiantao >> >> >> From: Xiantao Zhang >> Date: Tue, 13 Jan 2009 17:42:16 +0800 >> Subject: [PATCH] Always return latest pmsts instead of the old one. >> >> It may lead to the issue when booting windows guests with acpi=1 >> if return the old pmsts. >> >> Signed-off-by: Xiantao Zhang --- >> qemu/hw/acpi.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/qemu/hw/acpi.c b/qemu/hw/acpi.c >> > > Please resubmit against QEMU, not kvm-userspace. > > Regards, > > Anthony Liguori > >> index 0ff8851..8bde989 100644 >> --- a/qemu/hw/acpi.c >> +++ b/qemu/hw/acpi.c >> @@ -94,7 +94,7 @@ static int get_pmsts(PIIX4PMState *s) >> d = muldiv64(qemu_get_clock(vm_clock), PM_FREQ, ticks_per_sec); >> if (d >= s->tmr_overflow_time) >> s->pmsts |= TMROF_EN; >> - return pmsts; >> + return s->pmsts; >> } >> >> static void pm_update_sci(PIIX4PMState *s) --- 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 =================================================================== --- trunk/hw/acpi.c +++ trunk/hw/acpi.c @@ -92,7 +92,7 @@ static int get_pmsts(PIIX4PMState *s) d = muldiv64(qemu_get_clock(vm_clock), PM_FREQ, ticks_per_sec); if (d >= s->tmr_overflow_time) s->pmsts |= TMROF_EN; - return pmsts; + return s->pmsts; } static void pm_update_sci(PIIX4PMState *s)