Attachment 283691 Details for Bug 419291 – fix for the cpuspeed initscript

[patch] fix for the cpuspeed initscript

cpuspeed.init.diff (text/plain), 1.26 KB, created by Rik van Riel on 2007-12-11 07:16:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rik van Riel

Created: 2007-12-11 07:16:56 UTC

Size: 1.26 KB

patch

obsolete

>--- init.d/cpuspeed.old	2007-12-11 02:06:44.000000000 -0500
>+++ init.d/cpuspeed	2007-12-11 02:14:02.000000000 -0500
>@@ -109,7 +109,7 @@ stop_cpuspeed() {
> }
> 
> start() {
>-  if [ ! -f $lockfile ] && [ ! -d "$xendir" ]; then
>+  if [ ! -f $lockfile ]; then
>     if ! some_file_exist $testpat ; then
>       # Attempt to load scaling_driver if not loaded
>       # but it is configured
>@@ -163,9 +163,15 @@ start() {
> 
>     # Run cpuspeed daemon for userspace gov, kernel ones otherwise
>     if [ "${governor}" == "userspace" ]; then
>-      start_cpuspeed
>-      RETVAL=$?
>+      if [ ! -d "$xendir" ]; then
>+        start_cpuspeed
>+        RETVAL=$?
>+      else
>+        $logger "Xen does not support the userspace CPU freq scaling governor"
>+        RETVAL=6
>+      fi
>     else
>+      # the userspace governor does not know how to handle Xen
>       if [ -n "$MIN_SPEED" ]; then
>         adjust_cpufreq scaling_min_freq $MIN_SPEED
>       fi
>@@ -191,13 +197,8 @@ start() {
>     # doing some sort of cpu frequency scaling.
>     [ $RETVAL = 0 ] && touch $lockfile
>   else
>-    if [ -d "$xendir" ]; then
>-      $logger "CPU Frequency scaling is currently not supported on xen kernels"
>-      RETVAL=6
>-    else
>       # cpuspeed is already started
>       RETVAL=1
>-    fi
>   fi
>   return $RETVAL
> }

Actions: View | Diff

Attachments on bug 419291: 283691