diff options
author | Will Deacon <will.deacon@arm.com> | 2015-07-27 13:36:54 -0400 |
---|---|---|
committer | Will Deacon <will.deacon@arm.com> | 2015-07-27 13:36:54 -0400 |
commit | 6f883d10a18b971a3646303d12ef75138dfd31f9 (patch) | |
tree | 91e4001d36720193793ec9be6390ae81e9ca2666 /arch | |
parent | 662ba3dbceca3ca284885a464ecb8c936f417003 (diff) |
arm64: debug: rename enum debug_el to avoid symbol collision
lib/list_sort.c defines a 'struct debug_el', where "el" is assumedly a
a contraction of "element". This conflicts with 'enum debug_el' in our
asm/debug-monitors.h header file, where "el" stands for Exception Level.
The result is build failure when targetting allmodconfig, so rename our
enum to 'dbg_active_el' to be slightly more explicit about what it is.
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/arm64/include/asm/debug-monitors.h | 6 | ||||
-rw-r--r-- | arch/arm64/kernel/debug-monitors.c | 4 | ||||
-rw-r--r-- | arch/arm64/kernel/hw_breakpoint.c | 6 |
3 files changed, 8 insertions, 8 deletions
diff --git a/arch/arm64/include/asm/debug-monitors.h b/arch/arm64/include/asm/debug-monitors.h index e3f2bad788c9..279c85b5ec09 100644 --- a/arch/arm64/include/asm/debug-monitors.h +++ b/arch/arm64/include/asm/debug-monitors.h | |||
@@ -115,13 +115,13 @@ void unregister_break_hook(struct break_hook *hook); | |||
115 | 115 | ||
116 | u8 debug_monitors_arch(void); | 116 | u8 debug_monitors_arch(void); |
117 | 117 | ||
118 | enum debug_el { | 118 | enum dbg_active_el { |
119 | DBG_ACTIVE_EL0 = 0, | 119 | DBG_ACTIVE_EL0 = 0, |
120 | DBG_ACTIVE_EL1, | 120 | DBG_ACTIVE_EL1, |
121 | }; | 121 | }; |
122 | 122 | ||
123 | void enable_debug_monitors(enum debug_el el); | 123 | void enable_debug_monitors(enum dbg_active_el el); |
124 | void disable_debug_monitors(enum debug_el el); | 124 | void disable_debug_monitors(enum dbg_active_el el); |
125 | 125 | ||
126 | void user_rewind_single_step(struct task_struct *task); | 126 | void user_rewind_single_step(struct task_struct *task); |
127 | void user_fastforward_single_step(struct task_struct *task); | 127 | void user_fastforward_single_step(struct task_struct *task); |
diff --git a/arch/arm64/kernel/debug-monitors.c b/arch/arm64/kernel/debug-monitors.c index b056369fd47d..9b3b62ac9c24 100644 --- a/arch/arm64/kernel/debug-monitors.c +++ b/arch/arm64/kernel/debug-monitors.c | |||
@@ -82,7 +82,7 @@ early_param("nodebugmon", early_debug_disable); | |||
82 | static DEFINE_PER_CPU(int, mde_ref_count); | 82 | static DEFINE_PER_CPU(int, mde_ref_count); |
83 | static DEFINE_PER_CPU(int, kde_ref_count); | 83 | static DEFINE_PER_CPU(int, kde_ref_count); |
84 | 84 | ||
85 | void enable_debug_monitors(enum debug_el el) | 85 | void enable_debug_monitors(enum dbg_active_el el) |
86 | { | 86 | { |
87 | u32 mdscr, enable = 0; | 87 | u32 mdscr, enable = 0; |
88 | 88 | ||
@@ -102,7 +102,7 @@ void enable_debug_monitors(enum debug_el el) | |||
102 | } | 102 | } |
103 | } | 103 | } |
104 | 104 | ||
105 | void disable_debug_monitors(enum debug_el el) | 105 | void disable_debug_monitors(enum dbg_active_el el) |
106 | { | 106 | { |
107 | u32 mdscr, disable = 0; | 107 | u32 mdscr, disable = 0; |
108 | 108 | ||
diff --git a/arch/arm64/kernel/hw_breakpoint.c b/arch/arm64/kernel/hw_breakpoint.c index 7a1a5da6c8c1..003bc3d50636 100644 --- a/arch/arm64/kernel/hw_breakpoint.c +++ b/arch/arm64/kernel/hw_breakpoint.c | |||
@@ -156,7 +156,7 @@ static void write_wb_reg(int reg, int n, u64 val) | |||
156 | * Convert a breakpoint privilege level to the corresponding exception | 156 | * Convert a breakpoint privilege level to the corresponding exception |
157 | * level. | 157 | * level. |
158 | */ | 158 | */ |
159 | static enum debug_el debug_exception_level(int privilege) | 159 | static enum dbg_active_el debug_exception_level(int privilege) |
160 | { | 160 | { |
161 | switch (privilege) { | 161 | switch (privilege) { |
162 | case AARCH64_BREAKPOINT_EL0: | 162 | case AARCH64_BREAKPOINT_EL0: |
@@ -230,7 +230,7 @@ static int hw_breakpoint_control(struct perf_event *bp, | |||
230 | struct perf_event **slots; | 230 | struct perf_event **slots; |
231 | struct debug_info *debug_info = ¤t->thread.debug; | 231 | struct debug_info *debug_info = ¤t->thread.debug; |
232 | int i, max_slots, ctrl_reg, val_reg, reg_enable; | 232 | int i, max_slots, ctrl_reg, val_reg, reg_enable; |
233 | enum debug_el dbg_el = debug_exception_level(info->ctrl.privilege); | 233 | enum dbg_active_el dbg_el = debug_exception_level(info->ctrl.privilege); |
234 | u32 ctrl; | 234 | u32 ctrl; |
235 | 235 | ||
236 | if (info->ctrl.type == ARM_BREAKPOINT_EXECUTE) { | 236 | if (info->ctrl.type == ARM_BREAKPOINT_EXECUTE) { |
@@ -537,7 +537,7 @@ int arch_validate_hwbkpt_settings(struct perf_event *bp) | |||
537 | * exception level at the register level. | 537 | * exception level at the register level. |
538 | * This is used when single-stepping after a breakpoint exception. | 538 | * This is used when single-stepping after a breakpoint exception. |
539 | */ | 539 | */ |
540 | static void toggle_bp_registers(int reg, enum debug_el el, int enable) | 540 | static void toggle_bp_registers(int reg, enum dbg_active_el el, int enable) |
541 | { | 541 | { |
542 | int i, max_slots, privilege; | 542 | int i, max_slots, privilege; |
543 | u32 ctrl; | 543 | u32 ctrl; |