cpufreq Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Dominik Brodowski <linux@dominikbrodowski.de>
To: Marcus Grando <marcus@corp.grupos.com.br>
Cc: Ducrot Bruno <ducrot@poupinou.org>, cpufreq@www.linux.org.uk
Subject: Re: p4_clockmod or speedstep_ich | acpi?
Date: Wed, 28 Jan 2004 23:41:06 +0100	[thread overview]
Message-ID: <20040128224106.GA5987@dominikbrodowski.de> (raw)
In-Reply-To: <4015CFE8.1030002@corp.grupos.com.br>


[-- Attachment #1.1.1: Type: text/plain, Size: 1556 bytes --]

On Tue, Jan 27, 2004 at 12:41:44AM -0200, Marcus Grando wrote:
> Hi,
> 
> But why my kernel show this messages, if celeron not support speedstep-ich?
> 
> cpufreq: P4/Xeon(TM) CPU On-Demand Clock Modulation available
> cpufreq: Warning: Pentium 4-M detected. The speedstep-ich or acpi cpufreq
> cpufreq: modules offers voltage scaling in addition of frequency 
> scaling. You
> cpufreq: should use either one instead of p4-clockmod, if possible.
> 
> Regards
> 
> Ducrot Bruno wrote:
> >On Sat, Jan 24, 2004 at 01:53:57AM -0200, Marcus Grando wrote:
> >
> >>Hi,
> >>
> >>I have a toshiba satellite a10 127, and use p4 clockmod in cpufreq. I 
> >>look this kernel messages:
> >>
> >>cpufreq: P4/Xeon(TM) CPU On-Demand Clock Modulation available
> >>cpufreq: Warning: Pentium 4-M detected. The speedstep-ich or acpi cpufreq
> >>cpufreq: modules offers voltage scaling in addition of frequency 
> >>scaling. You
> >>cpufreq: should use either one instead of p4-clockmod, if possible.
> >>
> >
> >
> >...
> >
> >>CPU:     After generic identify, caps: bfebf9ff 00000000 00000000 00000000
> >>CPU:     After vendor identify, caps: bfebf9ff 00000000 00000000 00000000
> >>CPU: Trace cache: 12K uops, L1 D cache: 8K
> >>CPU: L2 cache: 256K
> >>CPU:     After all inits, caps: bfebf9ff 00000000 00000000 00000080
> >>CPU: Intel Mobile Intel(R) Celeron(R) CPU 2.00GHz stepping 09
> >
> >                             ^^^^^^^
> >		  A celeron is not speedstep capable.
> >

Could you try out this patch, please?

	Dominik

[-- Attachment #1.1.2: cpufreq-2.6.1-speedstep_lib_fix_p4m_detection --]
[-- Type: text/plain, Size: 2168 bytes --]

diff -ruN linux-original/arch/i386/kernel/cpu/cpufreq/speedstep-lib.c linux/arch/i386/kernel/cpu/cpufreq/speedstep-lib.c
--- linux-original/arch/i386/kernel/cpu/cpufreq/speedstep-lib.c	2004-01-16 20:30:36.000000000 +0100
+++ linux/arch/i386/kernel/cpu/cpufreq/speedstep-lib.c	2004-01-28 23:32:49.637733272 +0100
@@ -184,17 +184,55 @@
 		if (c->x86_model != 2)
 			return 0;
 
-		if ((c->x86_mask != 4) && /* B-stepping [M-P4-M] */
-			(c->x86_mask != 7) && /* C-stepping [M-P4-M] */
-			(c->x86_mask != 9))   /* D-stepping [M-P4-M or M-P4/533] */
-			return 0;
-
 		ebx = cpuid_ebx(0x00000001);
 		ebx &= 0x000000FF;
-		if ((ebx != 0x0e) && (ebx != 0x0f))
-			return 0;
 
-		return SPEEDSTEP_PROCESSOR_P4M;
+		dprintk(KERN_INFO "ebx value is %x, x86_mask is %x\n", ebx, c->86_mask);
+
+		switch (c->x86_mask) {
+		case 4: 
+			/*
+			 * B-stepping [M-P4-M] 
+			 * sample has ebx = 0x0f, production has 0x0e.
+			 */
+			if ((ebx == 0x0e) || (ebx == 0x0f))
+				return SPEEDSTEP_PROCESSOR_P4M;
+			break;
+		case 7: 
+			/*
+			 * C-stepping [M-P4-M]
+			 * needs to have ebx=0x0e, else it's a celeron:
+			 * cf. 25130917.pdf / page 7, footnote 5 even
+			 * though 25072120.pdf / page 7 doesn't say
+			 * samples are only of B-stepping...
+			 */
+			if (ebx == 0x0e)
+				return SPEEDSTEP_PROCESSOR_P4M;
+			break;
+		case 9:
+			/*
+			 * D-stepping [M-P4-M or M-P4/533]
+			 *
+			 * this is totally strange: CPUID 0x0F29 is
+			 * used by M-P4-M, M-P4/533 and(!) Celeron CPUs.
+			 * The latter need to be sorted out as they don't
+			 * support speedstep.
+			 * Celerons with CPUID 0x0F29 may have either
+			 * ebx=0x8 or 0xf -- 25130917.pdf doesn't say anything
+			 * specific.
+			 * M-P4-Ms may have either ebx=0xe or 0xf [see above]
+			 * M-P4/533 have either ebx=0xe or 0xf. [25317607.pdf]
+			 * So, how to distinguish all those processors with
+			 * ebx=0xf? I don't know. Sort them out, and wait
+			 * whether someone complains.
+			 */
+			if (ebx == 0x0e)
+				return SPEEDSTEP_PROCESSOR_P4M;
+			break;
+		default:
+			break;
+		}
+		return 0;
 	}
 
 	switch (c->x86_model) {

[-- Attachment #1.2: Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 143 bytes --]

_______________________________________________
Cpufreq mailing list
Cpufreq@www.linux.org.uk
http://www.linux.org.uk/mailman/listinfo/cpufreq

  parent reply	other threads:[~2004-01-28 22:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-01-24  3:53 p4_clockmod or speedstep_ich | acpi? Marcus Grando
2004-01-24  6:03 ` Ducrot Bruno
2004-01-27  2:41   ` Marcus Grando
2004-01-27  9:39     ` Ducrot Bruno
2004-01-28 16:53       ` Dominik Brodowski
2004-01-28 22:41     ` Dominik Brodowski [this message]
2004-01-28 23:06       ` Marcus Grando

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=20040128224106.GA5987@dominikbrodowski.de \
    --to=linux@dominikbrodowski.de \
    --cc=cpufreq@www.linux.org.uk \
    --cc=ducrot@poupinou.org \
    --cc=marcus@corp.grupos.com.br \
    /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