diff options
| author | Dirk Brandewie <dirk.j.brandewie@intel.com> | 2014-02-12 13:01:03 -0500 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2014-02-12 20:11:18 -0500 |
| commit | 709c078e176bd47227e89bb34de7c64b57aaaeab (patch) | |
| tree | b7d11870565827e4fb6f56a2e7110f8889c42b87 /include/trace | |
| parent | fcb6a15c2e7e76d493e6f91ea889ab40e1c643a4 (diff) | |
intel_pstate: Remove energy reporting from pstate_sample tracepoint
Remove the reporting of energy since it does not provide any useful
information about the state of the driver and will be a maintainance
headache going forward since the RAPL energy units register is not
architectural and subject to change between micro-architectures
References: https://bugzilla.kernel.org/show_bug.cgi?id=69831
Fixes: b69880f9ccf7 (intel_pstate: Add trace point to report internal state.)
Signed-off-by: Dirk Brandewie <dirk.j.brandewie@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'include/trace')
| -rw-r--r-- | include/trace/events/power.h | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/include/trace/events/power.h b/include/trace/events/power.h index 9e9475c85de5..e5bf9a76f169 100644 --- a/include/trace/events/power.h +++ b/include/trace/events/power.h | |||
| @@ -42,7 +42,6 @@ TRACE_EVENT(pstate_sample, | |||
| 42 | u32 state, | 42 | u32 state, |
| 43 | u64 mperf, | 43 | u64 mperf, |
| 44 | u64 aperf, | 44 | u64 aperf, |
| 45 | u32 energy, | ||
| 46 | u32 freq | 45 | u32 freq |
| 47 | ), | 46 | ), |
| 48 | 47 | ||
| @@ -51,7 +50,6 @@ TRACE_EVENT(pstate_sample, | |||
| 51 | state, | 50 | state, |
| 52 | mperf, | 51 | mperf, |
| 53 | aperf, | 52 | aperf, |
| 54 | energy, | ||
| 55 | freq | 53 | freq |
| 56 | ), | 54 | ), |
| 57 | 55 | ||
| @@ -61,7 +59,6 @@ TRACE_EVENT(pstate_sample, | |||
| 61 | __field(u32, state) | 59 | __field(u32, state) |
| 62 | __field(u64, mperf) | 60 | __field(u64, mperf) |
| 63 | __field(u64, aperf) | 61 | __field(u64, aperf) |
| 64 | __field(u32, energy) | ||
| 65 | __field(u32, freq) | 62 | __field(u32, freq) |
| 66 | 63 | ||
| 67 | ), | 64 | ), |
| @@ -72,17 +69,15 @@ TRACE_EVENT(pstate_sample, | |||
| 72 | __entry->state = state; | 69 | __entry->state = state; |
| 73 | __entry->mperf = mperf; | 70 | __entry->mperf = mperf; |
| 74 | __entry->aperf = aperf; | 71 | __entry->aperf = aperf; |
| 75 | __entry->energy = energy; | ||
| 76 | __entry->freq = freq; | 72 | __entry->freq = freq; |
| 77 | ), | 73 | ), |
| 78 | 74 | ||
| 79 | TP_printk("core_busy=%lu scaled=%lu state=%lu mperf=%llu aperf=%llu energy=%lu freq=%lu ", | 75 | TP_printk("core_busy=%lu scaled=%lu state=%lu mperf=%llu aperf=%llu freq=%lu ", |
| 80 | (unsigned long)__entry->core_busy, | 76 | (unsigned long)__entry->core_busy, |
| 81 | (unsigned long)__entry->scaled_busy, | 77 | (unsigned long)__entry->scaled_busy, |
| 82 | (unsigned long)__entry->state, | 78 | (unsigned long)__entry->state, |
| 83 | (unsigned long long)__entry->mperf, | 79 | (unsigned long long)__entry->mperf, |
| 84 | (unsigned long long)__entry->aperf, | 80 | (unsigned long long)__entry->aperf, |
| 85 | (unsigned long)__entry->energy, | ||
| 86 | (unsigned long)__entry->freq | 81 | (unsigned long)__entry->freq |
| 87 | ) | 82 | ) |
| 88 | 83 | ||
