aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPetr Tesarik <ptesarik@suse.cz>2008-02-11 16:42:34 -0500
committerTony Luck <tony.luck@intel.com>2008-03-05 18:48:16 -0500
commite868a55c2a8cb72b66d7137fbcc54b82016e98eb (patch)
tree05c1fc4e3d84537aa80709a1091c0ce2978f025d
parent08b23d74e07ac053fe4a5d6f4a48e8048fcfe52b (diff)
[IA64] remove find_thread_for_addr()
find_thread_for_addr() is no longer needed. It was only used to find the correct kernel RBS for a given memory address, but since the kernel RBS is not needed any longer, this function can go away. Signed-off-by: Petr Tesarik <ptesarik@suse.cz> Signed-off-by: Tony Luck <tony.luck@intel.com>
-rw-r--r--arch/ia64/kernel/ptrace.c56
1 files changed, 1 insertions, 55 deletions
diff --git a/arch/ia64/kernel/ptrace.c b/arch/ia64/kernel/ptrace.c
index 54b7ea551559..e82fe296c2c0 100644
--- a/arch/ia64/kernel/ptrace.c
+++ b/arch/ia64/kernel/ptrace.c
@@ -698,52 +698,6 @@ thread_matches (struct task_struct *thread, unsigned long addr)
698} 698}
699 699
700/* 700/*
701 * GDB apparently wants to be able to read the register-backing store
702 * of any thread when attached to a given process. If we are peeking
703 * or poking an address that happens to reside in the kernel-backing
704 * store of another thread, we need to attach to that thread, because
705 * otherwise we end up accessing stale data.
706 *
707 * task_list_lock must be read-locked before calling this routine!
708 */
709static struct task_struct *
710find_thread_for_addr (struct task_struct *child, unsigned long addr)
711{
712 struct task_struct *p;
713 struct mm_struct *mm;
714 struct list_head *this, *next;
715 int mm_users;
716
717 if (!(mm = get_task_mm(child)))
718 return child;
719
720 /* -1 because of our get_task_mm(): */
721 mm_users = atomic_read(&mm->mm_users) - 1;
722 if (mm_users <= 1)
723 goto out; /* not multi-threaded */
724
725 /*
726 * Traverse the current process' children list. Every task that
727 * one attaches to becomes a child. And it is only attached children
728 * of the debugger that are of interest (ptrace_check_attach checks
729 * for this).
730 */
731 list_for_each_safe(this, next, &current->children) {
732 p = list_entry(this, struct task_struct, sibling);
733 if (p->tgid != child->tgid)
734 continue;
735 if (thread_matches(p, addr)) {
736 child = p;
737 goto out;
738 }
739 }
740
741 out:
742 mmput(mm);
743 return child;
744}
745
746/*
747 * Write f32-f127 back to task->thread.fph if it has been modified. 701 * Write f32-f127 back to task->thread.fph if it has been modified.
748 */ 702 */
749inline void 703inline void
@@ -1520,7 +1474,6 @@ asmlinkage long
1520sys_ptrace (long request, pid_t pid, unsigned long addr, unsigned long data) 1474sys_ptrace (long request, pid_t pid, unsigned long addr, unsigned long data)
1521{ 1475{
1522 struct pt_regs *pt; 1476 struct pt_regs *pt;
1523 unsigned long peek_or_poke;
1524 struct task_struct *child; 1477 struct task_struct *child;
1525 struct switch_stack *sw; 1478 struct switch_stack *sw;
1526 long ret; 1479 long ret;
@@ -1532,19 +1485,12 @@ sys_ptrace (long request, pid_t pid, unsigned long addr, unsigned long data)
1532 goto out; 1485 goto out;
1533 } 1486 }
1534 1487
1535 peek_or_poke = (request == PTRACE_PEEKTEXT
1536 || request == PTRACE_PEEKDATA
1537 || request == PTRACE_POKETEXT
1538 || request == PTRACE_POKEDATA);
1539 ret = -ESRCH; 1488 ret = -ESRCH;
1540 read_lock(&tasklist_lock); 1489 read_lock(&tasklist_lock);
1541 { 1490 {
1542 child = find_task_by_pid(pid); 1491 child = find_task_by_pid(pid);
1543 if (child) { 1492 if (child)
1544 if (peek_or_poke)
1545 child = find_thread_for_addr(child, addr);
1546 get_task_struct(child); 1493 get_task_struct(child);
1547 }
1548 } 1494 }
1549 read_unlock(&tasklist_lock); 1495 read_unlock(&tasklist_lock);
1550 if (!child) 1496 if (!child)