RE: [Xen-devel][PATCH 3/5] Add cpufreq pwr mgmt to Xen

["Liu, Jinsong" <jinsong.liu@xxxxxxxxx>]

Keir,

I've updated my patches accroding to your comments.
[PATCH 2/5] [dom0] pm_dom0_1_px.patch: update public header files.
[PATCH 3/5] [xen] pm_xen_1_hypercall.patch: update public header files,
and common/domain.c

Thanks,
Jinsong

-----Original Message-----
From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx] 
Sent: Tuesday, May 13, 2008 4:56 PM
To: Liu, Jinsong; xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel][PATCH 3/5] Add cpufreq pwr mgmt to Xen

On 13/5/08 04:10, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:

> Get ACPI Px from dom0 and chose Px controller
> 
> Add platform op hypercall case to get ACPI Px info from dom0.
> Chose Px controller from dom0 (cpufreq=dom0-kernel)
> or hypervisor (cpufreq=xen-cpufreq)

No gcc-specific attributes in the public header files. This means no
__attribute__((packed)). Please fix that.

Also why declare a local variable 'a' in your changes to
common/domain.c?

 -- Keir

Attachment: pm_dom0_1_px.patch
Description: pm_dom0_1_px.patch

Attachment: pm_xen_1_hypercall.patch
Description: pm_xen_1_hypercall.patch

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