xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Tobias Geiger <tobias.geiger@vido.info>
Cc: xen-devel@lists.xen.org
Subject: Re: xen acpi cpufreq driver
Date: Thu, 26 Apr 2012 14:25:17 -0400	[thread overview]
Message-ID: <20120426182517.GA24459@phenom.dumpdata.com> (raw)
In-Reply-To: <4F973726.9010405@vido.info>

On Wed, Apr 25, 2012 at 01:28:38AM +0200, Tobias Geiger wrote:
> Am 24.04.2012 19:36, schrieb Konrad Rzeszutek Wilk:
> >On Tue, Apr 24, 2012 at 07:04:03PM +0200, Tobias Geiger wrote:
> >>Hi,
> >>
> >>i'm not sure if i understood the new acpi xen cpufreq driver - here's the
> >>output when loading  xen_acpi_processor module in linux 3.4:
> >>
> >>dom0 dmesg:
> >>
> >>[   32.728151] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU8
> >>[   32.728156] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU9
> >>[   32.728160] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU10
> >>[   32.728164] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU11
> >>[   32.728168] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU12
> >>[   32.728172] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU13
> >>[   32.728176] xen-acpi-processor: (CX): Hypervisor error (-22) for ACPI CPU14

So your DSDT has:

   External (\_PR_.CPUF, DeviceObj)
    External (\_PR_.CPUE, DeviceObj)
    External (\_PR_.CPUD, DeviceObj)
    External (\_PR_.CPUC, DeviceObj)
    External (\_PR_.CPUB, DeviceObj)
    External (\_PR_.CPUA, DeviceObj)
    External (\_PR_.CPU9, DeviceObj)
    External (\_PR_.CPU8, DeviceObj)
    External (\_PR_.CPU7, DeviceObj)
    External (\_PR_.CPU6, DeviceObj)
    External (\_PR_.CPU5, DeviceObj)
    External (\_PR_.CPU4, DeviceObj)
    External (\_PR_.CPU3, DeviceObj)
    External (\_PR_.CPU2, DeviceObj)
    External (\_PR_.CPU1, DeviceObj)
    External (\_PR_.CPU0, DeviceObj)

And along with some other stuff in the DSDT it advertises that
it has 16 CPUs and it sets up even sixteen _CST and _PST data structures.

But you only have eight. This is really a BIOS bug.

However, let me fix it in the driver so that you don't get that error.
(I had a similar fix in the driver for dealing with the P-states
but didn't do it for the C-states).

      reply	other threads:[~2012-04-26 18:25 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-24 17:04 xen acpi cpufreq driver Tobias Geiger
2012-04-24 17:36 ` Konrad Rzeszutek Wilk
2012-04-24 23:28   ` Tobias Geiger
2012-04-26 18:25     ` Konrad Rzeszutek Wilk [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120426182517.GA24459@phenom.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=tobias.geiger@vido.info \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).