2009/10/5 Dmitry V. Levin <l...@altlinux.org>:
>> -                     tcp->u_nargs = MAX_ARGS;
>> +                     /* Unknown syscall, assume maximum number of 
>> arguments.  */
>> +                     tcp->u_nargs = sizeof(args)/sizeof(args[0]);
>>               for (i = 0; i < tcp->u_nargs; i++) {
>> -                     if (upeek(tcp, PT_GR26-4*i, &tcp->u_arg[i]) < 0)
>> +                     if (upeek(tcp, args[i], &tcp->u_arg[i]) < 0)
>>                               return -1;
>>               }
>>       }
>
> This hunk does not look correct.
> If MAX_ARGS should be lowered on HPPA from 32 to 6, let's change it in
> the defs.h file.

Dmitry,

Lowering HPPA from 32 to 6 syscall args is correct.

Thank you for committing the hppa fixes listed below.

I have verified that upstream strace HEAD works correctly on hppa.

~~~
commit 67874d8393499e3c7f0d5a950f2707758560f61f
Author: Dmitry V. Levin <l...@altlinux.org>
Date:   Mon Oct 5 23:32:39 2009 +0000

    * defs.h [HPPA]: Lower MAX_ARGS from 32 to 6.

commit c0124e64489277d2d8a6c76ee66579ffa18e052e
Author: Dmitry V. Levin <l...@altlinux.org>
Date:   Mon Oct 5 23:31:54 2009 +0000

    * ipc.c [LINUX] (sys_shmat): HPPA does not use an IPC multiplexer.

    Based on patch from Carlos O'Donell.

commit 1dc2061c2ac7a73fa3d88fa0b10079151f7de530
Author: Carlos O'Donell <car...@systemhalted.org>
Date:   Mon Sep 14 02:06:20 2009 +0000

    * linux/hppa/syscallent.h: Update syscalls.

    Based on work by Kyle McMartin and Helge Deller.

commit 4677c8af94bcb3f39d5bf929ee44cd6724b7d0b8
Author: Carlos O'Donell <car...@systemhalted.org>
Date:   Wed Sep 9 18:13:19 2009 +0000

    Fix SA_HANDLER function pointer comparisons for hppa

    * signal.c (sys_sigaction): Cast SA_HANDLER function pointers to long.
    (sys_rt_sigaction): Likewise.
~~~

Cheers,
Carlos.

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to