diff options
| author | Ravi Bangoria <[email protected]> | 2020-05-14 16:47:40 +0530 |
|---|---|---|
| committer | Michael Ellerman <[email protected]> | 2020-05-19 00:14:45 +1000 |
| commit | 514db915e7b33e7eaf8e40192b93380f79b319b5 (patch) | |
| tree | 1610520c4f5fa51cf52d44e5973bebc9a8422614 | |
| parent | 29da4f91c0c1fbda12b8a31be0d564930208c92e (diff) | |
powerpc/watchpoint/xmon: Don't allow breakpoint overwriting
Xmon allows overwriting breakpoints because it's supported by only
one DAWR. But with multiple DAWRs, overwriting becomes ambiguous
or unnecessary complicated. So let's not allow it.
Signed-off-by: Ravi Bangoria <[email protected]>
Signed-off-by: Michael Ellerman <[email protected]>
Reviewed-by: Michael Neuling <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
| -rw-r--r-- | arch/powerpc/xmon/xmon.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c index 30b3e3d99c0d..094bf4715f2c 100644 --- a/arch/powerpc/xmon/xmon.c +++ b/arch/powerpc/xmon/xmon.c @@ -1399,6 +1399,10 @@ bpt_cmds(void) printf("Hardware data breakpoint not supported on this cpu\n"); break; } + if (dabr.enabled) { + printf("Couldn't find free breakpoint register\n"); + break; + } mode = 7; cmd = inchar(); if (cmd == 'r') |