cpu-freq broken on powerpc

[Dustin Kirkland ]

Binary package hint: screen-profiles

cpu frequency is reported differently on powerpc:

dustin@guestbr2:~$ cat /proc/cpuinfo
processor : 0
cpu : 7447A, altivec supported
clock : 1249.999995MHz
revision : 0.2 (pvr 8003 0102)
bogomips : 83.20
timebase : 41620907
platform : PowerMac
machine : PowerMac10,1
motherboard : PowerMac10,1 MacRISC3 Power Macintosh
detected as : 287 (Mac mini)
pmac flags : 00000010
L2 cache : 512K unified
pmac-generation : NewWorld

This makes screen-profiles cpu-freq command fail.

:-Dustin

[Launchpad Janitor (janitor)]

This bug was fixed in the package screen-profiles - 1.42-0ubuntu1

---------------
screen-profiles (1.42-0ubuntu1) jaunty; urgency=low

  [ Dustin Kirkland ]
  * bin/cpu-freq: fix cpu freq for powerpc hardware, LP: #352286;
    use 2 decimals for GHz (bring this down to 1 decimal when cpus hit 10GHz)
  * debian/postinst, debian/prerm, screen: remove some maintainer hacks
    that should be cleaned up before Jaunty GA, LP: #352291
  * po/*: screen-profiles-export translation template
  * profiles/common: added 'altscreen on', which cleans the screen for
    programs that support this, such as vi and emacs, LP: #349677
  * bin/logo, profiles/common, profiles/generate: dynamically print logo
    LP: #352632
  * screen-profiles-export, screen-profiles-export.1: drop distro selection,
    no longer needed
  * added logo for Arch linux

  [ Adi Roiban ]
  * debian/install, debian/rules, po/POTFILES.python, po/POTFILES.sh,
    po/es.po, po/fr.po, po/screen-profiles.pot: LP: #347521, i18n support
    in the screen-profiles build and install

 -- Dustin Kirkland <email address hidden> Tue, 31 Mar 2009 17:31:10 -0500

[Dustin Kirkland  (kirkland)]

This regressed, somehow...

:-Dustin

[Dustin Kirkland  (kirkland)]

Also a trivial fix, add clock to the egrep. Should be SRU'd if we can't get this into the release.

:-Dustin

[Steve Langasek (vorlon)]

These two bugs have been marked as 'low' importance, which seems accurate to me, so I'm rejecting this upload for jaunty final. Please consider this as a candidate for SRU instead (immediately, if you wish).

[Dustin Kirkland  (kirkland)]

SRU Justification:

 1. Impact: Minor impact on powerpc users, cpu frequency incorrectly reported; trivial to fix as a cherrypick from upstream
 2. How fixed: Simply need a better egrep regular expression; seems that different architectures report their clock speed differently in /proc/cpuinfo
 3. Patch: in the debdiff
 4. Test Case: on a powerpc system, run 'screen'. Observe that the cpu-frequency is not reported correctly in the status bar.
 5. Regression potential: Virtually no chance. I have tested this code on all architectures I can access. It's just a better regular expression.

:-Dustin

[Dustin Kirkland  (kirkland)]

Uploaded to jaunty-proposed.

:-Dustin

[Martin Pitt (pitti)]

Accepted screen-profiles into jaunty-proposed-proposed; please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

[Launchpad Janitor (janitor)]

This bug was fixed in the package screen-profiles - 1.44-0ubuntu1.1

---------------
screen-profiles (1.44-0ubuntu1.1) jaunty-proposed; urgency=low

  Cherry-picked fixes from upstream:
  * bin/cpu-freq: fix for powerpc LP: #352286, fix for arm LP: #362157,
    always report in GHz, round to nearest hundred
  * bin/uptime: don't print seconds, since we don't update often enough
  * Performance improvements, screen too slow to launch, LP: #360759
    - screen-profiles-status: use home bin scripts, if available first,
      then fallback to system-wide, allows for local overrides; comment
      out nice and ionice calls; this *really* slows down startup
    - bin/release: eliminate one expensive lsb_release call
    - bin/updates-available: background updates checking processes,
      let them write to cache, and let the next cache check pick up the
      changes

 -- Dustin Kirkland <email address hidden> Fri, 17 Apr 2009 12:40:29 -0500

[Brian Murray (brian-murray)]

Using screen-profiles version 1.44-0ubuntu1 I was not able to see any CPU speed on my powerpc system. After installing screen-profiles version 1.44-0ubuntu1.1 from -proposed I was able to see CPU speed.

By the way this verification occurred on a Playstation 3.