Bug 10493 - runtime/time.c: need autoconf for cpu_khz
Summary: runtime/time.c: need autoconf for cpu_khz
Status: RESOLVED FIXED
Alias: None
Product: systemtap
Classification: Unclassified
Component: runtime (show other bugs)
Version: unspecified
: P2 normal
Target Milestone: ---
Assignee: Unassigned
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-06 22:26 UTC by Frank Ch. Eigler
Modified: 2010-02-26 02:23 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed:


Attachments
patch to detect cpu_khz (833 bytes, patch)
2009-08-07 09:42 UTC, Wenji Huang
Details | Diff
Updated patch (1.19 KB, patch)
2010-01-12 09:18 UTC, Wenji Huang
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Frank Ch. Eigler 2009-08-06 22:26:24 UTC
Some kernels (old rhel4 x86-64 xen) don't have and/or export cpu_khz.
These would need to go back to the heuristic estimation code in
runtime/time.c:__stp_get_freq().
Comment 1 Wenji Huang 2009-08-07 09:42:17 UTC
Created attachment 4120 [details]
patch to detect cpu_khz
Comment 2 Frank Ch. Eigler 2009-08-07 18:04:31 UTC
Looks good, if cpu_khz is actually absent in these kernels, as opposed to merely
not EXPORT_SYMBOL'd.
Comment 3 Wenji Huang 2010-01-12 09:18:23 UTC
Created attachment 4507 [details]
Updated patch

Reading symbol from Module.symvers
Comment 4 Wenji Huang 2010-01-13 03:35:42 UTC
commit e25ab03ca60a5c6ca687b698502730ba2ad244dc
Author: Wenji Huang <wenji.huang@oracle.com>
Date:   Wed Jan 13 11:27:48 2010 +0800

    PR10493: autoconf for cpu_khz
    
    * buildrun.cxx (output_cpu_khz): New function to check cpu_khz.
      (compile_pass): Invoke function output_cpu_khz.
    * runtime/time.c : Use STAPCONF_CPU_KHZ.
Comment 5 Frank Ch. Eigler 2010-02-26 02:23:17 UTC
committed