Xen devel - Feb 2011 - [PATCH] hvmloader: add "wakeup from S4 on RTC alarm" bit to ACPI FADT

# HG changeset patch
# User Paolo Bonzini <pbonzini@redhat.com>
# Date 1296827299 -3600
# Node ID c923816861a96328aaf3d5ba8939fd9b4c441cd0
# Parent  4cd3bd56fc0343031085b7e466e3022816b1b5dc
hvmloader: add "wakeup from S4 on RTC alarm" bit to ACPI FADT

Testing Xen with Microsoft''s Windows logo tests reveals a non-fatal
failure in the "CHAOS" test (more precisely in the PwrTest subjob).

The test expects to be able to hibernate the machine and have it
power up again via the RTC.  We do not really care about implementing
the exact timing in the management (and neither does MS) so their
request can be easily by manually restarting the VM or even via
on_poweroff="restart" (granted, that''s not something
you''d usually do).

Ok?

Paolo

diff --git a/tools/firmware/hvmloader/acpi/static_tables.c
b/tools/firmware/hvmloader/acpi/static_tables.c
--- a/tools/firmware/hvmloader/acpi/static_tables.c
+++ b/tools/firmware/hvmloader/acpi/static_tables.c
@@ -71,7 +71,8 @@ struct acpi_20_fadt Fadt = {
     .flags = (ACPI_PROC_C1 |
               ACPI_WBINVD |
               ACPI_FIX_RTC | ACPI_TMR_VAL_EXT |
-              ACPI_USE_PLATFORM_CLOCK),
+              ACPI_USE_PLATFORM_CLOCK |
+              ACPI_RTC_S4),
 
     .reset_reg = {
         .address_space_id    = ACPI_SYSTEM_IO,



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On 04/02/2011 14:52, "Paolo Bonzini" <pbonzini@redhat.com>
wrote:
> hvmloader: add "wakeup from S4 on RTC alarm" bit to ACPI FADT
> 
> Testing Xen with Microsoft''s Windows logo tests reveals a
non-fatal
> failure in the "CHAOS" test (more precisely in the PwrTest
subjob).
> 
> The test expects to be able to hibernate the machine and have it
> power up again via the RTC.  We do not really care about implementing
> the exact timing in the management (and neither does MS) so their
> request can be easily by manually restarting the VM or even via
> on_poweroff="restart" (granted, that''s not something
you''d usually do).
> 
> Ok?
Does a non-fatal error actually matter to passing a Windows logo test? After
all it seems more correct to not advertise a feature we don''t actually
support, regardless of what Microsoft would prefer.

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

I believe the test *requires* S4 support. Some others require S3 support, with
similar RTC wake.

  Paul
> -----Original Message-----
> From: xen-devel-bounces@lists.xensource.com [mailto:xen-devel-
> bounces@lists.xensource.com] On Behalf Of Keir Fraser
> Sent: 04 February 2011 15:05
> To: Paolo Bonzini; xen-devel@lists.xensource.com
> Subject: Re: [Xen-devel] [PATCH] hvmloader: add "wakeup from S4 on
> RTC alarm"bit to ACPI FADT
> 
> On 04/02/2011 14:52, "Paolo Bonzini" <pbonzini@redhat.com>
wrote:
> 
> > hvmloader: add "wakeup from S4 on RTC alarm" bit to ACPI
FADT
> >
> > Testing Xen with Microsoft''s Windows logo tests reveals a
non-
> fatal
> > failure in the "CHAOS" test (more precisely in the PwrTest
> subjob).
> >
> > The test expects to be able to hibernate the machine and have it
> power
> > up again via the RTC.  We do not really care about implementing
> the
> > exact timing in the management (and neither does MS) so their
> request
> > can be easily by manually restarting the VM or even via
> > on_poweroff="restart" (granted, that''s not
something you''d usually
> do).
> >
> > Ok?
> 
> Does a non-fatal error actually matter to passing a Windows logo
> test? After all it seems more correct to not advertise a feature we
> don''t actually support, regardless of what Microsoft would prefer.
> 
>  -- Keir
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On 02/04/2011 04:04 PM, Keir Fraser wrote:
> Does a non-fatal error actually matter to passing a Windows logo test?
No, it doesn''t, though I expect the error to become fatal in future 
releases of the tests.

Paolo

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On 02/04/2011 04:22 PM, Paul Durrant wrote:
> I believe the test *requires* S4 support. Some others require S3 support,
with similar RTC wake.
It requires S4 support, but that''s not a problem.  S4 is just hibernate
and works nicely.

For the test, RTC wake is not needed (you can simply restart the machine 
yourself), but the test fails to run if it is not available.

Paolo

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Yes, you can restart the VM yourself if you want to wait around and watch the
test! Yes, ACPI does have to be set up correctly for the test to run at all
though.

  Paul
> -----Original Message-----
> From: Paolo Bonzini [mailto:pbonzini@redhat.com]
> Sent: 04 February 2011 15:57
> To: Paul Durrant
> Cc: Keir Fraser; xen-devel@lists.xensource.com
> Subject: Re: [Xen-devel] [PATCH] hvmloader: add "wakeup from S4 on
> RTC alarm"bit to ACPI FADT
> 
> On 02/04/2011 04:22 PM, Paul Durrant wrote:
> > I believe the test *requires* S4 support. Some others require S3
> support, with similar RTC wake.
> 
> It requires S4 support, but that''s not a problem.  S4 is just
> hibernate and works nicely.
> 
> For the test, RTC wake is not needed (you can simply restart the
> machine yourself), but the test fails to run if it is not available.
> 
> Paolo
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On 02/04/2011 05:00 PM, Paul Durrant wrote:
> Yes, you can restart the VM yourself if you want to wait around and watch
the test!
That''s why I mentioned the slighly unorthodox alternative of 
on_poweroff="restart", which is as close as Xen management gets to RTC
wake...

Paolo

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Please don''t top-post. 

At 16:00 +0000 on 04 Feb (1296835213), Paul Durrant
wrote:
> Yes, you can restart the VM yourself if you want to wait around and
> watch the test! Yes, ACPI does have to be set up correctly for the
> test to run at all though.
XenServer has code in the qemu acpi path to actually perform timed wake
from S4 by pulling the RTC state out of the hypervisor.  I thought this
had gone upstream to qemu-xen already; now that we''re transitioning to
upstream qemu I''m not sure where it should go.  It will certainly need
tidying for upstream qemu as it''s very xen-specific.

Cheers,

Tim.
>   Paul
> 
> > -----Original Message-----
> > From: Paolo Bonzini [mailto:pbonzini@redhat.com]
> > Sent: 04 February 2011 15:57
> > To: Paul Durrant
> > Cc: Keir Fraser; xen-devel@lists.xensource.com
> > Subject: Re: [Xen-devel] [PATCH] hvmloader: add "wakeup from S4
on
> > RTC alarm"bit to ACPI FADT
> > 
> > On 02/04/2011 04:22 PM, Paul Durrant wrote:
> > > I believe the test *requires* S4 support. Some others require S3
> > support, with similar RTC wake.
> > 
> > It requires S4 support, but that''s not a problem.  S4 is just
> > hibernate and works nicely.
> > 
> > For the test, RTC wake is not needed (you can simply restart the
> > machine yourself), but the test fails to run if it is not available.
> > 
> > Paolo
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
-- 
Tim Deegan <Tim.Deegan@citrix.com>
Principal Software Engineer, Xen Platform Team
Citrix Systems UK Ltd.  (Company #02937203, SL9 0BG)

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On 02/04/2011 05:17 PM, Tim Deegan wrote:
> XenServer has code in the qemu acpi path to actually perform timed wake
> from S4 by pulling the RTC state out of the hypervisor.  I thought this
> had gone upstream to qemu-xen already; now that we''re
transitioning to
> upstream qemu I''m not sure where it should go.  It will certainly
need
> tidying for upstream qemu as it''s very xen-specific.
There''s nothing like that indeed in qemu-xen-unstable.git.

Paolo

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

At 16:22 +0000 on 04 Feb (1296836548), Paolo Bonzini
wrote:
> On 02/04/2011 05:17 PM, Tim Deegan wrote:
> > XenServer has code in the qemu acpi path to actually perform timed
wake
> > from S4 by pulling the RTC state out of the hypervisor.  I thought
this
> > had gone upstream to qemu-xen already; now that we''re
transitioning to
> > upstream qemu I''m not sure where it should go.  It will
certainly need
> > tidying for upstream qemu as it''s very xen-specific.
> 
> There''s nothing like that indeed in qemu-xen-unstable.git.
Grrr. :(  I''ve attached a patch extracted from XenServer source trees,
against qemu-xen of Xen 3.4 vintage.  This is just as RFC, since I''m
sure it will need work even to apply to current qemu-xen.

Ian, what''s the best thing to do with this?  Is it worth rebasing it to
qemu-xen tip or should it wait for the switch to upstream? 

Tim

-- 
Tim Deegan <Tim.Deegan@citrix.com>
Principal Software Engineer, Xen Platform Team
Citrix Systems UK Ltd.  (Company #02937203, SL9 0BG)


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Tim Deegan writes ("Re: [Xen-devel] [PATCH] hvmloader: add "wakeup
from S4 on RTC alarm"bit to ACPI FADT"):
> At 16:22 +0000 on 04 Feb (1296836548), Paolo Bonzini wrote:
> > There''s nothing like that indeed in qemu-xen-unstable.git.
> 
> Grrr. :(  I''ve attached a patch extracted from XenServer source
trees,
> against qemu-xen of Xen 3.4 vintage.  This is just as RFC, since
I''m
> sure it will need work even to apply to current qemu-xen.
Hrm.
> Ian, what''s the best thing to do with this?  Is it worth rebasing
it to
> qemu-xen tip or should it wait for the switch to upstream? 
Well, it''s definitely not going into 4.1 and in 4.2 we''re
hoping to
switch to upstream qemu.  So I wouldn''t bother trying to rebase it to
current qemu-xen tip.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Mon, 7 Feb 2011, Tim Deegan wrote:
> At 16:22 +0000 on 04 Feb (1296836548), Paolo Bonzini wrote:
> > On 02/04/2011 05:17 PM, Tim Deegan wrote:
> > > XenServer has code in the qemu acpi path to actually perform
timed wake
> > > from S4 by pulling the RTC state out of the hypervisor.  I
thought this
> > > had gone upstream to qemu-xen already; now that we''re
transitioning to
> > > upstream qemu I''m not sure where it should go.  It will
certainly need
> > > tidying for upstream qemu as it''s very xen-specific.
> > 
> > There''s nothing like that indeed in qemu-xen-unstable.git.
> 
> Grrr. :(  I''ve attached a patch extracted from XenServer source
trees,
> against qemu-xen of Xen 3.4 vintage.  This is just as RFC, since
I''m
> sure it will need work even to apply to current qemu-xen.
> 
> Ian, what''s the best thing to do with this?  Is it worth rebasing
it to
> qemu-xen tip or should it wait for the switch to upstream? 
> 
I personally think it could be rebased now because it is self contained.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel