diff options
| author | Jason Wessel <jason.wessel@windriver.com> | 2012-08-08 22:12:19 -0400 |
|---|---|---|
| committer | Jason Wessel <jason.wessel@windriver.com> | 2012-10-12 07:37:34 -0400 |
| commit | 42c12213141d7c88e8f649b4bacd19ba14577658 (patch) | |
| tree | 3ac8cd09ea97332f18f71f68d8f219393e996579 | |
| parent | f0a996eeeda214f4293e234df33b29bec003b536 (diff) | |
kgdb,x86: fix warning about unused variable
When compiling without CONFIG_DEBUG_RODATA the following
compiler warning is generated:
arch/x86/kernel/kgdb.c: In function 'kgdb_arch_set_breakpoint':
arch/x86/kernel/kgdb.c:749: warning: unused variable 'opc'
The variable instantiation needs to be inside the #ifdef to
make the warning go away.
Reported-by: Thiago Rafael Becker <trbecker@trbecker.org>
Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
| -rw-r--r-- | arch/x86/kernel/kgdb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/arch/x86/kernel/kgdb.c b/arch/x86/kernel/kgdb.c index 3f61904365cf..836f8322960e 100644 --- a/arch/x86/kernel/kgdb.c +++ b/arch/x86/kernel/kgdb.c | |||
| @@ -746,7 +746,9 @@ void kgdb_arch_set_pc(struct pt_regs *regs, unsigned long ip) | |||
| 746 | int kgdb_arch_set_breakpoint(struct kgdb_bkpt *bpt) | 746 | int kgdb_arch_set_breakpoint(struct kgdb_bkpt *bpt) |
| 747 | { | 747 | { |
| 748 | int err; | 748 | int err; |
| 749 | #ifdef CONFIG_DEBUG_RODATA | ||
| 749 | char opc[BREAK_INSTR_SIZE]; | 750 | char opc[BREAK_INSTR_SIZE]; |
| 751 | #endif /* CONFIG_DEBUG_RODATA */ | ||
| 750 | 752 | ||
| 751 | bpt->type = BP_BREAKPOINT; | 753 | bpt->type = BP_BREAKPOINT; |
| 752 | err = probe_kernel_read(bpt->saved_instr, (char *)bpt->bpt_addr, | 754 | err = probe_kernel_read(bpt->saved_instr, (char *)bpt->bpt_addr, |
