WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel] [PATCH 2/10] PM interface between dom0 and Xen

To: "Keir Fraser" <keir@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 2/10] PM interface between dom0 and Xen
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Thu, 12 Jul 2007 10:01:12 +0800
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 11 Jul 2007 18:59:03 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C2BAAE96.12447%keir@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Ace4v14xFx0B/3G7T4i9ErPfdBh/EgLDIm7EABcAqkA=
Thread-topic: [Xen-devel] [PATCH 2/10] PM interface between dom0 and Xen
>From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx]
>Sent: 2007年7月11日 22:58
>
>Patches 1 and 2 are now in with only very slight changes.
>
>However, I have not yet enabled the new platform hypercalls. I believe
>they
>can be slimmed down significantly.
>
> 1. We don't need set_acpi_sleep() at all. The values it provides are all
>quite easily found in the fixed-format FADT and FACS structures. We
>don't
>have the code for finding and parsing those tables in Xen currently, and
>we
>don't want to pull in the whole Linux generic mechanism! *But* I expect it
>would not be hard to put together a small routine to discover
>waking_vector,
>waking_vector_length, pm1{a,b}_{evt,vnt}_port, based on the Linux code
>(which we want to crib from since it does all the right things for different
>ACPI versions) but written from scratch to be tailored for extracting just
>those values that we require for S-state support.

Sounds fair. Actually our first version in house was in this direction, but 
gave up later due to lazy copy based on file level which brought too much 
unused stuff into xen.

>
> 2. Enter_acpi_sleep() does not require the magic video_{flags,mode}
>fields.
>If this is just to support "acpi_sleep=" cmdline option, we can support that
>option directly in Xen and cut dom0 out entirely.

Sure.

>
>I'll carry on looking through the remainder of the patches, since I can
>apply those in parallel with any changes being made to this patch 2/10.
>

Thanks,
Kevin

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

<Prev in Thread] Current Thread [Next in Thread>