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: RE: RE: RE: RE: RE: [Xen-devel] No C-States any longer...

To: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Subject: RE: RE: RE: RE: RE: RE: [Xen-devel] No C-States any longer...
From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date: Wed, 15 Jun 2011 09:34:40 +0100
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Carsten Schiers <carsten@xxxxxxxxxx>, "Yu, Ke" <ke.yu@xxxxxxxxx>
Delivery-date: Wed, 15 Jun 2011 01:35:30 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <625BA99ED14B2D499DC4E29D8138F1505D210E0C1B@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Citrix Systems, Inc.
References: <H0000067000b16a1.1307971313.uhura.space.zz@MHS> <H0000067000b17e4.1307996827.uhura.space.zz@MHS> <625BA99ED14B2D499DC4E29D8138F1505D210E0C1B@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Wed, 2011-06-15 at 09:11 +0100, Tian, Kevin wrote:
> > From: Carsten Schiers [mailto:carsten@xxxxxxxxxx]
> > Sent: Tuesday, June 14, 2011 4:27 AM
> > 
> > One step further: the problem is that pr->pblk is not set, thus
> > acpi_processor_get_power_info_fadt fails.
> > Knowing this, I found an error in the ACPI_DEBUG output that
> > corresponds:
> > 
> > [   17.062739] processor_xen-0222 [00] xen_acpi_processor_get: Processor
> > [-1:0]
> > [   17.062902] processor_xen-0225 [00] xen_acpi_processor_get: No PBLK
> > (NULL address)
> 
> this looks a bit strange. how about the native log?
> 
> > 
> > It does this for all processors. pr_id is always -1, pr->acpi_id
> > counting up from 0 to 2.
> 
> what's your dom0 vcpu number? and how about physical cpu?

Wasn't there some oddity in the Xen ACPI PM support due to the
disconnect between VCPU and PCPU? I thought it resulted in some CPU id
or other being reported back as -1, in much this manner.

There's some tweaking of this stuff in e.g.
xen_acpi_processor_get_power_info(). But equally
xen_acpi_processor_get_info() has a bunch of cases for the != -1 case.

Does the dom0_vcpus_pin hypervisor option workaround this sort of thing?

This changeset refers to the -1 too:

commit 68320323a51c2378aca433c76157d9e66104ff1e
Author: Jiang, Yunhong <yunhong.jiang@xxxxxxxxx>
Date:   Tue Sep 14 14:41:52 2010 -0700

    xen/acpi: Add cpu hotplug support
    
    Add physical CPU hotplug support to origin/xen/next-2.6.32 branch.
    Please notice that, even with this change, the acpi_processor->id is
    still always -1. This is because several workaround in PM side depends
    on acpi_processor->id == -1. As the CPU hotplug logic does not depends
    on acpi_processor->id, I'd still keep it no changes.
    
    But we need change the acpi_processor->id in the future.
    
    Signed-off-by: Jiang, Yunhong <yunhong.jiang@xxxxxxxxx>
    Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx>

Ian.

> > Any help is welcome, but I will analyze further...
> 
> Current Dom0 depends on several Xen specific functions like 
> xen_acpi_get_power_info
> you mentioned earlier, which is a copy from native acpi_get_power_info with 
> xen
> specific tweaks added. there's possibility that in your environment general 
> ACPI code
> is changed which is not reflected in Xen specific versions. 
> 
> Thanks
> Kevin
> 
> > 
> > Carsten.
> > 
> > -----Ursprüngliche Nachricht-----
> > Von: Carsten Schiers
> > Gesendet: Montag, 13. Juni 2011 15:22
> > An: ke.yu; kevin.tian
> > Cc: xen-devel
> > Betreff: AW: RE: RE: RE: RE: RE: [Xen-devel] No C-States any longer...
> > 
> > >>I will try to boot native Linux in order to verify 100%
> > >> that the tables
> > >> are there.
> > >
> > >yes, that's interesting data to compare.
> > 
> > I have booted with a Live Linux and dumped acpi tables. Those are 100%
> > identical with those
> > I received from Dom0. I will now start looking into
> > acpi_processor_get_power_info_fadt And
> > check, why it is returning -ENODEV.
> > 
> > Carsten.
> > 
> > 
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@xxxxxxxxxxxxxxxxxxx
> > http://lists.xensource.com/xen-devel
> > 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



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