[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
  • Date: Wed, 14 May 2008 14:01:17 +0800
  • Delivery-date: Tue, 13 May 2008 23:02:04 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: Aci0pum5DchKJJryTjCAyBnEoEeG2AAMDsoTACvKrCA=
  • Thread-topic: [Xen-devel][PATCH 3/5] Add cpufreq pwr mgmt to Xen

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.