diff options
author | Bjoern B. Brandenburg <bbb@cs.unc.edu> | 2010-07-16 10:30:06 -0400 |
---|---|---|
committer | Bjoern B. Brandenburg <bbb@cs.unc.edu> | 2010-07-16 11:50:35 -0400 |
commit | 136a08dbe8c28e751b01e932420f715edb229f6b (patch) | |
tree | c988a0025de9455b622aabd0d3cc98596284060a /litmus | |
parent | cbc5d49e4973400737aab50b60dc5d86e71f5420 (diff) |
Bugfix: avoid link error in Feather-Trace on x86
If no events are defined but Feater-Trace support is enabled, then the current
implementation generates a link error because the __event_table sections is
absent.
> arch/x86/built-in.o: In function `ft_disable_all_events':
> (.text+0x242af): undefined reference to `__start___event_table'
As a simple work around, we force zero-element array to always be "allocated"
in the __event_table section. This ensures that we end up with a zero-byte
section if no events are enabled, and does not affect the layout of the section
if events are present.
> bbb@ludwig:~/dev/litmus2010$ nm vmlinux | grep event_table
> ffffffff81950cdc D __event_table_dummy
> ffffffff81950cdc A __start___event_table
> ffffffff81950cdc A __stop___event_table
Diffstat (limited to 'litmus')
-rw-r--r-- | litmus/Kconfig | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/litmus/Kconfig b/litmus/Kconfig index f3aa7478faf7..9888589ef126 100644 --- a/litmus/Kconfig +++ b/litmus/Kconfig | |||
@@ -90,6 +90,10 @@ config FEATHER_TRACE | |||
90 | 90 | ||
91 | Bottom line: to avoid increased overheads, choose DEBUG_RODATA=n. | 91 | Bottom line: to avoid increased overheads, choose DEBUG_RODATA=n. |
92 | 92 | ||
93 | Note that this option only enables the basic Feather-Trace infrastructure; | ||
94 | you still need to enable SCHED_TASK_TRACE and/or SCHED_OVERHEAD_TRACE to | ||
95 | actually enable any events. | ||
96 | |||
93 | config SCHED_TASK_TRACE | 97 | config SCHED_TASK_TRACE |
94 | bool "Trace real-time tasks" | 98 | bool "Trace real-time tasks" |
95 | depends on FEATHER_TRACE | 99 | depends on FEATHER_TRACE |