Re: [PATCH] Revert 9fc2105aeaaf56b0cf75296a84702d0f9e64437b to fix pyaudio (and probably more)
From: Nicolas Pitre
Date: Sun Jan 04 2015 - 15:03:14 EST
On Sun, 4 Jan 2015, Pavel Machek wrote:
> 9fc2105aeaaf56b0cf75296a84702d0f9e64437b breaks audio in python, and
> probably elsewhere, with message
>
> FATAL: cannot locate cpu MHz in /proc/cpuinfo
>
> I'm not the first one to hit it, see for example
>
> https://theredblacktree.wordpress.com/2014/08/10/fatal-cannot-locate-cpu-mhz-in-proccpuinfo/
> https://devtalk.nvidia.com/default/topic/765800/workaround-for-fatal-cannot-locate-cpu-mhz-in-proc-cpuinf/?offset=1
>
> Reading original changelog, I have to say "Stop breaking working
> setups. You know who you are!".
>
> Signed-off-by: Pavel Machek <pavel@xxxxxx>
NAK.
No setups actually relying on this completely fony bogomips value
bearing no links to hardware reality coule have been qualified as
"working".
The bogomips entry was removed from /proc/cpuinfo in 2013. We're now in
2015. You're apparently the first to suggest moving the kernel back to
providing random values via /proc/cpuinfo. So this removal must not
have inconvenienced that many people in the end.
Broken applications appear to have been fixed already as mentioned via
those links you provided above. So if you want a working setup, you may
stick with a kernel of the same vintage as your user space apps or
update the later.
If that is still unacceptable to you for whatever reason, then the least
wrong compromize should be:
seq_printf(m, "BogoMIPS\t: 1.00\n");
That'D allow for those broken applications to run while making clear
that the provided value is phony. I was about to suggest 0.00 but that
could trigger a divide by zero error somewhere I suppose.
> diff --git a/arch/arm/kernel/setup.c b/arch/arm/kernel/setup.c
> index ff6760a..940779c 100644
> --- a/arch/arm/kernel/setup.c
> +++ b/arch/arm/kernel/setup.c
> @@ -1048,6 +1048,15 @@ static int c_show(struct seq_file *m, void *v)
> seq_printf(m, "model name\t: %s rev %d (%s)\n",
> cpu_name, cpuid & 15, elf_platform);
>
> +#if defined(CONFIG_SMP)
> + seq_printf(m, "BogoMIPS\t: %lu.%02lu\n",
> + per_cpu(cpu_data, i).loops_per_jiffy / (500000UL/HZ),
> + (per_cpu(cpu_data, i).loops_per_jiffy / (5000UL/HZ)) % 100);
> +#else
> + seq_printf(m, "BogoMIPS\t: %lu.%02lu\n",
> + loops_per_jiffy / (500000/HZ),
> + (loops_per_jiffy / (5000/HZ)) % 100);
> +#endif
> /* dump out the processor features */
> seq_puts(m, "Features\t: ");
>
> diff --git a/arch/arm/kernel/smp.c b/arch/arm/kernel/smp.c
> index 5e6052e..86ef244 100644
> --- a/arch/arm/kernel/smp.c
> +++ b/arch/arm/kernel/smp.c
> @@ -387,6 +387,18 @@ asmlinkage void secondary_start_kernel(void)
>
> void __init smp_cpus_done(unsigned int max_cpus)
> {
> + int cpu;
> + unsigned long bogosum = 0;
> +
> + for_each_online_cpu(cpu)
> + bogosum += per_cpu(cpu_data, cpu).loops_per_jiffy;
> +
> + printk(KERN_INFO "SMP: Total of %d processors activated "
> + "(%lu.%02lu BogoMIPS).\n",
> + num_online_cpus(),
> + bogosum / (500000/HZ),
> + (bogosum / (5000/HZ)) % 100);
> +
> hyp_mode_check();
> }
>
>
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
>
>
--
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/