lkml.org 
[lkml]   [2009]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] Fix the multithread program core thread message error
From
Hi Amerigo,

Thanks for your reply.

On Wed, Jul 1, 2009 at 08:54, Amerigo Wang<xiyou.wangcong@gmail.com> wrote:
>
> Hi, Hui.
>
> On Tue, Jun 30, 2009 at 05:12:31PM +0800, Hui Zhu wrote:
>>Fix the multithread program core thread message error.
>>The thread message of core file is generated in
>>elf_dump_thread_status.  The register values is set by
>>elf_core_copy_task_regs in this function.
>>static inline int elf_core_copy_task_regs(struct task_struct *t,
>>                                          elf_gregset_t* elfregs)
>>{
>>
>>       return ELF_CORE_COPY_TASK_REGS(t, elfregs);
>>       return 0;
>>}
>>If a arch doesn't define ELF_CORE_COPY_TASK_REGS, This function will do
>>nothing.  Then the core file will not have the register message of
>>thread.
>>So add elf_core_copy_regs to set regiser values if
>>ELF_CORE_COPY_TASK_REGS doesn't define.
>
>
> You forgot your Signed-off-by line. :)

Signed-off-by: Hui Zhu <hui.zhu@windriver.com>

>
> Hmmm, this patch looks sane for me. But could you please
> send us your test program? i.e. how did you test this?
>

I test this issue in a arm board. My test code is:
#include <stdio.h>
#include <pthread.h>
#include <assert.h>

void td1(void * i)
{
while (1)
{
printf ("1\n");
sleep (1);
}

return;
}

void td2(void * i)
{
while (1)
{
printf ("2\n");
sleep (1);
}

return;
}

int
main(int argc,char *argv[],char *envp[])
{
pthread_t t1,t2;

pthread_create(&t1, NULL, (void*)td1, NULL);
pthread_create(&t2, NULL, (void*)td2, NULL);

sleep (10);

assert(0);

return (0);
}

The follow is how to reproduce this issue:
arm-xxx-gcc -g -lpthread 1.c -o 1
copy 1.c and 1 to a arm board.
Goto this board.
ulimit -c 1800000
./1
# ./1
1
2
1
...
...
1
1: 1.c:37: main: Assertion `0' failed.
Aborted (core dumped)
Then you can get a core file.
gdb 1 core.xxx

Without the patch:
(gdb) info threads
3 process 909 0x00000000 in ?? ()
2 process 908 0x00000000 in ?? ()
* 1 process 907 0x4a6e2238 in raise () from /lib/libc.so.6
You can found that the pc of 909 and 908 is 0x00000000.

With the patch:
(gdb) info threads
3 process 885 0x4a749974 in nanosleep () from /lib/libc.so.6
2 process 884 0x4a749974 in nanosleep () from /lib/libc.so.6
* 1 process 883 0x4a6e2238 in raise () from /lib/libc.so.6
The pc of 885 and 884 is right.


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

\
 
 \ /
  Last update: 2009-07-01 05:31    [W:0.068 / U:1.392 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site