Messages in this thread Patch in this message | | | From | Srikar Dronamraju <> | Date | Thu, 31 May 2012 17:16:30 +0530 | Subject | [PATCH 1/3] uprobes/core: Remove redundant lock_page/unlock_page |
| |
From: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
No need to lock the page when copying the opcode in read_opcode().
Signed-off-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com> --- kernel/events/uprobes.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/kernel/events/uprobes.c b/kernel/events/uprobes.c index 42b21eb..b3f3095 100644 --- a/kernel/events/uprobes.c +++ b/kernel/events/uprobes.c @@ -315,12 +315,10 @@ static int read_opcode(struct mm_struct *mm, unsigned long vaddr, uprobe_opcode_ if (ret <= 0) return ret; - lock_page(page); vaddr_new = kmap_atomic(page); vaddr &= ~PAGE_MASK; memcpy(opcode, vaddr_new + vaddr, UPROBE_SWBP_INSN_SIZE); kunmap_atomic(vaddr_new); - unlock_page(page); put_page(page);
| |