linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dave Jones <davej@redhat.com>
To: Christian <christiand59@web.de>
Cc: Alexey Starikovskiy <alexey_y_starikovskiy@linux.intel.com>,
	Adrian Bunk <bunk@stusta.de>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-acpi@vger.kernel.org
Subject: Re: [discuss] Linux 2.6.19-rc4: known unfixed regressions (v2)
Date: Sun, 5 Nov 2006 15:04:48 -0500	[thread overview]
Message-ID: <20061105200448.GE859@redhat.com> (raw)
In-Reply-To: <200611051832.13285.christiand59@web.de>

On Sun, Nov 05, 2006 at 06:32:12PM +0100, Christian wrote:
 > Am Freitag, 3. November 2006 16:56 schrieb Dave Jones:
 > > On Fri, Nov 03, 2006 at 11:25:37AM +0300, Alexey Starikovskiy wrote:
 > >  > Could this be a problem?
 > >  > --------------------
 > >  > ...
 > >  > CONFIG_ACPI_PROCESSOR=m
 > >  > ...
 > >  > CONFIG_X86_POWERNOW_K8=y
 > >
 > > Hmm, possibly.  Christian, does it work again if you set them both to =y ?
 > 
 > Yes, it works now! Only the change to CONFIG_ACPI_PROCESSOR=y made it work 
 > again!

So, the reasoning behind this, is that we have this construct..

config X86_POWERNOW_K8_ACPI
    bool
    depends on X86_POWERNOW_K8 && ACPI_PROCESSOR
    depends on !(X86_POWERNOW_K8 = y && ACPI_PROCESSOR = m)
    default y


Which makes us use the ACPI stuff if it's there, otherwise not,
and in your case, it seems your system _needs_ this enabled
to make powernow work.

Thing is, this was there in 2.6.18 too, so strictly speaking,
we haven't regressed here, and you're getting exactly what you asked for.
The problem is that it's completely silent as to why it then fails.

I'm open to improvements, but I'm not sure what the right thing to do
here is.. opinions ?

	Dave

-- 
http://www.codemonkey.org.uk

  reply	other threads:[~2006-11-05 20:05 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Pine.LNX.4.64.0610302019560.25218@g5.osdl.org>
2006-10-31 19:56 ` 2.6.19-rc4: known unfixed regressions Adrian Bunk
2006-10-31 20:11   ` Greg KH
2006-10-31 20:12   ` Arjan van de Ven
2006-11-03  2:41 ` Linux 2.6.19-rc4: known unfixed regressions (v2) Adrian Bunk
2006-11-03  2:56   ` [discuss] " Dave Jones
2006-11-03  8:25     ` Alexey Starikovskiy
2006-11-03 15:56       ` Dave Jones
2006-11-05 17:32         ` Christian
2006-11-05 20:04           ` Dave Jones [this message]
2006-11-06 17:35             ` Adrian Bunk
2006-11-06 17:49               ` Dave Jones
2006-11-06  6:00           ` Adrian Bunk
2006-11-06 15:43             ` Christian
2006-11-06 17:20               ` Dave Jones
2006-11-06 17:30                 ` Adrian Bunk
2006-11-06 17:37               ` Adrian Bunk
2006-11-05  6:48 ` 2.6.19-rc4: known unfixed regressions (v3) Adrian Bunk
2006-11-05  9:30   ` Jeff Chua
2006-11-06 17:47     ` [discuss] " Andi Kleen
2006-11-05 13:26   ` Michael S. Tsirkin
2006-11-05 13:57     ` Adrian Bunk
2006-11-07  2:17   ` Ernst Herzberg
2006-11-07  5:41     ` Len Brown
2006-11-07 20:05       ` Ernst Herzberg
2006-11-08  8:36         ` [linux-pm] " Adrian Bunk
2006-11-07 13:30 ` 2.6.19-rc4: known unfixed regressions (v4) Adrian Bunk

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=20061105200448.GE859@redhat.com \
    --to=davej@redhat.com \
    --cc=alexey_y_starikovskiy@linux.intel.com \
    --cc=bunk@stusta.de \
    --cc=christiand59@web.de \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.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).