RE: [patch] enable userspace cpu core voltage control withacpi-cpufreq
From: Pallipadi, Venkatesh
Date: Mon Sep 10 2007 - 17:05:30 EST
>-----Original Message-----
>From: linux-kernel-owner@xxxxxxxxxxxxxxx
>[mailto:linux-kernel-owner@xxxxxxxxxxxxxxx] On Behalf Of Dave Jones
>Sent: Monday, September 03, 2007 8:25 AM
>To: Andi Kleen
>Cc: colin.michael@xxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
>Subject: Re: [patch] enable userspace cpu core voltage control
>withacpi-cpufreq
>
>On Mon, Sep 03, 2007 at 12:56:13PM +0200, Andi Kleen wrote:
> > <colin.michael@xxxxxxxxxxx> writes:
> >
> > > i want to make a patch known that provides a userspace
>interface to control the core voltage of a computer processor(s).
> >
> > That would be essentially linux supported undervolting which
> > for stability is as bad as overclocking. The problem is that
> > such games tend to generate weird kernel crashes and then
> > chew up development issues when kernel hackers have to chase
> > ghost bugs. I don't think we should support it. Developer
> > time is too precious.
>
>Seconded. Exactly the same reasons I've refused to merge patches
>into cpufreq to allow arbitrary tables to override BIOS tables.
>Or patches to remove boundary checks. Even when correctly
>implemented, this stuff can be fragile as hell, so introducing
>more things that cast doubt over its stability isn't something
>I'm keen on at all.
>
And acpi-cpufreq does not seem to be the place to be doing this. I would
say it should be a new driver or go into speedstep-centrino which has
similar user defined freq voltage values. Ideally a standalone driver so
that users do not get confused with single driver working in different
modes and distributors can choose not to ship it in case where they do
not like the driver tainting the kernel.
Thanks,
Venki
-
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/