Bug 459441 - cpuspeed support for loading powernow_k8 module
Summary: cpuspeed support for loading powernow_k8 module
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: cpuspeed
Version: 5.2
Hardware: All
OS: Linux
medium
medium
Target Milestone: rc
: ---
Assignee: Jarod Wilson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 438835 RHEL5u3_relnotes
TreeView+ depends on / blocked
 
Reported: 2008-08-18 22:07 UTC by Brian Maly
Modified: 2009-01-20 20:55 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
cpuspeed has been updated to version 1.2.1-5. With this update, the cpuspeed init script now loads the speedstep-centrino module if all other module loads fail. In addition, a user-space bug that prevented the Powernow-k8 module from loading is now fixed.
Clone Of:
Environment:
Last Closed: 2009-01-20 20:55:25 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2009:0098 0 normal SHIPPED_LIVE cpuspeed enhancement update 2009-01-20 16:04:33 UTC

Description Brian Maly 2008-08-18 22:07:01 UTC
Description of problem: Powernow-k8 is being change from a builtin to a module. Userspace needs to support loading of the module.


Version-Release number of selected component (if applicable): 5.2


This change is a dependency for bug 438835 which is a kernelspace change

Comment 6 Don Domingo 2008-11-21 02:25:24 UTC
this bug is now documented in the RHEL5.3 release notes. please consult the following link to view the most recent build of this document:

http://documentation-stage.bne.redhat.com/docs/en-US/Red_Hat_Enterprise_Linux/5.3/html-single/Release_Notes/

this document is synced daily. as such, any recent additions are expected to show up within 24 hours or so.

Comment 7 Don Domingo 2008-11-21 02:25:24 UTC
Release note added. If any revisions are required, please set the 
"requires_release_notes" flag to "?" and edit the "Release Notes" field accordingly.
All revisions will be proofread by the Engineering Content Services team.

New Contents:
cpuspeed has been updated to version 1.2.1-5. With this update, the cpuspeed init script now loads the speedstep-centrino module if all other module loads fail. In addition, a user-space bug that prevented the Powernow-k8 module from loading is now fixed.

Comment 8 Zhang Kexin 2008-11-24 09:51:03 UTC
on AMD x86_64 arch machine hp-xw9400-01.rhts.bos.redhat.com:

[root@hp-xw9400-01 cpufreq]# lsmod | grep powernow
powernow_k8            58073  0 
freq_table             40889  2 powernow_k8,cpufreq_ondemand

I can use "modprobe -r powernow_k8" to remove the module, while on AMD i686
arch machine hp-xw9400-02.rhts.bos.redhat.com:

[root@hp-xw9400-02 function]# lsmod | grep powernow
powernow_k8            23009  2

I can not remove by "modprobe -r powernow_k8"
[root@hp-xw9400-02 function]# modprobe -r powernow_k8
FATAL: Module powernow_k8 is in use.

This should be not a big problem, just want to know whether need to improve or
change it, to make them look the same.

Comment 9 Zhang Kexin 2008-12-08 09:34:54 UTC
tested on hp-xw4550-01.rhts.bos.redhat.com bare metal kernel is OK with test case cpuspeed/function and test case in https://bugzilla.redhat.com/show_bug.cgi?id=451119#c1

cpuspeed is not supported under xen kernel, see kernel function powernowk8_init()

............
        if (preregister_acpi_perf == 1 && cpu_family == CPU_OPTERON) {
                char * dmi_data = dmi_get_system_info(DMI_BIOS_VENDOR);
                if (dmi_data && !strncmp(dmi_data, "Hewlett-Packard", 15)) {
#ifdef CONFIG_XEN
                        /* Disable cpufreq for HP AMD Opteron systems */
                        printk("%s: This BIOS is %s .... disabling cpufreq "
                               "support\n", __FUNCTION__, dmi_data);
                        return -EPERM;
..............

Comment 10 Zhang Kexin 2008-12-08 12:27:36 UTC
tested on sun-x4600m2-01.rhts.bos.redhat.com bare metal kernel is OK.
but on xen kernel, output of "service cpuspeed status" is:

[root@sun-x4600m2-01 ~]# service cpuspeed status
Frequency scaling enabled using performance governor

ondemand driver is not loaded by default:

[root@sun-x4600m2-01 cpufreq]# cat scaling_available_governors 
userspace performance 

after load  the ondemand driver and echo ondemand > scaling_governor,
then "service cpuspeed status" becomes to:

[root@sun-x4600m2-01 cpufreq]# service cpuspeed status
Frequency scaling enabled using ondemand governor

I don't know whether xen kernel on AMD machine is intended to use performance governor, but I found that on several intel cpu machine, all xen kernel use ondemand governor by default.

so put the status to ON_DEV.

Comment 12 Jarod Wilson 2008-12-08 18:16:37 UTC
From the initscript:

    # Frequency scaling on AMD rev F CPUs under Xen can result in
    # timekeeping problems for fully virtualized guests, so we disable
    # it by default.
    if [ -d /proc/xen ] && [ "$cpu_vendor" == AuthenticAMD ] \
       && [ "$cpu_family" -le 15 ]; then
      default_governor=performance
    fi

In other words, things are behaving as designed.

Comment 13 Ryan Lerch 2008-12-08 21:47:30 UTC
Is there a reason why the release notes flag was set back to ?

this note is currently in the 5.3 release notes...

Comment 16 errata-xmlrpc 2009-01-20 20:55:25 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHEA-2009-0098.html


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