On Monday, January 27, 2014 20:58:11 Ezequiel García wrote:
> On 27 January 2014 17:54, Mike Frysinger wrote:
> > On Sunday, January 26, 2014 21:59:07 Ezequiel Garcia wrote:
> >> --- a/system.c
> >> +++ b/system.c
> >> 
> >> +int sys_getpagesize(struct tcb *tcp)
> >> +{
> >> +     return 0;
> >> +}
> > 
> > that can't possibly be correct ?
> 
> Hm... why? The syscall receives no args and returns the page size through
> the get_error() path which retrieves the syscall result.

doesn't declaring it using printargs in the syscall list work ?

> After talking to Arnd Bergmann, we might rework the whole syscall thing
> for this arch to implement the generic syscall ABI.
> 
> Therefore, I'm not really sure what to do with this patch, as it works
> fine with current nios2 available kernel and toolchain.

i'd be fine with merging it since it works as-is, but i'm not an strace 
maintainer, so i don't get to say :)

> Is it possible to take this and then change the implementation once
> the generic syscall ABI is implemented?

i would think that the existing ptrace ABI would remain unchanged, just the 
syscall #'s would get rewritten (as well as some get added/removed).  so most 
of your patch would still work ... just update the syscall list file.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
WatchGuard Dimension instantly turns raw network data into actionable 
security intelligence. It gives you real-time visual feedback on key
security issues and trends.  Skip the complicated setup - simply import
a virtual appliance and go from zero to informed in seconds.
http://pubads.g.doubleclick.net/gampad/clk?id=123612991&iu=/4140/ostg.clktrk
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to