https://bugzilla.novell.com/show_bug.cgi?id=749114
https://bugzilla.novell.com/show_bug.cgi?id=749114#c6
Holger Macht changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |NEEDINFO
InfoProvider| |ManuelStol@operamail.com
--- Comment #6 from Holger Macht 2012-05-10 13:21:11 UTC ---
Had a look at the different scripts. I noticed that the CPU frequency stuff for
kernel >3.1 is already fixed in upstream git and via online update for 12.1
(see bnc#737319). At least the enable-profile part. Your enable-profile script
changes a lot of other stuff. What exactly? Can you rebase the patch on what is
upstream and mention the changes? Upstream code can be found at:
http://pm-profiler.git.sourceforge.net/git/gitweb.cgi?p=pm-profiler/pm-profi...
The following is rather not related to this bug report:
I like you changes for config.template, but have one question: Now we have
CPUFREQ_CONSERVATIVE_SAMPLING_RATE and CPUFREQ_ONDEMAND_SAMPLING_RATE. Wouldn't
it make sense to merge those two to only have one CPUFREQ_SAMPLING_RATE
variable?
Didn't have a closer look at get-current-settings which changes quite a bit.
What does it do? Why not only changing what's necessary to get the cpufreq
stuff >3.1 supported?
Same for pm-profiler wrapper. Why so many changes?
Anyway, thanks a lot for your efforts.
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.