timer: fix TSC frequency by not reading /proc/cpuinfo
authorBruce Richardson <bruce.richardson@intel.com>
Mon, 7 Apr 2014 12:57:28 +0000 (13:57 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Wed, 9 Apr 2014 12:21:36 +0000 (14:21 +0200)
commitd73d8f3ad4d42b04d6379dee830cc1726f9c94ef
treea65702609de2f9b49de9c54a1048f5a4d5470613
parent99f2cdf9ca10cac4af9a01f4934ea23d8d1bdacb
timer: fix TSC frequency by not reading /proc/cpuinfo

This reverts commit da6fd0759cbeb5fc14991a79e40105b9f6b99059.
"timer: get TSC frequency from /proc/cpuinfo"

The use of cpuinfo to determine the frequency of the TSC is not
advisable and leads to incorrect results when power management is
in use. This is because, while the TSC frequency does not change
in modern cpus with constant_tsc support, the frequency of the core,
and hence the frequency of the core reported by cpuinfo *does* change.

Depending on the current frequency of core 0 when an application is
started, the EAL can get a wildly incorrect value for the TSC freq.
Since frequency is scaled down for power saving, any incorrect value
is likely to be lower than the default, which means that any delay
loops inside the code which rely on the TSC will be shorter than
planned. This can cause issues (reported on the mailing list by a number
of people) where ports are not initialized correctly due to delays being
too short.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
lib/librte_eal/linuxapp/eal/eal_timer.c