uprobes/x86: Kill the "ia32_compat" check in handle_riprel_insn(), remove "mm" arg
authorOleg Nesterov <oleg@redhat.com>
Mon, 31 Mar 2014 16:09:36 +0000 (18:09 +0200)
committerOleg Nesterov <oleg@redhat.com>
Thu, 17 Apr 2014 19:58:17 +0000 (21:58 +0200)
Kill the "mm->context.ia32_compat" check in handle_riprel_insn(), if
it is true insn_rip_relative() must return false. validate_insn_bits()
passed "ia32_compat" as !x86_64 to insn_init(), and insn_rip_relative()
checks insn->x86_64.

Also, remove the no longer needed "struct mm_struct *mm" argument and
the unnecessary "return" at the end.

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Reviewed-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Reviewed-by: Jim Keniston <jkenisto@us.ibm.com>
Acked-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
arch/x86/kernel/uprobes.c

index 098e56ec79543490867cdc4950c676bde0a4626e..963c121c0307ac9f094e566b65c474559e186dba 100644 (file)
@@ -253,14 +253,11 @@ static int validate_insn_32bits(struct arch_uprobe *auprobe, struct insn *insn)
  *  - The displacement is always 4 bytes.
  */
 static void
-handle_riprel_insn(struct arch_uprobe *auprobe, struct mm_struct *mm, struct insn *insn)
+handle_riprel_insn(struct arch_uprobe *auprobe, struct insn *insn)
 {
        u8 *cursor;
        u8 reg;
 
-       if (mm->context.ia32_compat)
-               return;
-
        if (!insn_rip_relative(insn))
                return;
 
@@ -314,7 +311,6 @@ handle_riprel_insn(struct arch_uprobe *auprobe, struct mm_struct *mm, struct ins
                cursor++;
                memmove(cursor, cursor + insn->displacement.nbytes, insn->immediate.nbytes);
        }
-       return;
 }
 
 static int validate_insn_64bits(struct arch_uprobe *auprobe, struct insn *insn)
@@ -343,7 +339,7 @@ static int validate_insn_bits(struct arch_uprobe *auprobe, struct mm_struct *mm,
        return validate_insn_64bits(auprobe, insn);
 }
 #else /* 32-bit: */
-static void handle_riprel_insn(struct arch_uprobe *auprobe, struct mm_struct *mm, struct insn *insn)
+static void handle_riprel_insn(struct arch_uprobe *auprobe, struct insn *insn)
 {
        /* No RIP-relative addressing on 32-bit */
 }
@@ -376,7 +372,7 @@ int arch_uprobe_analyze_insn(struct arch_uprobe *auprobe, struct mm_struct *mm,
         * and annotate arch_uprobe->fixups accordingly. To start with, ->fixups
         * is either zero or it reflects rip-related fixups.
         */
-       handle_riprel_insn(auprobe, mm, &insn);
+       handle_riprel_insn(auprobe, &insn);
 
        switch (OPCODE1(&insn)) {
        case 0x9d:              /* popf */