----- Original Message -----
> From: "Mike Frysinger" <[email protected]>
> To: [email protected]
> Sent: Thursday, 9 January, 2014 2:03:38 PM
> Subject: Re: [LTP] [PATCH] clone/clone08.c: check whether clone supports 7
> arguments
>
> On Wednesday 08 January 2014 09:31:30 Zeng Linggang wrote:
> > +AC_DEFUN([LTP_CHECK_CLONE7ARGS],[
> > +AH_TEMPLATE(HAVE_CLONE7ARGS,
> > +[Define to 1 if clone() supports 7 arguments.])
> > +AC_MSG_CHECKING([for CLONE7ARGS])
> > +AC_TRY_LINK([#define _GNU_SOURCE
> > + #include <sched.h>
> > + #include <stdlib.h>],
> > + [
> > + #if !defined(__ia64__)
> > + clone(NULL, NULL, 0, NULL, NULL, NULL, NULL);
> > + #endif
> > + ],
> > + AC_DEFINE(HAVE_CLONE7ARGS) AC_MSG_RESULT(yes),
> > AC_MSG_RESULT(no))
> > +])
>
> you aren't really testing for "clone takes 7 args", you're testing for "clone
> has varargs support". rename the define (and use _ in its name), and drop
> the
> ia64 check as it isn't needed.
Why not? It should matter, according to clone(2) you should be using __clone2()
on ia64:
ia64
On ia64, a different interface is used:
int __clone2(int (*fn)(void *),
void *child_stack_base, size_t stack_size,
int flags, void *arg, ...
/* pid_t *ptid, struct user_desc *tls, pid_t *ctid */ );
# cat a.c
#define _GNU_SOURCE
#include <sched.h>
#include <stdlib.h>
int main()
{
clone(NULL, NULL, 0, NULL, NULL, NULL, NULL);
return 0;
}
# uname -m
ia64
# gcc a.c
/tmp/cc0QVVOU.o: In function `main':
a.c:(.text+0x42): undefined reference to `clone'
collect2: ld returned 1 exit status
So perhaps we should add #else and test for __clone2 on ia64.
Regards,
Jan
> -mike
>
> ------------------------------------------------------------------------------
> CenturyLink Cloud: The Leader in Enterprise Cloud Services.
> Learn Why More Businesses Are Choosing CenturyLink Cloud For
> Critical Workloads, Development Environments & Everything In Between.
> Get a Quote or Start a Free Trial Today.
> http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
> _______________________________________________
> Ltp-list mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>
------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
