From: Davidlohr Bueso <d...@stgolabs.net>

This becomes quite straightforward with the mmrange in place.

Signed-off-by: Davidlohr Bueso <dbu...@suse.de>
---
 arch/blackfin/kernel/ptrace.c | 5 +++--
 arch/blackfin/kernel/trace.c  | 7 ++++---
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/arch/blackfin/kernel/ptrace.c b/arch/blackfin/kernel/ptrace.c
index a6827095b99a..e6657ab61afc 100644
--- a/arch/blackfin/kernel/ptrace.c
+++ b/arch/blackfin/kernel/ptrace.c
@@ -121,15 +121,16 @@ is_user_addr_valid(struct task_struct *child, unsigned 
long start, unsigned long
        bool valid;
        struct vm_area_struct *vma;
        struct sram_list_struct *sraml;
+       DEFINE_RANGE_LOCK_FULL(mmrange);
 
        /* overflow */
        if (start + len < start)
                return -EIO;
 
-       down_read(&child->mm->mmap_sem);
+       mm_read_lock(child->mm, &mmrange);
        vma = find_vma(child->mm, start);
        valid = vma && start >= vma->vm_start && start + len <= vma->vm_end;
-       up_read(&child->mm->mmap_sem);
+       mm_read_unlock(child->mm, &mmrange);
        if (valid)
                return 0;
 
diff --git a/arch/blackfin/kernel/trace.c b/arch/blackfin/kernel/trace.c
index 151f22196ab6..9bf938b14601 100644
--- a/arch/blackfin/kernel/trace.c
+++ b/arch/blackfin/kernel/trace.c
@@ -33,6 +33,7 @@ void decode_address(char *buf, unsigned long address)
        struct mm_struct *mm;
        unsigned long offset;
        struct rb_node *n;
+       DEFINE_RANGE_LOCK_FULL(mmrange);
 
 #ifdef CONFIG_KALLSYMS
        unsigned long symsize;
@@ -124,7 +125,7 @@ void decode_address(char *buf, unsigned long address)
                        continue;
 
                mm = t->mm;
-               if (!down_read_trylock(&mm->mmap_sem))
+               if (!mm_read_trylock(mm, &mmrange))
                        goto __continue;
 
                for (n = rb_first(&mm->mm_rb); n; n = rb_next(n)) {
@@ -166,7 +167,7 @@ void decode_address(char *buf, unsigned long address)
                                        sprintf(buf, "[ %s vma:0x%lx-0x%lx]",
                                                name, vma->vm_start, 
vma->vm_end);
 
-                               up_read(&mm->mmap_sem);
+                               mm_read_unlock(mm, &mmrange);
                                task_unlock(t);
 
                                if (buf[0] == '\0')
@@ -176,7 +177,7 @@ void decode_address(char *buf, unsigned long address)
                        }
                }
 
-               up_read(&mm->mmap_sem);
+               mm_read_unlock(mm, &mmrange);
 __continue:
                task_unlock(t);
        }
-- 
2.13.6

Reply via email to