incorrect cpu information for ARM server

[Michael Rooney]

Binary package hint: screen-profiles

Hello, thanks for the wonderful package :)

The CPU information doesn't seem to be detected properly on my 1200MHZ ARM machine. I've attached a screenshot of what I see, and here is the hopefully relevant information:

$ cat /proc/cpuinfo
Processor : ARM926EJ-S rev 1 (v5l)
BogoMIPS : 1192.75
Features : swp half thumb fastmult edsp
CPU implementer : 0x56
CPU architecture: 5TE
CPU variant : 0x2
CPU part : 0x131
CPU revision : 1
Cache type : write-back
Cache clean : cp15 c7 ops
Cache lockdown : format C
Cache format : Harvard
I size : 16384
I assoc : 4
I line length : 32
I sets : 128
D size : 16384
D assoc : 4
D line length : 32
D sets : 128

Hardware : Feroceon-KW
Revision : 0000
Serial : 0000000000000000

[Dustin Kirkland  (kirkland)]

Hmm, interesting, thanks for the report, Michael.

I guess the BogoMIPS is the best number to use for cpu-freq on ARM?

:-Dustin

[Michael Rooney (mrooney)]

On Thu, Apr 16, 2009 at 10:06 AM, Dustin Kirkland
<email address hidden> wrote:
> I guess the BogoMIPS is the best number to use for cpu-freq on ARM?

Yes that does appear to be the correct number. I wonder if, based on
the potential inaccuracy of the number, it might be best to round it
to the nearest hundred?

[Dustin Kirkland  (kirkland)]

Fix committed, will be released in 1.45.

:-Dustin

[Michael Rooney (mrooney)]

On Thu, Apr 16, 2009 at 11:12 AM, Dustin Kirkland
<email address hidden> wrote:
> Fix committed, will be released in 1.45.

Thanks Dustin, will this make Jaunty or backports? I actually don't
think there is enough free space on the machine (512MB total disk) for
me to install build-deps and compile it myself!

[Dustin Kirkland  (kirkland)]

I'm actually going to try and get one SRU for screen-profiles.

Alternatively, you can either:
 a) use the screen-profiles PPA, where I will continue to publish
upstream packages
 b) use screen-profiles-export and just spit out a static tarball of a
good profile.

:-Dustin

[Dustin Kirkland  (kirkland)]

Trivial fix to the egrep, add 'bogomips' to the list of grepped terms.

Hopefully we can get this in. Very low priority. Should be SRU'd if not.

:-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 arm 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 an arm 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!

[Michael Rooney (mrooney)]

I can confirm that 1.48-0ubuntu1~ppa3 from the screen-profiles PPA
addresses the CPU speed issue with no apparent regressions (thanks
Dustin!) although I suppose that isn't particularly helpful for the
SRU.

[Dustin Kirkland  (kirkland)]

Michael-

Enable the -proposed repository and test that one. It should address
the problem in the same way, but if you could test that and post the
results, that would be valuable to the SRU.

:-Dustin

[Michael Rooney (mrooney)]

It is a little more complicated than that :) I had to enable the
proposed repository, disable the PPA, update, uninstall and then
reinstall, but I now can confirm that everything appears to work as
intended with the version from -proposed ("Installed:
1.44-0ubuntu1.1")

[Dustin Kirkland  (kirkland)]

Sweet, thanks, Michael. "As you were!" You may safely re-enable the
PPA repo's as I have already released some cool new stuff in 1.50 ;-)

:-Dustin

[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