diff options
author | Shreshtha Sahu <ssahu@nvidia.com> | 2013-07-19 10:57:26 +0530 |
---|---|---|
committer | Shreshtha Sahu <ssahu@nvidia.com> | 2013-10-23 22:09:17 -0700 |
commit | 99c9871c5793a023468f7d50b993ddb3199a61d6 (patch) | |
tree | b40e0e7539ff2a88c89b9c922398b05b19d27f8c /init | |
parent | b4941d8f5d3c2b6b10876087e861d301b21325a5 (diff) |
init: calibrate: don't print out bogomips value on boot
BogoMIPs is a confusing concept to the ill-informed, so allow
architectures to print it only if they find it worthwhile. The delay
calibration code should stick to lpj and avoid trying to draw any
correlation with BogoMIPs, which may be a fixed value derived from a
timer frequency independent of the CPU clock speed.
Signed-off-by: Will Deacon <will.deacon@arm.com>
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
Patch picked from - https://lkml.org/lkml/2013/5/3/399
Bug 1320222
Change-Id: I6f08d6eea075a7ee0ab98e7c04ceb29ba350ee60
Signed-off-by: Shreshtha Sahu <ssahu@nvidia.com>
Reviewed-on: http://git-master/r/302229
Reviewed-by: Kiran Adduri <kadduri@nvidia.com>
Reviewed-by: Matthew Pedro <mapedro@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>
Diffstat (limited to 'init')
-rw-r--r-- | init/calibrate.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/init/calibrate.c b/init/calibrate.c index fda0a7b0f06c..282a92ae45c6 100644 --- a/init/calibrate.c +++ b/init/calibrate.c @@ -95,7 +95,7 @@ static unsigned long __cpuinit calibrate_delay_direct(void) * >= 12.5% apart, redo calibration. */ if (start >= post_end) - printk(KERN_NOTICE "calibrate_delay_direct() ignoring " + pr_info("calibrate_delay_direct() ignoring " "timer_rate as we had a TSC wrap around" " start=%lu >=post_end=%lu\n", start, post_end); @@ -134,15 +134,13 @@ static unsigned long __cpuinit calibrate_delay_direct(void) good_timer_count = 0; if ((measured_times[max] - estimate) < (estimate - measured_times[min])) { - printk(KERN_NOTICE "calibrate_delay_direct() dropping " - "min bogoMips estimate %d = %lu\n", - min, measured_times[min]); + pr_info("%s() dropping min delay estimate %d = %lu\n", + __func__, min, measured_times[min]); measured_times[min] = 0; min = max; } else { - printk(KERN_NOTICE "calibrate_delay_direct() dropping " - "max bogoMips estimate %d = %lu\n", - max, measured_times[max]); + pr_info("%s() dropping max delay estimate %d = %lu\n", + __func__, max, measured_times[max]); measured_times[max] = 0; max = min; } @@ -160,7 +158,7 @@ static unsigned long __cpuinit calibrate_delay_direct(void) } - printk(KERN_NOTICE "calibrate_delay_direct() failed to get a good " + pr_info("calibrate_delay_direct() failed to get a good " "estimate for loops_per_jiffy.\nProbably due to long platform " "interrupts. Consider using \"lpj=\" boot option.\n"); return 0; @@ -292,9 +290,7 @@ void __cpuinit calibrate_delay(void) } per_cpu(cpu_loops_per_jiffy, this_cpu) = lpj; if (!printed) - pr_cont("%lu.%02lu BogoMIPS (lpj=%lu)\n", - lpj/(500000/HZ), - (lpj/(5000/HZ)) % 100, lpj); + pr_cont("lpj=%lu\n", lpj); loops_per_jiffy = lpj; printed = true; |