diff options
author | Paul Mackerras <paulus@samba.org> | 2010-06-29 12:50:32 +1000 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2010-06-30 13:54:58 +1000 |
commit | d09ec7387184eba9e3030496f0451204090ff610 (patch) | |
tree | c3a6f251bf695a20009d1cd3f0d7a9115fdb3018 /arch/powerpc/kernel/machine_kexec_64.c | |
parent | 76b0f1337690d223811c852ad3a5078eb89276c5 (diff) | |
download | talos-obmc-linux-d09ec7387184eba9e3030496f0451204090ff610.tar.gz talos-obmc-linux-d09ec7387184eba9e3030496f0451204090ff610.zip |
powerpc, hw_breakpoint: Tell generic code we have no instruction breakpoints
At present, hw_breakpoint_slots() returns 1 regardless of what
type of breakpoint is specified in the type argument. Since we
don't define CONFIG_HAVE_MIXED_BREAKPOINTS_REGS, there are
separate values for TYPE_INST and TYPE_DATA, and hw_breakpoint_slots()
returns 1 for both, effectively advertising instruction breakpoint
support which doesn't exist.
This fixes it by making hw_breakpoint_slots return 1 for TYPE_DATA
and 0 for TYPE_INST. This moves hw_breakpoint_slots() from the
powerpc hw_breakpoint.h to hw_breakpoint.c because the definitions
of TYPE_INST and TYPE_DATA aren't available in <asm/hw_breakpoint.h>.
They are defined in <linux/hw_breakpoint.h> but we can't include
that header in <asm/hw_breakpoint.h>, and nor can we rely on
<linux/hw_breakpoint.h> being included before <asm/hw_breakpoint.h>.
Since hw_breakpoint_slots() is only called at boot time, there is
no performance impact from making it a real function rather than
a static inline.
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/kernel/machine_kexec_64.c')
0 files changed, 0 insertions, 0 deletions