Re: [PATCH] powernow-k8: On load failure, remind the user to enable support in BIOS setup
From: Thomas Renninger
Date: Wed Jan 20 2010 - 10:57:08 EST
On Wednesday 20 January 2010 16:33:26 Marti Raudsepp wrote:
> arch/x86/kernel/cpu/cpufreq/powernow-k8.c | 15 +++++++++------
> 1 files changed, 9 insertions(+), 6 deletions(-)
> In most cases this is simply user error or bad BIOS defaults, rather
> than a firmware problem.
Bad BIOS defaults is a firmware problem.
But most often this happens if people upgrade their CPU and do not
update their BIOS.
Or the vendor does not recognise the new CPU even if the BIOS got
The itself message might be an enhancment, IMO it's not worth a patch.
But do not revert the FW_BUG part!
People complain to lkml or to the cpufreq list that cpufreq does not
work. The [Firmware Bug] (FW_BUG) already reminds them to do following
- Read up the BIOS setting description
- Update the BIOS
- If it still does not work complain to the vendor
Also this string gets picked up by the linuxfirmwarekit and it could not
detect anymore that the BIOS does not export cpufreq tables which is
very often *not* a wrong BIOS default, in which case the message is
> Signed-off-by: Marti Raudsepp <marti@xxxxxxxxx>
> diff --git a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> --- a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> +++ b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> @@ -789,17 +789,20 @@
> (struct pst_s *)(psb+1), maxvid);
> - * If you see this message, complain to BIOS manufacturer. If
> - * he tells you "we do not support Linux" or some similar
> - * nonsense, remember that Windows 2000 uses the same legacy
> - * mechanism that the old Linux PSB driver uses. Tell them it
> - * is broken with Windows 2000.
> + * If you see this message, check that Cool'N'Quiet is enabled
> + * in your BIOS setup. If that fails, complain to your BIOS
> + * manufacturer. If he tells you "we do not support Linux" or
> + * some similar nonsense, remember that Windows 2000 uses the
> + * same legacy mechanism that the old Linux PSB driver uses.
> + * Tell them it is broken with Windows 2000.
> * The reference to the AMD documentation is chapter 9 in the
> * BIOS and Kernel Developer's Guide, which is available on
> * www.amd.com
> - printk(KERN_ERR FW_BUG PFX "No PSB or ACPI _PSS objects\n");
> + printk(KERN_ERR PFX "No PSB or ACPI _PSS objects\n");
> + printk(KERN_ERR PFX "Make sure that Cool'N'Quiet support is "
> + "enabled in your BIOS setup\n");
> return -ENODEV;
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/