public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Erik Slagter <erik@slagter.name>
To: Juhani Rautiainen <juhani.rautiainen@gmail.com>
Cc: "Brown, Len" <len.brown@intel.com>, Joerg Sommrey <jo@sommrey.de>,
	Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
	tony@atomide.com, alan@lxorguk.ukuu.org.uk
Subject: Re: [PATCH] amd76x_pm: C3 powersaving for AMD K7
Date: Fri, 03 Feb 2006 18:02:15 +0100	[thread overview]
Message-ID: <1138986135.18273.56.camel@localhost.localdomain> (raw)
In-Reply-To: <fad2c7740602030759i65e45a6as29964b8e90aeecd7@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2082 bytes --]

On Fri, 2006-02-03 at 17:59 +0200, Juhani Rautiainen wrote:

> This comes from AMD-768 revision guide. In product errata there there
> is errata number
> 24 which seems to suggest that you can't enable POS, C2 or C3 states
> in single processor
> environments. This at the end of errata:
> ----- snip ----
> This workaround will not work for awaking from the C2/C3 state, since
> the operating
> system has full control. The (ACPI C2/C3) state support is not
> required for Microsoft-compatible workstation and server platforms.
> 
> It is recommended that BIOS should disable the C2 state by
> clearing bit C2EN (device B function 3 offset 4F'h). It is recommended
> that BIOS should disable the C3 state by clearing bit C3EN (device B
> function 3 offset 4F'h).
> ---- snip ----
> 
> If I understand errata correctly then enabling C2/C3 should be safe in
> SMP environment put
> not in single processor enviroment. Maybe module should check this
> before enabling C3.

This is a very cryptic text! What would they mean actually?

That you cannot use POS/C* states in a single processor system OR in SMP
system that only has one processor fitted? Also this bug seems very odd
to me, what would the number of processors have to do with the
availability of C states?

Why "is the C2/C3 state support not required for Microsoft-compatible
platforms"? Do they mean it is okay (in their vision) to have these
running at full power all of the time? Or do they mean there is some
sort of driver that replaces the functionality?

My BIOS (Tyan S2466) does something completely different. It offers an
option "ACPI power saving" (or something like that) in the configuration
screen. This seems to toggle the above mentioned registers. But at the
same time it always marks C2 and C3 unusable by offering impossible
timings. This looks to me that the intention was to have C2/C3 enabled,
but only with this "special" windows driver that does some "extra"
things.

Grmbl, it's all sick.

Does *anyone* have some sort of contact with AMD or Tyan?

[-- Attachment #2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 2771 bytes --]

  reply	other threads:[~2006-02-03 17:02 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-03  8:45 [PATCH] amd76x_pm: C3 powersaving for AMD K7 Brown, Len
2006-02-03  9:35 ` Erik Slagter
2006-02-03 10:31   ` Pavel Troller
2006-02-03 14:16 ` Joerg Sommrey
2006-02-03 15:59 ` Juhani Rautiainen
2006-02-03 17:02   ` Erik Slagter [this message]
2006-02-08 20:10   ` Pavel Machek
  -- strict thread matches above, loose matches on Subject: below --
2006-02-03 18:40 Brown, Len
2006-02-04 10:31 ` Erik Slagter
2006-02-04 10:49   ` Pavel Troller
2006-02-03 18:28 Brown, Len
2006-02-03 18:20 Brown, Len
2006-02-02 22:24 Joerg Sommrey
2006-02-02 22:37 ` Andrew Morton
2006-02-03  7:03   ` Joerg Sommrey
2006-02-02 22:43 ` Dave Jones
2006-02-02  1:35 Brown, Len
2006-02-02  1:50 ` Alan Cox
2006-02-02  9:50   ` Erik Slagter
2006-02-02 19:38     ` Tony Lindgren
2006-02-03  9:21 ` Erik Slagter
2006-02-03  9:33   ` Pavel Troller
2006-02-03  9:39     ` Erik Slagter
2006-02-03 10:14       ` Pavel Troller
2006-02-03 10:28         ` Erik Slagter
2006-02-01 18:11 Brown, Len
2006-02-01 18:18 ` Erik Slagter
2006-02-01 18:40   ` Tony Lindgren
2006-02-01 19:13 ` Joerg Sommrey
2006-01-31 18:55 Joerg Sommrey
2006-02-01  3:34 ` Andrew Morton
2006-02-01 10:25   ` Erik Slagter
2006-02-01 17:20   ` Tony Lindgren

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=1138986135.18273.56.camel@localhost.localdomain \
    --to=erik@slagter.name \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=jo@sommrey.de \
    --cc=juhani.rautiainen@gmail.com \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tony@atomide.com \
    /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