Subject: Re: port-m68k/37631: m68k libc broken
To: None <port-m68k-maintainer@netbsd.org, gnats-admin@netbsd.org,>
From: ITOH Yasufumi <itohy@NetBSD.org>
List: netbsd-bugs
Date: 12/28/2007 17:35:02
The following reply was made to PR port-m68k/37631; it has been noted by GNATS.

From: itohy@NetBSD.org (ITOH Yasufumi)
To: gnats-bugs@NetBSD.org
Cc: port-m68k-maintainer@NetBSD.org, gnats-admin@NetBSD.org,
	netbsd-bugs@NetBSD.org, martin@duskware.de
Subject: Re: port-m68k/37631: m68k libc broken
Date: Sat, 29 Dec 2007 02:34:39 +0900

 martin@duskware.de writes:
 
 > On Fri, Dec 28, 2007 at 03:50:01PM +0000, ITOH Yasufumi wrote:
 > >  This should be in libc to avoid adding clock cycles to unrelated
 > >  system calls.
 > >  See shmat(2) stub.
 > 
 > I agree, but that is not how it is done right now - and that it didn't
 > work for me was just a side effect of me not having COMPAT_AOUT_M68K
 > defined.
 
 Ah, I see.  The code to copy return value to a0
 didn't exist when I wrote userland ELF support.
 
 > What my change does is just remove the (bogus) dependency on that option.
 > All GENERIC kernels already duplicate rval[0] to a0.
 > 
 > So how about this: I commit the patch (with syscall_fancy added) for now
 > and then start collecting a table of missing stubs in the m68k libc.
 > Once we have added all missing stubs, we'll make the a0 copy #ifdef COMPAT_50?
 
 That looks fine.
 
 Thanks,
 -- 
 ITOH Yasufumi