On Thursday 05 February 2009 13:02:03 Ingo Molnar wrote:
>
> * Thomas Renninger <trenn@xxxxxxx> wrote:
>
> > Signed-off-by: Thomas Renninger <trenn@xxxxxxx>
> > ---
> > arch/x86/kernel/cpu/cpufreq/powernow-k8.c | 12 ++++++------
> > 1 files changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> > index 83515f1..5aa832f 100644
> > --- a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> > +++ b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
> > @@ -1247,12 +1247,12 @@ static int __cpuinit powernowk8_cpu_init(struct
cpufreq_policy *pol)
> > * thing gets introduced
> > */
> > if (!print_once) {
> > - WARN_ONCE(1, KERN_ERR FW_BUG PFX "Your BIOS "
> > - "does not provide ACPI _PSS objects "
> > - "in a way that Linux understands. "
> > - "Please report this to the Linux ACPI"
> > - " maintainers and complain to your "
> > - "BIOS vendor.\n");
> > + printk(KERN_ERR FW_BUG PFX "Your BIOS "
> > + "does not provide ACPI _PSS objects "
> > + "in a way that Linux understands. "
> > + "Please report this to the Linux ACPI"
> > + " maintainers and complain to your "
> > + "BIOS vendor.\n");
> > print_once++;
>
> hm, why the open-coded WARN_ONCE? (which print_once flag + the printk in
> essence is)
>
> So please use WARN_ONCE(), and indent it all one tab to the left which will
> solve at least part of that ugly 6-line split up thing. And if it's a
> WARN_ONCE() then kerneloops.org will pick it up too.
No.
This happens if your BIOS is older than your CPU and you then miss cpufreq.
This often happens on very new machines/CPUs. It could also happen that you
have to wait a month or so until your vendor offers a new BIOS.
We want to tell the user that it's not the kernel's fault, but we better do
not spit out a huge backtrace, which is worthless anyway as it's the BIOS
which is broken.
Thomas
--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
