Messages in this thread | | | Date | Tue, 7 Sep 2010 17:15:41 +0200 | From | Robert Richter <> | Subject | Re: [PATCH] perf, x86: Disable perf if the BIOS got its grubby paws on the PMU |
| |
> > > On 9/3/2010 2:13 AM, Peter Zijlstra wrote: > > > >+static void print_BIOS_fail(void) > > > >+{ > > > >+ printk(KERN_ERR "\n"); > > > >+ printk(KERN_ERR "=============================================\n"); > > > >+ printk(KERN_ERR "It appears the BIOS is actively using the PMU\n"); > > > >+ printk(KERN_ERR "this avoids Linux from using it, please de- \n"); > > > >+ printk(KERN_ERR "activate this BIOS feature or request a BIOS \n"); > > > >+ printk(KERN_ERR "update from your vendor. \n"); > > > >+ printk(KERN_ERR "=============================================\n");
I would rather prefer this:
BIOS bug, cpu 1, invalid <register=value> ...
... which is a much better information on one line, explains the bug and is also better parsable. I intend to implement messages like this. So maybe we could find consensus with this or something similar.
A simple grep of dmesg will then give a list of BIOS bugs.
-Robert
-- Advanced Micro Devices, Inc. Operating System Research Center
| |