diff options
author | Oleg Nesterov <oleg@redhat.com> | 2012-09-23 21:30:44 +0200 |
---|---|---|
committer | Oleg Nesterov <oleg@redhat.com> | 2012-09-29 21:21:54 +0200 |
commit | ed6f6a50dc5f183c53e7b3b7fed4794bc50d9aa7 (patch) | |
tree | 8ef3472db708804bb292a79752d0b04de8f2060d /kernel/panic.c | |
parent | uprobes: Introduce copy_opcode(), kill read_opcode() (diff) | |
download | linux-ed6f6a50dc5f183c53e7b3b7fed4794bc50d9aa7.tar.xz linux-ed6f6a50dc5f183c53e7b3b7fed4794bc50d9aa7.zip |
uprobes: Kill set_orig_insn()->is_swbp_at_addr()
Unlike set_swbp(), set_orig_insn()->is_swbp_at_addr() makes sense,
although it can't prevent all confusions.
But the usage of is_swbp_at_addr() is equally confusing, and it adds
the extra get_user_pages() we can avoid.
This patch removes set_orig_insn()->is_swbp_at_addr() but changes
write_opcode() to do the necessary checks before replace_page().
Perhaps it also makes sense to ensure PAGE_MAPPING_ANON in unregister
case.
find_active_uprobe() becomes the only user of is_swbp_at_addr(),
we can change its semantics.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/panic.c')
0 files changed, 0 insertions, 0 deletions