Bug 131015 - cpuspeed shouldn't require changes to config file to find the right driver
Summary: cpuspeed shouldn't require changes to config file to find the right driver
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: cpuspeed
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dave Jones
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2004-08-26 17:35 UTC by Bastien Nocera
Modified: 2015-01-04 22:09 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-16 01:53:06 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Bastien Nocera 2004-08-26 17:35:16 UTC
One shouldn't need to modify /etc/cpuspeed.conf to change the driver
to be able to use the cpuspeed daemon. It should be possible to detect
automatically which driver should be used.

For example, kudzu would detect the CPU in the machine, and write out
the correct alias to /etc/modprobe.conf:
alias cpufreq-driver acpi

The cpuspeed init script would then try to load that driver before
running.

(This is an RFE slightly larger than just this module, but upstream
doesn't seem to have a mailing-list or a forum to discuss this problem)

Comment 1 Bill Nottingham 2004-08-27 16:14:08 UTC
Also, we appear to have some modules built in, and some modules built
as modules. :/

Comment 2 Dave Jones 2006-07-16 01:53:06 UTC
the modular/non-modular thing is intentional.   Building them all in solves the
problem of figuring out which module to load from userspace.

Basically, by the time we get to userspace, we should have a valid driver setup,
and if we haven't then the cpuspeed init script will load acpi-cpufreq as a
fallback.

The other drivers that /are/ modular are done so because probing for them is
unsafe and has been known to cause problems on some boxes.

in the majority of cases, you should never have to set the DRIVER line in the
cpuspeed config.



Note You need to log in before you can comment on or make changes to this bug.