blackfin: remove dump_thread()

This patch removes the unused dump_thread().

Signed-off-by Adrian Bunk <[email protected]>

---

 arch/blackfin/kernel/bfin_ksyms.c  |    1
 arch/blackfin/kernel/process.c     |   45 -----------------------------
 include/asm-blackfin/bfin-global.h |    1
 3 files changed, 47 deletions(-)

c7746b10ac98a3166a7257f7117dd125e5ba4c7b

arch/blackfin/kernel/bfin_ksyms.c +0 -1

index 2198afe..99ea57c 100644
@@ -39,7 +39,6 @@
 EXPORT_SYMBOL(__ioremap);
 EXPORT_SYMBOL(strcmp);
 EXPORT_SYMBOL(strncmp);
-EXPORT_SYMBOL(dump_thread);
 
 EXPORT_SYMBOL(ip_fast_csum);
 

arch/blackfin/kernel/process.c +0 -45

index 9124467..1a8cf33 100644
@@ -239,51 +239,6 @@ copy_thread(int nr, unsigned long clone_flags,
 }
 
 /*
- * fill in the user structure for a core dump..
- */
-void dump_thread(struct pt_regs *regs, struct user *dump)
-{
-       dump->magic = CMAGIC;
-       dump->start_code = 0;
-       dump->start_stack = rdusp() & ~(PAGE_SIZE - 1);
-       dump->u_tsize = ((unsigned long)current->mm->end_code) >> PAGE_SHIFT;
-       dump->u_dsize = ((unsigned long)(current->mm->brk +
-                                        (PAGE_SIZE - 1))) >> PAGE_SHIFT;
-       dump->u_dsize -= dump->u_tsize;
-       dump->u_ssize = 0;
-
-       if (dump->start_stack < TASK_SIZE)
-               dump->u_ssize =
-                   ((unsigned long)(TASK_SIZE -
-                                    dump->start_stack)) >> PAGE_SHIFT;
-
-       dump->u_ar0 = (struct user_regs_struct *)((int)&dump->regs - (int)dump);
-
-       dump->regs.r0 = regs->r0;
-       dump->regs.r1 = regs->r1;
-       dump->regs.r2 = regs->r2;
-       dump->regs.r3 = regs->r3;
-       dump->regs.r4 = regs->r4;
-       dump->regs.r5 = regs->r5;
-       dump->regs.r6 = regs->r6;
-       dump->regs.r7 = regs->r7;
-       dump->regs.p0 = regs->p0;
-       dump->regs.p1 = regs->p1;
-       dump->regs.p2 = regs->p2;
-       dump->regs.p3 = regs->p3;
-       dump->regs.p4 = regs->p4;
-       dump->regs.p5 = regs->p5;
-       dump->regs.orig_p0 = regs->orig_p0;
-       dump->regs.a0w = regs->a0w;
-       dump->regs.a1w = regs->a1w;
-       dump->regs.a0x = regs->a0x;
-       dump->regs.a1x = regs->a1x;
-       dump->regs.rets = regs->rets;
-       dump->regs.astat = regs->astat;
-       dump->regs.pc = regs->pc;
-}
-
-/*
  * sys_execve() executes a new program.
  */
 

include/asm-blackfin/bfin-global.h +0 -2

index 0212e18..cd924bb 100644
@@ -50,7 +50,6 @@ extern unsigned long get_sclk(void);
 extern unsigned long sclk_to_usecs(unsigned long sclk);
 extern unsigned long usecs_to_sclk(unsigned long usecs);
 
-extern void dump_thread(struct pt_regs *regs, struct user *dump);
 extern void dump_bfin_regs(struct pt_regs *fp, void *retaddr);
 extern void dump_bfin_trace_buffer(void);
 
 
-

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Quoted 2 lines · Adrian Bunk

This patch removes the unused dump_thread().

Why only remove it from Blackfin? any more reason?

I found this in latest 2.6.24-rc2:

--
Cscope tag: dump_thread
# line filename / context / line
1 324 arch/alpha/kernel/process.c <<dump_thread>>
dump_thread(struct pt_regs * pt, struct user * dump)
2 373 arch/arm/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
3 244 arch/blackfin/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs *regs, struct user *dump)
4 321 arch/m68k/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
5 572 arch/sparc/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
6 731 arch/sparc64/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
7 306 arch/um/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs *regs, struct user *u)
8 519 arch/x86/kernel/process_32.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
--

Regards,
-Bryan

Quoted 4 lines · Bryan Wu

Quoted 2 lines · Adrian Bunk

This patch removes the unused dump_thread().

Quoted 1 line

Why only remove it from Blackfin? any more reason?

The only user is the a.out support.

It was therefore removed prior to the blackfin merge from all architectures not supporting a.out.

Quoted 23 lines

I found this in latest 2.6.24-rc2:

--
Cscope tag: dump_thread
# line filename / context / line
1 324 arch/alpha/kernel/process.c <<dump_thread>>
dump_thread(struct pt_regs * pt, struct user * dump)
2 373 arch/arm/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
3 244 arch/blackfin/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs *regs, struct user *dump)
4 321 arch/m68k/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
5 572 arch/sparc/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
6 731 arch/sparc64/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
7 306 arch/um/kernel/process.c <<dump_thread>>
void dump_thread(struct pt_regs *regs, struct user *u)
8 519 arch/x86/kernel/process_32.c <<dump_thread>>
void dump_thread(struct pt_regs * regs, struct user * dump)
--

Quoted 2 lines

Regards,
-Bryan

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

Quoted 4 lines · Adrian Bunk

Quoted 3 lines · Bryan Wu

Quoted 1 line · Adrian Bunk

This patch removes the unused dump_thread().

Quoted 6 lines

Quoted 1 line

Why only remove it from Blackfin? any more reason?

The only user is the a.out support.

It was therefore removed prior to the blackfin merge from all architectures not supporting a.out.

OK, make sense. Currently, Blackfin doesn't suppport a.out.